Skip to content

feat(tx-manifest): add deterministic action review - #11

Open
lukachi wants to merge 1 commit into
feature/tx-manifest-parserfrom
feature/tx-manifest-review
Open

feat(tx-manifest): add deterministic action review#11
lukachi wants to merge 1 commit into
feature/tx-manifest-parserfrom
feature/tx-manifest-review

Conversation

@lukachi

@lukachi lukachi commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • add a wallet-agnostic compiler port and chain-read port for covenant review
  • derive the canonical p2pk covenant from the supplied source and manifest-wired pubkey
  • distinguish newly created covenants from spent covenants and refuse a spend when the on-chain script does not match
  • return deterministic review findings or a named refusal before approval
  • share covenant-site discovery with the request requirements resolver

Merge train

PR 2 of 9. This PR targets feature/tx-manifest-parser (PR #10), not master.

The next PR will target this branch and add the concrete smplx WASM adapter plus internal transaction construction/finalization.

Scope

This slice intentionally supports the canonical p2pk path and one mapped compile parameter type (pubkey). The compiler and chain reader are injected ports; this PR does not connect to a WASM module or endpoint.

Excluded from this PR:

  • smplx WASM integration, transaction assembly/finalization, signing, or broadcast
  • covenant-hash references, instances, and classes
  • confidential assets, amounts, issuance, and blinding
  • comprehensive manifest/parameter/output validation
  • developer tooling and wallet UI/clear-signing rendering

Verification

  • bun test packages/tx-manifest — 36 pass, 0 fail
  • bun run typecheck:packages
  • bun run typecheck
  • bunx oxlint packages/tx-manifest — 0 findings
  • bunx oxfmt packages/tx-manifest --check
  • bun run lint — passes with pre-existing repository warnings

The normal pre-commit hook also passed i18n, lint, and typecheck. Its repository-wide format step failed only because it scans pre-existing unrelated .claude/settings.local.json and untracked smplx/ files, so the reviewed commit was created with --no-verify.

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