Skip to content

implement-transaction-building/transfers - #358

Merged
Cjay-Cyber-2 merged 4 commits into
ASTROIDX556:mainfrom
Hamda-gbade:feat/implement-transaction-building
Oct 4, 2026
Merged

Cjay-Cyber-2 merged 4 commits into
ASTROIDX556:mainfrom
Hamda-gbade:feat/implement-transaction-building

Conversation

@Hamda-gbade

@Hamda-gbade Hamda-gbade commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Type of change

  • Bug fix
  • New feature
  • Refactor
  • Docs

Related issue

Closes #250

Checklist

  • npm run build passes
  • npm run typecheck passes
  • npm run lint passes
  • No secrets in tracked files

@Cjay-Cyber-2

Copy link
Copy Markdown
Contributor

The �uild check fails in packages/transaction/src/builder.ts around lines 439–447: prose was parsed as executable TypeScript, producing unknown names such as XDR, When, and �mpty. Move that text into a comment/JSDoc or replace it with the intended XDR validation expression, then rerun the package build.

@Cjay-Cyber-2

Copy link
Copy Markdown
Contributor

CI Checks Failed - Action Needed. The build check fails in packages/transaction/src/builder.ts because explanatory prose is being parsed as TypeScript, producing unknown-name errors such as XDR. Move that text into a comment or JSDoc, then rerun build, typecheck, and lint.

@Cjay-Cyber-2
Cjay-Cyber-2 merged commit 9f497a8 into ASTROIDX556:main Oct 4, 2026
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.

Implement transaction building helper for multi-asset agent transfers

3 participants