fix(ci): skip no-commit-to-branch in the full-suite CI run

[nstarman]

Urgent: fixes broken CI on main

no-commit-to-branch (added in ##881) is currently failing every push to main — CI checks out a real local branch literally named main for push events, so the hook (which exists to stop a human committing directly to a protected branch) fires every single time, unconditionally. This is a hotfix, not a revert: the hook stays fully active for local git commit/git push (via prek install), which is its actual purpose — it's only skipped in the "run every hook over all files" CI invocation, where it can never do anything but fail.

Test plan

• SKIP=no-commit-to-branch prek run --all-files (and prek run --skip no-commit-to-branch --all-files) verified locally to skip only this hook and run everything else normally
• prek run no-commit-to-branch --all-files (without SKIP) still fails on main / versions/* and passes on a feature branch — confirming the hook itself is untouched

🤖 Generated with Claude Code

[Copilot]

🟡 Changes recommended

The current implementation overrides any existing SKIP setting and the developer docs now disagree with nox -s precommit behavior, so it should be adjusted for correctness and consistency.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

This PR hotfixes CI on main by preventing the no-commit-to-branch prek hook (which is intended for local developer workflows) from failing the “run all hooks over all files” nox/CI lint invocation.

Changes:

• Update the precommit nox session to run prek run --all-files with SKIP=no-commit-to-branch.
• Add inline rationale in noxfile.py explaining why this hook is skipped in the nox session.

File summaries

File	Description
noxfile.py	Skips no-commit-to-branch during nox -s precommit runs to avoid CI failures on protected branches.

Review details

• Files reviewed: 1/1 changed files
• Comments generated: 2
• Review effort level: Lite

---

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[codspeed]

Merging this PR will not alter performance

✅ 20 untouched benchmarks
⏩ 12 skipped benchmarks¹

---

_{Comparing fix-no-commit-to-branch-ci (27ca60c) with main (1f5e6d3)}

Footnotes

1. 12 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

[github-code-quality]

Code Coverage Overview

Languages: Python

Python / code-coverage-agent

The overall line coverage in commit 59c074a in the fix-no-commit-to-bra... branch remains at 91%, unchanged from commit c3415ba in the main branch.

Show a line coverage summary of the most impacted files.

File	main c3415ba	fix-no-commit-to-bra... 59c074a	+/-
packages/coordi...src/__init__.py	100%	100%	0%
src/coordinax/_...rts/jacobian.py	100%	100%	0%
packages/coordi...signedplanar.py	0%	54%	+54%

---

_{Updated September 15, 2026 17:40 UTC}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.38%. Comparing base (c3415ba) to head (27ca60c).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #885      +/-   ##
==========================================
+ Coverage   97.34%   97.38%   +0.04%     
==========================================
  Files         270      271       +1     
  Lines        9529     9692     +163     
==========================================
+ Hits         9276     9439     +163     
  Misses        253      253              

☔ 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.