Repository navigation
Remove the Claude Code adapter - #19
Conversation
Delete the adapter file and the install paragraphs that point at it. Co-authored-by: Kaitian Xie <[email protected]>
caelaxie
left a comment
There was a problem hiding this comment.
verdict: changes requested (posted as a comment since author and reviewer share an account).
content is exactly the #14 smallest fix: adapter file gone, both claude code paragraphs gone, docs/harness/ empty, Closes #14 set, not a draft. two things before merge:
- each one-line delete leaves two blank lines in a row (see inline). drop one adjacent blank line in each file so the surrounding text reads as before.
- harness proof.
skills/poteto-help/SKILL.mdis skill content, so show at the final head sha:grok plugin install --trust+grok plugin list+grok plugin validate(orgrok inspect) and the installedpoteto-help/SKILL.mdwith the claude code paragraph gone. a green ci run that does validate + inspect at the final head also works for this text-only change; no check runs exist on 2751268 yet.
| enabled = ["pstack"] | ||
| ``` | ||
|
|
||
| In Claude Code, run `/plugin marketplace add cursor/plugins`, then `/plugin install pstack@cursor-plugins`. Claude Code prefixes plugin commands, so each command in this guide takes a `pstack:` prefix there, such as `/pstack:setup-pstack`, which writes its model rule to `~/.claude/rules/pstack-models.md`. The [Claude Code adapter](../harness/claude-code.md) covers the other differences. |
There was a problem hiding this comment.
deleting just this line leaves the blank line above and below it back to back. delete one of them too.
|
|
||
| This checkout is the Grok Build port. Workflow skills spawn with `spawn_subagent`. Only the parent chat can spawn. `/loop` is a Grok Build built-in. There is no custom mode to pin. Type `/poteto-mode` per task. | ||
|
|
||
| pstack is built for Cursor, and it also installs in Claude Code with `/plugin marketplace add cursor/plugins`, then `/plugin install pstack@cursor-plugins`. There, each command takes a `pstack:` prefix, such as `/pstack:poteto-mode`. The [Claude Code adapter](../../docs/harness/claude-code.md) maps the Cursor names, and a step that needs a Cursor-only feature, such as Bugbot or Automations, is skipped with a one-line note. |
There was a problem hiding this comment.
same here: lines 47 and 49 are both blank, so this leaves a double blank line between the grok port paragraph and ## Start a task. delete one.
Co-authored-by: Kaitian Xie <[email protected]>
|
Harness proof on head
{
"name": "poteto-help",
"description": "Guides users through pstack setup, /poteto-mode, and picking the skill, playbook, or principle for a task. Type /poteto-help with a question.",
"source": {
"type": "plugin",
"plugin_name": "pstack",
"path": "/tmp/grok-pr19-m91eq2im/.grok/installed-plugins/grok-pstack-619a4a2c/skills/poteto-help/SKILL.md"
},
"userInvocable": true
}Installed Installed CI on this sha: https://github.com/pstack-ports/grok-pstack/actions/runs/39532457549 (Validate plugin, success). |
caelaxie
left a comment
There was a problem hiding this comment.
verdict: passes (comment review, same account). double blank lines are gone in docs/guide/01-setup.md and skills/poteto-help/SKILL.md; diff is now exactly #14's smallest fix. proof on head 3ef144e: grok 1.0.62 validate + install --trust + list, grok inspect loads poteto-help from the install, installed L44-48 show one blank line and no claude code paragraph. validate plugin ci green. squash-merging.
Removes the Cursor-to-Claude Code adapter from this grok-only port.
docs/harness/claude-code.mdand the emptydocs/harness/directory.docs/guide/01-setup.md.skills/poteto-help/SKILL.md. The grok port paragraph above it stays.CI does not reference the deleted file. It checks
plugin.json, skill and agent frontmatter, the poteto-mode index, andgrok plugin validate/grok inspect.Closes #14