Skip to content

chore: bump verifier to v0.1.26 (plugin_id enum to text)#157

Open
RaghavSood wants to merge 1 commit intomainfrom
bump-verifier-plugin-id-text
Open

chore: bump verifier to v0.1.26 (plugin_id enum to text)#157
RaghavSood wants to merge 1 commit intomainfrom
bump-verifier-plugin-id-text

Conversation

@RaghavSood
Copy link
Collaborator

@RaghavSood RaghavSood commented Feb 8, 2026

Summary

  • Bump github.com/vultisig/verifier from v0.1.20-0.20260206093101-7552132a5cd0 to v0.1.26

Context

This picks up the system migration from vultisig/verifier#547 that converts the plugin_id PostgreSQL ENUM type to TEXT.

The system migration:

  • Dynamically discovers all columns using the plugin_id enum via pg_catalog
  • Converts them to TEXT
  • Drops the enum and creates CREATE DOMAIN plugin_id AS TEXT for forward-compatibility

Note: This repo uses plugin.WithMigrations() which runs plugin-specific migrations (scheduler, policy, tx_indexer, safety) but does not run system migrations. If the database this app connects to has already had system migrations applied by the verifier, this bump ensures the embedded plugin migrations are compatible with the new schema.

Test plan

  • go build ./... passes
  • Existing tests pass
  • Verify app starts and connects to database successfully

Summary by CodeRabbit

  • Chores
    • Updated project dependencies to latest stable versions for improved security and stability.

Picks up the system migration that converts the plugin_id PostgreSQL
ENUM type to TEXT with a domain alias. See vultisig/verifier#547.

Co-Authored-By: Claude (claude-opus-4-6) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings February 8, 2026 10:25
@coderabbitai
Copy link

coderabbitai bot commented Feb 8, 2026

Walkthrough

The pull request updates multiple Go module dependencies in go.mod. The primary update is the verifier library from v0.1.20 to v0.1.26. Additionally, several golang.org standard library packages are upgraded, including crypto, sync, and indirect dependencies for net, sys, term, and text modules.

Changes

Cohort / File(s) Summary
Dependency Updates
go.mod
Bumps verifier from v0.1.20-0.20260206093101-7552132a5cd0 to v0.1.26 and upgrades golang.org/x dependencies (crypto v0.45.0→v0.46.0, sync v0.18.0→v0.19.0, net v0.47.0→v0.48.0, sys v0.38.0→v0.39.0, term v0.37.0→v0.38.0, text v0.31.0→v0.33.0). Indirect dependencies updated correspondingly.
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically identifies the main change: bumping the verifier module to v0.1.26 and notes the key schema change (plugin_id enum to text conversion).
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch bump-verifier-plugin-id-text

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Bumps the github.com/vultisig/verifier dependency to pick up upstream changes related to the plugin_id PostgreSQL type migration (ENUM → TEXT / domain), keeping this service’s embedded plugin migrations aligned with verifier’s expected schema.

