[ENHANCEMENT] Skip binary/image/asset files and deprioritize tests in Overture PRojection

[John McCall (lowlydba)]

Closes #35

Summary

Improves token budget utilization in Overture PRojection so meaningful source files aren't crowded out by assets, test files, and data that don't benefit from code review.

Changes

Ignore outright (no reviewable content)

Extended the default ignore-files list to skip:

• Images/SVGs: *.png, *.jpg, *.jpeg, *.gif, *.ico, *.webp, *.bmp, *.tiff, *.svg
• Fonts: *.woff, *.woff2, *.ttf, *.eot, *.otf
• Archives/binaries: *.zip, *.tar, *.gz, *.tgz, *.jar, *.war, *.class, *.pyc, *.pyo
• Media: *.mp4, *.mp3, *.wav, *.pdf
• Minified: *.min.js, *.min.css
• Geospatial data: *.geojson, *.parquet, *.geoparquet, *.pmtiles, *.mbtiles, *.csv

Deprioritized (reviewed when budget allows, cut first when tight)

• Test files: __tests__/, *.test.*, *.spec.*, test_*, *_test.*
• Fixture/data dirs: fixtures/, testdata/, test-fixtures/, __fixtures__/

Config files (package.json, tools.json, tsconfig.json, etc.) are unaffected.

Tests: signal fix

When test files are deprioritized out of the diff budget, the model was incorrectly seeing Tests: ❌ none in diff. Now checks budgetSkippedFiles too and shows one of:

• Tests: ✅ — tests reviewed in diff
• Tests: ✅ (not reviewed — budget-cut) — tests present in PR but cut for context
• Tests: ❌ none in diff — genuinely no tests in the PR

workflow_dispatch UX

• model, selection_model, model_provider are now type: choice dropdowns with annotated options
• Claude/Anthropic options are clearly labeled as requiring model_provider: anthropic

GitHub Models sunset notice

GitHub announced GitHub Models retirement on 2026-06-16 (existing customers unaffected for now, timeline TBD). Noted in README.md and defaults.js with the migration path (model-provider: anthropic + ANTHROPIC_API_KEY).

Tests

41 tests passing across diff.js and prompt.js (6 new tests added).

[overture-projection]

Overture PRojection Review

This PR enhances the Overture PRojection GitHub Action to skip binary/image/asset files and deprioritize test/fixture files when selecting files for code review, improving token budget utilization and review relevance. It also updates the test signal logic and improves workflow UX for model selection.

✅ Checks Passed

• The ignore list now comprehensively skips non-reviewable assets (images, fonts, binaries, media, minified files, geospatial data, CSVs).
• Test and fixture files are deprioritized, ensuring source files are reviewed first when context is tight.
• The Tests: signal now accurately reflects whether tests were present but skipped due to budget, reviewed, or absent.
• Workflow UX improvements (dropdowns, clear provider/model mapping, sunset notice for GitHub Models).
• 41 tests passing, with 6 new tests covering the new logic.

🚩 Flags

• None observed. The changes are well-targeted, with clear separation between outright ignored files and deprioritized files. No logic errors, security issues, or process violations are visible.

❓ Open Questions

• None. The intent and implementation are clear and well-documented.

---

No further action required. This is a solid enhancement that will improve review quality and efficiency.

[Copilot]

Pull request overview

Extends the default ignore-files patterns for the .github/actions/overture-projection composite action so that common binary / asset file types (images, fonts, archives, media, minified assets) are excluded from the diff content sent to the review model, preserving diff/token budget for actual source files.

Changes:

• Expanded the default ignore-files block in action.yml to include common image/font/archive/media/minified file patterns.
• Keeps the existing override behavior intact (callers can set ignore-files: "" to disable exclusions).

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Francisco Jiménez (jjcfrancisco)]

Lgtm. I don't know if AI finds unit tests useful, I do as a human!