Skip to content

docs(skills): bring matrix-communication and matrix-announcement under the word cap - #119

Merged
CybotTM merged 2 commits into
mainfrom
chore/skill-md-word-cap
Aug 19, 2026
Merged

docs(skills): bring matrix-communication and matrix-announcement under the word cap#119
CybotTM merged 2 commits into
mainfrom
chore/skill-md-word-cap

Conversation

@CybotTM

@CybotTM CybotTM commented Aug 19, 2026

Copy link
Copy Markdown
Member

Closes #118.

Both files were over the 500-word SKILL.md cap and had been for a long time. CI never said so, because validate-skill.sh stopped at the first SKILL.md it found and this repo ships three — fixed in netresearch/skill-repo-skill#241, which is why the failure appears now.

skill before after
matrix-communication 1348 496
matrix-announcement 624 492
matrix-administration 498 498 (untouched)

The cap is on what loads with the skill, not on what the skill knows, so almost nothing is deleted — the overflow moves into references/, which has no cap.

matrix-communication

Three new references take the bulk:

  • command-reference.md — every script and flag, the E2EE/non-E2EE fallback table, the config keys. SKILL.md keeps the seven commands that appear in almost every session.
  • troubleshooting.md — the error table and the mistakes that cost the most time.
  • agent-governance.md — who may change the agent's function, and reading a room log as events rather than a story. Both are retro-born and both keep their burned-case paragraph, which is the part that makes a rule stick.

What deliberately stays in SKILL.md, because being one file away would cost more than the words:

  • the ⛔ access-token warning — the one mistake in this skill that damages something outside it, silently, with nothing failing at the moment you paste;
  • the governance rule in short form, since it applies to messages the agent is about to send;
  • the M_UNKNOWN_TOKEN / Room not found pair, because those two errors look alike and mean different things (dead token vs dead E2EE credential).

matrix-announcement

Rule 4's naming example moves to html-subset.md, which owns code wrapping; the findings-report heading rule and the sending recipe move to structure.md; the pre-send checklist merges three items that only pointed back at rules 3–5. The five rules, the type tags, the glyph set and the linking checklist item — the operative core — stay.

Checked

  • Every script name in the old SKILL.md still appears somewhere in the skill — extracted the set and grepped each, no misses.
  • validate-skill.sh (the merged multi-skill version): all three skills under the cap, 0 errors.
  • markdownlint-cli2 clean across all 19 touched files.
  • CLAUDE.md repointed at the moved sections — it linked two SKILL.md headings that no longer exist, and now lists the three new references.

matrix-communication lands at 496 rather than at 499 on purpose: the next person to add a sentence should not be blocked by the commit that made room.

Assisted by claude-code:claude-opus-5 — Session

…r the cap

Both were over the 500-word SKILL.md cap and had been for a long time -- CI
never said so, because validate-skill.sh stopped at the first SKILL.md it found
and this repo ships three (netresearch/skill-repo-skill#214, fixed and merged).
matrix-communication measured 1348 words, matrix-announcement 624.

The cap is on what loads with the skill, not on what the skill knows, so almost
nothing is deleted: the overflow moves into references/, which has no cap.

matrix-communication 1348 -> 496:
  - references/command-reference.md (new) -- every script and flag, the E2EE
    fallback table, the config keys. SKILL.md keeps the seven commands that
    appear in almost every session.
  - references/troubleshooting.md (new) -- the error table and the mistakes that
    cost the most time.
  - references/agent-governance.md (new) -- who may change the agent's function,
    and reading a room log as events rather than a story. Both are retro-born
    and both keep their burned-case paragraph, which is what makes them stick.

What deliberately stays in SKILL.md: the access-token warning (the one mistake
here that damages something outside this skill), the governance rule in short
form, and the M_UNKNOWN_TOKEN / "Room not found" pair, because those two errors
look alike and mean different things.

matrix-announcement 624 -> 492: rule 4's naming example moves to
html-subset.md, which owns code wrapping; the findings-report heading rule and
the sending recipe move to structure.md; the checklist merges three items that
only pointed back at rules 3-5.

CLAUDE.md now points at the moved sections rather than at SKILL.md headings that
no longer exist.

Every script name in the old SKILL.md still appears somewhere in the skill --
checked by extracting the set and grepping each. Validator: all three skills
under the cap, 0 errors. markdownlint clean over all 19 touched files.

Refs #118.

Assisted-by: claude-code:claude-opus-5
Agent-Session: https://claude.ai/code/session_01CT41JfSGYEJJaBUZxg7xzu
Agent-Host: 32116e
Signed-off-by: Sebastian Mendel <github@sebastianmendel.de>
Copilot AI lite review requested due to automatic review settings August 19, 2026 14:14
@github-actions github-actions Bot added documentation Improvements or additions to documentation skill labels Aug 19, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Listing the three new references one per line pushed AGENTS.md to 151 lines,
and harness-verify fails at 150 -- the file is meant to be an index, not
documentation. The same links now sit on two lines.

Assisted-by: claude-code:claude-opus-5
Agent-Session: https://claude.ai/code/session_01CT41JfSGYEJJaBUZxg7xzu
Agent-Host: 32116e
Signed-off-by: Sebastian Mendel <github@sebastianmendel.de>
@sonarqubecloud

Copy link
Copy Markdown

@CybotTM

CybotTM commented Aug 19, 2026

Copy link
Copy Markdown
Member Author

Self-review — Copilot has no review quota left this month.

A 1348 → 496 cut is where this kind of change goes wrong, so the check that matters is what left the file. Nothing did: every script name in the old SKILL.md still appears somewhere in the skill — extracted the set and grepped each, no misses. The three new references hold the material verbatim rather than paraphrased, so the commands, flags and the two burned-case paragraphs read the same as before.

What stayed in SKILL.md was chosen by one question — does being one file away cost more than the words? The access-token warning stayed because it is the only mistake here that damages something outside the skill, silently, with nothing failing at the moment you paste. The governance rule stayed because it applies to messages the agent is about to send. The M_UNKNOWN_TOKEN / Room not found pair stayed because the two look alike and mean different things.

One thing this PR found on the way: adding three reference lines to AGENTS.md pushed it to 151 lines and harness-verify fails at 150. That is the check doing its job — the file is an index, not documentation — so the links now sit on two lines and it is back to 146.

21 checks pass, 0 unresolved threads, mergeState=CLEAN.

Assisted by claude-code:claude-opus-5 — Session

@CybotTM
CybotTM merged commit bcce630 into main Aug 19, 2026
22 checks passed
@CybotTM
CybotTM deleted the chore/skill-md-word-cap branch August 19, 2026 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation skill

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Two SKILL.md files are over the 500-word cap — CI now sees them

2 participants