Skip to content

[NA] [DOCS] Drop ask_ollie from the MCP server page - #8061

Open
aswynz wants to merge 1 commit into
mainfrom
aswynz/docs/drop-ask_ollie-from-mcp
Open

[NA] [DOCS] Drop ask_ollie from the MCP server page#8061
aswynz wants to merge 1 commit into
mainfrom
aswynz/docs/drop-ask_ollie-from-mcp

Conversation

@aswynz

@aswynz aswynz commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Details

comet-ml/opik-mcp#170 disabled ask_ollie everywhere behind a kill switch, so the hosted MCP no longer exposes it, but the MCP server page still presented it as an available tool. This removes it and repairs everything that referenced it. Touches prompt_engineering/mcp-server.mdx only.

  • Tool table — dropped the ask_ollie row and corrected the count ("six tools" → "five tools").
  • "Ollie & auto-approve" section — removed entirely. OPIK_MCP_AUTO_APPROVE only ever governed the mid-stream writes Ollie performed, so it is moot without the tool.
  • Self-hosted <Tip> — now names run_experiment as the only Cloud-only call.
  • Cursor 60s timeout notes (tip and "Known client limits") — the example of a long call is now a large run_experiment instead of an ask_ollie turn.
  • Example conversation — the investigative turn calls list + read.

Deliberately unchanged: run_experiment, still Cloud-only and available; and the web-UI Ollie pages, which document the Opik UI's Explain (Ollie) action and are unaffected by the MCP kill switch.

Change checklist

  • User facing
  • Documentation update

Issues

  • [NA] — no ticket. Stale-docs cleanup following comet-ml/opik-mcp#170.

AI-WATERMARK

AI-WATERMARK: yes

  • Tools: Claude Code
  • Model(s): Claude Opus 5
  • Scope: prose edits to a single .mdx docs page; no code
  • Human verification: author reviewed the full diff; the auto_approve claim was checked against the server source — all usage lives in ask_ollie.py / audit.py

Testing

Docs-only change, so no unit or integration tests apply and none were run.

  • The factual claim behind removing the auto-approve section — that OPIK_MCP_AUTO_APPROVE governs nothing once ask_ollie is gone — was checked by grepping the server for auto_approve; it appears only in ask_ollie.py and audit.py.
  • Remaining verification is the Fern docs build on this PR, which must render the page without broken anchors (the removed section had no inbound links from this page).

Documentation

This PR is the documentation change. No follow-up docs work required.

