Skip to content

Docs/issue 11 host compatibility - #23

Merged
egdev6 merged 7 commits into
egdev6:mainfrom
aam9063:docs/issue-11-host-compatibility
Oct 1, 2026
Merged

egdev6 merged 7 commits into
egdev6:mainfrom
aam9063:docs/issue-11-host-compatibility

Conversation

@aam9063

@aam9063 aam9063 commented Sep 30, 2026

Copy link
Copy Markdown
Contributor

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:

  • Documented project and user skill locations for each host, sourced from current official documentation (accessed 2026-09-30).
  • Documented session-resume entry points per host, keeping Pi JSONL resume behavior explicitly Pi-specific.
  • Marked unverified operational behavior (including Herdr integration with CLI resume flows) as unverified rather than inferred.
  • Distinguished Herdr process persistence from CLI-owned agent conversation recovery.
  • Linked the reference from the canonical SKILL.md References section and README.md.

Verification

  • npm run pack:check: exit 0; tarball contains the expected 8 entries including agent-hosts.md.
  • node scripts/validate-package.mjs: all checks pass except the tarball comparison step, which reports spawnSync npm ENOENT a 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-existing detector.pathname path bug that reproduces on unmodified main (expected output verified manually; details in odd/tasks/issue-11-host-compatibility-docs.md).

Task/evidence document: odd/tasks/issue-11-host-compatibility-docs.md.

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
@egdev6

egdev6 commented Sep 30, 2026 •

Copy link
Copy Markdown
Owner

@aam9063 Found two documentation issues worth correcting before merging:

  1. Pi skill locations (skills/mobile-agent-orchestrator/references/agent-hosts.md:11): please add .pi/skills/ for project scope and ~/.pi/agent/skills/ for user scope (the default agent directory). Pi v0.99.1 documents both in docs/configuration.md. Also clarify that skills/ and pi.skills describe package discovery, not a general project skill root.
  2. OpenCode resume (skills/mobile-agent-orchestrator/references/agent-hosts.md:25): --fork is not an independent resume entry point. The official CLI reference requires combining it with --continue or --session. Use opencode --continue --fork or opencode --session <id> --fork instead of listing it as an alternative.

Verification of head 2915d14 in a temporary Linux snapshot passed: npm test (package validation and 15/15 tests) and npm run pack:check (8 entries, including the new reference). These checks do not establish end-to-end operational compatibility for the hosts.

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.
@egdev6

egdev6 commented Sep 30, 2026

Copy link
Copy Markdown
Owner

One minor correction in skills/mobile-agent-orchestrator/references/agent-hosts.md:11: the Pi Project scope cell includes ~/.agents/skills/, but that is a user-level location. Please remove it from the project column and keep it in User scope, where it is already correctly listed. This avoids presenting a global installation as project-local.

Reviewed commit bb2e06d: npm test passed (package validation and 15/15 tests), and npm run pack:check passed with the expected 8 entries. The documented skill locations and resume commands were checked against official documentation. No blocking findings; I did not perform end-to-end verification of each CLI or reproduce the reported Windows failures.

Per review: ~/.agents/skills/ is a user-level location; keep only
.pi/skills/ and the .agents/skills/ project root in the project column.
@egdev6

egdev6 commented Oct 1, 2026

Copy link
Copy Markdown
Owner

Reviewed commit 66df0e0: no blocking findings. The documented skill locations and resume commands match the official documentation, and the corrections from the earlier reviews are incorporated.

Verification on an exact Linux snapshot passed: npm test (package validation and 15/15 tests) and npm run pack:check (8 entries, including the new reference). I did not perform end-to-end verification of each CLI or reproduce the reported Windows failures.

One non-blocking documentation note: the checklist in odd/tasks/issue-11-host-compatibility-docs.md:21 marks npm test as passing, while the evidence below reports pre-existing Windows failures. Please distinguish the successful Linux verification from those Windows limitations so the checklist and evidence are consistent.

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.
@egdev6

egdev6 commented Oct 1, 2026

Copy link
Copy Markdown
Owner

Ready to merge! Good job @aam9063 ! <3

@egdev6
egdev6 merged commit 9183524 into egdev6:main Oct 1, 2026
1 check passed
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