Docs/issue 11 host compatibility - #23
Conversation
Add an agent-hosts reference citing current official documentation for Pi, Claude Code, Codex, Gemini CLI, and OpenCode: project and user skill locations, session-resume entry points, and the shared Agent Skills specification. Keep Pi JSONL resume explicitly Pi-specific, mark unverified operational behavior as unverified, and distinguish Herdr process persistence from CLI-owned conversation recovery. Link the reference from the canonical SKILL.md and README. Refs egdev6#11
|
@aam9063 Found two documentation issues worth correcting before merging:
Verification of head |
Per review: document Pi's .pi/skills/ project root and ~/.pi/agent/skills/ default user agent directory, and clarify that skills/ and pi.skills describe package discovery rather than a general project skill root. Present OpenCode --fork as a modifier of --continue/--session instead of an independent resume entry point.
|
One minor correction in Reviewed commit |
Per review: ~/.agents/skills/ is a user-level location; keep only .pi/skills/ and the .agents/skills/ project root in the project column.
|
Reviewed commit Verification on an exact Linux snapshot passed: One non-blocking documentation note: the checklist in |
Per review note: the checklist now states that npm test and pack:check pass on the maintainer's Linux snapshot of 66df0e0, while local Windows verification is limited to pack:check plus the validator checks outside the pre-existing spawnSync npm limitation; details remain in Evidence.
|
Ready to merge! Good job @aam9063 ! <3 |
What
Add
skills/mobile-agent-orchestrator/references/agent-hosts.md, an evidence-backed host compatibility reference covering Pi, Claude Code, Codex CLI, Gemini CLI, and OpenCode:SKILL.mdReferences section andREADME.md.Verification
npm run pack:check: exit 0; tarball contains the expected 8 entries includingagent-hosts.md.node scripts/validate-package.mjs: all checks pass except the tarball comparison step, which reportsspawnSync npm ENOENTa pre-existing Windows-only limitation (see task doc evidence).node --test: 13/13 pure-logic tests pass. The 2 CLI-spawn tests fail locally on Windows only, due to a pre-existingdetector.pathnamepath bug that reproduces on unmodifiedmain(expected output verified manually; details inodd/tasks/issue-11-host-compatibility-docs.md).Task/evidence document:
odd/tasks/issue-11-host-compatibility-docs.md.