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 #845) 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. Two things landed on this branch:

• The SKIP=no-commit-to-branch fix in this CI job / the nox -s lint session, so the "run every hook over all files" invocation stops re-litigating a hook that can never do anything but fail there (git commit/push never happens in that context).
• A follow-up stages: [pre-commit] restriction on the hook itself in .pre-commit-config.yaml (matching the same fix applied org-wide after a review comment on unxt#929): with default_stages: [pre-commit, pre-push], this hook would also fire as a pre-push git hook — which checks the currently checked-out branch, not the ref being pushed, so pushing a release tag while on main would be wrongly blocked. It's now scoped to the pre-commit git hook stage only, where it still fully enforces the original intent (blocking a direct git commit to main/versions/*).

Test plan

• SKIP=no-commit-to-branch prek run --all-files verified locally to skip only this hook and run everything else normally
• prek run no-commit-to-branch --all-files --stage pre-commit still fails on main / versions/* and passes on a feature branch — confirming the hook itself is untouched at the stage it's meant for
• prek run no-commit-to-branch --all-files --stage pre-push no longer runs the hook at all, confirming it won't false-positive on a tag-only push from main

🤖 Generated with Claude Code

[Copilot]

🟢 Approval recommended

The change is narrowly scoped to CI-style --all-files prek runs and uses standard SKIP behavior without altering the hook itself.

Pull request overview

Hotfix to unblock main CI by skipping the no-commit-to-branch pre-commit hook only in “run all hooks over all files” CI-style invocations, where GitHub Actions checks out a local branch named main and the hook would otherwise fail unconditionally.

Changes:

• Set SKIP=no-commit-to-branch for the prek run --all-files invocation in the Nox lint session.
• Set SKIP=no-commit-to-branch for the j178/prek-action CI step that runs --all-files.
• Add inline comments explaining why this hook must be skipped in these contexts.

File summaries

File	Description
noxfile.py	Skips no-commit-to-branch during the Nox “run prek over all files” lint session via env={"SKIP": ...}.
.github/workflows/ci.yml	Skips no-commit-to-branch in the CI pre-commit job by setting step-level SKIP.

Review details

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

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.32%. Comparing base (3246221) to head (1c6dea1).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #847   +/-   ##
=======================================
  Coverage   96.32%   96.32%           
=======================================
  Files         164      164           
  Lines        6769     6769           
=======================================
  Hits         6520     6520           
  Misses        249      249           

☔ 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

The noxfile.py change currently skips the hook for local nox -s lint runs too, which conflicts with the PR’s stated “CI-only skip” intent unless explicitly gated or clarified.

Get a fresh assessment by requesting another Copilot review.

Review details

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

[Copilot]

🟢 Approval recommended

The changes are narrowly scoped to CI execution behavior, preserve local hook enforcement, and are consistent across nox, pre-commit config, and the CI workflow.

Review details

• Files reviewed: 3/3 changed files
• Comments generated: 0 new
• Review effort level: Lite

[Copilot]

🟢 Approval recommended

The changes are narrowly scoped to CI/pre-commit execution behavior and appear to resolve the unconditional failure without altering the hook’s intended commit-time enforcement.

Review details

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

[Copilot]

🟢 Approval recommended

The changes are narrowly scoped to CI/pre-commit configuration and are internally consistent with the repository’s default_stages and the stated intent to preserve commit-time enforcement.

Review details

• Files reviewed: 3/3 changed files
• Comments generated: 0 new
• Review effort level: Lite

[Copilot]

🟢 Approval recommended

The changes directly address the described CI failure mode while preserving branch protection at the appropriate git hook stage.

Review details

• Files reviewed: 3/3 changed files
• Comments generated: 0 new
• Review effort level: Lite