Skip to content

fix(always-on): apply snapshot ignores before size checks#265

Open
snvtac wants to merge 1 commit into
OpenBMB:mainfrom
snvtac:snvtac/262-snapshot-copy-nested-ignores
Open

fix(always-on): apply snapshot ignores before size checks#265
snvtac wants to merge 1 commit into
OpenBMB:mainfrom
snvtac:snvtac/262-snapshot-copy-nested-ignores

Conversation

@snvtac

@snvtac snvtac commented Jun 20, 2026

Copy link
Copy Markdown

Summary

  • Apply snapshot ignore rules to every path segment during size estimation and filtered copies.
  • Recursively prune ignored files and directories after clonefile/reflink snapshots.
  • Fall back to filtered fs.cp if clonefile/reflink pruning cannot complete.
  • Add a regression test for nested node_modules, dist, and .git paths that should not count toward maxBytes or remain in the snapshot.

Fixes #262

Testing

  • npm test
  • npm exec tsc -- -p tsconfig.json --noEmit
  • node --import tsx --test --test-force-exit --test-timeout 60000 tests/always-on/workspace/SnapshotCopyProvider.test.ts

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.

bug(always-on): snapshot size cap does not apply nested ignored directories

1 participant