Skip to content

Split the Over-Length Sentences in scripts/README.md - #3594

Merged
ptr727 merged 8 commits into
developfrom
feature/3531-md-scripts-readme
Oct 29, 2026
Merged

ptr727 merged 8 commits into
developfrom
feature/3531-md-scripts-readme

Conversation

@ptr727

@ptr727 ptr727 commented Oct 29, 2026 •

Copy link
Copy Markdown
Owner

Splits the over-length sentences in scripts/README.md without changing what they say.

prose_lint.py --check sentence-length: 303 hits before, 2 after. The two left are copies held in files other slices own: README line 64 ("On the operational model a promotion runs the rule...", copy in OPERATIONS.md, still long form) and README line 169 ("Layering is tighten-only...", copy in STANDUP.md, owned by slice D).

The splits were checked against the originals in six strict review rounds. Three sentences that review kept flagging with opposite readings (README lines 9, 50 and the --check paragraph) were restructured as the orchestrator ruled. prose_lint.py --diff origin/develop reports nothing.

Copies left for the comment sweep

These README sentences are split here. A near-verbatim copy in a code comment, docstring or test still holds the long form, for the next step to align:

  • README 64 "A trailing comment is out of scope..." and README 108 "comment-case reads the first character...": tests/test_prose_lint.py
  • README 151 "sha-pin resolves the pin..." and "Resolving also catches the neighboring case...": .github/actions/repo-gate/repo_gate.py
  • README 268 "A branch is finished when...": scripts/pr_cleanup.py
  • README 288, 294 (--repo, --track, the fork exception, the in-process refusal): scripts/handoff.py
  • README 315 (two sentences), 317, 321, 323: scripts/local_review.py (321 also tests/test_local_review.py)
  • README 340, 354 (lock): scripts/canonical_review.py
  • README 364 (skill and region): scripts/build_dist.py
  • README 387, 389: scripts/carry.py

Copies in AGENTS.md, .github/copilot-instructions.md and the Skills already read zero, and this README follows their wording where they carry the same sentence.

Part of #3531

🤖 Generated with Claude Code

ptr727 and others added 6 commits October 28, 2026 20:13
Part of #3531.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Part of #3531.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Part of #3531.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Part of #3531.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Part of #3531.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Part of #3531.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings October 29, 2026 03:34

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request. Check if the Files changed in this pull request are included in default exclusions.

@coderabbitai

coderabbitai Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

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

Run configuration
  • 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

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

@ptr727

ptr727 commented Oct 29, 2026

Copy link
Copy Markdown
Owner Author

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

…pts/README.md

Part of #3531.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@codecov

codecov Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.60%. Comparing base (7364e14) to head (16e206b).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #3594   +/-   ##
========================================
  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.

Part of #3531.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@ptr727

ptr727 commented Oct 29, 2026

Copy link
Copy Markdown
Owner Author

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

@ptr727
ptr727 merged commit 68d34fd into develop Oct 29, 2026
13 checks passed
@ptr727
ptr727 deleted the feature/3531-md-scripts-readme branch October 29, 2026 04:07
ptr727 added a commit that referenced this pull request Oct 29, 2026
…down (#3600)

## Summary

Promotes `develop` to `main`, carrying the sentence-length splits of the
hub-only Markdown:

- #3594: splits them in scripts/README.md.
- #3593: splits them in STANDUP, AUDIT, RESYNC, and OPERATIONS.
- #3591: splits them in TODO.md.
- #3596: splits them in the agent-safety Markdown.
- #3595: splits them in the host setup docs.
- #3592: splits them in reusable-workflows and fleet-map.
- #3598: splits them in the remaining host setup sentences.
- #3597: splits them in the spec and repo-config docs.
- #3599: splits them in the docs, catalog READMEs, and HISTORY.

This is part of #3531, which stays open.

🤖 Generated with [Claude Code](https://claude.com/claude-code)
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.

2 participants