Improve SKILL.md v1.1.0: token budgeting, model allocation, Mermaid, language-agnostic#2
Open
yelban wants to merge 1 commit intoblader:mainfrom
Open
Improve SKILL.md v1.1.0: token budgeting, model allocation, Mermaid, language-agnostic#2yelban wants to merge 1 commit intoblader:mainfrom
yelban wants to merge 1 commit intoblader:mainfrom
Conversation
…language-agnostic Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
d06c01a to
d12be3f
Compare
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
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.
Summary
Compared schematic with the Cartographer skill to identify engineering improvements while preserving schematic's core strengths (semantic file grouping, deep product inference, cross-validation).
Changes to SKILL.md
Phase 1 — Token budget estimation: Added
wc -cstep to estimate diff size, with tiered agent scaling strategy:Why: Previously no guidance on how many agents to spawn. Large branches wasted time with too few agents; small branches wasted cost with too many.
Phase 2 — Explicit model allocation: Specified
subagent_type: "Explore"withmodel: "sonnet"for file reading agents, with Opus reserved for orchestration (planning, synthesis, inference).Why: Sonnet offers the best cost/capability ratio for file analysis. Without explicit guidance, the skill might use Opus for all agents, increasing cost 5x with no quality benefit for file reading tasks.
Phase 3 — Language-agnostic file listing: Replaced hardcoded
git diff --stat -- '*.ts' '*.tsx' | awk '{print $1}'withgit diff --name-only, removing the TypeScript/React assumption.Why: The original command only caught
.ts/.tsxfiles during cross-check, silently missing changes in Python, Go, Rust, or any other language. This broke the completeness guarantee for non-TypeScript projects.Phase 4 — Mermaid diagrams: Changed architecture output instructions from ASCII diagrams to Mermaid (
graph TBfor system diagrams,sequenceDiagramfor data lifecycle).Why: Mermaid renders natively on GitHub/GitLab, producing cleaner, more maintainable diagrams. ASCII art is hard to edit and doesn't scale well for complex architectures.
What was intentionally NOT changed
Test plan
🤖 Generated with Claude Code