[WRONG BRANCH] chore(release): promote verified candidate to 2.43.0 - #3678
Merged
Conversation
* fix(subagents): migrate featured roster to Astra first and 5.5 last * test(subagents): exercise malformed arrays and failed migration persistence * fix(subagents): retain rebased config through later startup migrations --------- Co-authored-by: t <a@b.com>
…dence (#3614) Owner-authorized corrective admin merge. Revalidates post-wait combo presence and matches fatal UTF-8 quota evidence handling. Typecheck/static verification passed; no local tests. Final dev Linux CI is the batch gate.
Owner-authorized admin squash of child-only #3551 carry onto corrected #3608. Scheme-matched proxy snapshot, explicit proxy transport binding, exact IPv6 prefix and NO_PROXY denial retained. Existing source security assessment and current static/typecheck evidence recorded. No local tests; final dev Linux CI is the batch gate.
Owner-authorized corrective admin merge. Verifies copied runtime bytes and rejects unexpected sources and symlinks without including Git metadata. Typecheck/static checks passed; no local tests or Docker execution. Final dev HEAD CI is the gate.
Owner-authorized final-CI repair: concrete Linux quota route and fixture failures corrected; typecheck/static checks passed, no local tests. Final dev HEAD CI is the gate.
Owner-authorized diagnostic follow-up for the actual Linux restart failure. Only the fixture launcher is instrumented; production behavior, assertions and time budgets unchanged. Raw output redacted. No local tests; next dev HEAD CI is the execution gate.
…ccount-quota-api # Conflicts: # devlog/_plan/260905_provider_usage_quota_parity/040_stack_landing.md
fix(quota): repair Windows reset fixtures and route inventory
fix(responses): preserve caller cancellation in eager SSE relay
…ttribution # Conflicts: # skills/ocx/references/01_management_surface.md # src/cli/capabilities.ts # src/server/management/route-registry.ts # tests/usage/quota-reset-notify.test.ts
…ttribution # Conflicts: # tests/usage/quota-reset-notify.test.ts
* feat(gui): unify quota activation under advanced settings * docs: attach privacy-safe unified quota settings screenshot * docs(gui): explain ordered quota activation writes * fix(gui): preserve unavailable opt-ins when enabling quota activation * docs: refresh quota control verification screenshot * fix(i18n): clarify per-account quota activation scope --------- Co-authored-by: t <a@b.com>
…cycle feat(responses): bounded canonical upstream WS reuse
…oseout-split-train
refactor: consolidate verified modularization train and closeout records
* feat: enable Fast selectors by default across external clients * fix: count Fast variants in OpenCode launch summary * fix: align suffix-shaped Fast discovery with ingress --------- Co-authored-by: t <a@b.com>
…#3670) * feat(xai): default Grok Responses and migrate legacy Chat choices * fix(responses): carry code-mode output guidance on the first turn * fix(xai): retain post-upgrade Chat choice through reauthentication * docs(xai): record first-output and isolated control verification * test(responses): build signature scrub fixture through the real parser * fix(xai): preserve OAuth account rotation on native Responses * test(xai): use a short synthetic refresh credential in the fixture * test(xai): explicitly select Chat in the Chat streaming regression --------- Co-authored-by: t <a@b.com>
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. |
github-actions
Bot
added
the
chore
Contributor
|
Important Review skippedToo many files! This PR contains 1959 files, which is 1659 over the limit of 300. To get a review, reduce the PR to 300 files or fewer by splitting it into smaller PRs or changing its base branch. Usage-priced reviews support at most 300 files. ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Team Run ID: ⛔ Files ignored due to path filters (40)
📒 Files selected for processing (1959)
You can disable this status message by setting the |
Contributor
|
✅ Deterministic PR hygiene checks passed. |
Owner
Author
리뷰 · 우선순위 78 / 80이 PR은 형제 PR 역할이 분명합니다. 이번에 실리는 tip 줄기는 Grok Responses 기본 + Chat opt-in( 경로 tip 메인테이너의 판단이 필요한 지점
너의 추천 이 댓글은 grok-bot이 작성했습니다 |
github-actions
Bot
changed the title
Contributor
⏳ DRAFT
What to do
Its title has been prefixed with |
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
Comment on lines
+51
to
+54
| return createHash("sha256").update(JSON.stringify([ | ||
| "quota-key", name, provider.adapter, provider.baseUrl, provider.authMode ?? "key", | ||
| provider.disabled === true, id, key, | ||
| ])).digest("hex"); |
Comment on lines
+1811
to
+1815
| return createHash("sha256").update(JSON.stringify([ | ||
| provider, accountId, credential.access, credential.refresh, credential.expires, | ||
| credential.accountId, credential.projectId, credential.source, | ||
| target.adapter, target.baseUrl, target.authMode, target.disabled === true, | ||
| ])).digest("hex"); |
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
2 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
Promote the owner-approved development candidate af50c6d to main as 2.43.0. The owner explicitly directed shipping this candidate now and leaving pending fixes for later.
The promotion preserves the pinned candidate ancestry. The promotion branch is exactly the pinned candidate. No new production behavior is introduced by this promotion.
This is a maintainer-controlled release promotion. The dev-only target check is an expected promotion exception. The owner-requested administrative PR merge will be recorded here; no self-approval is claimed. Functional release gates remain required on the final main commit.
Verification
Promotion carries the already-reviewed interface change from #3670; original implementation screenshot:
CodeQL residual (not claimed green): the promotion scan reports 72 alerts, 38 under tests/ and 34 under src/. All 34 reported source lines exist verbatim in prior main; the large promotion and test-path moves change alert attribution. This is not proof that every alert is false. The owner directed shipping the current candidate and deferring fixes; the owner-authorized administrative promotion preserves these outstanding alerts without dismissing them. Exact push-event Cross-platform CI and lifecycle still gate publication.
Checklist