Skip to content

docs(hooks): add immutable escrow hook risk warnings#37

Merged
dev-jodee merged 1 commit intomainfrom
docs/hook-immutability-warnings
Apr 7, 2026
Merged

docs(hooks): add immutable escrow hook risk warnings#37
dev-jodee merged 1 commit intomainfrom
docs/hook-immutability-warnings

Conversation

@dev-jodee
Copy link
Copy Markdown
Collaborator

Summary

  • add a dedicated hook warning section to README covering immutable escrow risk
  • add warning language to Program Overview for SetHook and Hook extension behavior
  • update security considerations to call out hook liveness dependency and HookRejected
  • add UI warning copy to Set Hook and Set Immutable forms
  • add an in-code comment documenting fail-closed hook invocation semantics

Test Plan

  • pnpm install
  • just generate-clients
  • just build (needed in this environment to produce SBF artifacts)
  • just fmt (fails on existing repository-wide ESLint baseline: 86 @typescript-eslint/no-unsafe-* errors across web/demo files unrelated to this PR)

Breaking Changes

  • none

@dev-jodee dev-jodee requested a review from amilz April 7, 2026 14:59
@dev-jodee dev-jodee merged commit c5788d1 into main Apr 7, 2026
6 checks passed
@dev-jodee dev-jodee deleted the docs/hook-immutability-warnings branch April 7, 2026 15:00
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.

1 participant