perf(repo-health): trim the family's skills without changing behaviour - #1467
Conversation
Kaap10
left a comment
There was a problem hiding this comment.
LGTM on the optimization pass! All quoted routing triggers are strictly preserved, frontmatters/bodies sit well within budgets, and static checks pass.
Minor nit:
In dependency-license-audit/ (audit-tool-setup.md, license-normalization.md, and scan-commands.md), the # <Title> top-level heading is immediately repeated as a ## <Title> sub-heading below the companion preamble. You can drop the duplicate ## <Title> for cleaner markdown structure.
Also, please rebase on latest main to resolve the merge conflict so this can be merged.
Apply the apache#1342 setup-family recipe to the repo-health family (apache#1349): - Always-on frontmatter (description / when_to_use) tightened on the five skills over the 200-token budget; every quoted routing phrase kept. - dependency-license-audit: tool-setup, scan-command, and license- normalization sections moved byte-identically into sibling files with one-line pointers (5244 -> 3453 measured tokens; 523 -> 315 lines). - audit-finding-fix: Step 6 and Step 8 moved byte-identically into compose-commit.md / draft-pr-procedure.md / pre-pr-adversarial-review.md (6170 -> 4663 measured tokens; 591 -> 461 lines). - measured_tokens re-stamped via skill-token-count; surface_hash re-stamped where structural anchors moved; family total 27979 -> 24554 tokens. Generated-by: ZCode (GLM, coding agent) Authored-by: liwenjie200543 <liwenjie200543@users.noreply.github.com>
…on figure - dependency-license-audit siblings ended with a stray blank line (end-of-file-fixer). - draft-pr-procedure.md had consecutive blank lines (markdownlint MD012). - docs/setup/marketplace.md published ~0.7k always-on tokens for magpie-repo-health; the tightened frontmatter measures ~0.6k. Generated-by: ZCode (GLM, coding agent) Authored-by: liwenjie200543 <liwenjie200543@users.noreply.github.com>
69b5260 to
4e29a59
Compare
|
Both done: the duplicated |
…dependency-license-audit companions
4e29a59 to
d311158
Compare
Apply the #1342 setup-family recipe to the repo-health family (#1349),
as previously applied to setup (#1353), pr-management (#1390), and issue
(#1400).
What changed
Always-on frontmatter — the five skills over the 200-token budget
(
dependency-license-audit~256,audit-finding-fix~211,license-compliance-audit~214,flaky-test-triage~210,ci-runner-audit~207,workflow-security-audit~204) had theirdescription/when_to_usetightened; every quoted routing phrase iskept.
Bodies — the two skills over the 5,000-token / 500-line budgets moved
sub-action-specific sections byte-for-byte into sibling files, leaving
one-line pointers:
dependency-license-auditaudit-tool-setup.md,scan-commands.md,license-normalization.mdaudit-finding-fixcompose-commit.md,draft-pr-procedure.md,pre-pr-adversarial-review.mdThe moved sections are byte-identical to the text on
main(verified byscript, not by eye). Family total: 27979 → 24475 measured tokens (−12.5%).
All seven skills now sit inside the 5,000-token body budget, the 500-line
cap, and the 200-token always-on budget.
measured_tokenswas re-stamped viaskill-token-count;surface_hashwas re-stamped only where structural anchors moved between files
(
dependency-license-audit) — the other five keep their hashes.How checked
tools/dev/estimate-skill-tokens.py --check: OK (published figuresmatch the live frontmatter).
check-placeholders.sh,check-duplication.py,check-companion-skills.py: OK.skill-evalssuites foraudit-finding-fixanddependency-license-auditneed the--climodel runner, so they areleft to CI rather than this offline environment.
check-skill-config.pyflags pre-existingrequires_configtabledrift in
docs/{security,setup,utilities}/README.md— present onmainbefore this PR and untouched by it.Gen-AI disclosure
Authored with ZCode (GLM) under
docs/ai-contribution-policy.md;
every line reviewed and the checks above run locally by me before posting.