fix(content-lane): limit registry provider companions#1312
Conversation
|
Superagent didn't find any vulnerabilities or security issues in this PR. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1312 +/- ##
=======================================
Coverage 95.38% 95.38%
=======================================
Files 201 201
Lines 21598 21598
Branches 7807 7808 +1
=======================================
Hits 20601 20601
Misses 416 416
Partials 581 581
🚀 New features to boost your workflow:
|
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
gittensory-ui | 940ce41 | Commit Preview URL Branch Preview URL |
Jun 25 2026, 11:05 PM |
940ce41 to
bbdd03e
Compare
|
Tip 🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩 ✅ Gittensory review — safe to merge
✅ Approved — safe to merge
Nits — 2 non-blocking
Review context
Contributor next steps
Signal definitions
🟩 Safe / merged · 🟦 Advisory · 🟨 Held for review · 🟥 Blocked / closed 💰 Earn for open-source contributions like this. Gittensor lets GitHub contributors earn for the work they already do — register to start earning →. Checked by Gittensory, a quiet PR intelligence layer for OSS maintainers.
|
|
Closing — the condition it guards can't occur. The orchestrator already routes any changed file that isn't the single |
Motivation
Description
classifyRegistryPrScopeto requireproviderFiles.length <= 1when an entry file is present so an entry-submission may have at most one flat provider companion (changed theisEntryPrpredicate insrc/review/content-lane/registry-logic.ts).src/review/content-lane/registry-logic.ts.test/unit/content-lane-registry-logic.test.tsthat asserts an entry plus two flat provider files is classified asnot-direct-submissionwithdirectFilenull.Testing
npx vitest run test/unit/content-lane-registry-logic.test.tsand the unit suite (147 tests) passed.npm run typecheck(tsc --noEmit) and it completed successfully.npx vitest --coverageand tests passed but coverage post-processing failed withTypeError: jsTokens is not a functionoriginating inast-v8-to-istanbul.npm audit --audit-level=moderatecould not complete in this environment due to an HTTP 403 from the npm audit endpoint.Codex Task