Skip to content

fix: remove .ai directory and harden discovery, runner temp file cleanup, and report line endings - #49

Merged
Mresyzz merged 1 commit into
mainfrom
fix/cleanup-ai-and-harden-edge-cases
Oct 11, 2026
Merged

Mresyzz merged 1 commit into
mainfrom
fix/cleanup-ai-and-harden-edge-cases

Conversation

@Mresyzz

@Mresyzz Mresyzz commented Oct 11, 2026

Copy link
Copy Markdown
Owner

Summary

  1. Completely remove .ai/ directory from repository and ignore .ai/ in .gitignore.
  2. Discovery readability probe: is_shell_script now probes read access first, preventing unreadable files from being returned as candidates.
  3. Temp file leak prevention: wrapped run_on_distro post-preparation code in try-finally to guarantee own_temp_file cleanup on any error.
  4. Matrix preflight check: run_matrix validates script existence and rejects directories before Docker daemon connection and thread pool creation.
  5. Consistent POSIX LF: publish_output enforces newline='\n' across platforms.

Verification

  • Added 4 targeted tests in tests/test_audit_coverage.py.
  • Full pytest suite passes: 205 tests (193 passed, 12 skipped, 0 failures).

@Mresyzz
Mresyzz merged commit fe6152f into main Oct 11, 2026
@Mresyzz
Mresyzz deleted the fix/cleanup-ai-and-harden-edge-cases branch October 11, 2026 10:52
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