Skip to content

Chore: Clean deprecated Claude Code references from .opencode/docs/ - #25

Merged
striderZA merged 4 commits into
developmentfrom
chore/docs-cleanup
May 2, 2026
Merged

Chore: Clean deprecated Claude Code references from .opencode/docs/#25
striderZA merged 4 commits into
developmentfrom
chore/docs-cleanup

Conversation

@striderZA

Copy link
Copy Markdown
Owner

Closes #24

Deleted

  • CLAUDE-local-template.md (Claude Code specific, no OpenCode equivalent)
  • settings-local-template.md (Claude Code permission modes)

Rewritten for OpenCode

  • setup-requirements.md — OpenCode CLI, Node.js dep, editor setup
  • hooks-reference.md — TS plugin table replacing bash hooks
  • context-management.md — OpenCode session terminology

Fixed references

  • quick-start.md: Claude Code → OpenCode
  • coordination-rules.md: Claude Code session → OpenCode session
  • director-gates.md: AskUserQuestion → question
  • templates/ (3 files): AskUserQuestion → question, Write/Edit → write and edit

@github-actions

github-actions Bot commented May 2, 2026

Copy link
Copy Markdown
Contributor

PR Review: chore/docs-cleanup

Overall verdict: Clean, well-scoped, and correct. Approve with one minor nit.

What's good

  • Consistent rename sweep — All 11 files updated in lockstep, no partial changes
  • Correct tool namesAskUserQuestionquestion matches OpenCode's API
  • Correct Write/Edit wording — Templates now say write and edit instead of tool names
  • Clean deletions — Both deleted files are Claude Code–specific with no OpenCode equivalent
  • hooks-reference.md — Thorough rewrite from bash hooks table to TS plugin table with test commands

One concern

coordination-rules.md:49 — still references the Claude Code env var:

Requires CLAUDE_CODE_EXPERIMENTAL_AGENT_TEAMS=1 environment variable.

Should be updated to reference whatever OpenCode's equivalent is (or dropped if not applicable). Minor — the section is labeled "experimental — opt-in" so it's plausible there's no direct OpenCode equivalent yet.

Two documentation nits (not blocking)

  1. setup-requirements.md — Linux apt install nodejs npm may install an older distro version. Consider adding a note about nodesource.com for Node 18+, or a version check command.

  2. hooks-reference.md — Test paths like node .opencode/plugins/tests/test-validate-commit.mjs are referenced but no test files exist in-diff. Verify the test files exist on the target branch or add a note to generate them.

Summary

Category Count
Correct renames 16
Tool name fixes (AskUserQuestionquestion) 10
Tool name fixes (Write/Editwrite and edit) 3
Leftover Claude refs 1
Missed files 0

New%20session%20-%202026-05-02T11%3A35%3A12.500Z
opencode session  |  github run

@github-actions

github-actions Bot commented May 2, 2026

Copy link
Copy Markdown
Contributor

PR Review: chore/docs-cleanup

Verdict: Approved with one fix required


✅ What's correct

  • Complete sweep — All 11 files updated, no partial renames
  • Tool name fixesAskUserQuestionquestion (10 occurrences across 4 files), Write/Editwrite and edit (3 occurrences in protocol templates)
  • Deleted files are purely Claude Code–specific with no OpenCode equivalent
  • hooks-reference.md — Thorough rewrite from bash hooks table to TS plugin event table
  • Test files existnode .opencode/plugins/tests/test-validate-commit.mjs and 10 other test files are present on disk (the earlier comment's concern is resolved)

🔴 Must fix (1 item)

.opencode/docs/coordination-rules.md:51 — This is the only remaining Claude Code reference in the docs directory:

Requires `CLAUDE_CODE_EXPERIMENTAL_AGENT_TEAMS=1` environment variable.

The Agent Teams section is labeled as experimental/opt-in. Either:

  • Replace with OpenCode's equivalent env var (if one exists)
  • Or drop the line entirely if OpenCode doesn't have agent teams yet

💡 Suggestions (non-blocking)

  1. setup-requirements.md:38apt install nodejs npm may install an older version (e.g., Node 12 on Ubuntu 20.04). Consider adding a note about using nodesource.com for Node 18+, or a node --version check in the "Verifying Your Setup" section.

  2. quick-start.md:292 — The directory listing references settings-local-template.md which was deleted in this PR. This stale reference should be cleaned up.

Summary

Category Count
Correct AskUserQuestionquestion 10
Correct Write/Editwrite and edit 3
Leftover CLAUDE_CODE_* ref 1
Stale file listing in quick-start.md 1

New%20session%20-%202026-05-02T11%3A44%3A18.271Z
opencode session  |  github run

@github-actions

github-actions Bot commented May 2, 2026

Copy link
Copy Markdown
Contributor

I've reviewed the full diff (4 commits, 11 files, +85/-186). Here's my analysis:

Verdict: Approve with one remaining issue

✅ What's done well

  • Complete name sweep — All AskUserQuestionquestion (10 occurrences across 4 protocol files), Write/Editwrite and edit (3 occurrences), Claude Code sessionOpenCode session (2 files)
  • hooks-reference.md is a thorough rewrite — from a flat bash hooks table to a clean TS plugin event table with test commands
  • setup-requirements.md correctly replaces jq/Python/Bash dep with Node.js 18+, reflecting the CCGS → TypeScript plugin migration
  • Deleted files are correct — both are Claude Code–specific with no OpenCode equivalent

🔴 One remaining issue

quick-start.md:292 — The directory listing still references the deleted settings-local-template.md:

    settings-local-template.md     -- Personal settings.local.json guide

This line should be removed since the file was deleted in commit 78a7ab0. The stale reference will confuse users trying to navigate the project.

✅ Other concerns from previous reviews are resolved

  • The CLAUDE_CODE_EXPERIMENTAL_AGENT_TEAMS=1 ref in coordination-rules.md was fixed in commit 16e1d88
  • The hooks test files exist on disk (verified by the second reviewer)

Fix suggestion: Remove line 292 from quick-start.md.

New%20session%20-%202026-05-02T11%3A47%3A35.151Z
opencode session  |  github run

@striderZA striderZA added this to the v0.3.1 milestone May 2, 2026
@striderZA
striderZA merged commit 1c0402a into development May 2, 2026
1 check passed
@striderZA
striderZA deleted the chore/docs-cleanup branch May 2, 2026 20:48
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.

1 participant