Changes:

  • Bump github.com/vultisig/verifier from a pseudo-version to v0.1.26.
  • Refresh Go stdlib-adjacent dependency versions (golang.org/x/*) as a result of the module bump.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
go.mod Updates verifier and related golang.org/x/* module versions.
go.sum Updates checksums to reflect the new module versions.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@go.mod`:
- Line 30: Do not change the verifier version to v0.1.26 until you verify that
v0.1.26 is actually published; instead follow a staged upgrade: first upgrade to
v0.1.25, run and verify DB migrations, then upgrade to v0.1.26. If you proceed
with v0.1.26, update code to remove any PostgreSQL casts like ::plugin_id in SQL
queries and replace SQLC's PluginID wrapper type with plain string across
generated code and call sites (search for PluginID() and PluginID types),
recreate any custom indexes that referenced the old enum plugin_id, and confirm
that the system migration that converts plugin_id ENUM→TEXT will not conflict
with the app's migration mechanism (check usages of plugin.WithMigrations()).
Also verify the existence of the v0.1.26 release on GitHub before committing the
version bump.

github.com/vultisig/mobile-tss-lib v0.0.0-20250316003201-2e7e570a4a74
github.com/vultisig/recipes v0.0.0-20260129020926-577976dfb292
github.com/vultisig/verifier v0.1.20-0.20260206093101-7552132a5cd0
github.com/vultisig/verifier v0.1.26
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🔴 Critical

🧩 Analysis chain

🌐 Web query:

vultisig verifier v0.1.26 release github

💡 Result:

I couldn’t find any GitHub release/tag for vultisig/verifier v0.1.26 via public indexing/search.

The most recent verifier releases I can confirm from Vultisig’s aggregated GitHub snapshot are v0.1.20v0.1.23, all dated 2026-02-06. [1] Libraries.io also shows the repo is actively releasing, but it doesn’t surface a v0.1.26 tag in the accessible results. [2]

To double-check directly against GitHub, run (from any machine with git):

git ls-remote --tags https://github.com/vultisig/verifier.git | grep -F "v0.1.26"

Sources: [1] [2]


🌐 Web query:

vultisig verifier v0.1.20 v0.1.26 breaking changes migration

💡 Result:

Breaking changes between v0.1.20v0.1.26 (and how to migrate)

1) Plugin images: legacy columns removed (DB + API payload change)

  • v0.1.20 introduced a plugin image management API (presigned S3 upload + confirm flow) and image endpoints under /plugins/:id/images/*. [1]
  • v0.1.24 then dropped legacy inline plugin image fields/columns (e.g., logo_url, thumbnail_url, images) and moved to/normalized around a PluginImage model; “installed plugins” responses get enriched with associated image records. This is a breaking schema+DTO change if you were reading/writing those old fields. [2]

Migration actions

  • Deploy and run DB migrations through v0.1.24 so the schema matches (legacy columns are removed). [2]
  • Update any code/queries/DTOs that referenced the removed fields (logo_url, thumbnail_url, images) to use the image records API/model instead. [2]
  • If you relied on the new image endpoints from v0.1.20, they remain the intended approach; note v0.1.21/v0.1.22 briefly reverted/reinstated #532, but upgrading to v0.1.26 lands you on the “images API + normalized storage” path. [1]

2) plugin_id type: PostgreSQL ENUM → TEXT (this is the big breaking change)

  • v0.1.26 converts plugin_id from a PostgreSQL ENUM to TEXT (via a dynamic “system” migration), removes enum casts, and removes the generated queries.PluginID() type usage—call sites become plain string. [3][4]

Important sequencing

  • The v0.1.26 migration is described as “Branch 2” and is intended to be deployed after #546 (which shipped in v0.1.25). So don’t jump straight from v0.1.20 to v0.1.26 in one step—go via v0.1.25 first. [4][5]

Migration actions (recommended path)

  1. Upgrade to v0.1.25, deploy, and run migrations. (v0.1.25 contains the preparatory removal of ::plugin_id casts, etc.). [5]
  2. Upgrade to v0.1.26, deploy, and run migrations. [3][4]
  3. In downstream/external consumers (anything using the shared common DB schema/migrations):
    • Remove ::plugin_id casts from your SQL. [4]
    • Replace any usage of SQLC’s removed PluginID wrapper type (queries.PluginID(...)) with plain string. [4]
    • Recreate any custom indexes that referenced the old enum type (the dynamic migration rebuilds FK/PK constraints, but not custom indexes). [4]

Source highlights

  • v0.1.20 release notes (new image endpoints + upload/confirm flow): [1]
  • v0.1.24 PR (drops legacy image columns; DTO/model changes): [2]
  • v0.1.25 release notes (#546 prep work): [5]
  • v0.1.26 release notes + PR #547 detailed migration guide (ENUM→TEXT): [3][4]

References:
[1] (github.com)
[2] (github.com)
[3] (github.com)
[4] (github.com)
[5] (github.com)

Citations:


🌐 Web query:

vultisig verifier pull request 547 plugin_id migration

💡 Result:

I couldn’t reliably retrieve vultisig/verifier Pull Request #547 itself (the PR page didn’t show up in accessible search results from GitHub), so I can’t quote or confirm the exact diff/commit message for “plugin_id migration”.

What I can confirm from Vultisig’s published docs/code is what that migration is almost certainly referring to:

  • plugin_id is a first-class field in plugin→verifier workflows, e.g., the plugin API’s /vault/reshare request includes a plugin_id string. [1]
  • The verifier-side plugin framework includes explicit DB migration support, via WithMigrations(...) and a MigrationManager intended to run “plugin-specific migrations”. [2]
  • Vultisig’s developer docs point plugin authors to build on the verifier plugin server foundation (github.com/vultisig/verifier/plugin). [3]

Practical implication / what to do for a “plugin_id migration”

  1. Update any plugin calls that now require/expect plugin_id (notably reshare flows) to always send it. [1]
  2. Deploy verifier with migrations enabled (use the verifier plugin framework’s migration runner) so the database schema/data changes associated with plugin_id are applied. [2]
  3. If you share more context (e.g., the PR link, title, or the failing error/log line), I can map it to the exact migration and describe the before/after schema and rollout order precisely.

Sources:
[1] Vultisig plugin repo docs showing plugin_id in /vault/reshare request: https://github.com/vultisig/plugin (see “Reshare Request”). (github.com)
[2] Go package docs for github.com/vultisig/verifier/plugin showing WithMigrations and MigrationManager. (pkg.go.dev)
[3] Vultisig “Build Your Plugin” docs referencing github.com/vultisig/verifier/plugin. (docs.vultisig.com)

Citations:


Do not upgrade directly to v0.1.26; v0.1.26 release cannot be confirmed to exist.

Web search indicates that while v0.1.20–v0.1.23 (dated 2026-02-06) are confirmed released on GitHub, v0.1.26 does not appear in public GitHub releases. Verify that v0.1.26 exists before proceeding.

If upgrading to v0.1.26, the upgrade path and breaking changes are more complex than noted:

  1. Upgrade sequencing: Do not jump directly from v0.1.20 to v0.1.26. The migration for plugin_id (ENUM → TEXT) requires deploying v0.1.25 first as a preparatory step, then v0.1.26.

  2. Breaking changes (v0.1.20 → v0.1.26):

    • v0.1.24: Plugin image schema changes—legacy logo_url, thumbnail_url, images columns are removed (breaking schema and DTO change)
    • v0.1.26: plugin_id type converts from PostgreSQL ENUM to TEXT via system migration (PR #547)
  3. Code updates required if this repo shares the verifier DB schema:

    • Remove ::plugin_id casts from SQL queries
    • Replace SQLC's PluginID() wrapper type with plain string
    • Recreate any custom indexes on plugin_id
  4. Migration compatibility: The system migration does run outside plugin.WithMigrations() context. Confirm that app-recurring's migration strategy (via plugin.WithMigrations()) will not conflict with or duplicate the system migration.

🤖 Prompt for AI Agents
In `@go.mod` at line 30, Do not change the verifier version to v0.1.26 until you
verify that v0.1.26 is actually published; instead follow a staged upgrade:
first upgrade to v0.1.25, run and verify DB migrations, then upgrade to v0.1.26.
If you proceed with v0.1.26, update code to remove any PostgreSQL casts like
::plugin_id in SQL queries and replace SQLC's PluginID wrapper type with plain
string across generated code and call sites (search for PluginID() and PluginID
types), recreate any custom indexes that referenced the old enum plugin_id, and
confirm that the system migration that converts plugin_id ENUM→TEXT will not
conflict with the app's migration mechanism (check usages of
plugin.WithMigrations()). Also verify the existence of the v0.1.26 release on
GitHub before committing the version bump.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants