Skip to content

Fix: Replace .claude/ path references with .opencode/ in skills and docs#10

Merged
striderZA merged 3 commits intodevelopmentfrom
fix/claude-paths
May 1, 2026
Merged

Fix: Replace .claude/ path references with .opencode/ in skills and docs#10
striderZA merged 3 commits intodevelopmentfrom
fix/claude-paths

Conversation

@striderZA
Copy link
Copy Markdown
Owner

Closes #5

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 1, 2026

fatal: could not read Username for 'https://github.com': No such device or address

opencode session  |  github run

striderZA added 2 commits May 1, 2026 21:38
Changed permissions for contents from read to write and updated checkout action to persist credentials.
Changed persist-credentials to true and added GITHUB_TOKEN.
@striderZA striderZA merged commit e572f7b into development May 1, 2026
1 check passed
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 1, 2026

PR Review: fix/claude-pathsdevelopment

Summary

Find-and-replace PR updating .claude/ path references to .opencode/ across 55 files in .opencode/skills/ and .opencode/docs/. The changes are correct where applied.

Issues Found

1. 🐛 Functional bug: workflow-catalog.yaml:35 — missed functional path

glob: ".claude/docs/technical-preferences.md"  # should be .opencode/docs/

This is a functional reference (artifact detection glob), not just documentation. The /setup-engine workflow step will fail to find this file since it now lives under .opencode/. This file was not included in the PR.

2. ⚠️ Incomplete update: quick-start.md:265-283 — directory tree not updated
The entire file structure reference still shows .claude/ tree (settings.json, agents, skills, hooks, rules, docs). The file was in the PR's changed list, but this section was missed.

3. ⚠️ Minor: ccgs-hooks.ts:8 — comment still references .claude/hooks/

 * Translates the 12 bash hooks from .claude/hooks/ into TypeScript plugin hooks.

This is a harmless documentation-of-origin comment, but could be updated for consistency.

4. 📝 Root docs still reference .claude/ (out of scope but noted)

  • README.md:202 — directory tree shows .claude/docs/
  • README.md:60-66 — migration table shows .claude/ paths in "CCGS" column (intentional)

Verdict

Severity Count Details
🐛 Bug 1 workflow-catalog.yaml glob path will fail to match
⚠️ Incomplete 1 quick-start.md directory tree not updated
📝 Minor 1 ccgs-hooks.ts comment

Recommendation: Fix item #1 before merging — it's a functional path that can cause the engine-setup artifact detection to silently miss its target. Items #2 and #3 are cosmetic but should be fixed for consistency.

New%20session%20-%202026-05-01T19%3A38%3A47.275Z
opencode session  |  github run

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