share: Show the master fingerprint after the threshold - #100
BenWestgate wants to merge 2 commits into
Conversation
054e8d9 to
115f2c2
Compare
`ms32 secret` shows the master fingerprint with the recovered secret, but `ms32 share` never did, so the operator couldn't compare it with the wallet record before writing a new card. Print it above the derived share on a terminal, whether every input is a share or one of them is the secret. `--plain` and redirected output stay unchanged. Closes #86 Claude-Session: https://claude.ai/code/session_015CuLXqAvAovfoVcUmmogwa
eb37865 to
8aaf0fa
Compare
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Ben authorized raising the budget so #91 fits. The stack tip with the open fix PRs was at 5,197 of 5,200, and #91 adds 26 lines. Update the enforcing test and both places that document the number. Claude-Session: https://claude.ai/code/session_015CuLXqAvAovfoVcUmmogwa
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
BenWestgate
left a comment
BenWestgate
left a comment
There was a problem hiding this comment.
AI-generated review (Codex), posted at the maintainer's request.
Not ACKing a3a27b0 yet: this raises the installed-code budget from 5,200 to 5,250 without the explicit authorization AGENTS.md requires. The fingerprint-after-threshold behavior itself looks sound.
BenWestgate
added
area: cli
This branch has not been deployed
2 participants
Requested by Ben · project thread
Before:
ms32 secretprinted the master fingerprint, butms32 sharenever did, so the operator couldn't check it against the wallet record before writing a new card.After: on a terminal,
ms32 shareprintsMaster fingerprint: XXXXXXXXabove the derived share, whether every input is a share or one of them is the secretS.--plainand redirected output are unchanged.How: four lines in
_share_command. The fingerprint comes from the entered secret, or from recovering the secret from the entered shares. #57's typed-fingerprint gate forwalletandcreate --existingis untouched.Budget: Ben authorized raising the limit from < 5200 to < 5250 in the project thread ("raise the budget", 2026-10-01, and "You can raise the budget to 5250", 2026-10-02). The last commit makes that change in
tests/test_cli.py,docs/developer/api.mdandAGENTS.md; it is the same commit #104 carries. This branch counts 5200 on #57 at115f2c2. 926 tests pass; ruff and mypy are clean.Closes #86
🤖 Generated with Claude Code
https://claude.ai/code/session_015CuLXqAvAovfoVcUmmogwa
Generated by Claude Code