The regression that motivated #76 — partial deletion inside a folder that still has files — is fixed by the entrypoint fetch, but the code path itself has no test coverage in the test suite.
This is awkward to unit-test since it depends on git ref state. The right home is an integration fixture in the eval harness (#65). The fixture should:
- Set up a repo with a docs subfolder containing multiple files in a folder
- Delete some (not all) files from the folder on the base branch
- Run
[review-docs] against a stale origin/main
- Verify the action correctly identifies remaining docs need updating
Follow-up from #76.
The regression that motivated #76 — partial deletion inside a folder that still has files — is fixed by the entrypoint fetch, but the code path itself has no test coverage in the test suite.
This is awkward to unit-test since it depends on git ref state. The right home is an integration fixture in the eval harness (#65). The fixture should:
[review-docs]against a staleorigin/mainFollow-up from #76.