Repository navigation
Register the pending model routing layer for its covering sweep - #1126
Closed
seathatflowsinourveins wants to merge 2 commits into
Closed
seathatflowsinourveins wants to merge 2 commits into
seathatflowsinourveins wants to merge 2 commits into
Conversation
3 of 4 tasks
5 of 6 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Scope
Register foundation/model-routes with the CC's exact provider/model/account-pool/effort scope so a covering sweep can start after review and landing. The row remains pending_lanes with no candidates or winners; the validator now admits that explicitly unsealed, gap-bearing registration without inventing a selection.
a339425ca56ec3fb87dff0fa8ad5dffc83354168(origin/main).lane:foundation.--slots 2after this PR lands and the CC read is complete. This PR dispatches no model, installs no runtime, changes no gateway/account state and makes no terminal claim.SOTA sources
252babe4040e39173cc178317c67699371806df6, catalogs/landscape/final-stack.json foundation/model-routes entry and catalogs/landscape/angles/foundation-model-routes.json: retained unswept, angle-only source review with no catalog winner.5692d4bfb6f30dbee76d79c45733c3ac2236f995bb690e69edfe40caa851562c; ROWS SHA256872e4b9b748b4c0967465ac08c90ead44e9f8f6c8d7455d2825e4c335c902c1a; CC D1/F5 and 00:29Z pending-empty ruling retained in the lane questions report.Evidence-class table
Local commands run
The final outcomes are recorded in the lane R blocks:
Final
python3 scripts/validate.pyexits 1 with ONLY the nine inherited model-currency date-rollover lines (nemotron embed, qmd reranker/query expansion, Potion, MinerU, the Kompress pair, Magika and background-haiku). The co-op confirmed this CI-wide incident; #1124 supplies its separately owned fix. No own file/hash/coverage errors remain and no currency-age or scanner bypass is added. Native landscape and final-stack checks pass; the final 44681-byte staged diff passed redacted gitleaks with zero findings.Decision record
docs/decisions/2026-10-15-model-routes-catalog-registration.md. The scope text is verbatim, the older native-clients sentence is explicitly superseded, and the readiness input is review plus landing before the paced covering sweep.Host evidence
No evidence/hosts receipt or platform-status change.
Checklist
Review correction at 0f9e90b
Fix round 1 addresses all three local-review P1 findings from reviewed head
5da12bca3a90237c2f4155814c05b2498c333062(review SHA2565a03c17995ca4684322d00b452355390c1d3e2deaea92fece93736eb8af49764), in one commit:registration_onlyscope has no wave. The shared reader returns null; frozen-wave output excludes it. The 20260922 wave, registry and handbook remain byte-identical. Final-stack haswave: nullandwave_doc: null.Each finding has a regression executed before its fix: foundation
20 required layersrefusal, wrong20260922wave identity, andmissing report layers: model-routes. After the fix, 195 affected consumer/landscape tests and 31 final-stack repository/status/registration tests pass. The three originally failing required commands now pass, along with standalone landscape, matrix, final-stack, catalog validation andvalidate_convergence.py --all-recorded.Full
validate.pywas run again before this commit and fails with ONLY the same nine inherited model-currency lines tracked by #1124. No own integrity failure is hidden by that incident; the individual required consumer checks were run directly. Final 36632-byte staged diff SHA2562eb7bda1a175d3e8f94bfa8a8cae118e8d7d8dc40c89ce5a54168f76bbf6a601passed redacted gitleaks idle-class, with no findings. The exact CC scope and empty candidates/winners remain unchanged.Additional pinned mechanism sources at the reviewed head: validate_foundation.py:35-42, its taxonomy/capability coverage at :88-94 and :216; build_verdicts.py:164-170 and frozen-wave checks :467-486; new_host_grand_list.py:91-151. The dated decision records the correction and boundaries; no upstream E2E/adoption or sweep result is claimed.