Skip to content

feat(plugins): use harness setup across agent hosts - #15

Merged
eshwar-sundar-glean merged 3 commits into
mainfrom
eshwar/remove-connect-skill-claude-codex
Aug 19, 2026
Merged

feat(plugins): use harness setup across agent hosts#15
eshwar-sundar-glean merged 3 commits into
mainfrom
eshwar/remove-connect-skill-claude-codex

Conversation

@eshwar-sundar-glean

@eshwar-sundar-glean eshwar-sundar-glean commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Remove the connect-glean skill from Claude Code, Codex, and Cursor.
  • Replace setup references with a prompt to the harness, e.g. Set up Glean for me.
  • Keep Cursor's manual MCP configuration instructions because Cursor does not bundle the local Glean MCP server.

Target behavior

Target connect-glean Setup guidance
Claude Code Removed Prompt the harness to set up Glean and complete OAuth
Codex Removed Prompt the harness to set up Glean and complete OAuth
Cursor Removed Prompt Cursor to set up Glean; use the plugin README's MCP configuration steps if an MCP URL is required

Updated the shared skill guidance (using-glean, project-handoff, and skill creation) so missing Glean tools direct the user to prompt the harness instead of referencing connect-glean.

Historical changelog entries remain unchanged.

Verification

  • npm test — build and validation passed for Claude, Cursor, and Codex
  • MCP typecheck passed
  • 195 MCP tests passed
  • No-shell guard passed
  • Confirmed generated outputs: Claude absent, Codex absent, Cursor absent
  • Confirmed no operational connect-glean references in any generated target
  • git diff --check passed

@eshwar-sundar-glean
eshwar-sundar-glean marked this pull request as ready for review August 19, 2026 09:38
Comment thread overrides/codex/root/README.md Outdated
```

Before starting Codex, connect the MCP server used by the plugin you installed:
Before starting Codex, set up Glean by prompting the harness, for example:

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Issue: The setup instructions tell users to prompt the harness before starting Codex, but the harness must be running to receive the prompt.

Suggested fix: Start Codex first, prompt the harness to set up Glean, complete OAuth, then start a new Codex task.

🔧 Tag @ glean-for-engineering to fix or click here to fix in Glean

💬 Help us improve! Was this comment helpful? React with 👍 or 👎

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed in 821a894. The Codex marketplace README now tells users to start Codex first, prompt the harness to set up Glean, complete OAuth, and only then start a new Codex task.

— sent via Glean Pi

Comment thread overrides/codex/root/README.md Outdated

Then start a new Codex task so the bundled skills and MCP tools are available.
The **Glean developer docs** plugin uses a separate public MCP server. Run
`codex mcp add glean-dev-docs --url https://developers.glean.com/mcp` if you

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Issue: The developer-docs MCP server is added after the documented task restart, with no subsequent restart, so its tools may not be available in the current Codex task.

Suggested fix: Run the codex mcp add command before restarting, or explicitly instruct users to start another Codex task afterward.

🔧 Tag @ glean-for-engineering to fix or click here to fix in Glean

💬 Help us improve! Was this comment helpful? React with 👍 or 👎

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed in 821a894. The developer-docs MCP command now appears before the documented new-task restart, followed by an explicit instruction to start a new Codex task so its tools are available.

The developer-docs guidance remains in this file because overrides/codex/root/README.md is the marketplace-level README for both the glean and glean-dev-docs plugins; the plugin-specific READMEs remain separate.

— sent via Glean Pi

Comment thread overrides/codex/glean/README.md Outdated
- [Codex](https://developers.openai.com/codex) with plugin support
- A Glean account with MCP access
- Your Glean MCP server URL and server name
- A Glean account with MCP access

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nit: duplicate

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed in 821a894 by removing the duplicate Glean MCP access requirement bullet.

— sent via Glean Pi

@eshwar-sundar-glean eshwar-sundar-glean changed the title feat(plugins): use harness setup for Claude and Codex feat(plugins): use harness setup across agent hosts Aug 19, 2026
@eshwar-sundar-glean
eshwar-sundar-glean merged commit 36ec522 into main Aug 19, 2026
1 check passed
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