Skip to content

Support for oh-my-pi coding agent#119

Closed
muneale wants to merge 2 commits into
zzet:mainfrom
muneale:main
Closed

Support for oh-my-pi coding agent#119
muneale wants to merge 2 commits into
zzet:mainfrom
muneale:main

Conversation

@muneale

@muneale muneale commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Summary

Registers the Oh My Pi (oh-my-pi) agent adapter to configure MCP servers under .omp/mcp.json. Also registers the client ID omp-coding-agent to automatically negotiate the GCX1 wire format. Includes unit testing, integration smoke tests, and global installation mode (ModeGlobal / gortex install) skip logic to prevent project-scoped file writes.

Changes

  • Adapter: Added internal/agents/ohmypi/adapter.go and internal/agents/ohmypi/adapter_test.go.
  • Format Negotiation: Added omp-coding-agent format negotiation in internal/mcp/server.go and updated test coverage in internal/mcp/format_negotiation_test.go.
  • CLI Commands: Registered the adapter in the CLI and wizard via cmd/gortex/init.go and cmd/gortex/init_wizard.go.
  • Smoke Tests: Added smoke testing integration in .github/workflows/init-smoke.yml.
  • Docs: Updated docs/agents.md.

Testing

  • All tests pass (go test -race ./...)
  • New tests added for new functionality
  • Benchmarks run if performance-relevant

Checklist

  • Code follows existing patterns in the codebase
  • No unnecessary abstractions added
  • Language extractor includes Meta["methods"] for interfaces (if applicable)
  • Methods have EdgeMemberOf edges to their containing type (if applicable)

muneale added 2 commits June 18, 2026 23:39
Register Oh My Pi as a new agent adapter writing config to `.omp/mcp.json`.
Includes detection in the init wizard, smoke test coverage, docs update,
and GCX default-format recognition for the `omp-coding-agent` client ID.
@zzet zzet added this to the v0.51.0 milestone Jun 20, 2026
@zzet

zzet commented Jun 20, 2026

Copy link
Copy Markdown
Owner

Thanks for this! CI was red because a new adapter must ship its agent-render drift-fence golden (TestAgentsRenderGolden → missing cmd/gortex/testdata/agent-render/oh-my-pi.txt), which also failed the full test matrix.

I've rebased your two commits onto current main and completed the integration in #130: added the render golden, fixed the adapter count in docs/agents.md (now seventeen), and sorted the ohmypi import. Your commits are preserved with authorship. #130 is green and mergeable, so this can land via that PR.

zzet added a commit that referenced this pull request Jun 20, 2026
Complete oh-my-pi agent adapter (supersedes #119)
@zzet zzet closed this Jun 20, 2026
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