Skip to content

chore: acknowledge reviewed skill-budget warning - #103

Merged
CodeSigils merged 2 commits into
mainfrom
docs/skill-budget-baseline
Sep 15, 2026
Merged

CodeSigils merged 2 commits into
mainfrom
docs/skill-budget-baseline

Conversation

@CodeSigils

Copy link
Copy Markdown
Owner

Adopts the designed advisory-baseline mechanism for the new skill-budget
cron-health warning rather than trimming the shipped payload.

What changed

  • advisory-baseline.json: reviewed 2026-09-15; records the 380-line
    SKILL.md warning as a known, accepted advisory.
  • proposals/ROADMAP.md: new 2026-09-15 entry documenting the decision.

Rationale

skills/skill-discovery/SKILL.md is 380 lines against the 350-line warning
threshold. The content is dense guardrail and reporting-contract prose — not
recoverable bloat — and is stable since the last payload edit (2026-09-09).
The 350-line threshold remains active for future regression detection; the
exact-string baseline still flags any growth beyond 380 lines.

Validation

UV_CACHE_DIR=/tmp/skill-discovery-uv-cache uv run python .github/scripts/validate-docs.py
# PASS: documentation and shipped payload are valid

UV_CACHE_DIR=/tmp/skill-discovery-uv-cache uv run python scripts/cron-health.py
# ℹ️ 1 known warning(s) suppressed by baseline, exit 0

UV_CACHE_DIR=/tmp/skill-discovery-uv-cache uv run python scripts/check-readme-tree.py
# PASS: all 45 files in README tree exist on disk, no unlisted files

git diff --check
# clean

What: Record the 380-line SKILL.md budget warning as a reviewed advisory with a 2026-09-15 review date.

Why: The weekly cron-health check surfaced a new skill-budget warning; the payload is dense guardrail prose at a stable size, so the current state is accepted via the repo's advisory-baseline mechanism while keeping the 350-line threshold active for future regression detection.
What: Add a 2026-09-15 dated entry to the implementation record describing the advisory-baseline decision.

Why: The roadmap maintenance discipline requires a concise dated entry for repository structure decisions.
@CodeSigils
CodeSigils merged commit bbfb7ac into main Sep 15, 2026
6 checks passed
@CodeSigils
CodeSigils deleted the docs/skill-budget-baseline branch September 15, 2026 08:37
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