Skip to content

Fix cursor, codex, claude installer output#700

Open
acunniffe wants to merge 5 commits intomainfrom
fix/codex-reporting-installed
Open

Fix cursor, codex, claude installer output#700
acunniffe wants to merge 5 commits intomainfrom
fix/codex-reporting-installed

Conversation

@acunniffe
Copy link
Collaborator

@acunniffe acunniffe commented Mar 15, 2026

While building some background agent containers I realized that cursor, codex and claude were reporting themselves as installed, even on empty containers

This was caused but the skills installer running before the hooks installer and creating ~/.{agent} folders.

This PR changes the order and ensures skill repositories are only setup if the agent is installed.


Open with Devin

Copy link
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 4 additional findings.

Open in Devin Review

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