chore: move agent skills to .claude/skills, drop Copilot, add workflow skills - #329
Merged
Merged
Conversation
Claude Code only auto-discovers skills under .claude/skills/<name>/SKILL.md; the flat .github/skills/*.md files were never listed and depended on the agent choosing to read them from a table. That is how the Docker-only rule for robot tests got missed in #328. - 19 flat files -> 14 skills with name/description frontmatter written as triggers. Merged: testing (robot + python + js, with Docker rule and CI diagnosis), dev-workflow, coding-standards. Folded architecture and gotchas into CLAUDE.md. - New skills: gh-issue (issue -> branch -> fix -> test -> review, commit gated on approval), add-db-column, add-cli-argument; add-graph checklist in dashboard-graphs. - Fixed stale content: script names (unittests/jstests), test paths, filter pipeline stages and tagMode, JS test coverage table, docs page map, example.sh, module tree, camelCase/snake_case contradiction. - CLAUDE.md: robot tests Docker-only with the reason (suites run the installed CLI, not the working tree); no Copilot references. - scripts/docker/run-in-container.{sh,bat}: only allocate a TTY when one exists so the wrappers work from CI and agent shells. - .gitignore: .claude/settings.local.json. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
Claude Code only auto-discovers project skills under
.claude/skills/<name>/SKILL.md. The flat.github/skills/*.mdfiles were never listed by the tool and depended on the agent choosing to read them from a table in CLAUDE.md — which is how the "robot tests run in Docker" rule got missed while working #327/#328.What
name+ trigger-styledescriptionfrontmatter.testing(robot + python + js references, Docker hard rule, CI diagnosis viagh run download),dev-workflow,coding-standards.project-architectureandconventions-and-gotchasfolded intoCLAUDE.md.gh-issue(fetch → branch → root cause → fix → regression test in Docker →/code-review→ report; commit only on explicit approval),add-cli-argument, plus an add-a-graph checklist indashboard-graphs.scripts\unittests.bat/jstests.bat, wrong test helper paths, filter pipeline stages (custom filters, suite path,tagModevsuseOrTags), JS unit-test coverage table, docs page map,example.sh,js-bundlingmodule tree, and the camelCase-vs-snake_case contradiction (functionssnake_case, variablescamelCase, per the actual code)..github/copilot-instructions.md,.github/skills/deleted).scripts/docker/run-in-container.shallocates a TTY only when stdin is one;.batskips-tunderCLAUDECODE/CI. Verified from a non-TTY shell..gitignore:.claude/settings.local.json.Not included
PreToolUsehook enforcing the Docker rule (discussed, deferred).CONTRIBUTING.mdstill documents local robot runs for human contributors.🤖 Generated with Claude Code