Repository navigation
Fix issue #759 - #760
Open
cenetex[bot] wants to merge 1 commit into
Open
Fix issue #759#760cenetex[bot] wants to merge 1 commit into
cenetex[bot] wants to merge 1 commit into
Conversation
![github-agent[bot]](https://github.githubassets.com/images/gravatars/gravatar-user-420.png){kind=avatar}
Contributor
Author
|
🔍 Automated PR Review Started This PR was opened by the coding agent and is now being automatically reviewed. |
cenetex
Bot
added
review:error
Contributor
Author
|
🔧 Automated Review: ERROR The automated review encountered an error and could not complete. Error Details: This PR will require manual review. 🔄 Labels Applied: Task ID: |
atimics
added a commit
that referenced
this pull request
Oct 23, 2026
…eview image (#762) * fix(review): run the metered review model and ship the review image The review script hardcoded z-ai/glm-5.2 and ignored the MODEL the Lambda passes, so reviews were reserved at Haiku's 4 credits but ran on glm-5.2. Read MODEL, keeping glm-5.2 as the fallback when it is absent. The Lambda's default anthropic/claude-haiku-4-5 is not an OpenRouter model ID (OpenRouter uses dotted IDs). Use anthropic/claude-haiku-5.5, whose 1M context matches the model_context_window in the shared Codex config. deploy.yml never built Dockerfile.review, so the github-agent-review:latest image has been frozen since 2026-08-29 (#707). Build and push it on every deploy alongside the agent image. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * fix(review): parse findings a model wrapped in a Markdown fence Every automated review since 2026-09-10 ended in review:error with exit code 4. Codex completed and the model wrote a decision, but glm-5.2 does not honor --output-schema through OpenRouter and fenced its JSON in ```json. jq rejected the fenced file and set -e aborted the review (CloudWatch, task_mvkh3yby_ah82dd on #760). Strip a leading/trailing fence when the findings are not already JSON, and fail with an explicit message when they still are not. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> --------- Co-authored-by: atimics <atimics@ratimics.com> Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
This was referenced Oct 24, 2026
This branch has not been deployed
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.
Fixes #759
Automated implementation published by the trusted control-plane broker.