Skip to content

fix(tooling): hydrate LFS snapshots during worktree setup - #231

Merged
kyleve merged 1 commit into
mainfrom
codex/hydrate-lfs-worktrees
Aug 10, 2026
Merged

fix(tooling): hydrate LFS snapshots during worktree setup#231
kyleve merged 1 commit into
mainfrom
codex/hydrate-lfs-worktrees

Conversation

@kyleve

@kyleve kyleve commented Aug 10, 2026

Copy link
Copy Markdown
Owner

Posted by an AI agent on kve's behalf.

Summary

  • hydrate every current Git LFS object during ./ide --bootstrap, the setup path Codex-managed macOS worktrees already run
  • preflight snapshot test scopes with one stable Git LFS inventory and fail before generation or building when references are still pointer text
  • document the deterministic setup and recovery behavior in the repository guidance and running-tests skill

Why

Codex worktrees can be created with LFS smudging skipped even when git-lfs is installed. Snapshot runs then compare against pointer files, producing misleading incomparable results and allowing downstream aggregate references to be missed during targeted review. Bootstrap is already the networked setup boundary, while the test runner should remain read-only and explain how to repair an interrupted or manually-created checkout.

Review focus

  • bootstrap intentionally fails when git lfs pull cannot hydrate the checkout; worktrees share the repository LFS cache, so repeated setup reuses downloaded objects
  • snapshot preflight checks every tracked __Snapshots__/*.png reference without performing network work and reports the first unresolved paths plus git lfs pull

Testing

  • bash -n ide test sync-agents
  • pre-hydration snapshot invocation: failed before generation/build and reported 323 pointer-backed references with the recovery command
  • git lfs pull
  • ./ide --bootstrap --no-open
  • ./test --snapshots --review --only 'FlyoverSnapshotTests/FlyoverSnapshotTests' (1 test passed; every capture matched byte-for-byte)
  • ./test --all (1,838 tests passed)
  • ./swiftformat --lint
  • ./snapshot-shards check
  • ./sync-agents

@kyleve
kyleve enabled auto-merge (squash) August 10, 2026 03:47
@kyleve
kyleve merged commit bbddcd9 into main Aug 10, 2026
11 of 12 checks passed
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