feat(skills): add devin platform target#630
Merged
Merged
Conversation
Add Devin (Cognition's coding agent) as a supported platform for `pup skills install`. Devin reads skills from `.agents/skills/` (project-local) and `~/.agents/skills/` (user-global); like cursor and codex it uses `SKILL.md` files, not Claude-style `<name>.md` subagents. - Register `devin` in PLATFORMS and add the SkillsPlatform::Devin variant (+ as_canonical arm), kept in sync by platform_enum_matches_table - Auto-detect Devin sessions via DEVIN_SESSION_ID. Because that variable is a per-session id rather than a truthy flag, detect it by presence of a non-empty value, without changing the existing truthy detector table - Surface `devin` in the `skills` --help text, the resolve error hints, the README Agent Skills section, and docs/COMMANDS.md - Add positive and negative unit tests: lookup, install paths for both scopes, SKILL.md (not subagent) format, and env detection Generated with [Devin](https://devin.ai) Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
samfert
force-pushed
the
feat/devin-skills-platform
branch
from
2f18455 to
b333373
Compare
platinummonkey
approved these changes
Jul 2, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Add Devin (Cognition's coding agent) as a supported platform for
pup skills install, so pup's skills and domain agents install into Devin's skills directories.Changes
devinin the platform registry (PLATFORMS) and the CLISkillsPlatformenum +as_canonicalarm insrc/skills.rs. Both project- and user-scope skills resolve to.agents/skills(i.e.<repo>/.agents/skills/and~/.agents/skills/). Modeled on thecursor/codexentries: no aliases, no extensions,uses_agent_md: false— agents install asSKILL.md, not Claude-style<name>.mdsubagents.devineverywhere the supported platforms are surfaced to users: theskills--helptext (src/main.rs), the resolve-error hints (src/commands/skills.rs), the README "Agent Skills" section (incl. install paths), anddocs/COMMANDS.md.Testing
cargo test— added positive + negative unit tests:lookup_platform("devin"),skills_dirfor both scopes, and theSKILL.md-not-subagent format for agent entries. The existingplatform_enum_matches_tablepasses with the new variant. (The only failures in the full suite are 5 pre-existing, network/DNS-dependent tests unrelated to this change, verified identical on cleanmain.)cargo clippy --all-targets -- -D warnings— clean.cargo fmt --check— clean.pup skills install devin --projectwrites 59SKILL.mdfiles under<repo>/.agents/skills/andpup skills install devinwrites them under~/.agents/skills/— zero<name>.mdfiles in both. Verifiedpup skills path devinfor user and--projectscopes.Related Issues
N/A
Generated with Devin