Skip to content

feat: installation guide, command comparison, defaults preview, summa… - #222

Merged
0xsamuel1 merged 2 commits into
soroban-testkit:mainfrom
Darkvader-ship-it:feat/install-comparison-defaults-summary
Oct 19, 2026
Merged

0xsamuel1 merged 2 commits into
soroban-testkit:mainfrom
Darkvader-ship-it:feat/install-comparison-defaults-summary

Conversation

@Darkvader-ship-it

Copy link
Copy Markdown
Contributor

-closes #155
closes #151
closes #150
closes #156

Implements upstream issues #150, #151, #155, and #156 as one coherent enhancement of the CLI workbench.

Also:

  • add @testing-library/user-event and explicit @testing-library/dom; the committed lockfile had also gone stale (npm ci failed on main because @playwright/test was missing), so package-lock.json is refreshed.
  • focused unit tests for the new helpers plus component tests covering copy flows, keyboard activation, diff markers, and live summaries.
  • README features list updated; visual snapshots regenerated.

…ry card

Implements upstream issues soroban-testkit#150, soroban-testkit#151, soroban-testkit#155, and soroban-testkit#156 as one coherent
enhancement of the CLI workbench.

- soroban-testkit#155 First-time installation: new section with six ordered steps (WASM
  target, CLI, coverage wrapper, dev-dependency, verification, first
  command), each with a keyboard-accessible copy button; Install nav
  link; README installation section.
- soroban-testkit#150 Effective defaults preview: before copying, the workbench lists
  every field that falls back to a default (batch_payout, recipients,
  ./src, ...), shows the value the command will use, and flags it as
  "Default applied" or "From your input". Fallback values now live in a
  shared fallbackValues constant used by buildCommand/formatCommandLine.
- soroban-testkit#156 Command summary card: live recap of the active workflow's
  effective settings, marking values that come from defaults.
- soroban-testkit#151 Comparison view: aria-expanded/aria-controls disclosure opens a
  second configuration editor (Configuration B), a table that marks each
  changed setting ("Changed"/"Matches"), both generated commands, focus
  management, and a "Match B to A" sync button.

Also:
- add @testing-library/user-event and explicit @testing-library/dom; the
  committed lockfile had also gone stale (npm ci failed on main because
  @playwright/test was missing), so package-lock.json is refreshed.
- focused unit tests for the new helpers plus component tests covering
  copy flows, keyboard activation, diff markers, and live summaries.
- README features list updated; visual snapshots regenerated.
@drips-wave

drips-wave Bot commented Oct 18, 2026

Copy link
Copy Markdown

@Darkvader-ship-it 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

@0xsamuel1
0xsamuel1 merged commit 21b53d3 into soroban-testkit:main Oct 19, 2026
0 of 2 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

2 participants