Skip to content

Add path-aware native CI gates - #306

Merged
yyjeqhc merged 2 commits into
mainfrom
ci/path-aware-native-gates
Sep 5, 2026
Merged

Add path-aware native CI gates#306
yyjeqhc merged 2 commits into
mainfrom
ci/path-aware-native-gates

Conversation

@yyjeqhc

@yyjeqhc yyjeqhc commented Sep 5, 2026

Copy link
Copy Markdown
Owner

Summary

  • add trusted-base path-aware native CI classification for owner PRs while keeping push-to-main, external PRs, and run-ci on the full native matrix
  • split macOS core vs Desktop/package coverage and preserve stable aggregate required checks (test, test-macos, test-windows, test-native)
  • harden classification against body-only edits inside existing platform cfg / Cargo target sections, npm Windows installer changes, Runner-owned process surfaces, arbitrary workflow policy changes, renames/deletions, and bounded-input overflow
  • keep first landing fail-safe: if the trusted base does not yet contain the classifier, run the full native matrix

Review follow-up

Independent review added 21033798 (Harden native CI path classification) after rebase onto current main. It closes false-negative cases for existing platform cfg context, nested Cargo target sections, npm/webcodex/**, Runner process owners, and new workflow files.

Validation

  • python3 -m unittest scripts.tests.test_ci_path_risk — 25 passed
  • bash scripts/test_python_tooling.sh — syntax-checked 26 Python files; 148 tests passed
  • PyYAML parse of .github/workflows/ci.yml — passed
  • git diff --check / staged check — passed
  • real Git range classifier checks, rename fixture, platform-context and bounded-fallback regressions — passed
  • worktree hygiene clean

actionlint passed in the implementation handoff. The fresh reviewer environment did not have actionlint on PATH; reviewer did not modify ci.yml after that handoff.

@yyjeqhc
yyjeqhc merged commit 7a35332 into main Sep 5, 2026
2 checks passed
@yyjeqhc
yyjeqhc deleted the ci/path-aware-native-gates branch September 5, 2026 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant