[ptr727]

Part of #3531

Splits the over-length comment and docstring sentences in the host menu, bootstrap, and setup scripts, in their paired .ps1 and .sh forms and the three test files that read them. Only comment lines and Python docstrings change. An AST comparison of the three test files and a changed-line check of the scripts confirm no code or printed string moved.

Verification

• prose_lint.py --check sentence-length over the 11 files: 261 before, 0 after.
• bash -n, shfmt -d, and shellcheck clean on the touched .sh files. setup-github.sh keeps its SC2119/SC2120 findings on key_body, which are present at develop too.
• PowerShell parser: 0 errors on the four touched .ps1 files.
• tests/test_bootstrap.py (150 tests, 5 skipped) and tests/test_setup_github.py (40 tests): OK.
• Seven local adversarial review passes at opus. The first six each found meaning drift (mostly a joint reason left attached to only one half), fixed in the commits that follow. The seventh found none, and its pass is recorded against the head.

Known borderline wording left

• host-setup/windows/upgrade-host.ps1:200-201: "That setting would also corrupt..." can be misread as referring to WSL_UTF8 rather than the console encoding.

Copies left for the comment sweep

• "Every probe refuses to write a host key" and "A native command writes to this function" also appear in host-setup/bootstrap.ps1 (split here) and in install-agents.ps1, install-tools.ps1, and setup-wsl.ps1 (slice F1).
• The WSL_UTF8 comment in upgrade-host.ps1 has an unsplit copy near setup-wsl.ps1:121 (slice F1).
• The "script parameter reached only from a nested scope" and "Under pipefail" sentences have copies in files outside this slice. They were not individually located.

🤖 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-cmt-host-menu-bootstrap
• File: .coderabbit.yaml
• Commit: 21ec5a0
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: 9b21d4cd-ca01-49e9-9e3d-6f60f5618880

• 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 e82923d986754b6a6f18eb49965d2e52e4389466, the content this pull request carries at that commit against develop, and it recorded 0 findings.

[Copilot]

🟡 Changes recommended

The change still contains an unresolved comment ambiguity that could invert the rationale for the console-encoding conversion.

1 open issue

• Explicitly name the console-encoding setting · New

What changed in this PR

This pull request splits over-length comments and Python docstrings across host menu, bootstrap, setup scripts, and related tests while preserving executable behavior and printed output.

Changes:

• Reflows long comments and docstrings into shorter sentences and bullet lists.
• Applies equivalent updates across Bash, PowerShell, and Python files.
• Improves sentence-length lint compliance without changing runtime logic.

Final review comments note comment-marker consistency in bootstrap.sh and upgrade-host.ps1, an ambiguous encoding reference in upgrade-host.ps1, and wording about the desired registration state in setup-github.sh.

File	Description
tests/​test_setup_github.py	Splits the test module docstring.
tests/​test_bootstrap.py	Splits bootstrap test docstrings and explanatory comments.
tests/​host_capability.py	Splits host capability docstrings.
host-setup/​windows/​upgrade-host.ps1	Splits Windows upgrade comments. Final comments: [P2] encoding reference ambiguity (severity: nit, 4 votes) and [P2] prescribed comment-marker usage (severity: nit, 1 vote).
host-setup/​windows/​setup-github.ps1	Splits Windows GitHub setup comments.
host-setup/​menu.sh	Splits Bash menu and locking comments.
host-setup/​menu.ps1	Splits PowerShell menu and locking comments.
host-setup/​linux/​upgrade-host.sh	Splits Linux upgrade comments.
host-setup/​linux/​setup-github.sh	Splits Linux GitHub setup comments. Final comment: [P1] desired registration-state wording (severity: nit, 1 vote).
host-setup/​bootstrap.sh	Splits Bash bootstrap comments. Final comment: [P2] prescribed comment-marker usage (severity: nit, 1 vote).
host-setup/​bootstrap.ps1	Splits PowerShell bootstrap comments.

_{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 (21ec5a0).
⚠️ Report is 7 commits behind head on develop.

Additional details and impacted files

@@           Coverage Diff            @@
##           develop    #3613   +/-   ##
========================================
  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.

[ptr727]

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

[Copilot]

🟢 Approval recommended

The pull request is limited to documentation formatting with no executable behavior changes, and no active review issues remain.

No open issues 🎉

1 resolved since last review

• Explicitly name the console-encoding setting

_{Review effort: Lite}