Wrap overlong lines in Claude wrappers per the 100-char limit#35
Merged
Conversation
Fold every wrapper `description:` into a `>` block scalar wrapped near 80 columns — wording byte-identical, verified by parsing old and new frontmatter and comparing the strings — and re-wrap the overlong body lines in the command wrappers. `allowed-tools:` lines stay single-line even beyond 100 characters: the Claude Code docs define no format for the field beyond single-line examples and a live parsing check could not be run, so re-wrapping a machine-consumed permission list is not provably safe. The exemption, and the always-fold `description:` convention, are now recorded in docs/authoring-skills.md (wrapper-frontmatter section, SKILL.md rules, and the pre-PR checklist). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR standardizes Claude wrapper Markdown formatting to comply with the repo’s 100-character line-length limit by converting wrapper description: fields to folded block scalars and re-wrapping overlong body lines, and it documents the convention in docs/authoring-skills.md to make it durable for future edits.
Changes:
- Replaced single-line
description:values in Claude command/agent wrappers with wrappeddescription: >folded scalars. - Re-wrapped overlong lines in wrapper bodies while keeping
allowed-tools:/ agenttools:single-line. - Documented the Claude wrapper frontmatter conventions and added validation checklist bullets in
docs/authoring-skills.md.
Reviewed changes
Copilot reviewed 16 out of 16 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| docs/authoring-skills.md | Documents wrapper frontmatter wrapping rules and adds a validation checklist note. |
| claude/commands/write-docs.md | Wraps description: into a folded scalar and reflows an overlong body line. |
| claude/commands/update-copyright.md | Wraps description: into a folded scalar and reflows an overlong body line. |
| claude/commands/run-build.md | Wraps description: into a folded scalar and reflows overlong bullet lines. |
| claude/commands/review-docs.md | Wraps description: into a folded scalar. |
| claude/commands/raise-coverage.md | Wraps description: into a folded scalar. |
| claude/commands/pre-pr.md | Wraps description: into a folded scalar. |
| claude/commands/move-files.md | Wraps description: into a folded scalar and reflows an overlong body line. |
| claude/commands/java-to-kotlin.md | Wraps description: into a folded scalar and reflows an overlong body line. |
| claude/commands/dependency-update.md | Wraps description: into a folded scalar and reflows overlong body lines. |
| claude/commands/bump-version.md | Wraps description: into a folded scalar and reflows an overlong body line. |
| claude/commands/bump-gradle.md | Wraps description: into a folded scalar and reflows an overlong body line. |
| claude/agents/spine-code-review.md | Wraps description: into a folded scalar. |
| claude/agents/review-docs.md | Wraps description: into a folded scalar. |
| claude/agents/kotlin-engineer.md | Wraps description: into a folded scalar. |
| claude/agents/dependency-audit.md | Wraps description: into a folded scalar. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
The document itself establishes `.agents/guidelines/<file>.md` as the way to reference shared guidance; the new wrapper-frontmatter prose used bare `guidelines/...` paths. Addresses Copilot review feedback on PR #35. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 6c35facc28
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
armiol
approved these changes
Jul 17, 2026
alexander-yevsyukov
added a commit
that referenced
this pull request
Jul 17, 2026
Master's line-wrapping (PR #35) and this branch's model tiers crossed; the merge keeps master's wrapping style everywhere both sides only differed in wrap points, plus this branch's semantic changes. Doc reconciliation: both new sections are kept — "Claude wrappers and model tiers" (this branch) and "Claude wrapper frontmatter" (master). Master's allowed-tools bullet said folding was unverified and must be avoided; this branch has since verified equivalence (strict YAML parse over every wrapper plus live registration of folded commands), so the bullet now sanctions both the single-line and folded forms. The four new commands' short descriptions are folded per master's convention. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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.
What changed
description:— the four agents and all elevencommands — into a
>block scalar wrapped near 80 columns. Wording isbyte-identical (verified by parsing old and new frontmatter and comparing the
strings), so the descriptions that drive dispatch matching are unaffected.
allowed-tools:lines stay single-line even beyond 100 characters. TheClaude Code docs define no format for the field beyond single-line examples,
and its value is machine-consumed as permission rules, so re-wrapping is not
provably safe across harness versions (a
>-scalar folds to a byte-identicalstring under spec YAML, but a live permission-gating check could not be run).
docs/authoring-skills.mdnow records both conventions — a "Claude wrapperfrontmatter" section (always fold
description:near 80 columns; keepallowed-tools:/tools:on one line), a matchingSKILL.mdrules bullet,and a "Validate before opening a PR" checklist bullet — so future agent
sessions apply them without re-deriving.
Why
guidelines/coding.mdsetsmax-line-length: 100, andguidelines/documentation.mdapplies it to docs. #34 fixed the newly addedwrappers; this covers the pre-existing ones and makes the convention durable.
Reviewer notes
awk 'length($0)>100 {print FILENAME":"FNR}' claude/agents/*.md claude/commands/*.mdreturns exactly the three documented
allowed-tools:exemptions(
dependency-update,raise-coverage,review-docscommands).model:changes; expect one trivial adjacent-insertion conflict indocs/authoring-skills.md(both add a section afteragents/openai.yaml).Assign Claude model tiers to skill wrappers #34's new wrappers already match these conventions.
review-docsreturned APPROVE WITH CHANGES — three cosmetic re-flowsuggestions (runt last lines), left for maintainer judgment.
🤖 Generated with Claude Code