Skip to content

[107] ISSUE-080: Add Offline Transaction Preparation Workflow Example - #141

Merged
Dantama022 merged 3 commits into
Dantama022:mainfrom
esthertitilayo-dev:fix/107-add-offline-transaction-preparation-workflow-example
Jul 29, 2026
Merged

[107] ISSUE-080: Add Offline Transaction Preparation Workflow Example#141
Dantama022 merged 3 commits into
Dantama022:mainfrom
esthertitilayo-dev:fix/107-add-offline-transaction-preparation-workflow-example

Conversation

@esthertitilayo-dev

Copy link
Copy Markdown
Contributor

Summary

  • Adds src/examples/80-offline-transaction-workflow.ts: build an unsigned tx, export unsigned XDR, simulate an air-gapped transfer to an offline signer, sign offline, and gracefully handle corrupted/invalid XDR before reconstructing and submitting the signed tx.
  • Adds tests/offline-transaction-workflow.test.ts covering XDR reconstruction (valid, truncated, garbage, and fee-bump inputs), offline signing, describeTransaction, isLikelyCorruptedXDR, and runner/README registration.
  • Registers the example in src/runner/catalog.ts (with an optional amount param) and documents it in README.md (roadmap list + Direct Run section).
  • Also fixes a pre-existing JSON syntax error in src/validation/validation.config.json (missing },\n{ between the 67-soroban-contract-events and 50-asset-issuer-discovery exclusion entries) — needed for the test suite to parse the config and pass, and adds an exclusion entry for the new example.

Test plan

  • npx eslint and npx tsc --noEmit on changed files (pre-existing unrelated errors in 72-contract-interface-inspection.ts / 73-complex-data-types.ts / 75-deploy-precompiled-wasm.ts left untouched)
  • npx jest tests/offline-transaction-workflow.test.ts tests/ledger-effects.test.ts tests/runner.test.ts
  • npm test — full suite green (22 suites, 257 tests passing)

Closes #107

- Add 80-offline-transaction-workflow.ts: build unsigned tx, export XDR,
  simulate an offline signing hop, sign, and gracefully handle
  corrupted/invalid XDR before reconstructing and submitting.
- Add tests/offline-transaction-workflow.test.ts covering reconstruction,
  offline signing, corrupted XDR handling, and runner/README registration.
- Register the example in the runner catalog and document it in README.md.
- Fix a pre-existing JSON syntax error in validation.config.json (missing
  closing brace between the 67 and 50 exclusion entries) that was breaking
  the test suite, and add an exclusion entry for the new example.
@drips-wave

drips-wave Bot commented Jul 29, 2026

Copy link
Copy Markdown

@esthertitilayo-dev Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Dantama022
Dantama022 merged commit 2c98a88 into Dantama022:main Jul 29, 2026
0 of 3 checks passed
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.

ISSUE-080: Add Offline Transaction Preparation Workflow Example

2 participants