Skip to content

fix: Correct /prototype and /hybrid-prototype command docs (#29)#31

Merged
striderZA merged 5 commits intodevelopmentfrom
fix/command-docs
May 2, 2026
Merged

fix: Correct /prototype and /hybrid-prototype command docs (#29)#31
striderZA merged 5 commits intodevelopmentfrom
fix/command-docs

Conversation

@striderZA
Copy link
Copy Markdown
Owner

Summary

Fixes the misleading /prototype command description in AGENTS.md and adds /hybrid-prototype to the command index.

Changes

  • AGENTS.md: Changed /prototype description from "rapid prototype a concept (hybrid workflow)" to "rapid prototype a concept"
  • AGENTS.md: Added /hybrid-prototype to Getting Started section
  • AGENTS.md: Added Prototyping category to Available Commands list

Related

Closes #29

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 2, 2026

Review: fix/command-docsdevelopment

Verdict: APPROVED with minor concerns

Changes (3 insertions, 0 deletions):

  1. Added /prototype and /hybrid-prototype to the "Getting Started" section
  2. Added a "Prototyping" category to "Available Commands"

Issues Found

1. PR description inaccuracy
The PR body says it "Changed /prototype description from 'rapid prototype a concept (hybrid workflow)'" — but in the parent commit, /prototype didn't exist in the Getting Started section at all. The change was an addition, not a modification. The phrase "(hybrid workflow)" doesn't appear anywhere in the parent tree. The issue or description may be referencing a state that never existed in this branch.

2. Missing skill file for /hybrid-prototype
AGENTS.md:101 references /hybrid-prototype as a command, but .opencode/skills/hybrid-prototype/ does not exist on disk. The prototype/ skill exists, but there's no implementation for hybrid-prototype. Adding it to the docs creates a dead link that will confuse users who try to run it.

3. Description drift between skill and index
AGENTS.md says /prototype is "rapid prototype a concept", but the skill's own SKILL.md:3 describes itself as "Rapid prototyping workflow. Skips normal standards to quickly validate a game concept or mechanic. Produces throwaway code and a structured prototype report." The AGENTS.md descriptions are very abbreviated compared to the skill metadata — this is cosmetic but could be aligned.

Suggestions

Priority Action
High Either create .opencode/skills/hybrid-prototype/ before merging, or remove the /hybrid-prototype references from AGENTS.md
Low Update PR body to accurately describe the changes (additions, not modifications)
Low Consider aligning AGENTS.md descriptions with the skill frontmatter description fields so users get consistent info /-completion vs reading the doc

New%20session%20-%202026-05-02T20%3A12%3A09.970Z
opencode session  |  github run

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 2, 2026

Review: fix/command-docsdevelopment

Verdict: APPROVED with one concern

Changes (152 additions, 0 deletions):

  1. AGENTS.md — Added /prototype and /hybrid-prototype to Getting Started + new Prototyping section in Available Commands
  2. .opencode/skills/hybrid-prototype/SKILL.md — New 149-line skill definition file

Issues Found

1. Dangling reference to docs/hybrid-workflow.md (hybrid-prototype/SKILL.md:13)

This skill implements the **Discovery Phase fast lane** from `docs/hybrid-workflow.md`.

This file does not exist. The skill references a document that hasn't been created, creating a dead link.

2. PR body inaccuracy (cosmetic)
The PR body says /prototype was "changed from 'rapid prototype a concept (hybrid workflow)'" — but there's no prior state with that description. The line was added, not modified. Low impact since the actual code is correct.


Minor Observations

  • The SKILL.md is well-structured: proper YAML frontmatter, clear phase breakdown with time budgets, explicit constraints, and a useful comparison table against the full /prototype skill.
  • Agent references (creative-director, game-designer, prototyper, godot-specialist) all have corresponding agent definition files — no dead references there.
  • The AGENTS.md integration is clean and follows existing formatting conventions.

Recommendation

Fix the dangling reference before merging — either create docs/hybrid-workflow.md or remove the citation. Everything else is clean.

New%20session%20-%202026-05-02T20%3A20%3A35.666Z
opencode session  |  github run

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 2, 2026

Here's my review of fix/command-docsdevelopment:


Review Summary

Severity Count
🔴 Critical 1
🟡 Medium 1
🟢 Low 2

🔴 Critical: Duplicate /hybrid-prototype in skills-reference.md

.opencode/docs/skills-reference.md lists /hybrid-prototype in two sections:

  • Line 24 — under "Game Design"
  • Line 113 — under "Creative & Content" (alongside /prototype)

This will confuse users who see the command listed twice with different descriptions. Fix: remove the entry from one section. Since /prototype lives under "Creative & Content", /hybrid-prototype should logically live there too.


🟡 Medium: Duplicate hybrid-workflow.md files with drift

Two nearly identical files exist:

  • .opencode/docs/hybrid-workflow.md (the PR's file)
  • docs/hybrid-workflow.md (pre-existing)

They differ in 3 places:

  • .opencode/docs/ is missing the producer role note
  • Section title says /prototype fast lane vs /hybrid-prototype fast lane
  • Closing paragraph references /prototype vs /hybrid-prototype

Having parallel docs with divergent content is a maintenance trap. Either consolidate into one location or clearly document which one is canonical.


🟢 Low: PR body inaccuracy

The PR body says /prototype description was "changed from 'rapid prototype a concept (hybrid workflow)'" — but this line was added to AGENTS.md, not modified. No prior state with "(hybrid workflow)" exists in the branch history.


🟢 Low: Command count not updated

AGENTS.md lines 24 and 105 both say "72 slash commands". If this PR effectively adds /hybrid-prototype as a documented command (it's now in AGENTS.md Getting Started + Available Commands), the count should be 73.


Overall Verdict: Changes requested

The duplicate entry in skills-reference.md should be fixed before merging. The duplicate workflow docs should be consolidated or the relationship between them clarified.

New%20session%20-%202026-05-02T20%3A29%3A36.900Z
opencode session  |  github run

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 2, 2026

Review: fix/command-docsdevelopment

Verdict: APPROVED (minor concern remains)

What changed

The latest commit ee72b31 addressed nearly all issues from prior reviews:

  • Removed duplicate /hybrid-prototype entry from "Game Design" in skills-reference.md
  • Fixed dangling reference — SKILL.md now correctly points to .opencode/docs/hybrid-workflow.md
  • Added missing producer role note to .opencode/docs/hybrid-workflow.md (resolved the drift from docs/hybrid-workflow.md)
  • Removed stale "72" hardcoded count from AGENTS.md (now reads "all available commands")
  • Added docs/hybrid-workflow.md as a companion file (new, identical content structure)
  • Added opencode.json entry and prototypes/.gitkeep

Remaining concern

There's a naming drift between the two hybrid-workflow.md files:

Location Line 115 Line 179
.opencode/docs/hybrid-workflow.md The \/prototype` Fast Lane` The \/prototype` skill`
docs/hybrid-workflow.md The \/hybrid-prototype` Fast Lane` The \/hybrid-prototype` skill`

Since the entire premise of this PR is to distinguish /prototype from /hybrid-prototype, the .opencode/docs/ version should use /hybrid-prototype for consistency. The SKILL.md already references this file, so a reader following the link would see "/prototype Fast Lane" which conflicts with the skill name.

Also, having two nearly-identical copies of the same workflow document (docs/ vs .opencode/docs/) is a maintenance risk. Consider consolidating to one canonical location and symlinking or cross-referencing the other.

New%20session%20-%202026-05-02T20%3A33%3A25.312Z
opencode session  |  github run

@striderZA striderZA merged commit 681011c into development May 2, 2026
1 check passed
@striderZA striderZA deleted the fix/command-docs branch May 2, 2026 20:47
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