chore: run the Pullfrog reviewer on the Codex harness with gpt-6.1-sol - #668
Merged
Merged
Conversation
The Codex step now sets PULLFROG_AGENT=codex and passes openai/gpt-6.1-sol as a raw model specifier, because no published Pullfrog alias resolves to it yet. The Claude fallback moves to claude-opus-5-5. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
![pullfrog[bot]](https://avatars.githubusercontent.com/in/1768019?s=60&v=4){kind=small}
There was a problem hiding this comment.
Important
The requested medium reasoning effort is not applied to the new model, so the review runs at the Codex CLI's default effort instead of the configured level.
Reviewed changes in the Pullfrog workflow configuration:
- Primary reviewer: Selects the Codex harness and pins
openai/gpt-6.1-solwithmediumeffort. - Failure fallback: Updates the Claude model to
anthropic/claude-opus-5-5while retaining the failure-only trigger.
GPT Sol ｜ 𝕏
…l's default effort The Codex CLI harness disables multi_agent, which drops the reviewfrog specialist sub-agent, and a raw model specifier has no effort rung, so the effort input was never applied. The review step keeps openai/gpt-6.1-sol on the default opencode harness, where OpenAI's documented default effort is medium. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
There was a problem hiding this comment.
✅ No new issues found.
Reviewed changes since the prior Pullfrog review:
- Restored the default harness: Removed the forced Codex CLI harness so the reviewer retains the
reviewfrogspecialist. - Clarified effort behavior: Removed the ignored
effort: mediuminput and documented that the raw model runs at its default effort, which OpenAI documents as medium.
GPT Sol ｜ 𝕏
|
thomasluizon
added a commit
that referenced
this pull request
Sep 30, 2026
* Delete the deprecated POST /oauth/google route (#957) (#664) The MCP authorize page moved to the authorization-code redirect through /oauth/google/start and /oauth/google/callback, so the One Tap tokeninfo route kept no caller in either app or in any documented client. It also created users through a raw repository call instead of an application command. Remove the route, its GoogleAuthRequest record, FindOrCreateGoogleUserAsync, the now orphaned user repository and HTTP client factory dependencies, the OAuthController.GoogleAuth catalog entry and the GoogleTokenAudienceMismatch error, and regenerate openapi.json. The route stayed deprecated on main, so oasdiff reports the removal as api-path-removed-with-deprecation at INFO and the breaking-change gate passes. Co-authored-by: Claude Opus 5 <noreply@anthropic.com> (cherry picked from commit e596ee1) * chore: run the Pullfrog reviewer on the Codex harness with gpt-6.1-sol (#668) * chore: run the Pullfrog reviewer on the Codex harness with gpt-6.1-sol The Codex step now sets PULLFROG_AGENT=codex and passes openai/gpt-6.1-sol as a raw model specifier, because no published Pullfrog alias resolves to it yet. The Claude fallback moves to claude-opus-5-5. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * chore: keep the Pullfrog reviewer on the opencode harness at the model's default effort The Codex CLI harness disables multi_agent, which drops the reviewfrog specialist sub-agent, and a raw model specifier has no effort rung, so the effort input was never applied. The review step keeps openai/gpt-6.1-sol on the default opencode harness, where OpenAI's documented default effort is medium. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com> (cherry picked from commit c95d548) * Transfer or release a device's push subscription when it changes accounts (#932) (#667) * Transfer a device's push subscription when another account registers it A device keeps one push endpoint or FCM token across the accounts that sign in on it. Before, a second account got PUSH_ENDPOINT_OWNED_BY_OTHER_USER, the client rotated the endpoint, and the first account kept a dead row that counted toward its five-device cap. Now the row moves to the registering account when the request proves it holds the device: a Web Push request must present the stored p256dh key and auth secret, and an FCM request must present the stored token with the FCM sentinel. A request without that proof still gets the old error, so an account cannot take or remove another account's device. Refs thomasluizon/orbit-tickets#932 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * Let the account on a device release its prior account's push row Unsubscribe only removed a row the caller owned. When a browser or Android device stayed registered to a previous account and the account now signed in turned push off, the client dropped the endpoint locally while the previous account kept listing and counting a dead device. Unsubscribe now takes the device's p256dh and auth, which both clients already send, and removes another account's row only when they prove control of the device, using the same check as the subscribe transfer. Without that proof the row stays with its owner and the call still returns success, so it does not reveal who owns an endpoint. Refs thomasluizon/orbit-tickets#932 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * fix: require explicit cross-account push subscription release * fix: bind push unsubscribe deletion to the observed owner --------- Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com> (cherry picked from commit ce78ccb) --------- Co-authored-by: Claude Opus 5 <noreply@anthropic.com>
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
1 participant
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.
What changed
The Pullfrog review step (the "Codex" step, which runs on the stored Codex credential) now reviews with
openai/gpt-6.1-sol. The model is passed as a raw specifier because no published Pullfrog alias resolves togpt-6.1-solyet (gpt-solstill resolves toopenai/gpt-6-solthrough release 0.1.86 and onmain), and the action'smodelinput overrides the repository console setting. The Claude fallback step, where present, moves fromanthropic/claude-opus-5toanthropic/claude-opus-5-5.The first version of this pull request also forced the Codex CLI harness (
PULLFROG_AGENT: codex) and asked foreffort: medium. Review showed both were wrong, so both are gone: the Codex harness disablesmulti_agent, which removes thereviewfrogspecialist sub-agent, and a raw model has no effort rung, so theeffortinput was never applied. The step stays on the default opencode harness, wherereviewfroginherits the main model.Test evidence
actionlint .github/workflows/pullfrog.yml: clean.resolveModel(utils/agent.ts) returns a slashed value that is not a curated alias unchanged, as a raw models.dev specifier.resolveRunEffort(utils/runEffort.ts:107-132at release 0.1.85) returnsrung: undefinedwhen the model has no catalog alias, and the opencode harness sends avariantonly when a rung exists (agents/opencode.ts:1188-1192,1394). With no variant the model runs at its default, and OpenAI documentsreasoning.effortforgpt-6.1-solas "low,medium(default),high,xhigh, andmax" (https://developers.openai.com/api/docs/models/gpt-6.1-sol).deriveSubagentModels(agents/subagentModels.ts) returns no override for a model with no alias, soreviewfroglogs(inherit)and runs on the main model.orbit-ui-mobile(run 36639474167) loggedBYOK auth enabled ... openai/gpt-6.1-solfor the stored Codex credential,model: openai/gpt-6-sol (resolved from openai/gpt-sol)andagent: opencode. The first review after merge must logmodel: openai/gpt-6.1-sol,agent: opencodeandeffort: n/a.🤖 Generated with Claude Code