fix: track sprint gate corpus fixture - #743
Conversation
Co-Authored-By: brainlayerCodex-a6df3baf running gpt-5.6-sol <noreply@anthropic.com>
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_8d742154-cf85-4aad-9e1d-cfd56af6ee86) |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Team Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@codex review — brainlayerCodex-a6df3baf (worker) · codex/gpt-5.6-sol |
|
|
Overall Grade |
Security Reliability Complexity Hygiene |
Code Review Summary
| Analyzer | Status | Updated (UTC) | Details |
|---|---|---|---|
| Python | Sep 1, 2026 2:30p.m. | Review ↗ | |
| Swift | Sep 1, 2026 2:30p.m. | Review ↗ | |
| JavaScript | Sep 1, 2026 2:30p.m. | Review ↗ | |
| Shell | Sep 1, 2026 2:30p.m. | Review ↗ | |
| Secrets | Sep 1, 2026 2:30p.m. | Review ↗ |
Important
AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.
|
Codex Review: Didn't find any major issues. More of your lovely PRs please. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Summary
docs.local/to the tracked test-fixture path and update both path constantsdocs.localCI guard (1tracked file before,0after)The corpus is a 100% Git rename. Its SHA-256 remains
aa79c2dc8bdf8fc3b6eab65f503aa8fab06e0ffe296cb8f17bf291fbb70e2c04; gate logic and corpus content are unchanged.Previously, corpus availability depended on a force-add that conflicted with both
.gitignoreand the CI guard. Removing the trackeddocs.localfile to satisfy the guard would also remove the gate's required input. Moving it to an uncontested tracked path removes that latent fragility.Test plan
pytest tests/test_sprint_gate.py -q— 11 passeddocs.local/tests/fixtures/sprint_gate/corpus.jsonReview
— brainlayerCodex-a6df3baf (worker) · codex/gpt-5.6-sol
Note
Low Risk
Path-only change with identical corpus content; no gate logic or threshold changes.
Overview
Relocates the frozen sprint-gate
corpus.jsonfromdocs.local/plans/...totests/fixtures/sprint_gate/corpus.json(content unchanged) and updates theCORPUSconstant inscripts/sprint_gate.pyandtests/test_sprint_gate.pyso the gate and tests read the same tracked fixture.This removes reliance on a force-added file under
docs.local, which conflicted with the tracked-docs.localCI guard and could break clean clones or M1 runs when the guard dropped that path. Gate behavior and corpus thresholds/queries are not modified—only where the JSON is loaded from.Reviewed by Cursor Bugbot for commit 0c17d73. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Move sprint gate corpus fixture to
tests/fixtures/sprint_gate/corpus.jsonRelocates the corpus data file out of
docs.local/plans/...intotests/fixtures/sprint_gate/and updates theCORPUSconstant in both sprint_gate.py and test_sprint_gate.py to match.Macroscope summarized 0c17d73.