[ptr727]

Summary

Clears the rest of #2488, the 9 Windows failures left once #2521 cleared the test_local_review cluster. With both changes, python3 -m unittest discover -s tests passes on the Windows host from Git Bash.

• spec/validate.py writes UTF-8. On Windows a piped stdout uses the ANSI code page, so the validator crashed reporting a registry name that code page cannot encode, a zero-width space for one, and wrote others, such as ß, in a form the suite's UTF-8 decode rejected. It now reconfigures both streams to UTF-8 at the top of main, through the same utf8_console helper scripts/pr_review.py already carries. On Linux the streams were already UTF-8, so CI output is unchanged.
• test_release_guards runs its guard script from a file. The handoff read this failure as jq 1.8.2 rejecting \S, and that diagnosis is wrong: winget's jq 1.8.2 compiles the filter and gives every expected verdict when it receives it intact. The script went through bash -c, and Git Bash's own parse of the Windows command line collapses each doubled backslash inside a quoted argument, so the filter arrived holding an invalid escape and every value was refused. The test now writes the script to a temporary file and runs that, and host_capability.bash_path's docstring records the trap for the next test.
• test_host_gate asserted POSIX forms where Windows prints a backslash path, the PowerShell call operator, and -Repo. Each assertion now takes the host's form, and on Linux each expected string is still byte-identical to the literal it replaces.
• test_docker_lint expected a forward-slash mount source from a native Windows Path. The expectation takes the separator from the host, and the doubled-quote escaping it pins is unchanged.

Part of #2488, which the promotion pull request closes.

Verification

• On the Windows host from Git Bash, test_spec_validate, test_host_gate, test_docker_lint, and test_release_guards each pass, and the python-versions guard case runs rather than skips.
• The full suite on this branch failed only in test_local_review, which Absorb a Windows Closed Pipe and Remove Read-Only Staged Objects in local_review #2521 fixes and which has since merged into develop.
• ruff format --check, ruff check, and mypy pass.

🤖 Generated with Claude Code

[coderabbitai]

Important

• 🔍 Trigger review

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

⚙️ Run configuration

• Configuration used: Repository: ptr727/ProjectTemplate/.coderabbit.yaml
• Review profile: CHILL
• Plan: Advanced
• Run ID: d572b4c5-0d4b-4aae-921a-a078c4ec418a

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

[codecov]

Codecov Report

❌ Patch coverage is 16.66667% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 58.40%. Comparing base (8bdfa25) to head (d438d25).
⚠️ Report is 2 commits behind head on develop.

Files with missing lines	Patch %	Lines
spec/validate.py	16.66%	5 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #2523      +/-   ##
===========================================
- Coverage    58.50%   58.40%   -0.10%     
===========================================
  Files           16       16              
  Lines         8066     8090      +24     
===========================================
+ Hits          4719     4725       +6     
- Misses        3347     3365      +18     

Flag	Coverage Δ
python-3.13	58.40% <16.66%> (-0.10%)	⬇️

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.

[Copilot]

🟡 Changes recommended

Address the two moderate review findings before approval.

2 open findings

• Add UTF-8 regression test for non-UTF-8 console diagnostics · New
• Keep command-prefix assertion strict · New

What changed in this PR

This PR fixes remaining Windows portability issues and ensures validator output is UTF-8 encoded.

Changes:

• Runs release guard scripts from temporary files.
• Makes host and Docker assertions platform-aware.
• Configures validator streams for UTF-8.
• Requires a UTF-8 regression test and restoration of strict command-prefix validation.

File	Summary
tests/​test_release_guards.py	Executes guard scripts from files.
tests/​test_host_gate.py	Updates Windows-specific expectations.
tests/​test_docker_lint.py	Uses host-native path separators.
tests/​host_capability.py	Documents Git Bash quoting behavior.
spec/​validate.py	Configures UTF-8 output streams.

🧠 Review effort: Lite

---

Give feedback about Copilot approvals in this survey to enter a drawing for a $150 gift card.

[ptr727]

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