docs: add CHANGELOG entries for #43/#44 and sync demo /help snapshot - #45
Conversation
|
Thanks for the follow-up! Most of this already landed on 1. The CHANGELOG half is redundant — please drop it.
A rebase onto 2. The 3. Stale premises in the description (worth updating).
|
Per maintainer feedback on shauryagangrade#45: keep only the make_demo.py change. The snapshot now matches _print_help (adds /ollama, /pull, /setup, /diff, and the 'Any other input' line); CHANGELOG entries were already merged to main, and the stale docstring is updated to note the vhs migration.
ef131c0 to
d19b42b
Compare
|
Thanks for the detailed review — all three points addressed:
Branch now merges cleanly against |
Problem
Follow-up to the review comments on the two GCode PRs merged today:
for
demo/make_demo.pyto stop hardcoding the old flat/helpsnapshot andinstead mirror the categorized
/helpoutput.Change
demo/make_demo.py: the/helpdemo block now mirrors the categorizedoutput from
_print_helpingcode/cli.py, including/ollama,/pull,/setup, and/diff, which were missing from the flat snapshot, plus the"Any other input is sent to the agent." line.
This branch intentionally touches only
demo/make_demo.py— the CHANGELOGentries for #43/#44 already landed on
mainat merge time.Why this approach
Keeps the recorded demo in sync with the real
/helpoutput. Note the demopipeline has since migrated to vhs (
vhs demo/demo.tape→docs/demo.gif,documented in the README's "Demo GIF" section);
make_demo.pyis a legacyself-contained renderer kept because it reproduces the exact Rich markup of
_print_help, and its docstring now says so.Testing
The demo
/helpmarkup was rendered through Rich to confirm it parses andproduces the categorized layout (verified by the maintainer).
Documentation and release impact
mainfrom fix(tools): make execute_bash non-interactive-safe #43/feat(cli): categorize /help into General/Model/Session/Git groups #44)