Skip to content

feat: Add reasonix and pi support#183

Merged
Pringled merged 5 commits into
mainfrom
add-reasonix
Jun 5, 2026
Merged

feat: Add reasonix and pi support#183
Pringled merged 5 commits into
mainfrom
add-reasonix

Conversation

@Pringled
Copy link
Copy Markdown
Member

@Pringled Pringled commented Jun 4, 2026

This PR adds an install path for both reasonix and pi, which resolves both #79 and #170.

@Pringled Pringled requested a review from stephantul June 4, 2026 15:53
@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

Files with missing lines Coverage Δ
src/semble/installer/agents.py 100.00% <ø> (ø)
src/semble/version.py 100.00% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented Jun 4, 2026

Confidence Score: 4/5

Safe to merge after verifying whether Pi requires an explicit tools declaration in sub-agent frontmatter.

The Reasonix and Pi installer entries are well-structured and consistent with how other agents are integrated. The one concrete gap is pi.md lacking a tools field — every other agent file in this directory declares allowed tools, and without it the installed Pi sub-agent may silently be unable to run shell commands. If Pi's sub-agent system does not require an explicit tools list, the change is clean; if it does, users who install the Pi sub-agent will get a broken agent.

src/semble/agents/pi.md — the missing tools declaration is worth confirming against Pi's sub-agent spec before merging.

Reviews (1): Last reviewed commit: "Update test and docs" | Re-trigger Greptile

Comment thread src/semble/agents/pi.md
@Pringled Pringled merged commit 49bd6e2 into main Jun 5, 2026
16 checks passed
@Pringled Pringled deleted the add-reasonix branch June 5, 2026 04:45
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