[ptr727]

Summary

Answers the finding promotion #3476 raised on the vocabulary check #3474 added to tests/test_prose_lint.py.

• The is_word docstring said "letters alone", while its pattern joins letter runs with single spaces, hyphens, or apostrophes. It now says that. It also names which paths and environment variable names are not words, those with a separator or an underscore, since an all-letter name such as HOME is one. The test covering them is renamed to match.
• A word tuple keying a mapping beside a value holding no words was never reported (class sweep, found by the local pass). The mapping does not count such a key, yet marked it nested, so the key literal was skipped as well. Only a key the mapping counts is nested now, and the docstring says a literal reports at the outermost literal that counts it. One case proves each half: KEYED reports at the tuple, and COUNTED reports once, at the mapping.

Verification

• tests/test_prose_lint.py: 629 tests, OK. ruff format and check, mypy: clean. prose_lint.py --diff against the merge base, with and without --check sentence-length: clean.
• Three local adversarial passes at the frontier tier. The first found the remaining environment-name overclaim and the skipped key, the second two unproven halves and two wording gaps, each fixed. The third found nothing.

🤖 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: is-word-docstring
• File: .coderabbit.yaml
• Commit: 0999fc8
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: 3b4f29b0-a50c-4b9e-b60a-7a5cdfaa1406

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

[Copilot]

🟢 Approval recommended

No unresolved review issues were identified, and all reviewed changes have approval readiness.

No open issues 🎉

What changed in this PR

This pull request clarifies vocabulary-check documentation and fixes nested-literal handling for mapping keys, with regression coverage for counted and uncounted tuple keys.

Changes:

• Clarifies the is_word contract and renames the environment-name test.
• Marks mapping keys as nested only when their associated values contribute words.
• Adds regression cases for counted and uncounted mapping keys.

File	Description
tests/​test_prose_lint.py	Updates vocabulary-check documentation, nested mapping logic, and regression tests.

_{Review effort: Lite}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.98%. Comparing base (aa85f74) to head (0999fc8).

Additional details and impacted files

@@           Coverage Diff            @@
##           develop    #3477   +/-   ##
========================================
  Coverage    80.98%   80.98%           
========================================
  Files           25       25           
  Lines         8895     8895           
========================================
  Hits          7204     7204           
  Misses        1691     1691           

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

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.