Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CHIA-1552] Proposed set of drivers for multi-sig custody #18686

Draft
wants to merge 141 commits into
base: main
Choose a base branch
from

Conversation

Quexington
Copy link
Contributor

@Quexington Quexington commented Oct 9, 2024

I think this PR probably should not be merged as it has no effect on anything running in production. This PR is instead a reference implementation for a new inner puzzle meta-standard that has instructions for how a wallet can handle arbitrary authorization trees of so called "members" to be used as inner puzzles for assets. The full spec is here: https://gist.github.com/Quexington/32936e9a03bbf4f1956f3538096f2a83 .

@Quexington Quexington added clvm Added Required label for PR that categorizes merge commit message as "Added" for changelog labels Oct 9, 2024
@Chia-Network Chia-Network deleted a comment from github-actions bot Oct 10, 2024
@Chia-Network Chia-Network deleted a comment from coveralls-official bot Oct 10, 2024
…8690)

An optimization of inner puzzle construction when an M of N is involved
and M=1
…18694)

An optimization of inner puzzle construction when an M of N is involved
and M=N
A small bug where the API of member puzzles was not taken into account
when they are the only puzzle in the stack.
Copy link
Contributor

File Coverage Missing Lines
chia/wallet/puzzles/custody/custody_architecture.py 97.6% lines 180, 275, 310, 325, 341
chia/wallet/puzzles/custody/member_puzzles/member_puzzles.py 99.1% lines 69
chia/wallet/puzzles/custody/restriction_puzzles/restrictions.py 94.0% lines 45, 61, 80, 113
Total Missing Coverage
913 lines 10 lines 98%

matt-o-how and others added 10 commits November 19, 2024 17:17
…sh (#18898)

### Purpose:

Add a new Fixed Puzzle Member puzzle, corresponding class, and test


### Testing Notes:

Added a test to `test_member_puzzles.py` which tests failure and success
cases
…e hash (#18886)

Mostly a duplication of the passkey member puzzle but with
`ASSERT_MY_PUZZLE_HASH` instead.
@matt-o-how matt-o-how force-pushed the quex.multi_sig_chialisp_drivers branch from 25a5727 to 4652380 Compare November 20, 2024 14:58
matt-o-how and others added 10 commits November 20, 2024 15:00
… ID commitment (#18885)

This is a near duplication of the restriction that forces a coin
announcement. The purpose is the same.
### Purpose:

Add a BLS With Taproot chialisp puzzle, member class, and tests
### Testing Notes:

New tests have been added for taproot and non-taproot spend cases in
`test_member_puzzles.py`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Added Required label for PR that categorizes merge commit message as "Added" for changelog clvm coverage-diff
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants