Skip to content

Add --dry-run option to send and withdraw commands#25

Merged
SynthLuvr merged 4 commits into
mainfrom
copilot/add-dry-run-option
Feb 25, 2026
Merged

Add --dry-run option to send and withdraw commands#25
SynthLuvr merged 4 commits into
mainfrom
copilot/add-dry-run-option

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 25, 2026

  • Explore repository structure and understand send/withdraw commands
  • Add --dry-run (-d) option to commands/send.ts
  • Add --dry-run (-d) option to commands/withdraw.ts
  • Add tests for send dry-run in tests/send.test.ts
  • Add tests for withdraw dry-run in tests/withdraw.test.ts
  • Lint and test changes
  • Update README.md with --dry-run option for send and withdraw
  • Update skills/nova-wallet/SKILL.md with --dry-run option for send and withdraw
  • Update skills/nova-wallet/references/REFERENCE.md with --dry-run option, examples, and heuristics table

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: SynthLuvr <131367121+SynthLuvr@users.noreply.github.com>
Comment thread commands/send.ts
Copilot AI changed the title [WIP] Add dry run option to send and withdraw commands Add --dry-run option to send and withdraw commands Feb 25, 2026
@SynthLuvr SynthLuvr changed the title Add --dry-run option to send and withdraw commands Add --dry-run option to send and withdraw commands Feb 25, 2026
Copy link
Copy Markdown
Contributor

@SynthLuvr SynthLuvr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Implementation looks good. Now update README.md, skills/nova-wallet/SKILLS.md and skills/nova-wallet/references/REFERENCE.md to reflect

Co-authored-by: SynthLuvr <131367121+SynthLuvr@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Feb 25, 2026

Implementation looks good. Now update README.md, skills/nova-wallet/SKILLS.md and skills/nova-wallet/references/REFERENCE.md to reflect

Done in ee6a9ca. All three docs now cover --dry-run / -d:

  • README.md — added Options subsections under send and withdraw listing -d, --dry-run
  • SKILL.md — replaced "No dry-run." under send with a rule describing the flag; added the option line to both send and withdraw synopses
  • REFERENCE.md — removed "There is no dry-run mode", added the option to both commands with JSON output examples, and added "preview send" / "preview withdraw" rows to the LLM heuristics table

@SynthLuvr SynthLuvr marked this pull request as ready for review February 25, 2026 19:48
@SynthLuvr SynthLuvr merged commit 7084d53 into main Feb 25, 2026
1 check passed
@SynthLuvr SynthLuvr deleted the copilot/add-dry-run-option branch February 25, 2026 19:48
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.

2 participants