Repository navigation
ci: lengthen npm registry verify retry window - #114
Merged
Merged
Conversation
Give Verify package.json version resolves on npm ~20 minutes (40 attempts, 30s apart) and raise the job timeout so that wait can finish after a slow registry propagation. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Rebuild so the committed agent-bundle manifest records the updated sha256 for .github/workflows/release.yml. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
ScriptedAlchemy
added a commit
that referenced
this pull request
Sep 29, 2026
) * feat: scaffold ChatGPT Desktop CDP adapter and MCP/CLI tools Add a ChatGptDesktopAdapter with a local-only CDP implementation, DOM selectors in one module, macOS relaunch helper, chatgpt_desktop_* MCP tools, gbot chatgpt-desktop CLI commands, and app-server fallback that reports backend. Includes rstest coverage with a fake adapter and a minor changeset. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com> * chore: rebuild artifact with chatgpt_desktop MCP tools Include the compiled grok-bot MCP server and gbot CLI routes for the ChatGPT Desktop CDP scaffold so packed installs expose the new tools. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com> * fix: align ChatGPT Desktop DOM module with exploration results Tighten cdp-dom selectors against the macOS Desktop probe: exact main window URL filtering, _MainContentSurface thread root, named experimental composer constants, and focus → Input.insertText → submit send order. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com> * test: cover CDP-first facade with app-server backend fallback Prove chatgpt_desktop_* prefer CDP, reuse codex-bridge when CDP is unreachable, and tag results with backend cdp|app-server. NotImplemented DOM gaps do not trigger fallback. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com> * feat: align ChatGPT Desktop CDP send/wait/read with live DOM Update cdp-dom and the adapter to the verified Desktop behaviors: project new-chat button, composer focus+insertText+Enter, Stop/final-assistant reply detection, conversation-id annotation, Loading task wait, and column-reverse mouseWheel full history reads with limit/full. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com> * docs: document chatgpt-desktop project send and full read Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com> * feat: deep-read and list via app-server; keep send/wait on CDP Desktop local:<conversationId> maps to bare app-server thread ids. Visible reads stay on CDP; full/deep history uses thread/read (wheel crawl last-resort). List prefers app-server and merges CDP sidebar fields. Send, new-thread-in-project, and wait-for-reply stay CDP-only. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com> * chore: retrigger Package CI for app-server deep-read redesign Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com> * feat: app-server list/search/read per 0.158.0; fix modelProviders Strip local: before app-server RPCs. Read via thread/read metadata plus thread/turns/list (itemsView full); never resume for reads. List and codex list-threads pass modelProviders:[]. Surface richer thread fields, remote-thread typed errors, and search. CDP keeps send/wait/selected. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com> * feat: merge remote-control threads into desktop list Parse ~/.codex/.codex-global-state.json remote-thread-summaries-v3 defensively and merge into chatgpt_desktop_list_threads with location, hostId, hostName, host filter, and groupBy=host. read_thread returns a typed remote error with a host app-server hint. Fixture-backed tests. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com> * feat: separate host and modelProvider list filters; add list_hosts Keep host (all|local|id/name) and modelProvider→modelProviders as independent list/search filters. Add chatgpt_desktop_list_hosts / gbot chatgpt-desktop hosts with thread counts, friendly names, and modelProviders discovered via app-server list method or distinct thread/list values (hostsSource / modelProvidersSource reported). Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com> * fix: include remotes in listHosts even without global-state env When listRemoteThreads is injected (or remotes arrive without a matching CODEX_HOME), still surface those hostIds with accurate thread counts. Rebuild artifacts for chatgpt_desktop_list_hosts. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com> * fix: codex list/read/send bugs from desktop work - list-threads: modelProviders:[], nextCursor paging, useStateDbOnly with full-scan fallback when the state DB returns nothing - normalize Desktop local: thread ids on every codex threadId acceptor - map remote-control "thread not loaded" to REMOTE_THREAD_NOT_LOADED with hostId/hostName on codex send and conversation open - readCodexThread uses thread/read metadata + thread/turns/list (never deprecated includeTurns) - outcomeFromError forwards hostId/hint/code; allowlist matches normalized ids Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com> * fix: CDP production risks from Ripwire audit Resolve new-thread send to data-response-annotation-conversation (never return local:client-new-thread:* as threadId). Harden column-reverse history wheels, Loading task timeouts, local:/bare sidebar dedupe+open, exact MAIN_WINDOW_URL attach, and keep loopback + useStateDbOnly cold-list guards covered by tests. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com> * fix: host/modelProvider are freeform strings with dynamic discovery Zack hard requirement: no hardcoded hosts, machines, connections, or providers in schemas, enums, docs defaults, or tests beyond fixtures. host and modelProvider accept any string; descriptions point to chatgpt_desktop_list_hosts. Add a test that injects an unknown hostId into the global-state fixture and proves list_hosts + filtering work with no code change. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com> * chore: regenerate artifact manifest after rebase onto main Rebased onto main (#114 verify retry + #96 @rstest/core 0.12.2) and rebuilt agent-bundle artifacts so the committed snapshot matches source. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com> * test: include chatgpt_desktop_list_hosts in MCP tool inventory Package CI failed because tools.test.ts still expected the pre-list_hosts tool set after rebase. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com> * chore: refresh artifact source snapshot after tools.test fix The package CI snapshot check requires the committed manifest sourceInputs hash to match tests/route-unit/tools.test.ts. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com> * Fix Desktop CDP isolation, reply tracking, and dynamic thread inventory * chore: refresh artifact snapshot after CDP review fixes Rebuild agent-bundle evidence/manifest so sourceInputs match the Desktop CDP isolation/reply-tracking commit that just landed. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com> * Exclude local plugin checkout from artifact source snapshots * Fix ChatGPT Desktop e2e discovery and thread handling * Expose complete ChatGPT Desktop MCP workflow and skill * fix: page ChatGPT Desktop MCP results below document cap * fix: page merged Desktop inventory and resolve project scope * fix: address Codex review of Desktop search, read fallback and backend provenance --------- Co-authored-by: Cursor Agent <cursoragent@cursor.com> Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.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
2 participants
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.
Summary
timeout-minutesfrom 15 to 35 so a full verify wait can complete after build/publish.artifact/agent-bundle.manifest.jsonsource snapshot so it matches the updatedrelease.ymlhash.This addresses run 36468899576, where npm took ~6 minutes to show 0.10.1 and the verify step timed out even though publish succeeded.
No publish, OIDC, or token changes.
Test plan
npm run build,validate:artifact, andtest/release-config.test.jspass after regenerating the artifact snapshot.