spectr(proposal): remove-vhs-echo-commands (#119)

Proposal for review: spectr/changes/remove-vhs-echo-commands/

Generated by: spectr pr proposal

Author: connerohnesorge

spectr/changes/remove-vhs-echo-commands/proposal.md

@@ -0,0 +1,27 @@
+# Remove VHS Echo Commands
+
+## Summary
+Remove all unnecessary `Type "echo ..."` commands from VHS tape files in `assets/vhs/`. These typed echo lines add visual clutter and extend GIF duration without providing meaningful value - the actual spectr commands are self-explanatory.
+
+## Motivation
+The current VHS tape files contain 16 typed echo commands across 5 files that:
+1. Add ~10+ seconds of typing animation to each GIF
+2. State the obvious (e.g., "=== Running validation ===" before running validation)
+3. Create visual noise that distracts from the actual spectr commands
+4. Make maintenance harder when updating demos
+
+The demos should focus on showcasing spectr's actual commands and output, not on watching echo statements being typed.
+
+## Scope
+- **In scope**: Remove all `Type "echo ..."` lines and their associated `Enter` and `Sleep` commands from all 5 VHS tape files
+- **Out of scope**: Comments (lines starting with `#`) which serve as documentation within the tape files
+
+## Files Affected
+- `assets/vhs/archive.tape` - 4 echo commands
+- `assets/vhs/init.tape` - 2 echo commands
+- `assets/vhs/list.tape` - 3 echo commands
+- `assets/vhs/partial-match.tape` - 4 echo commands
+- `assets/vhs/validate.tape` - 3 echo commands
+
+## Expected Outcome
+Cleaner, faster demo GIFs that focus on spectr functionality without redundant section headers being typed out.

spectr/changes/remove-vhs-echo-commands/specs/documentation/spec.md

@@ -0,0 +1,22 @@
+# Documentation Specification (Delta)
+
+## MODIFIED Requirements
+
+### Requirement: VHS Tape Output Clarity
+
+VHS tape files SHALL NOT contain typed echo statements that display section headers or commentary. Demos SHALL let the spectr commands and their output speak for themselves. Comments within the tape file (lines starting with `#`) SHOULD be used to document sections for maintainers, but these are not displayed in the recording.
+
+#### Scenario: No typed echo section headers
+- **WHEN** a VHS tape file is reviewed
+- **THEN** it SHALL contain no `Type "echo ..."` commands for section headers
+- **AND** visual context SHALL be provided through VHS comments (starting with `#`) which are not recorded
+
+#### Scenario: No useless echo statements
+- **WHEN** a VHS tape file is reviewed
+- **THEN** it SHALL contain no `Type "echo ''"` commands
+- **AND** visual spacing SHALL be achieved through `Sleep` commands only
+
+#### Scenario: Commands are self-documenting
+- **WHEN** a user views a demo GIF
+- **THEN** they SHALL see only the actual spectr commands being typed
+- **AND** they SHALL NOT see preparatory echo statements being typed out

spectr/changes/remove-vhs-echo-commands/tasks.md

@@ -0,0 +1,8 @@
+# Tasks
+
+- [x] Remove echo commands from archive.tape (4 occurrences)
+- [x] Remove echo commands from init.tape (2 occurrences)
+- [x] Remove echo commands from list.tape (3 occurrences)
+- [x] Remove echo commands from partial-match.tape (4 occurrences)
+- [x] Remove echo commands from validate.tape (3 occurrences)
+- [x] Regenerate all GIFs using vhs to verify changes work correctly