Skip to content

Align desktop MCP onboarding and complete notarization gate - #80

Merged
trissim merged 2 commits into
mainfrom
chatgpt-desktop-onboarding
Jul 28, 2026
Merged

Align desktop MCP onboarding and complete notarization gate#80
trissim merged 2 commits into
mainfrom
chatgpt-desktop-onboarding

Conversation

@trissim

@trissim trissim commented Jul 28, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Keep the existing codex registration target as the single owner of the shared ~/.codex/config.toml, while correctly presenting that registration to both ChatGPT desktop and Codex.
  • Update Windows and macOS installer UI, user documentation, installer documentation, website copy, and focused tests to match current local desktop versus ChatGPT web behavior.
  • Retrieve the authoritative Apple notarization log for the exact accepted submission, emit it, and independently require its status to remain Accepted before stapling, validation, Gatekeeper assessment, or upload.
  • Document that notarytool API-key authentication requires a team App Store Connect key.

No duplicate ChatGPT registration target or parallel client registry was added.

Current authorities

  • OpenAI MCP documentation states that ChatGPT desktop, Codex CLI, and the IDE extension share MCP configuration and that ChatGPT desktop supports local STDIO servers.
  • Apple notarization workflow requires reviewing the submission log even after success.
  • Apple packaging guidance confirms inside-out signing, outermost-container notarization, stapling, and distributed-product testing.

Verification

  • Combined installer, release, registration, and website suite: 90 passed
  • Focused cross-platform onboarding suite: 55 passed
  • Complete installer plus release-workflow gate: 56 passed
  • Strict Sphinx dummy build: all 149 sources passed with -W --keep-going
  • Ruff, Bash syntax, workflow YAML ordering, and diff hygiene: passed
  • Windows installer source: parsed successfully with Microsoft PowerShell 7.5

A native credentialed Apple notarization was not claimed: the repository currently has no Apple signing/notary secrets, and public v0.6.4 remains unsigned.

Copilot AI review requested due to automatic review settings July 28, 2026 01:41

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@augmentcode

augmentcode Bot commented Jul 28, 2026

Copy link
Copy Markdown

PR Risk Analyzer Agent🛡️

👀 Human Input Needed
A pair-review briefing is ready for you.

→ Pair Review Briefing

@augmentcode augmentcode Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Deep Code Review Agent🐛

Review completed with 0 suggestions.

@trissim
trissim merged commit b2a13a9 into main Jul 28, 2026
10 of 34 checks passed
@trissim
trissim deleted the chatgpt-desktop-onboarding branch July 28, 2026 01:47
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