[ptr727]

Summary

This answers the sixth round of review findings on the promotion pull request #3521:

• A repeated tier name. model_tier_errors compared each tool's models against tiers as sets, so a tiers list that repeats a name passed. It now reports "tiers must not repeat a name", and a test covers it.
• The installer's account of its hooks. The module docstring named one PreToolUse hook and two registered hooks. It now names both PreToolUse hooks, the model-tier guard's mapping, and the three hooks it registers. A class sweep of the same file found more wording of this kind, and this fixes it:
  • The --report check now labels the write guard as such, rather than as "the PreToolUse hook" or a bare "guard".
  • The settings-file messages now say "the hooks" where several hooks go unregistered.

🤖 Generated with Claude Code

[ptr727]

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

[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: promotion-3521-tiers-and-installer-doc
• File: .coderabbit.yaml
• Commit: 1b59b7c
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: 698ded67-8921-40f1-a76d-252f095c6ab5

• 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.}

[Copilot]

🟡 Changes recommended

Unresolved review comments request removing incidental hook counts and explicitly naming the write guard in the absent-registration message.

1 open issue

• Remove incidental hook counts from docstring · New

What changed in this PR

This pull request rejects repeated model-tier names and updates the Claude installer’s hook documentation and registration diagnostics.

Changes:

• Adds validation and regression coverage for duplicate tier names.
• Documents the installer’s model-tier guard and registered hooks.
• Clarifies hook-registration and write-guard diagnostics.

File	Summary
tests/​test_spec_validate.py	Adds coverage for repeated model-tier names.
spec/​validate.py	Rejects duplicate entries in the tiers configuration.
host-setup/​agent-safety/​claude/​install.py	Updates hook documentation and registration diagnostics.

_{Review effort: Lite}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.58%. Comparing base (557ac28) to head (1b59b7c).

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #3527      +/-   ##
===========================================
+ Coverage    81.56%   81.58%   +0.02%     
===========================================
  Files           26       26              
  Lines         8970     8972       +2     
===========================================
+ Hits          7316     7320       +4     
+ Misses        1654     1652       -2     

Flag	Coverage Δ
python-3.13	81.58% <100.00%> (+0.02%)	⬆️

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