chore: no-issue: add Claude Code skills and top-level specs/ scaffold#10131
chore: no-issue: add Claude Code skills and top-level specs/ scaffold#10131tcaiger wants to merge 6 commits into
Conversation
Adds the spec/design/plan/test/audit skill cluster under .claude/skills plus shared convention docs, ported and adapted from the workhorse repo. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Want higher recall? High effort reviews run extra passes and find more bugs. A team admin can switch effort levels in the Cursor dashboard.
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit fe62a58. Configure here.
|
|
||
| Ask focused questions — one or two at a time, not long lists. **Number your questions** so the user can reply by number. | ||
|
|
||
| For UI-heavy features, proactively create an HTML mockup when a visual would help. Read the relevant existing implementation in the repo first so it's grounded in what exists. |
There was a problem hiding this comment.
Interview mockups skip design rules
Medium Severity
The interview skill creates HTML mockups but only says to read existing UI code. It omits design-sourcing.md, so mockups may miss the required path, inline format, and “preserve unchanged aspects” rules used elsewhere.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit fe62a58. Configure here.
|
🦸 Review Hero Summary No issues found. Looks good! |
Specs live in a top-level specs/<area>/ tree (e.g. specs/invoicing), not docs/specs/. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…olders Seeds specs/<area>/overview.md placeholders for Tamanu's top-level product/platform domains, each with a one-sentence summary. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
🦸 Review Hero Summary No issues found. Looks good! |
Can I suggest using |
tcaiger
force-pushed
the
chore/workhorse-skills
branch
from
1400160 to
9b4f899
Compare
Relocates the skill cluster to the tool-agnostic .agents/skills directory and adds a .claude/skills symlink so Claude Code still discovers them. Internal cross-references updated to .agents/skills for tool-neutrality. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
tcaiger
force-pushed
the
chore/workhorse-skills
branch
from
9b4f899 to
80f6ce8
Compare
tcaiger
changed the title
2 participants
Changes
Adds a cluster of Claude Code skills under
.agents/skills/, ported and adapted from the workhorse repo, plus a top-levelspecs/directory the skills write to. The skills sit in the tool-agnostic.agents/directory, with a.claude/skillssymlink so Claude Code discovers them. Markdown only — no application code, no runtime impact.Skills — covering the spec → design → plan → test → implement → audit lifecycle:
interview,draft-spec-changes,spec-review,workshop,mock-this-up,ascii-designtech-design,plan-implementation,plan-cardsdraft-test-cases,automate-test-casesimplement,investigate-and-fixbug-review,design-audit,security-audit,acceptance-auditPlus shared convention docs under
.agents/skills/shared/(spec-format,plan-format,test-cases-format,card-plan-format,design-sourcing) that the skills reference. Everything is adapted to Tamanu — Linear tickets,mainas the base branch, and the healthcare security rules fromllm/project-rules/.Specs location — the skills read and write specs in a new top-level
specs/<area>/tree (e.g.specs/invoicing), notdocs/specs/. This PR scaffolds that directory withoverview.mdplaceholders (a one-sentence summary each) for the obvious top-level domains: administration, encounters, fhir, imaging, immunisation, invoicing, labs, medication, patient, program-registry, programs, referrals, reports, scheduling, sync, vitals.