Skip to content

feat: add acp_create_session tool to submit hpc-style jobs via mcp#26

Merged
jeremyeder merged 7 commits intoambient-code:mainfrom
jeremyeder:feature/acp-create-session
Feb 16, 2026
Merged

feat: add acp_create_session tool to submit hpc-style jobs via mcp#26
jeremyeder merged 7 commits intoambient-code:mainfrom
jeremyeder:feature/acp-create-session

Conversation

@jeremyeder
Copy link
Contributor

@jeremyeder jeremyeder commented Feb 16, 2026

Summary

  • Add acp_create_session MCP tool for submitting AgenticSessions with custom prompts (complements the existing template-only acp_create_session_from_template)
  • Extract shared _apply_manifest() helper to eliminate duplication between the two creation methods
  • Fix pod label selector bug in get_session_logs (agenticsessionagentic-session) that caused log retrieval to silently return empty
  • Add first live integration test (test_hello_acp) that creates a session, polls for marker output in logs, and cleans up
  • Register integration pytest marker so live-cluster tests are excluded from default pytest runs

Demo

asciicast

Test plan

  • 56 existing unit tests pass (0 regressions)
  • Integration test test_hello_acp passes against vteam-stage / jeder-workspace
  • Dry-run mode returns correct manifest without hitting cluster
  • get_session_logs now correctly discovers pods via agentic-session label

🤖 Generated with Claude Code

jeremyeder and others added 4 commits February 15, 2026 19:01
Add acp_create_session MCP tool for submitting AgenticSessions with
custom prompts (vs template-only). Extract shared _apply_manifest()
helper from create_session_from_template. Fix pod label selector bug
in get_session_logs (agenticsession → agentic-session). Add first
live integration test that creates a session, polls for marker output
in logs, and cleans up.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Claude Code UI simulation showing the full acp_create_session flow:
dry-run, live creation, status polling, log verification, cleanup,
and automated pytest run.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Replace scripted typing-effects demo with raw pytest execution.
Simpler script, smaller artifacts (GIF: 1.1MB → 42KB).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Replace deprecated class Config with model_config = SettingsConfigDict()
to eliminate PydanticDeprecatedSince20 warning. Re-record demo with
100-col terminal to prevent line wrapping.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@jeremyeder jeremyeder changed the title feat: add acp_create_session tool and hello-acp integration test feat: add acp_create_session tool to submit hpc-style jobs via mcp Feb 16, 2026
jeremyeder and others added 2 commits February 15, 2026 21:58
New demo shows the actual user workflow:
1. Display the plan
2. Submit via acp_create_session (real API call)
3. Disconnect — session runs autonomously on cluster
4. Check session status via acp_list_sessions
5. Verify output via acp_get_session_logs

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…ecture

Port acp_create_session feature from oc-CLI to HTTP-based public API:
- create_session() now uses POST /v1/sessions instead of _apply_manifest + oc create
- format_session_created formatter updated for API DTO format
- Integration test updated to use public API patterns
- All 46 tests pass

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…lint

Merge QUICKSTART.md and TRIGGER_PHRASES.md into a single comprehensive
README.md with TOC. Delete the redundant files — users no longer need to
hop between 4 docs to get started. Update CLAUDE.md cross-references.

Also includes pending fixes for ruff F541 (unnecessary f-strings) in
demos/hello-acp-workflow.py that were failing CI, plus prior uncommitted
work on the branch: public-api gateway docs, settings, and test updates.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@codecov
Copy link

codecov bot commented Feb 16, 2026

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

ℹ️ You can also turn on project coverage checks and project coverage reporting on Pull Request comment

Thanks for integrating Codecov - We've got you covered ☂️

@jeremyeder jeremyeder merged commit 4671c81 into ambient-code:main Feb 16, 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