Skip to content

chore(shell): drop AppKit fake agent chrome - #21

Open
Jimthetaxguy wants to merge 1 commit into
mainfrom
cursor/drop-appkit-fake-agent-chrome-e4b5
Open

Jimthetaxguy wants to merge 1 commit into
mainfrom
cursor/drop-appkit-fake-agent-chrome-e4b5

Conversation

@Jimthetaxguy

@Jimthetaxguy Jimthetaxguy commented Sep 14, 2026

Copy link
Copy Markdown
Owner

After #12, native ask/agent is gone. This removes leftover fake AppKit agent/approval/policy chrome that never talked to Rust.

  • AppKit hostAdapters no longer stubs startAgentRun / submitApproval / cancelAgentRun / getActionPolicy / setActionPolicy (or setProvider).
  • Compact AppKit React tree is browser commands + stats only: no ChatPanel, no provider/policy selects.
  • Tauri primary path still has real agent run / approval / policy.
  • No AppKit↔Rust agent bridge; dual browser chrome left alone.

Proof: AppKit adapter grep is clean of fake-success stubs; compact tree has no ChatPanel/selects. npm run build and npm run build:appkit both succeed. Browser preview: AppKit surface is commands + stats only; Tauri still shows provider/policy + AI Assistant.

AppKit compact shell without agent chrome
Tauri shell still has agent, provider, and policy UI

To show artifacts inline, enable in settings.

Open in Web Open in Cursor 

Remove AppKit adapter stubs that faked agent runs, approvals, and policy.
Compact AppKit React tree is browser commands and stats only until a real
Rust bridge exists. Tauri agent/approval/policy path is unchanged.

Co-authored-by: Jimthetaxguy <Jimthetaxguy@users.noreply.github.com>
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

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