Skip to content

Fix: restore the Summary tab in Pocodex - #17

Open
xIGBClutchIx wants to merge 1 commit into
davej:mainfrom
xIGBClutchIx:fix/restore-summary-tab
Open

Fix: restore the Summary tab in Pocodex#17
xIGBClutchIx wants to merge 1 commit into
davej:mainfrom
xIGBClutchIx:fix/restore-summary-tab

Conversation

@xIGBClutchIx

Copy link
Copy Markdown
Contributor

Summary

This restores the missing Summary tab in Pocodex so the current Codex desktop bundle once again shows the full side-panel tab set for git-backed threads.

What changed

  • plumb the real Codex bundle version, buildFlavor, and buildNumber through extension-info so the bundle evaluates the current Summary gate against the correct app identity
  • propagate local host kind and host_config through the CLI runtime, app-server bridge, and bootstrap payload so the desktop webview sees the workspace as the correct host type during startup
  • expose the shared-object snapshot fields and browser bridge compatibility hooks that the current bundle expects, including host_config, remote_connections, and the newer theme/menu bridge methods
  • add the current desktop git-worker platform-family RPC support that the newer bundle now calls during its startup flow
  • cover the restored bridge and bootstrap contracts with focused regression tests for extension info, host config, shared-object snapshots, and the git-worker RPC

Root cause

The missing Summary tab came from two newer Codex bundle assumptions that Pocodex was not satisfying:

  • Pocodex reported placeholder extension-info, so the bundle evaluated the Summary Statsig gate against the wrong app identity
  • Pocodex also missed newer desktop compatibility contracts around host_config, shared-object snapshots, and the git-worker startup RPCs
  • once those startup contracts were incomplete, the side-panel shell did not fully initialize the git-backed thread UI that exposes Summary alongside Review

Impact

  • git-backed thread side panels show Summary, Review, and Unstaged again in Pocodex
  • the current desktop bundle gets the extension metadata and host bridge shape it now expects during startup
  • focused regressions now cover the restored desktop compatibility surface so future bundle updates are easier to catch

Validation

  • pnpm run check:commit
  • pnpm exec vitest run test/app-server-bridge-basic.test.ts test/app-server-bridge-host-config.test.ts test/bootstrap-script.test.ts test/codex-desktop-git-worker.test.ts
  • pnpm exec tsx src/cli.ts --dev --listen 127.0.0.1:8812
  • live local verification on http://127.0.0.1:8812/ that opening the affected thread and toggling the side panel shows Summary, Review, and Unstaged

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.

1 participant