Repository navigation
Summarize with the Codex sign-in, and make Test setup deterministic - #1404
Merged
Merged
Conversation
Summaries longer than max_length_ratio (default 0.67, as before) of the body are discarded. Test setup lifts the limit when diffr supports it, so it checks the credential and model instead of the sample's length. The code fence models add despite the prompt is stripped first. Agent-Session: 02fbf542-41e9-4dde-86c4-2f75d6db972d Agent-Session: 7d7ae491-dbed-4fe3-8bc2-fe4851572ede Agent-Session: 5625cb14-3381-4fef-8d68-2cdc27ef7155 Agent-Session: ab3076ed-7808-4738-a770-0261b6018801 Agent-Session: 9a4dd689-814b-4ec4-aee5-d93cc4647727 Agent-Session: 99993da5-66b9-4872-b400-ba7f8c41628b Agent-Session: e05d0ee1-ebec-4824-922f-319222ec3817
It calls the ChatGPT backend Codex uses: the streamed Responses API with a ChatGPT token. provider_details.login names the sign-in the host reads when no key is set. Agent-Session: 02fbf542-41e9-4dde-86c4-2f75d6db972d Agent-Session: 7d7ae491-dbed-4fe3-8bc2-fe4851572ede Agent-Session: 5625cb14-3381-4fef-8d68-2cdc27ef7155 Agent-Session: ab3076ed-7808-4738-a770-0261b6018801 Agent-Session: 9a4dd689-814b-4ec4-aee5-d93cc4647727 Agent-Session: 99993da5-66b9-4872-b400-ba7f8c41628b Agent-Session: e05d0ee1-ebec-4824-922f-319222ec3817
diffr reads $CODEX_HOME/auth.json on every run, never refreshing it, and passes the token in the summarizer's options. `diffr config logins` reports whether it is ready, and `diffr config init` prefers Codex when it is. Agent-Session: 02fbf542-41e9-4dde-86c4-2f75d6db972d Agent-Session: 7d7ae491-dbed-4fe3-8bc2-fe4851572ede Agent-Session: 5625cb14-3381-4fef-8d68-2cdc27ef7155 Agent-Session: ab3076ed-7808-4738-a770-0261b6018801 Agent-Session: 9a4dd689-814b-4ec4-aee5-d93cc4647727 Agent-Session: 99993da5-66b9-4872-b400-ba7f8c41628b Agent-Session: e05d0ee1-ebec-4824-922f-319222ec3817
Settings shows the sign-in's status in place of the key field, accepts it as a credential, and picks Codex when it is signed in and no other provider has a key. Agent-Session: 02fbf542-41e9-4dde-86c4-2f75d6db972d Agent-Session: 7d7ae491-dbed-4fe3-8bc2-fe4851572ede Agent-Session: 5625cb14-3381-4fef-8d68-2cdc27ef7155 Agent-Session: ab3076ed-7808-4738-a770-0261b6018801 Agent-Session: 9a4dd689-814b-4ec4-aee5-d93cc4647727 Agent-Session: 99993da5-66b9-4872-b400-ba7f8c41628b Agent-Session: e05d0ee1-ebec-4824-922f-319222ec3817
thesiti92
force-pushed
the
diffr-agent-auth
branch
2 times, most recently
from
October 26, 2026 20:47
e1674bf to
7e357d4
Compare
Agent-Session: 02fbf542-41e9-4dde-86c4-2f75d6db972d Agent-Session: 7d7ae491-dbed-4fe3-8bc2-fe4851572ede Agent-Session: 5625cb14-3381-4fef-8d68-2cdc27ef7155 Agent-Session: ab3076ed-7808-4738-a770-0261b6018801 Agent-Session: 9a4dd689-814b-4ec4-aee5-d93cc4647727 Agent-Session: 99993da5-66b9-4872-b400-ba7f8c41628b Agent-Session: e05d0ee1-ebec-4824-922f-319222ec3817
thesiti92
force-pushed
the
diffr-agent-auth
branch
from
October 26, 2026 20:49
7e357d4 to
0743d89
Compare
sidkmenon
approved these changes
Oct 26, 2026
thesiti92
enabled auto-merge (squash)
October 26, 2026 20:51
thesiti92
added a commit
that referenced
this pull request
Oct 26, 2026
Bumps diffr to 0.1.22, the same set of files as #1365. This ships: - #1404 the `codex` summary provider, which uses the Codex ChatGPT sign-in, plus `max_length_ratio` - #1373 gix 0.90, #1374 wasmtime 49.0.4 Whiteboard's Settings shows the Codex provider once `@dev.fast/diffr-*` is bumped to this release. After merge, push the `diffr/0.1.22` tag on the merge commit to trigger Diffr Release.
thesiti92
added a commit
that referenced
this pull request
Oct 26, 2026
Pins the diffr platform packages to 0.1.22 (#1408), which adds the `codex` summary provider from #1404. With it, Settings → AI summaries offers Codex. Checked: frozen install passes, the lockfile lists all five platform packages, and the resolved binary reports `diffr 0.1.22` with `codex` in `diffr config schema`.
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.
diffr's summarizer gets a
codexprovider that uses the ChatGPT sign-in Codex already saved, so summaries need no API key. diffr reads$CODEX_HOME/auth.jsonon every run and never refreshes it, so Codex's own session is untouched; an expired token turns summaries off for that run and says to open Codex. The token goes into the summarizer's options, not the environment other plugins see.diffr config loginsreports whether the sign-in is ready.codex, and strips the code fence models wrap answers in despite the prompt.max_length_ratio(default unchanged). Test setup lifts it when diffr's schema has it, so the button checks the credential and model instead of failing on a short sample.diffr config initprefers it too.docs/ai-summaries.mdcovers setup.Whiteboard runs the pinned diffr, so the provider appears after a diffr release and bump; with the current pin, Settings behaves as before.
Checked in an isolated Desktop with a throwaway diffr config: Codex was preselected, saving wrote
provider = "codex", and a review's diff showed Codex summaries. Test setup with the lifted limit returned a summary 3 of 3 times against Codex (0 of 3 before).A Claude Code provider was tried and tabled: only
claude-haiku-4-5answers through its sign-in, and its summaries mostly fail the length rule.