Skip to content

Agents pane: CLI-only MCP, yaak mcp --http, and plugin deprecation - #761

Open
gschier wants to merge 3 commits into
mainfrom
agents-pane
Open

gschier wants to merge 3 commits into
mainfrom
agents-pane

Conversation

@gschier

@gschier gschier commented Oct 15, 2026

Copy link
Copy Markdown
Member

The agent story becomes one sentence: install the Yaak CLI, and every agent talks to it.

  • The app no longer hosts an MCP server. mcp_ext.rs, its token, and the port toast are gone; the mcp_server_enabled column stays in the database, unread.
  • yaak mcp --http serves the same tools over HTTP on 127.0.0.1 with --port (default 64343) and --token; without a token it generates one and prints it to stderr. For agents that cannot run the host's CLI, such as one inside a container. Two end-to-end tests.
  • Settings > Agents is the picker, a CLI status line (re-checked on window focus), and two command blocks. Each is marked done when the agent's config or skills directory already has it, through a new cmd_agent_setup_status command. Claude Code snippets use -s user so the registration is global and detectable. A banner with Uninstall appears while the deprecated @yaak/mcp-server plugin is installed.
  • The plugin API reads the registry's new deprecated note (mountain-loop/yaak-web#110) and makes yanked optional. Discover hides deprecated plugins; Installed shows the note with Uninstall. The mcp-server plugin carries the note, a reworded toast, and a README whose Claude Desktop config is yaak mcp with no mcp-remote. Not published yet.

Left for later: rand and yaak-mcp are unused in the app crate's Cargo.toml (lockfile change), and publishing the plugin.

Agents pane with both steps detected

Installed plugins with the deprecation note

@greptile-apps

greptile-apps Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

RetriggerView in GreptileConfidence Score: 5/5

[High impact] The PR appears safe to merge, with non-blocking gaps in setup detection and deprecation warnings.

Findings

  1. P2 Older plugins hide deprecation warnings ▶
  2. P2 VS Code setup looks incomplete ▶
  3. P2 Multiline Codex arguments go undetected ▶

Summary

The PR moves agent connections to the CLI and adds yaak mcp --http.

  • The Yaak CLI serves MCP tools over stdio or HTTP.
  • Settings > Agents shows setup commands and marks steps it detects.
  • Deprecated plugins are hidden from search and come with replacement guidance.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  Settings[Agents settings] --> Config[Read agent configuration]
  Agent[Agent] --> Stdio[yaak mcp]
  Agent --> HTTP[yaak mcp --http]
  Stdio --> Tools[Shared MCP tools]
  HTTP --> Token[Check bearer token]
  Token --> Tools
  Tools --> Data[Local Yaak data]
Loading

Reviews (3) · Last reviewed commit: "Detect npx launches and multiline Codex ..." · Reviewed by Greptile

Comment thread apps/yaak-client/lib/agentSetup.ts Outdated
Comment thread crates/yaak-commands/src/agents.rs Outdated
The agent story is one sentence: install the Yaak CLI, and every agent
talks to it. The app no longer hosts an MCP server; `yaak mcp` serves
stdio and `yaak mcp --http` serves HTTP with a bearer token for agents
that cannot run the host's CLI.

Settings > Agents is the picker, a CLI status line, and two command
blocks (add the MCP server, install the skill), each marked done when
the agent's config and skills directory already have it. A banner
appears when the deprecated @yaak/mcp-server plugin is installed.

The plugin API reads the registry's new `deprecated` note; Discover
hides deprecated plugins and the Installed tab shows the note with an
Uninstall button. The mcp-server plugin carries the note, a reworded
toast, and a README that points at `yaak mcp`.
`yaak agent install` takes the agent as `--agent`, so the pane's copied
command failed. Detection now requires an entry whose command is yaak
with mcp in its args; a `yaak` entry pointing at a URL is the deprecated
plugin's server and no longer shows the step as done.
@gschier

gschier commented Oct 15, 2026

Copy link
Copy Markdown
Member Author

@greptileai review

Comment thread apps/yaak-client/components/Settings/SettingsPlugins.tsx
Comment thread crates/yaak-commands/src/agents.rs
Comment thread crates/yaak-commands/src/agents.rs Outdated
Comment thread crates/yaak-commands/src/agents.rs Outdated
`npx -y @yaakapp/cli mcp` runs the same binary, so it counts; the
mcp-remote bridge still does not. The Codex table reader now collects
`args` across lines instead of requiring the array on one line.
@gschier

gschier commented Oct 15, 2026

Copy link
Copy Markdown
Member Author

@greptileai review

This branch has not been deployed

No deployments
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