Skip to content

feat: add cwd and args info to snapshot headers#175

Merged
Brooooooklyn merged 1 commit intomainfrom
02-27-feat-snapshot-info-headers
Feb 27, 2026
Merged

feat: add cwd and args info to snapshot headers#175
Brooooooklyn merged 1 commit intomainfrom
02-27-feat-snapshot-info-headers

Conversation

@branchseer
Copy link
Copy Markdown
Member

@branchseer branchseer commented Feb 27, 2026

Why

Snapshot files currently only show source, expression, and input_file in the YAML header. To understand what a snapshot is testing, you have to cross-reference snapshots.toml. Adding cwd and args to the header makes each snapshot self-describing, making human review easier.

Summary

  • Add info field to plan snapshot headers with args (array) and optional cwd
  • Add info field to E2E snapshot headers with cwd when non-default
  • Makes snapshots self-describing without cross-referencing snapshots.toml

Test plan

  • cargo test -p vite_task_plan --test plan_snapshots passes
  • cargo test -p vite_task_bin --test e2e_snapshots passes
  • Verified headers in generated .snap files contain correct info

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Copy Markdown
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@Brooooooklyn Brooooooklyn merged commit 995f80c into main Feb 27, 2026
7 checks passed
@Brooooooklyn Brooooooklyn deleted the 02-27-feat-snapshot-info-headers branch February 27, 2026 09:03
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.

3 participants