Skip to content

feat(agents): Add Pi as a supported agent#78

Closed
chargome wants to merge 1 commit intomainfrom
feat/add-pi-agent
Closed

feat(agents): Add Pi as a supported agent#78
chargome wants to merge 1 commit intomainfrom
feat/add-pi-agent

Conversation

@chargome
Copy link
Member

Add Pi (https://github.com/badlogic/pi-mono) to the agent registry. Pi reads .agents/skills/ natively and does not support MCP or hooks.

Make McpConfigSpec optional on AgentDefinition so agents without MCP support are modeled honestly at the type level rather than using dummy specs with runtime throws. The mcp-writer loops skip agents with mcp: undefined via guard clauses.

Add Pi (https://github.com/badlogic/pi-mono) to the agent registry.
Pi reads .agents/skills/ natively and does not support MCP or hooks.

Make McpConfigSpec optional on AgentDefinition so agents without MCP
support are modeled honestly at the type level rather than using dummy
specs with runtime throws. The mcp-writer loops skip agents with
mcp: undefined via guard clauses.

Co-Authored-By: Claude <noreply@anthropic.com>
@chargome chargome requested a review from gricha March 20, 2026 18:36
@chargome chargome self-assigned this Mar 20, 2026
@vercel
Copy link

vercel bot commented Mar 20, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
dotagents Ready Ready Preview, Comment Mar 20, 2026 6:36pm

Request Review

@gricha
Copy link
Member

gricha commented Mar 20, 2026

We talked about having this in here: #25
but i don't think there's value in it?

pi can read from .agents/skills and doesn't support anything else so it should work out of the box unless I'm missing something. For MCP support I think the way would be to have a Pi extension that reads from common mcp.json file.

Feel free to reopen if you think that's wrong and can talk more. Also cc @bruno-garcia

@gricha gricha closed this Mar 20, 2026
@chargome
Copy link
Member Author

Would have been purely cosmetic I agree, it was just unintuitive for me not have this in the agent list.

@gricha
Copy link
Member

gricha commented Mar 20, 2026

Hm.. I could argue that's fine. Pi seems to be big enough @bruno-garcia thoughts?

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