Skip to content

Solution: LP-0002 — Private M-of-N Multisig for LEZ - #115

Open
duongja wants to merge 3 commits into
logos-co:masterfrom
duongja:solution/lp-0002-private-multisig
Open

Solution: LP-0002 — Private M-of-N Multisig for LEZ#115
duongja wants to merge 3 commits into
logos-co:masterfrom
duongja:solution/lp-0002-private-multisig

Conversation

@duongja

@duongja duongja commented Jul 30, 2026

Copy link
Copy Markdown

Summary

This PR submits a solution for LP-0002: Private M-of-N Multisig.

It adds:

  • solutions/LP-0002.md

The solution links to the public implementation repository, narrated demo video, hosted testnet execution evidence, reviewer commands, and a checklist mapping the implementation to the prize success criteria.

Submission

Key Evidence

  • Example hosted testnet execute_private transaction:
    • e1805cae3ab6e46239cc823cc45d79e45a78ba80e57168e8fc50e240b9e70b59
  • Hosted testnet final state includes:
    • proposal_status: Executed
    • proposal_approval_count: 2
    • target_account_data_utf8: threshold-approved

Reviewer Notes

The linked implementation repository includes:

  • LEZ v0.2 program handlers for:
    • create_multisig
    • propose
    • execute_private
  • private approval aggregation with proposal-scoped nullifiers
  • reproducible standalone local sequencer flow with RISC0_DEV_MODE=0
  • hosted LEZ testnet execution evidence
  • Basecamp GUI that drives the real execution path
  • reviewer-facing docs and cost evidence

Minimal reviewer command in the implementation repo:

./scripts/preflight.sh

Full local evidence:

export RISC0_DEV_MODE=0
./scripts/preflight.sh --with-localnet

Hosted testnet evidence:

export RISC0_DEV_MODE=0
./scripts/testnet-evidence.sh .local/testnet-evidence/latest

Notes

The lambda-prize fork and this PR branch are updated onto upstream master with the validator fix. The PR validation check is green on the latest submission commit.

The current implementation documents one known ecosystem limitation explicitly:

  • official hosted-testnet per-transaction CU values depend on what the current LEZ explorer/RPC exposes

Proof-cycle and local transaction-cost evidence are included in the implementation repository.

@github-actions

github-actions Bot commented Jul 30, 2026

Copy link
Copy Markdown

⚠️ Warnings

Warnings (non-blocking):

  • ⚠️ No test files detected in linked repo.

ℹ️ Solution submission for LP-0002.
ℹ️ Checking repo: https://github.com/duongja/private-multisig


Automated check. See solution template and TERMS.

@duongja

duongja commented Jul 30, 2026

Copy link
Copy Markdown
Author

The current failing check is blocked before the validation script reads this solution file.

Failing run: https://github.com/logos-co/lambda-prize/actions/runs/30554437421

The failing step is Checkout PR head (untrusted data, never executed), with:

Refusing to check out fork pull request code from a 'pull_request_target' workflow

I opened a small workflow fix here:

I also ran the repository validator locally against this PR branch after the LP-0002 fixes. The submission content passes the validator; the only remaining validator note is a non-blocking warning about test-file naming in the linked implementation repo.

@duongja

duongja commented Jul 30, 2026

Copy link
Copy Markdown
Author

Update: this is the same upstream validator issue already tracked in #109.

Canonical fix PR:

I had opened #116 with the same one-line workflow change, but closed it as a duplicate to avoid splitting the thread. Once #109 is merged into master, the validator can be rerun on this LP-0002 solution PR.

@weboko

weboko commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

@duongja can you update your fork? ci should be working

@duongja
duongja force-pushed the solution/lp-0002-private-multisig branch from 82a7687 to 7d4b058 Compare August 1, 2026 00:20
@duongja

duongja commented Aug 1, 2026

Copy link
Copy Markdown
Author

Hello @weboko , done

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