Skip to content

fix(cli): preserve snapshot times and clarify occlusion opt-out#2434

Open
miguel-heygen wants to merge 2 commits into
mainfrom
fix/cli-feedback-remediation-1784047790
Open

fix(cli): preserve snapshot times and clarify occlusion opt-out#2434
miguel-heygen wants to merge 2 commits into
mainfrom
fix/cli-feedback-remediation-1784047790

Conversation

@miguel-heygen

Copy link
Copy Markdown
Collaborator

Summary

Repeated snapshot --at flags previously discarded every timestamp except the last one. Repeated flags, comma-separated lists, and mixed forms now combine in order so every requested frame is captured.

text_occluded also suggested the intentional-occlusion opt-out without identifying its target, while the skill reference implied it belonged on the covering element. Both now state that data-layout-allow-occlusion belongs on the covered text or one of its ancestors, never the covering element.

Test plan

  • bun run --cwd packages/cli test — 1,750 passed, 2 skipped
  • bun run --cwd packages/cli typecheck
  • bun run lint
  • bun run format:check
  • bun run build
  • Built CLI: snapshot ... --at 0.2 --at 0.8 --no-end captured both requested PNGs

Compound Engineering
Codex

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