Skip to content

fix(ci): correct desktop path filter scope - #26

Merged
BrianInAz merged 1 commit into
mainfrom
fix/ci-desktop-path-filter
Aug 10, 2026
Merged

fix(ci): correct desktop path filter scope#26
BrianInAz merged 1 commit into
mainfrom
fix/ci-desktop-path-filter

Conversation

@BrianInAz

Copy link
Copy Markdown
Owner

Summary

Correct the CI Desktop path filter so mobile, documentation, and Beads-only changes do not activate Desktop jobs.

Root cause

The filter used desktop/** plus !desktop/src-tauri/** while dorny/paths-filter uses predicate-quantifier: some. Each rule is therefore independent, and the negative rule matches every file outside desktop/src-tauri.

TDD remediation

  • Red: 3/3 new routing contracts failed on untouched main
  • Green: 3/3 new routing contracts passed
  • Existing file-size contracts: 6/6 passed
  • All existing Detect Changed Paths contract commands passed
  • Workflow YAML parsing, Biome, and diff checks passed
  • No dependency changes

The replacement is one positive extglob: desktop/!(src-tauri)/**. Tauri paths remain covered by the existing desktop-rust filter and downstream job conditions.

Documentation

Promotion

The user explicitly authorized this PR and promotion after review of the hosted-runner consequence. Require all checks green before squash merge.

Signed-off-by: Brian Charbonneau <github@briancharbonneau.com>
@cursor

cursor Bot commented Aug 10, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@BrianInAz
BrianInAz merged commit ef72743 into main Aug 10, 2026
24 checks passed
@BrianInAz
BrianInAz deleted the fix/ci-desktop-path-filter branch August 11, 2026 22:26
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