Record why Grok and Copilot CLI get no harness entry - #11
Conversation
Both were candidates for the harness table. Neither should be added, and the reasons are worth writing down before someone re-derives them wrongly. Grok already has engram's commands. It reads other vendors' directories on purpose: `[compat.claude]` in `~/.grok/config.toml` defaults every cell to `true`, so `~/.claude/commands/` and `~/.claude/skills/` are scanned as user-tier sources. `grok inspect` lists engram-context, engram-ingest and engram-save-chat as `user [claude]`, and `engram (stdio)` under MCP Servers — so engram's Claude Code install already serves Grok, and a Grok entry would write a second copy that Grok's own name-deduplication would shadow. Copilot CLI takes plugins, not files. `copilot plugin install` accepts a marketplace, a GitHub repository, or a git URL; there is no directory engram can drop a command into. Its previous one-line reason said only that there was no writable directory, which invited exactly the wrong follow-up: this machine has a `~/.copilot/skills` symlink, but it belongs to the user's own skills bundle rather than to Copilot, and reading it as a surface would have shipped commands nothing loads — the Codex prompts mistake again. Gates: fmt, clippy -D warnings, 257 tests, REUSE 3.3, makeinfo clean. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016i16R4GhdSffsboRYq97Fs
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8369b0d4eb
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| - **Grok needs no harness entry, and adding one would duplicate its commands.** | ||
| Grok reads other vendors' directories on purpose: `[compat.claude]` in | ||
| `~/.grok/config.toml` defaults every cell to `true`, so `~/.claude/commands/` | ||
| and `~/.claude/skills/` are scanned as user-tier sources, as are |
There was a problem hiding this comment.
Add a path for Grok-only installations
For a user who has Grok but not Claude Code, this premise fails: Harness::ALL has no Grok entry, while install skips Claude Code when its ~/.claude probe is absent, so nothing creates commands in any directory Grok scans. The cited grok inspect result only demonstrates a machine where Claude commands already exist. Add a detected Grok surface or otherwise arrange a Claude-compatible target; without one, Grok-only users cannot install these commands despite this paragraph saying no entry is needed.
AGENTS.md reference: AGENTS.md:L180-L180
Useful? React with 👍 / 👎.
Both were candidates for the harness table after the Kimi/Qwen survey. Neither should be added, and the reasons are worth writing down before someone re-derives them wrongly.
Grok already has engram's commands
Grok reads other vendors' directories on purpose. From its own bundled
docs/user-guide/08-skills.md:~/.grok/skills/,~/.grok/commands/~/.claude/skills/,~/.claude/commands/~/.cursor/skills/And
05-configuration.md: "Every cell defaults totrue."Verified non-interactively with
grok inspect:Engram's Claude Code install already serves Grok, on both surfaces. A Grok entry would write a second copy that Grok's own name-deduplication would shadow. Its own
~/.grok/skillsis a home-manager symlink into the store here anyway.Copilot CLI takes plugins, not files
There is no directory engram can drop a command into. The previous reason — "no user-writable command or prompt directory" — invited exactly the wrong follow-up: this machine does have
~/.copilot/skills, but it is a symlink belonging to the user's own skills bundle, not to Copilot. Reading it as a surface would have shipped commands nothing loads — the Codexprompts/mistake a second time. The detail now says what Copilot actually accepts.Also surveyed, no change warranted
ompon PATH is oh-my-posh 16.0.10.)~/.config/Code/User/mcp.json;prompts/exists but is empty. A real surface — but an editor, not a CLI harness. Scope decision, not a defect.skills-cursor/is vendor-shipped (Grok filters these as vendor defaults); no user~/.cursor/skills, no engram MCP registration.~/.cline/data/settingsdoes not exist — never configured. Its skills come fromnpx skills, not a directory.Gates
fmt·clippy -D warnings· 257 tests · REUSE 3.3 ·makeinfoclean🤖 Generated with Claude Code
https://claude.ai/code/session_016i16R4GhdSffsboRYq97Fs