Skip to content

docs(references): point no-editorializing at its canonical copy - #159

Merged
CybotTM merged 1 commit into
mainfrom
chore/no-editorializing-pointer
Aug 19, 2026
Merged

docs(references): point no-editorializing at its canonical copy#159
CybotTM merged 1 commit into
mainfrom
chore/no-editorializing-pointer

Conversation

@CybotTM

@CybotTM CybotTM commented Aug 19, 2026

Copy link
Copy Markdown
Member

Part of the A23 generic-bloat sweep (netresearch/skill-repo-skill#157): one canonical home for no-editorializing.md, copies become pointers.

Measured before cutting, since the issue's own list is marked unverified. Across the skill repos checked out locally the file exists in five variants:

sha (12) lines repos
3e256d0bd498 102 git-workflow — the canonical
faeaeb38ccfe 41 github-project, github-release, netresearch-jira (internal)
a44643f0ea25 43 german-technical-writing
02cf1f04dc6a 3 matrix, netresearch-branding, nr-gitlab — already pointers

diff between the 41-line copy and the canonical reports additions only (41a42,102): this file was a strict prefix, the same rule minus the line-wrapping section the canonical has been gaining. So the pointer loses no rule and gains that section.

german-technical-writing is deliberately left alone — it is not a copy. Its scope paragraph limits the rule to the German artifacts that skill governs and explicitly excludes the English ones, which is a different statement from the canonical's "applies to every written artifact".

Nothing breaks offline: SKILL.md already states the rule in a sentence where it is used, and the reference file is the long form. The pointer is byte-identical to the two that already exist in the catalog, so the convention stays one shape rather than two.

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

The rule lived in five repos as three different texts. This copy was a strict
prefix of the canonical one in git-workflow-skill -- `diff` reports additions
only, no divergence -- so it was the same rule minus the line-wrapping section
that copy has been gaining since.

matrix-skill and netresearch-branding already carry the pointer; this brings the
convention to the last two public repos holding the full copy (A23,
netresearch/skill-repo-skill#157). german-technical-writing keeps its own file:
that one is not a copy, its scope paragraph deliberately limits the rule to the
German artifacts that skill governs.

Nothing is lost when offline: SKILL.md states the rule in a sentence at the
point of use, and the reference is the long form.

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 11:58
@sonarqubecloud

Copy link
Copy Markdown

@github-actions

Copy link
Copy Markdown
Contributor

Dependency Review

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

Scanned Files

None

@github-actions github-actions Bot added documentation Improvements or additions to documentation skill labels Aug 19, 2026

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.

@CybotTM

CybotTM commented Aug 19, 2026

Copy link
Copy Markdown
Member Author

Self-review — Copilot has no review quota left this month, so no bot review will land here or anywhere else until the reset.

The diff replaces a 41-line file with a 3-line pointer. The check that decides whether that is safe is whether the removed text said anything the canonical copy does not: diff against git-workflow/.../no-editorializing.md reports 41a42,102 — additions only, no divergence — so this file was a strict prefix and the pointer is a superset, not a substitute.

The heading line is unchanged, so SKILL.md's reference table and any anchor into this file still resolve. The pointer is byte-identical to the two already in the catalog (matrix-skill, netresearch-branding), so the convention stays one shape.

CI green, no unresolved threads, mergeState=CLEAN.

@CybotTM
CybotTM merged commit ee7ade1 into main Aug 19, 2026
20 of 21 checks passed
@CybotTM
CybotTM deleted the chore/no-editorializing-pointer branch August 19, 2026 12:13
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.

2 participants