[ptr727]

Summary

Make the prose gate's sentence-length rule a default check. The whole hub tree now reads zero over-cap sentences on develop, which was the precondition the opt-in existed for.

• prose-gate.json sets the rule's default to true. DEFAULT_RULES is computed from that flag, so a bare prose_lint.py run, the husky and pre-commit hooks, and the CI prose-gate action all pick it up with no list to edit.
• The comment-and-doc-style skill no longer calls the check opt-in. It states that the check runs by default, and that --diff <base> scopes it to the lines a change touches. sentence-split stays opt-in.
• The local-strict-review brief drops its second --check sentence-length run, since the first run now covers it.
• GOVERNANCE.md "Sentence Structure", OPERATIONS.md, docs/fleet-map.md, and scripts/README.md stop describing the rule as opt-in. The local verification block names it, so that block still covers every default rule except comment-added.
• The test asserting the rule is opt-in is replaced by one asserting a bare run reports an over-cap sentence.
• The regenerated skill copies under .github/skills/ and .claude-plugin/ come with it.

Fleet effect

A carrier's CI runs the hub's prose-gate action, which passes no --check, so it now gates the word cap on each sentence a pull request touches. The diff scope means an existing over-cap sentence in a carrier fails no pull request until a change edits one of its lines. The fetched-fresh pre-commit hooks gate the same way, diff-scoped. A whole-tree run, such as the Lint: Prose editor task, reports a carrier's existing backlog.

A carrier cannot turn the rule off through .prose-gate.json. That override only adds exemptions and refuses any other key. The only repository-level escape is .github/prose-gate-excludes, which drops whole paths and is meant for vendored content.

#3531 asked what a carrier's review pass does with a sentence-length hit on hub-verbatim text. The hub's text now reads zero, and the gate now holds every changed line to the cap. So a resync carries no such hits to dispose of, and the brief's separate opt-in run is gone.

Verification

• prose_lint.py . over the whole tree with the new default: 0 violations.
• prose_lint.py --diff origin/develop: 0 violations.
• tests/test_prose_lint.py: 629 tests OK. The new test fails with the flag flipped back.
• tests/test_build_dist.py, tests/test_hub_fetch_run.py, tests/test_pre_commit_hook.py: OK.
• build_dist.py --check: current.

Closes #3531

🤖 Generated with Claude Code

[coderabbitai]

Important

• 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration

• Source mode: Auto
• Repository: ptr727/ProjectTemplate
• Branch: feature/3531-sentence-length-default
• File: .coderabbit.yaml
• Commit: 8015552
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: 62fadadb-33df-449a-8ed2-79f86f6463d3

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[ptr727]

A recorded local strict-review pass covers head 8015552c438a7461ca3dcc1607cdbe33bf5479f4, the content this pull request carries at that commit against develop, and it recorded 0 findings.

[Copilot]

🟢 Approval recommended

No actionable review findings remain, and the rule promotion is consistently reflected across configuration, tests, documentation, and generated skill distributions.

No open issues 🎉

What changed in this PR

This pull request promotes the sentence-length prose rule from opt-in to a default check across local runs, hooks, CI, documentation, review guidance, and generated skill distributions.

Changes:

• Enables sentence-length by default in prose-gate.json.
• Updates tests, governance, operations, and fleet documentation.
• Removes redundant opt-in review instructions and refreshes generated skill copies and digests.
• No actionable findings were identified.

File	Description
tests/​test_prose_lint.py	Verifies that bare runs enforce sentence-length.
scripts/​README.md	Documents the rule as a default check.
OPERATIONS.md	Adds the rule to local verification guidance.
GOVERNANCE.md	Updates sentence-structure policy.
docs/​fleet-map.md	Records the rule’s promotion to default enforcement.
.github/​skills/​local-strict-review/​SKILL.md	Removes the redundant explicit sentence-length pass.
.github/​skills/​comment-and-doc-style/​SKILL.md	Documents default sentence-length enforcement.
.github/​actions/​prose-gate/​prose-gate.json	Enables sentence-length by default.
.claude-plugin/​fleet-skills/​skills/​local-strict-review/​SKILL.md	Regenerated local-review skill distribution.
.claude-plugin/​fleet-skills/​skills/​comment-and-doc-style/​SKILL.md	Regenerated documentation-style skill distribution.
.claude-plugin/​fleet-skills/​.source-digests/​local-strict-review	Updates the generated skill digest.
.claude-plugin/​fleet-skills/​.source-digests/​comment-and-doc-style	Updates the generated skill digest.
.agents/​skills/​local-strict-review/​SKILL.md	Updates the canonical local-review skill source.
.agents/​skills/​comment-and-doc-style/​SKILL.md	Updates the canonical documentation-style skill source.

_{Review effort: Lite}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.60%. Comparing base (b38e084) to head (8015552).
⚠️ Report is 10 commits behind head on develop.

Additional details and impacted files

@@           Coverage Diff            @@
##           develop    #3618   +/-   ##
========================================
  Coverage    81.60%   81.60%           
========================================
  Files           26       26           
  Lines         9070     9070           
========================================
  Hits          7402     7402           
  Misses        1668     1668           

Flag	Coverage Δ
python-3.13	81.60% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.