Uh oh!
There was an error while loading. Please reload this page.
chore(contracts): correct false TESTING.md → declared absence (no test command) - #35
Conversation
…eclared absence in .contracts-absent.json
Your free trial PR review limit of 300 PRs has been reached. Please upgrade your plan to continue using CodeAnt AI. |
ⓘ Qodo reviews are paused because your workspace is out of credits. Ask your workspace admin to add credits to resume reviews. Manage billing |
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_0978702c-ec37-4e64-8dd3-a5d8cf923f33) |
Reviewer's guide (collapsed on small PRs)Reviewer's GuideCorrects the contract metadata to reflect that this repository has no inferable test command: the invalid TESTING.md is removed and absence is explicitly recorded for fleet census and future contract regeneration. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Warning Review limit reachedNext included review available in 17 minutes. View limit detailsLimit details: You’ve used the included review currently available. Your 94 included PR review attempts over the past 7 days set your current allowance at 1 review per hour. Your organization has reached its usage spending cap. Adjust your spending cap in the billing tab. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
Comment |
Note Automatic reviews are paused because your team has used its included automatic processing for this billing period (headroom scales with your seat count). You can still comment "Gitar review" to run one anytime, and automatic reviews resume on their own by September 1. Add seats for more headroom. Code Review✅ ApprovedReplaces the misleading OptionsDisplay: compact → Showing less information. Comment with these commands to change the behavior for this request:
Was this helpful? React with 👍 / 👎 | Gitar |
ApprovabilityVerdict: Would Approve Macroscope's review found this PR approvable — This is a two-file contract-metadata cleanup that removes a misleading test instruction and records its intentional absence for tooling. It does not modify Python code, package behavior, customer-facing APIs, or production workflows. Not approved because:
Review your spending limits in Billing settings. You can add or adjust custom eligibility rules. Learn more. |
Uh oh!
There was an error while loading. Please reload this page.
chore(contracts): correct a false TESTING.md → declared absence
What was wrong
The contract-files rollout (E5-FLEET, 2026-08-30) landed a
TESTING.mdhere whose fencedtest-contractblock declaredentry: npm testand a suite runningnpm test. This repository has no npm test script (re-verified live at clone time:package.jsonabsent or withoutscripts.test), so the contract named a command that does not exist. The scaffold in contract-files ≤ v0.5.1 rendered that line unconditionally; bot reviewers caught it on the held PRs, and a fleet-wide census found 73 of 166 repos in the same state. A contract that lies is worse than no contract: in observe mode the Stop gate just logs, in block mode it would refuse every close on a command that cannot run.What this does
TESTING.md(the scaffold at contract-files v0.5.2 now REFUSES to render a test-contract when no test command is inferable —no-test-command— and that refusal was exercised here, not assumed)..contracts-absent.jsondeclaringtest-contractabsent with reasonno-test-command. That record is what the fleet census reads: this repo is now explicitly without a test contract, rather than silently missing one or falsely carrying one. When a real test command is added, delete the record and runcontracts init --families test-contract.PERMISSIONS.mdfrom the same rollout is untouched — it was correct.Guarantees
One reversible commit; reverting it restores the rollout state exactly. Nothing executes; no gate mode changes. Body kept above the fleet's measured 1,200-character PR floor deliberately.
Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.Note
Low Risk
Documentation and contract-metadata only; no runtime, gate mode, or test execution changes.
Overview
Replaces a misleading test contract with an explicit “no test contract” declaration because the removed
TESTING.mdfenced block pointed atnpm test/ suite commands that do not exist in this repo.The PR deletes
TESTING.mdentirely and adds.contracts-absent.jsonrecordingtest-contractas absent with reasonno-test-command, so fleet tooling treats the repo as intentionally without a machine test contract rather than carrying a broken one that could fail gates in block mode.Reviewed by Cursor Bugbot for commit 580394d. Bugbot is set up for automated code reviews on this repo. Configure here.
Summary by Sourcery
Replace the misleading test contract with an explicit no-test-command declaration.
Bug Fixes:
Enhancements: