Skip to content

feat(openai): stabilize ChatGPT subscription and add dynamic model discovery - #2

Merged
robitec97 merged 2 commits into
robitec97:mainfrom
claudi47:experimental/openai-chatgpt-oauth
Aug 21, 2026
Merged

robitec97 merged 2 commits into
robitec97:mainfrom
claudi47:experimental/openai-chatgpt-oauth

Conversation

@claudi47

@claudi47 claudi47 commented Aug 9, 2026

Copy link
Copy Markdown
Collaborator

Summary

Stabilizza l’integrazione OpenAI/ChatGPT Subscription e aggiorna la selezione dei modelli.

  • aggiunge discovery dinamica dei modelli OpenAI, con fallback locale GPT-5.6;
  • rende il reasoning capability-aware;
  • separa /model e /effort;
  • rafforza refresh OAuth, streaming/replay SSE, error handling e diagnostica sicura;
  • blocca redirect cross-origin per evitare l’inoltro di dati sensibili;
  • preserva risultati di errore per tool call saltate dopo Ctrl-C;
  • limita l’anteprima del diff durante overwrite di file grandi, evitando letture/allocation illimitate.

Validation

  • go test -count=1 ./...
  • go test -race -count=1 ./...
  • go vet ./...
  • build macOS, Linux e Windows
  • gofmt e git diff --check

Remaining live validation

Richiede credenziali reali:

  • sessioni ChatGPT Subscription consecutive con testo e tool use;
  • refresh token e riavvio;
  • /compact su Platform e Subscription;
  • verifica catalogo Subscription senza client_version.

@robitec97 robitec97 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed locally on the head SHA (06a984d): gofmt/vet/build clean, go test -race ./... green, git diff --check clean, dependency policy still only the MCP SDK subtree. CI is green. Checked the GPT-5.6 catalog against OpenAI's model guidance (gpt-5.6 alias → Sol; efforts none…max) — the static fallback matches. The Ctrl-C skipped-tool-result fix and the bounded overwrite preview both look right and are regression-tested.

Merging as-is. Live Subscription checks (M10 acceptance list) remain open until real credentials are available. Thanks Claudio!

@robitec97
robitec97 merged commit cc0d5f6 into robitec97:main Aug 21, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants