Skip to content

test(scripts): cover session backup tmp filtering#206

Merged
skulidropek merged 1 commit intoProverCoderAI:mainfrom
skulidropek:issue-198-session-backup-tests
Mar 26, 2026
Merged

test(scripts): cover session backup tmp filtering#206
skulidropek merged 1 commit intoProverCoderAI:mainfrom
skulidropek:issue-198-session-backup-tests

Conversation

@skulidropek
Copy link
Contributor

Summary

  • make scripts/session-backup-gist.js directly testable via a require.main === module guard and explicit helper exports
  • add a typed declaration for those exported helpers so packages/app tests can import the script cleanly
  • add direct regression coverage proving session backup skips tmp directories while preserving persistent session files

Context

  • #205 already merged the broader runtime fix that skips tmp directories in recursive scans
  • this follow-up keeps the useful testability pieces from closed #204, adapted to the broader semantics now on main

Validation

  • node --check scripts/session-backup-gist.js
  • pnpm --filter ./packages/app exec vitest run tests/hooks/session-backup-gist.test.ts tests/hooks/pre-commit-ai-dirs.test.ts
  • pnpm --filter ./packages/app exec eslint tests/hooks/session-backup-gist.test.ts --ext .ts
  • pnpm --filter ./packages/app lint:effect
  • pnpm --filter ./packages/app typecheck

@skulidropek skulidropek merged commit 546c727 into ProverCoderAI:main Mar 26, 2026
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