-
Notifications
You must be signed in to change notification settings - Fork 1.1k
chore(lanes): land the adapters and providers lane (#4682, #4670, #4668, #4622, #4224, #4652) #4719
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
24 commits
Select commit
Hold shift + click to select a range
fc260e9
fix(cursor): normalize localized native-shell routing claims
luvs01 588c333
fix(cursor): bound Korean shell aliases at token boundaries
luvs01 11e343e
fix(google): allow structured output for Gemini models on Cloud Code …
agentHits 4931e85
fix(kiro): send native reasoning effort for the GPT-5.6 family and re…
wentao-ma2 c68682d
docs(kiro): spell the full claude-opus-5 effort field in translated p…
wentao-ma2 6c8f5d1
fix(google): allow structured output for Gemini models on Cloud Code …
agentHits ed9f0b9
fix(providers): declare Anthropic image input capabilities
TykanN 179aa98
fix(claude): forward done-only tool arguments
RHODIZSECURITY 1a2b415
fix(claude-desktop): expand synthetic date alias slots from 365 to 3652
537e32b
Merge commit 'aa91958e3b050084e1edc07dcd66b05ef6eac604' into agent/cu…
luvs01 24adf12
fix(clients): propagate image capabilities across integrations
TykanN 22965dc
test(google): pin CCA image refusal and typed schema envelope
lidge-jun 46e5f55
fix(kiro): keep unverified luna and terra effort rungs emulated
lidge-jun ed5ba2c
merge pr #4682 into lane L3
lidge-jun e23534e
merge pr #4670 into lane L3
lidge-jun 4a3d39a
merge pr #4668 into lane L3
lidge-jun 3c9700d
merge pr #4622 into lane L3
lidge-jun 0553eb5
merge pr #4224 into lane L3
lidge-jun 75bdb1a
merge pr #4652 into lane L3
lidge-jun 3169dfe
Merge branch 'fix/cca-gemini-structured-output' of https://github.com…
agentHits bf07b72
docs(lanes): correct the Kiro reasoning-text claim and the Desktop al…
lidge-jun 4353ddb
merge pr #4670 current head into lane L3
lidge-jun f92312a
merge dev into lane L3 after the Codex account and catalog lane landed
lidge-jun 4f61edc
docs(cursor): record why the localized native-shell match has no righ…
lidge-jun File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
🔎 Supported by static analysis
🤖 get_repo_knowledge executed:
get_repo_knowledge lidge-jun/opencodex /tmp/coderabbit-repo-knowledge/lidge-jun-opencodex-7afea732/conventionsLength of output: 9786
Run the required
docs-sitebuild before merge. This change updates public documentation indocs-site/src/content/docs/ru/reference/adapters.md:192-197. The repository convention requires:cd docs-site bun install --frozen-lockfile bun run buildDo not claim documentation validation passed until this build completes successfully.
🤖 Prompt for AI Agents
Source: Coding guidelines
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This batch is verified by hosted CI at an exact head SHA, not by local commands, and that is a deliberate instruction from the repository owner rather than an oversight. A previous local run in this environment destroyed real data under the user's home directory, so the standing rule for this landing train is that the only admissible evidence is static source reading against the source-of-truth files plus a completed hosted CI run at the exact commit under review.
I am not going to claim I ran the commands you asked for. What exists instead is the aggregate
cicheck on this pull request at its current head, which composes the Linux test shards, the macOS legs,gates,storage policy,api usage,docker smoke, the npm-global smokes and the keyring smokes on Linux, macOS and Windows. Becauseplatform-windowsis dispatch-only and is always skipped on apull_requestevent, aworkflow_dispatchrun withlane=allis additionally started on the same head so the six Windows test shards actually execute: https://github.com/lidge-jun/opencodex/actions/runs/35048785887That run is not decoration. On the previous heads it caught a real
windows 2/6failure, which turned out to belong todevrather than to this lane and was fixed by #4564 in the sibling lane. The pull request description records the comparison.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🧩 Analysis chain
🏁 Script executed:
Length of output: 2670
@lidge-jun, I did not run local repository commands.The hosted
Cross-platform CIrun is for the current PR head SHA4f61edc49c823080a0b61294512c148e4b0114a4. However, workflow run35048785887is currentlyqueuedwith no conclusion. The PR check rollup also contains queued checks.Static inspection confirms that
docs-site/src/content/docs/ru/reference/adapters.mdLines 192-197 describes the intended Kiro effort behavior. Hosted validation is still incomplete. I will keep this finding open until the hosted CI run for this exact SHA completes successfully.🐇
🧠 Learnings used
You are interacting with an AI system.