Skip to content

feat(tx-manifest): add request parser foundation - #10

Open
lukachi wants to merge 1 commit into
masterfrom
feature/tx-manifest-parser
Open

feat(tx-manifest): add request parser foundation#10
lukachi wants to merge 1 commit into
masterfrom
feature/tx-manifest-parser

Conversation

@lukachi

@lukachi lukachi commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator
  • This PR suggests a bug fix and I've added the necessary tests.
  • This PR introduces a new feature and I've discussed the update in an Issue or with the team.
  • This PR is just a minor change like a typo fix.

Summary

This is PR 1 of the ordered merge train that replaces the oversized #5.

It establishes the source-faithful request/parser foundation for @humid/tx-manifest:

  • adds the package test and typecheck boundary;
  • validates the existing six-part processConfidentialTransaction request contract;
  • defaults broadcast to off and rejects caller-supplied fee fields;
  • reads the selected action's required contract sources and prompted parameters;
  • distinguishes wallet-funded inputs and newly created covenant outputs from covenant inputs that require state;
  • carries the canonical p2pk manifest fixture unchanged.

The implementation is reconstructed from the existing feature/conf-tx lineage rather than introducing a parallel parser API.

Stack

  • Base: master
  • Next PR will target: feature/tx-manifest-parser

Deliberate exclusions

This PR does not add chain reads, transaction planning or construction, smplx/WASM integration, cross-covenant references/classes, issuance or blinding policy, comprehensive semantic validation, rendering, signing, broadcast, or release activation. Those land in later train cars.

Verification

  • bun install --frozen-lockfile
  • bun test packages/tx-manifest — 24 passed, 0 failed
  • bun run typecheck:packages
  • bun run typecheck
  • bun run lint
  • bunx oxfmt packages/tx-manifest packages/tsconfig.json packages/bun-test-env.d.ts package.json .oxfmtrc.json --check
  • git diff master...HEAD --check

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