opik-mcp#170 disabled ask_ollie everywhere behind a kill switch, so the MCP no
longer exposes it. This removes it from the MCP server docs: the tool table
(now five tools), the self-hosted and Cursor-timeout notes, the example
conversation, and the "Ollie & auto-approve" section (auto-approve only ever
governed Ollie's mid-stream writes, so it is moot without the tool).
run_experiment is untouched — it is still Cloud-only and available.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@aswynz
aswynz requested review from a team as code owners August 28, 2026 19:19
@github-actions github-actions Bot added documentation Improvements or additions to documentation 🟢 size/S labels Aug 28, 2026
@CometActions

Copy link
Copy Markdown
Collaborator

No test needed here.

No product surface in this PR — only docs, CI, tests, lockfiles or other non-product files.

Run

Advisory, from the QA test radar. Nothing here blocks this PR, and anything it proposes is a draft for review.

@github-actions

Copy link
Copy Markdown
Contributor

⏱️ pre-commit per-hook timing

No linted files changed — nothing to run.

⏭️ 44 skipped (no matching files changed)
Hook Description Result
🐍 trim trailing whitespace — python sdk Strip trailing whitespace ⏭️
🐍 fix end of files — python sdk Ensure files end in a newline ⏭️
🐍 ruff — python sdk Lint + autofix Python (ruff) ⏭️
🐍 ruff-format — python sdk Format Python code (ruff) ⏭️
🐍 mypy — python sdk Static type check ⏭️
🤖 trim trailing whitespace — optimizer Strip trailing whitespace ⏭️
🤖 fix end of files — optimizer Ensure files end in a newline ⏭️
🤖 check yaml — optimizer Validate YAML syntax ⏭️
🤖 check json — optimizer Validate JSON syntax ⏭️
🤖 check toml — optimizer Validate TOML syntax ⏭️
🤖 check for added large files — optimizer Block large files (>1MB) ⏭️
🔐 detect private key — optimizer Block committed private keys ⏭️
🤖 check for merge conflicts — optimizer Block merge-conflict markers ⏭️
🤖 check for case conflicts — optimizer Block case-only name clashes ⏭️
🤖 pyupgrade — optimizer Modernize Python syntax ⏭️
🤖 ruff — optimizer Lint + autofix Python (ruff) ⏭️
🤖 ruff-format — optimizer Format Python code (ruff) ⏭️
🤖 mypy — optimizer Static type check ⏭️
📓 nbstripout — optimizer notebooks Strip notebook output ⏭️
📝 markdownlint — optimizer Lint Markdown ⏭️
🔤 codespell — optimizer Fix common misspellings ⏭️
📊 radon cc — optimizer Cyclomatic-complexity gate ⏭️
📊 radon raw — optimizer Raw size metrics gate ⏭️
📊 xenon — optimizer Fail on complexity thresholds ⏭️
📊 lizard — optimizer Cyclomatic-complexity gate ⏭️
🧹 vulture — optimizer Find dead code ⏭️
🛡️ trim trailing whitespace — guardrails Strip trailing whitespace ⏭️
🛡️ fix end of files — guardrails Ensure files end in a newline ⏭️
🛡️ ruff — guardrails Lint + autofix Python (ruff) ⏭️
🛡️ ruff-format — guardrails Format Python code (ruff) ⏭️
🛡️ mypy — guardrails Static type check ⏭️
⚓ helm-docs Regenerate Helm chart README ⏭️
block non-public FE plugins Block non-public FE plugins ⏭️
☕ spotless — java backend Format Java code ⏭️
🧪 pre-commit wrapper smoke tests Self-test the wrapper scripts ⏭️
🧪 rebaseline script tests Self-test the changelog re-baseline script ⏭️
🌐 eslint — frontend Lint + autofix JS/TS ⏭️
🌐 typecheck — frontend Whole-project tsc type check ⏭️
📘 eslint — typescript sdk Lint + autofix JS/TS ⏭️
📘 typecheck — typescript sdk Whole-project tsc type check ⏭️
⚙️ actionlint — github workflows Lint GitHub Actions workflows ⏭️
🐳 hadolint — dockerfiles Lint Dockerfiles ⏭️
🌈 zizmor — github workflows security Security-scan GitHub Actions workflows ⏭️
🛡️ semgrep — java backend sql Block SQL injection-prone string formatting ⏭️

@github-actions

Copy link
Copy Markdown
Contributor

🌿 Preview your docs: https://opik-preview-01a049d0-823d-774c-84d0-d999eacac334.docs.buildwithfern.com/docs/opik

No broken links found

Unverified links (timeout / rate-limited / server error — not failing the check)

https://aistudio.google.com/apikey (401)
↳ on page: /docs/opik/administration/workspace-settings/ai_providers
https://console.cloud.google.com/iam-admin/iam (401)
↳ on page: /docs/opik/administration/workspace-settings/ai_providers
https://console.cloud.google.com/iam-admin/roles (401)
↳ on page: /docs/opik/administration/workspace-settings/ai_providers
https://console.cloud.google.com/iam-admin/serviceaccounts (401)
↳ on page: /docs/opik/administration/workspace-settings/ai_providers
https://console.mistral.ai/api-keys/ (timeout)
↳ on page: /docs/opik/integrations/mistral
https://console.x.ai/ (403)
↳ on page: /docs/opik/integrations/xai-grok
https://docs.predibase.com/integrations/comet (403)
↳ on page: /docs/opik/integrations/predibase
https://opik-preview-01a049d0-823d-774c-84d0-d999eacac334.docs.buildwithfern.com/docs/opik/reference/rest-api/llm-provider-key/find-llm-provider-keys (500)
↳ on page: /docs/opik/reference/overview
https://portal.azure.com/ (403)
↳ on page: /docs/opik/administration/workspace-settings/ai_providers
https://x.ai/ (403)
↳ on page: /docs/opik/integrations/xai-grok


📌 Results for commit 88cb667

@aswynz aswynz changed the title docs(mcp): drop ask_ollie from the MCP server page [NA] [DOCS] Drop ask_ollie from the MCP server page Aug 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation 🟢 size/S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants