Skip to content

docs(demo): refresh demo materials for TUI feature drift (#1157–#1173)#1174

Open
GarthDB wants to merge 1 commit into
mainfrom
demo/tui-drift-refresh
Open

docs(demo): refresh demo materials for TUI feature drift (#1157–#1173)#1174
GarthDB wants to merge 1 commit into
mainfrom
demo/tui-drift-refresh

Conversation

@GarthDB

@GarthDB GarthDB commented Jun 17, 2026

Copy link
Copy Markdown
Member

Description

Refreshes all demo materials to match the current TUI after ~10 feature PRs (#1157#1173) landed since the last update (early June / #1107#1113), leaving the demo partly broken and out of date.

No TUI source changes/ fuzzy-find removal is treated as intentional (superseded by the :find wizard).

Related Issue

N/A — maintenance follow-up to #1157#1173.

Motivation and Context

The demo recording scripts (auto-demo.sh A/B) had three broken areas:

  1. A4 beat broken: /accentbg fuzzy-find no longer exists in the TUI (QueryView::fuzzy is never constructed, no / key handler). The Fuzzy marker sentinel would never anchor.
  2. B wizard sentinels stale: Screen titles changed from Wizard · N/4 · <Name> to Step N of 4 — <Name> (feat(tui): context-sensitive help + wizard step indicator #1169/feat(tui): wizard help overlay, Esc back-stack, did-you-mean, [SEL] badge, readline #1172). The N/4 regex fallbacks survived but comments/markers were wrong.
  3. B Esc behavior wrong: Esc now back-navigates one screen at a time (fix(tui): esc goes back one screen in wizard instead of cancelling #1160/feat(tui): wizard help overlay, Esc back-stack, did-you-mean, [SEL] badge, readline #1172) instead of canceling. A single Esc from Screen 4 (Confirm) only steps to Screen 3 — the old beats left the wizard open after "cancel".

Also refreshes documentation to document new features: :find wizard + faceted count badges (#1173), :validate grouping (#1165), g/G jump (#1164), y/Y yank (#1170), h/l horizontal scroll (#1166), Esc back-navigation (#1160/#1172), fuzzy palette autocomplete (#1168).

How Has This Been Tested?

  • Beat scripts (A.beats.sh, B.beats.sh) reviewed against current TUI source (sdk/tui/src/find.rs, model/views.rs, wizard.rs) for sentinel accuracy
  • remark linter passed on all four .md files (pre-commit hook)
  • Re-recording (moon run demo:auto-record) and CI verify (moon run demo:auto-verify) should be run after merge on a machine with rmux + asciinema + CLI pre-built

Screenshots (if appropriate):

N/A — documentation/script changes only.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@changeset-bot

changeset-bot Bot commented Jun 17, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: c2e1df1

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions

github-actions Bot commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Run report for c2e1df1a

Total time: 3m 25s | Comparison time: 3m 14s | Estimated loss: 10.8s (5.3% slower)

Action Time Status Info
⬛️ SetupProto(0.51.4) 0.1ms Skipped
🟩 SyncWorkspace 7.9ms Passed
🟩 SyncProject(sdk) 0.5ms Passed
🟩 SyncProject(design-data) 0.6ms Passed
🟩 SyncProject(sdk-wasm) 0.6ms Passed
🟩 SyncProject(design-data-glue) 0.3ms Passed
🟩 SyncProject(token-naming-audit) 0.4ms Passed
🟩 SyncProject(design-data-agent-mcp) 0.3ms Passed
⬛️ SetupToolchain(rust:1.88.0) 1.2s Skipped
🟩 InstallProjectDeps(rust:1.88.0, sdk-wasm) 202.1ms Passed
⬛️ SetupToolchain(node:20.17.0) 1.9s Skipped
🟩 InstallProjectDeps(node:20.17.0, sdk) 7.1s Passed
🟩 RunTask(sdk:codegen-check) 256.1ms Passed
🟩 InstallWorkspaceDeps(node:20.17.0) 7.7s Passed
🟩 RunTask(sdk-wasm:cache-build) 1m 6s Passed
🟩 RunTask(sdk-wasm:build) 2m 4s Passed SLOW
🟩 RunTask(token-naming-audit:test) 3.6s Passed
🟩 RunTask(token-naming-audit:ci) 17.4ms Passed
🟩 RunTask(design-data-glue:test) 3.7s Passed
🟩 RunTask(design-data-glue:ci) 17.3ms Passed
And 3 more...
Expanded report
Action Time Status Info
🟩 RunTask(design-data-agent-mcp:test) 4s Passed
🟩 RunTask(sdk-wasm:test) 4.8s Passed
🟩 RunTask(sdk-wasm:ci) 7ms Passed
Touched files
sdk/tui/DEMO.md
sdk/tui/README.md
tools/demo/auto/beats/A.beats.sh
tools/demo/auto/beats/B.beats.sh
tools/demo/presentation/RECORDING.md
tools/demo/presentation/slides.md

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.

1 participant