Skip to content

refactor(snapshot): remove element detection#23

Merged
msmps merged 1 commit into
mainfrom
refactor/remove-element-detection
Jul 11, 2026
Merged

refactor(snapshot): remove element detection#23
msmps merged 1 commit into
mainfrom
refactor/remove-element-detection

Conversation

@msmps

@msmps msmps commented Jul 11, 2026

Copy link
Copy Markdown
Owner

Summary

  • delete heuristic UI element classification, segmentation, style-grid adapters, tests, and direct dependency
  • reduce snapshots to terminal size, cursor state, optional text, and optional content hash
  • remove the unused snapshot ID counter from live and finalized capture paths
  • update CLI help, documentation, npm guidance, and the bundled skill around text-first navigation
  • introduce protocol v3 for the breaking snapshot request/response contract while retaining existing minimum versions for output and status

Breaking changes

Full JSON snapshots no longer contain elements or snapshot_id. Snapshot requests now require protocol v3, so mismatched client/daemon binaries receive the existing actionable upgrade guidance instead of attempting to decode incompatible evidence.

Verification

  • cargo fmt --all
  • cargo test --workspace (188 passed)
  • cargo clippy --all --benches --tests --examples --all-features
  • ./scripts/version-sync.sh --check
  • bash -n skills/pilotty/templates/dialog-interaction.sh skills/pilotty/templates/multi-session.sh skills/pilotty/templates/vim-workflow.sh
  • git diff --check

@msmps
msmps merged commit cb77c45 into main Jul 11, 2026
6 checks passed
@msmps
msmps deleted the refactor/remove-element-detection branch July 11, 2026 18:49
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