Skip to content

test: add compact option to plan snapshot tests#174

Merged
branchseer merged 2 commits intomainfrom
02-26-test_compact_plan_snapshots
Feb 27, 2026
Merged

test: add compact option to plan snapshot tests#174
branchseer merged 2 commits intomainfrom
02-26-test_compact_plan_snapshots

Conversation

@branchseer
Copy link
Copy Markdown
Member

@branchseer branchseer commented Feb 26, 2026

Summary

  • Add compact: bool field to [[plan]] in plan snapshot test config
  • When enabled, snapshots only show the graph topology ("relative_path#task_name" → [neighbors]), omitting execution details like spawn commands, env vars, and cache metadata
  • Nodes with nested Expanded items include an items array of sub-graphs
  • Enable compact mode on 3 existing test cases that only verify graph structure: nested-tasks, pnpm-workspace-packages-optional, vpr-shorthand

Test plan

  • cargo test -p vite_task_plan --test plan_snapshots passes
  • cargo clippy -p vite_task_plan --test plan_snapshots -- -D warnings passes
  • Compact snapshots show correct graph structure
  • Non-compact snapshots unchanged

🤖 Generated with Claude Code

branchseer and others added 2 commits February 26, 2026 21:43
When `compact = true` in `[[plan]]`, the snapshot only contains the
top-level graph topology as `"relative_path#task_name" -> [neighbors]`,
omitting execution details like spawn commands, env vars, and cache
metadata. Nodes with nested `Expanded` items include an `items` array.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Enable compact = true for nested-tasks, pnpm-workspace-packages-optional,
and vpr-shorthand fixtures since they only verify graph structure, not
execution details like cache metadata or env vars.

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.

@branchseer branchseer changed the title feat: add compact option to plan snapshot tests feat: add compact option to plan snapshot tests Feb 26, 2026
@branchseer branchseer changed the title feat: add compact option to plan snapshot tests test: add compact option to plan snapshot tests Feb 26, 2026
@branchseer branchseer requested a review from fengmk2 February 26, 2026 14:07
@branchseer branchseer merged commit 2c07f2a into main Feb 27, 2026
7 checks passed
@branchseer branchseer deleted the 02-26-test_compact_plan_snapshots branch February 27, 2026 08:30
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.

2 participants