Skip to content

feat(mcp/ocs): expose ocs_get_chatbot_pipeline_id atom#353

Merged
jjackson merged 1 commit into
mainfrom
emdash/clean-up-26v2d
May 19, 2026
Merged

feat(mcp/ocs): expose ocs_get_chatbot_pipeline_id atom#353
jjackson merged 1 commit into
mainfrom
emdash/clean-up-26v2d

Conversation

@jjackson
Copy link
Copy Markdown
Owner

Summary

  • New MCP atom `ocs_get_chatbot_pipeline_id({ experiment_id }) → { pipeline_id }` that wraps the existing `PlaywrightBackend.pipelineIdFor` method.
  • Closes the last orphan-storage class on `/ace:sweep ocs` — the 2026-05-18 sweep deleted 9 orphan chatbots + 12 collections cleanly but had to skip all 10 paired Pipeline rows because there was no atom to discover `pipeline_id` from `experiment_id`.
  • `sweep-ocs/SKILL.md` Step 3 now calls the new atom per chatbot; Step 9 warns against skipping the pipeline delete.
  • Thin 1-line delegate in the composite backend; one new vitest case (9/9 tests passing).

Test plan

  • `npx vitest run test/mcp/ocs/composite.test.ts test/mcp/ocs/capability-map.test.ts` (9/9 passing)
  • Post-merge: follow-up sweep against the 10 zombie pipelines from chatbots 11839, 11994, 11996, 12000, 12003, 12018, 12042, 12050, 12101 + historical 12027 versions

🤖 Generated with Claude Code

@jjackson jjackson enabled auto-merge May 18, 2026 21:01
New MCP atom that wraps the existing PlaywrightBackend.pipelineIdFor method.
Resolves an experiment_id to its working-version pipeline_id (integer) so
/ace:sweep ocs can pair each orphan chatbot with its per-opp Pipeline row
before deletion. Without this, the 2026-05-18 sweep had to skip all 10
paired pipelines — they're now reachable.

OCS's REST /api/experiments/<id>/ response omits pipeline_id by design; the
atom scrapes it from the pipeline-builder HTML (already cached). 1-line
delegate in the composite backend; one new vitest case in composite.test.ts.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@jjackson jjackson force-pushed the emdash/clean-up-26v2d branch from 79c5a6f to 569b57e Compare May 19, 2026 19:25
@jjackson jjackson merged commit 5ccedd4 into main May 19, 2026
2 checks 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.

1 participant