Skip to content

Move the design-tree skill here from Whitefoot - #1

Merged
mbbill merged 2 commits into
mainfrom
initial-skill
Oct 6, 2026
Merged

mbbill merged 2 commits into
mainfrom
initial-skill

Conversation

@mbbill

@mbbill mbbill commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

Moves the design-tree skill out of Whitefoot so that every project takes it from one place, as a Git submodule.

What moves

  • SKILL.md, lint.py, test_lint.py: from Whitefoot cf6b84ff3 design/skill/, byte-identical to Snowghost-wf's copy.
  • review-base.sh: moved in from .github/design-review-base.sh, identical in both projects.

The skill sits at the repository root, so a project that mounts this repository at design/skill keeps its paths, its .claude/skills and .agents/skills links and its Makefile targets.

The only changes made in moving

  • test_lint.py finds lint.py and review-base.sh beside itself instead of at a project's design/skill/ and .github/ paths. Nine of its cases run the base script, so the script has to move with the tests. This settles Whitefoot's TODO item "The design-tree skill's tests also test this project's CI script".
  • review-base.sh: header comment rewritten for its new place, usage string renamed; logic unchanged.
  • SKILL.md: one sentence in Lint names review-base.sh.
  • New: the README, and .github/workflows/test.yml, which runs test_lint.py.

Validation

  • The test workflow on this PR (CI). Nothing was run locally.

Follow-ups

  • Whitefoot and Snowghost-wf switch to this repository as a submodule in their own Draft PRs, pinned to this PR's head until it merges, then to its commit on main.

🤖 Generated with Claude Code

Bai Ming and others added 2 commits October 5, 2026 22:46
SKILL.md, lint.py and test_lint.py come from Whitefoot cf6b84ff3, identical to
Snowghost's copy. The CI base script moves in from .github/design-review-base.sh
as review-base.sh, since nine of test_lint.py's cases run it, and test_lint.py
finds both scripts beside itself instead of at a project's paths.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
A project mounts this repository at design/skill, so its paths, links and
Makefile targets stay as they are.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@mbbill
mbbill marked this pull request as ready for review October 6, 2026 05:55
@mbbill
mbbill merged commit 2accfb8 into main Oct 6, 2026
2 checks passed
@mbbill
mbbill deleted the initial-skill branch October 6, 2026 06:02
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