Skip to content

ci: print decoded transaction fixture summaries - #87

Open
tokenjunkielabs wants to merge 3 commits into
Gryd-lock:mainfrom
tokenjunkielabs:main
Open

tokenjunkielabs wants to merge 3 commits into
Gryd-lock:mainfrom
tokenjunkielabs:main

Conversation

@tokenjunkielabs

Copy link
Copy Markdown

Closes #27

Adds scripts/describe-transactions.mjs to discover and decode every transactions/*.xdr fixture using the network passphrase recorded in transactions/index.json.

The summary prints, per fixture, envelope type, source account, fee, sequence, memo, time bounds, and concise operation-specific fields. Fee-bump envelopes report the outer fee source/fee while decoding the inner transaction for source, sequence, memo, bounds, and operations. Decode failures are collected across all fixtures and make the command fail after reporting every bad file.

The repository currently has no committed npm dependencies and its existing PR CI invokes npm run validate. To preserve that contract, the decoder bootstraps a pinned @stellar/stellar-sdk@17.1.0 only when the SDK is absent, without saving a dependency or changing the lockfile. npm run validate now includes the decoder, so the human-readable transaction summary appears in the existing CI validation log for every PR. A standalone npm run describe:transactions entry is also provided.

This keeps new .xdr fixtures automatically covered by directory discovery with no per-fixture script changes.

@tokenjunkielabs

Copy link
Copy Markdown
Author

Maintainer review requested for claimant-owned bounty carrier fixing #27. This PR is OPEN and GitHub currently reports it mergeable at exact head a0033fbae89be4b9f8b109eb19a832d74879dfa6. Claimant-side payout setup is complete. Please merge if the implementation meets #27, or leave concrete change requests here so this carrier can be updated directly; on acceptance, please complete the bounty award/payment path tied to this issue.

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.

Add a CI step that decodes every transactions/*.xdr file and prints a human-readable summary

1 participant