Skip to content

fix: ignore nested snapshot directories in size cap#287

Open
C1-BA-B1-F3 wants to merge 1 commit into
OpenBMB:mainfrom
C1-BA-B1-F3:fix/issue-262
Open

fix: ignore nested snapshot directories in size cap#287
C1-BA-B1-F3 wants to merge 1 commit into
OpenBMB:mainfrom
C1-BA-B1-F3:fix/issue-262

Conversation

@C1-BA-B1-F3

Copy link
Copy Markdown

Summary

  • Apply snapshot-copy ignore rules at any path depth for size estimation, copy filtering, and post-copy pruning.
  • Replace the raw du size estimate with a directory walk that skips ignored directories.
  • Add regression coverage for nested node_modules and dist directories.

Closes #262

Tests

  • npx tsc -p tsconfig.json --noEmit
  • node --import tsx --test tests/always-on/workspace/SnapshotCopyProvider.test.ts
  • npm test

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

2 participants