Skip to content

Change the number of signers from 11 to 3 - #982

Merged
automergerpr-permission-manager[bot] merged 1 commit into
feature/p-tokenfrom
mk/num-signers-3
Mar 10, 2026
Merged

Change the number of signers from 11 to 3#982
automergerpr-permission-manager[bot] merged 1 commit into
feature/p-tokenfrom
mk/num-signers-3

Conversation

@mariaKt

@mariaKt mariaKt commented Mar 9, 2026

Copy link
Copy Markdown
Contributor

This PR makes the necessary changes in symbolic/p-token.md to have 3 MAX_SIGNERS instead of 11. It looks that this is something we are working towards (e.g., runtimeverification/solana-token#169, https://runtimeverification.slack.com/archives/C03QD0KTDMJ/p1772645076956859).

@mariaKt

mariaKt commented Mar 9, 2026

Copy link
Copy Markdown
Contributor Author

Relevant PRs:

@mariaKt
mariaKt marked this pull request as ready for review March 9, 2026 20:16
@mariaKt
mariaKt requested review from Stevengre and dkcumming March 9, 2026 20:17

@Stevengre Stevengre left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@automergerpr-permission-manager
automergerpr-permission-manager Bot merged commit 1413178 into feature/p-token Mar 10, 2026
7 checks passed
@automergerpr-permission-manager
automergerpr-permission-manager Bot deleted the mk/num-signers-3 branch March 10, 2026 05:57
dkcumming added a commit to runtimeverification/solana-token that referenced this pull request Mar 10, 2026
- fix(rt): handle fun-type closure env refs in callee setup
[956](runtimeverification/mir-semantics#956)
- fix(decode): accept signed enum tag scalars in _extract_tag
[954](runtimeverification/mir-semantics#954)
- Added check for FunType in setupCalleeClosure
[969](runtimeverification/mir-semantics#969)
- Merge in latest master
[973](runtimeverification/mir-semantics#973)
- fix(kompile): map linked NormalSym functions to monoItemFn(noBody)
[953](runtimeverification/mir-semantics#953)
- fix(rt): generalize direct-tag enum decoding to any variant count and
discriminant
[955](runtimeverification/mir-semantics#955)
- fix(spl-token): encode multisig signers as pubkey aggregates
[958](runtimeverification/mir-semantics#958)
- fix(rt): closure aggregate + #setTupleArgs fallback
[952](runtimeverification/mir-semantics#952)
- fix(rt): bug fix for castKindPtrToPtr
[974](runtimeverification/mir-semantics#974)
- fix(rt): repair closure callee setup for iter-eq repro
[957](runtimeverification/mir-semantics#957)
- Change the number of signers from 11 to 3
[982](runtimeverification/mir-semantics#982)
- Merging latest master into feature/p-token
[981](runtimeverification/mir-semantics#981)
Stevengre added a commit that referenced this pull request Mar 13, 2026
## Summary
Align the symbolic SPL multisig model with the 3-signer proof layout
used on `feature/p-token`:
- Reduce symbolic multisig signer array from 11 to 3 entries (buffer
size 355 → 99 bytes)
- Keep `m` and `n` in raw `u8` range without validity constraints
- Update `spl-multisig-signer-index.rs` prove-rs test to match

Related: #982

## Test
`uv --project kmir run pytest
kmir/src/tests/integration/test_integration.py -k
'spl-multisig-signer-index'`
dkcumming added a commit to runtimeverification/kompass that referenced this pull request Mar 15, 2026
[977](runtimeverification/mir-semantics#977)
- SPL-token reduce `Multisig` signers from 11 to 3
- SPL-token reduce `Multisig` size from 355 to 99

[982](runtimeverification/mir-semantics#982)
- P-token reduce `Multisig` signers from 11 to 3
- P-token reduce `Multisig` size from 355 to 99
automergerpr-permission-manager Bot pushed a commit to runtimeverification/kompass that referenced this pull request Mar 15, 2026
### Collapse `prove-rs` into `prove`
[986](runtimeverification/mir-semantics#986)
- Removed `ProveRsOpts` for `KMirProveOpts`
- `ProveOpts` is a collision so `KMirProveOpts` is used

### fix(symbolic-spl): align multisig cheatcode with 3-signer layout
[977](runtimeverification/mir-semantics#977)
- SPL-token reduce `Multisig` signers from 11 to 3
- SPL-token reduce `Multisig` size from 355 to 99

### Change the number of signers from 11 to 3
[982](runtimeverification/mir-semantics#982)
- P-token reduce `Multisig` signers from 11 to 3
- P-token reduce `Multisig` size from 355 to 99

---------

Co-authored-by: devops <devops@runtimeverification.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: dkcumming <daniel.cumming@runtimeverification.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants