Skip to content

Index initial vault co-owners on Mainnet - #29

Merged
maykonmichel merged 5 commits into
mainfrom
lndgalante/fix-multisig-discovery
Sep 28, 2026
Merged

maykonmichel merged 5 commits into
mainfrom
lndgalante/fix-multisig-discovery

Conversation

@lndgalante

@lndgalante lndgalante commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

Description

  • Add the validated mapping for discovering initial vault co-owners on Mainnet.
  • Fix multisig address types and add transaction timestamps in both processor configs.
  • Document the owner lookup query and deployment considerations.
  • Keep this PR limited to the two indexer YAML files and README. Deployment and mobile integration will follow separately.

Test Plan

  • Validated YAML syntax and checked that every table column has a source mapping.
  • Confirmed the Mainnet creation mapping matches the tested probe, apart from the production table name.
  • Checked the README's GraphQL example and formatting.
  • The probe previously found two known Mainnet vault creations. Production deployment is still pending.

maykonmichel and others added 4 commits September 25, 2026 15:09
…ion timestamp

remapping-processor v0.0.21 validates that every event_fields ($.x) target
column is a MoveType. Both multisig_account columns were declared
EventMetadata(AccountAddress) yet are fed by the $.multisig_account payload
field on the modern module events (Vote, AddOwners, RemoveOwners and the
execution events), so the processor crash-looped on boot with "Column
multisig_owner_activities.multisig_account should have a MoveType column_type"
and stopped indexing entirely.

Declare both multisig_account columns as MoveType(Address). The legacy *Event
variants that populate them from account_address metadata stay compatible, since
that metadata is the same underlying address.

Also map the previously-idle transaction_metadata.timestamp hook into a new
multisig_transactions.timestamp column, so execution history can be rendered
straight from the indexer, and bump spec_version to 0.0.21 to match the running
processor.

(cherry picked from commit b83a3a1)
@lndgalante
lndgalante requested a review from a team as a code owner September 25, 2026 18:15
@vercel

vercel Bot commented Sep 25, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
petra-vault-web Ready Ready Preview Sep 25, 2026 6:29pm UTC

Request Review

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 56f1c21. Configure here.

Comment thread apps/web/app/(authenticated)/vault/[vaultId]/(dashboard)/transactions/page.tsx Outdated
Comment thread apps/web/components/TransactionRow.tsx Outdated
@lndgalante lndgalante changed the title Fix vault history and initial co-owner discovery Index initial vault co-owners on Mainnet Sep 25, 2026
@maykonmichel
maykonmichel merged commit 9600232 into main Sep 28, 2026
9 checks passed
@maykonmichel
maykonmichel deleted the lndgalante/fix-multisig-discovery branch September 28, 2026 17:18

This branch was successfully deployed

1 active deployment
Preview — 0ba827fb Deployed Sep 25, 2026 by vercel[bot]
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.

2 participants