chore(triage): batch-land 15 reviewed PRs (lab CL-01..CL-10, adapters, providers, models) - #1736
Merged
Merged
Conversation
Rebased onto current dev with the reviewed public evidence trust core, consumer durability recovery, sparse-array JCS hardening, and required Windows publisher-key ACL hardening.
Rebase the reviewed CL-10 operator/community layer onto the current public-evidence core. This squashes the child history onto cl10-public-core while preserving the exact conflict-free GitHub merge tree, including the final review fixes.
Contributor
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (134)
📝 WalkthroughWalkthroughThis PR adds CL-10 local public evidence export and community import flows, centralizes adapter construction, adds custom-model reasoning metadata, tightens provider discovery and transport validation, improves Lab locking and purge durability, and adds broad regression coverage and planning documentation. ChangesCL-10 public evidence
Provider and platform controls
Triage documentation
Estimated code review effort: 5 (Critical) | ~180 minutes ✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
📝 Generate docstrings
🧪 Generate unit tests (beta)
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 |
github-actions
Bot
added
the
chore
Contributor
|
✅ Deterministic PR hygiene checks passed. |
Contributor
⏳ DRAFT
What to do
Automatic draft conversion failed (token cannot change draft status). Please convert this pull request to a draft manually. The required |
9 tasks
agentHits
pushed a commit
to agentHits/opencodex
that referenced
this pull request
Sep 17, 2026
chore(triage): batch-land 15 reviewed PRs (lab CL-01..CL-10, adapters, providers, models)
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
3 participants
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.
Summary\n\nBatch-lands 15 reviewed PRs onto dev via one integration branch, preserving each PR head as an ancestor so GitHub auto-marks every source PR as merged (attribution intact, incl. fork PR #1716):\n\n- Lab CL fixes: #1708 #1709 #1710 #1712 #1715 #1717 #1719 #1720\n- Lab CL-10 stack: #1705 #1706 (#1706/#1722/#1723 already auto-merged via stack-base fast-forward)\n- Providers/adapters: #1714, adapter authority stack #1721 #1722 #1723\n- Contributor feature: #1716 (per-custom-model reasoning effort)\n\nConflict resolutions: src/lab/ledger/purge.ts (#1706 export-purge deferral re-expressed inside #1709 withLedgerMutation, plus a TS closure-narrowing-safe cell for the deferred error), gui/.eslint/i18n-allowlist.ts (took #1714 case-insensitive HTTP match). Test-fixture fixes: mimo-free fixtures pointed at the canonical /chat endpoint required by #1714's guard (3 test files).\n\nTriage record: devlog/_plan/260815_open_pr_triage/.\n\n## Verification\n\n- bun run typecheck: green on integration tip ba20ce1 (local).\n- Full suite on remote Linux host (ssh lidge, /tmp/ocx-triage-260815-int @ ba20ce1): 12204 pass / 11 skip / 0 fail, 12215 tests across 777 files [448.99s]; bun run privacy:scan green.\n- Exact-head CI was green on every source PR before integration; zero head drift re-verified at push time.\n\n## Checklist\n\n- [x] Targets dev\n- [x] No token/secret logging added\n- [x] Regression tests included (each source PR ships its own; fixture fixes keep them green under #1714's canonical-endpoint guard)
Summary by CodeRabbit
New Features
Bug Fixes
Documentation