Skip to content

Commit 9c7b29e

Browse files
nodeeeeeeclaude
andcommitted
Codex default to gpt-5.2; always route verify through gpt-4o
- Default AUTONOTE_CODEX_MODEL=gpt-5.2 — gpt-5.1 needs an API-key codex login; gpt-5.2 works on ChatGPT-plan logins. - Verify pass no longer piggybacks on the generator model when NOTE_MODEL is codex-cli. Verification is a short review call where gpt-4o via the OpenAI API is cheaper and doesn't eat codex quota. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent d9b1d2f commit 9c7b29e

1 file changed

Lines changed: 9 additions & 5 deletions

File tree

note_generation.py

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -595,9 +595,10 @@ def _call(model: str, system: str, user: str, max_tokens: int,
595595
# have to parse the streaming event log on stdout.
596596
# The caller's ~/.codex/config.toml default (e.g. gpt-5.2-codex) is
597597
# often not available on a ChatGPT-plan login, so we override to
598-
# gpt-5.1 (matches NOTE_MODEL used elsewhere in the pipeline).
599-
# Set AUTONOTE_CODEX_MODEL to pick a different one when needed
600-
# (e.g. gpt-5.4, gpt-5.5, gpt-5.4-mini on ChatGPT-only accounts).
598+
# gpt-5.2 which is broadly available on ChatGPT plans. Set
599+
# AUTONOTE_CODEX_MODEL to pick a different one — gpt-5.1 requires
600+
# an API-key codex login, gpt-5.4 / gpt-5.5 / gpt-5.4-mini work on
601+
# ChatGPT-only accounts.
601602
if _provider(model) == "codex-cli":
602603
import subprocess as _sp
603604
import tempfile as _tf
@@ -606,7 +607,7 @@ def _call(model: str, system: str, user: str, max_tokens: int,
606607
_os2.close(out_fd)
607608
try:
608609
prompt_text = f"{system}\n\n{user}" if system else user
609-
codex_model = _os2.environ.get("AUTONOTE_CODEX_MODEL", "gpt-5.1")
610+
codex_model = _os2.environ.get("AUTONOTE_CODEX_MODEL", "gpt-5.2")
610611
cmd = [
611612
"codex", "exec",
612613
"-m", codex_model,
@@ -1051,7 +1052,10 @@ def generate_section(
10511052
terms.add(t)
10521053
term_list = ", ".join(sorted(terms)[:30])
10531054
v_user = _P("verify").format(term_list=term_list, draft=draft[:2500])
1054-
_vmodel = NOTE_MODEL if NOTE_MODEL in ("claude-cli", "codex-cli") else VERIFY_MODEL
1055+
# Always route the verify/revision pass through VERIFY_MODEL
1056+
# (gpt-4o) — cheap and fast, avoids burning codex quota on a
1057+
# short review call. Caller must have an OpenAI key configured.
1058+
_vmodel = VERIFY_MODEL
10551059
v_result = _call(_vmodel, "", v_user, 1500)
10561060
if not v_result.strip().upper().startswith("APPROVED"):
10571061
if len(v_result) > len(draft) * 0.3:

0 commit comments

Comments
 (0)