Skip to content

refactor: .opencode/skill/ vs .opencode/skills/ path inconsistency #21

@yvonnedevlinrh

Description

@yvonnedevlinrh

Summary

divisor-curator.md and review-council.md use path-matching heuristics to classify files as user-facing or code files. Both reference .opencode/skill/ (singular) in their classification lists, but the scaffolded skills from replicator init are written to .opencode/skills/ (plural).
This means files under .opencode/skills/ (where the scaffold writes) do not match the heuristic patterns:

  • divisor-curator.md:82 - .opencode/skill/ used in user-facing path classification. Changes to scaffolded skills under .opencode/skills/ would not trigger documentation checks.
  • review-council.md:58 - .opencode/skill/ used in code-vs-spec file classification. Scaffolded skill changes could be misclassified as spec files instead of code files.
    Both .opencode/skill/ and .opencode/skills/ directories exist on disk:
  • .opencode/skill/ - contains speckit-workflow/SKILL.md (legacy, scaffolded by uf)
  • .opencode/skills/ - contains 4 openspec-related skill directories (scaffolded by replicator init)

To Reproduce

  1. Modify a file under .opencode/skills/ (e.g., .opencode/skills/openspec-propose/SKILL.md)
  2. Run the divisor-curator agent - the change will not match the user-facing path list at divisor-curator.md:82
  3. Run the review council - the change may be classified as a spec file rather than a code file per review-council.md:58

Expected behavior

Agent heuristics should match files under both .opencode/skill/ and .opencode/skills/, or the directories should be consolidated to a single canonical name (preferably skills/ plural, matching commands/ and agents/).

Suggested Fix

Either:

  1. Add the plural form to both heuristic lists so both directories are covered
  2. Consolidate .opencode/skill/ into .opencode/skills/ (preferred - aligns with the command/ - commands/ rename in PR refactor: migrate .opencode/command/ to .opencode/commands/ #20 and the existing agents/ plural convention)

Location

  • .opencode/agents/divisor-curator.md:82 - user-facing path classification
  • .opencode/commands/review-council.md:58 - code file classification
  • .opencode/skill/ - legacy singular directory
  • .opencode/skills/ - scaffold target directory
    Discovered during review of PR refactor: migrate .opencode/command/ to .opencode/commands/ #20.

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No fields configured for Bug.

Projects

Status
Done ✔️

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions