Skip to content

maintainers: match wildcard parents in single-level globs - #746

Draft
OllieinCanada wants to merge 1 commit into
sashiko-dev:mainfrom
OllieinCanada:fix/maintainers-nested-globs
Draft

OllieinCanada wants to merge 1 commit into
sashiko-dev:mainfrom
OllieinCanada:fix/maintainers-nested-globs

Conversation

@OllieinCanada

Copy link
Copy Markdown
Contributor

Fixes #745.

MAINTAINERS patterns such as arch//net/ and arch//events/ fail because the single-level directory shortcut treats wildcard parents as literal prefixes. Restrict the shortcut to literal parents and use the existing glob matcher otherwise.

Rust 1.90.0: both new offline regressions fail before the fix and pass afterward. All 13 maintainer-index tests pass. Full make check-pr passed: 1150 tests passed, two ignored; formatting and three Clippy profiles passed. The inherited RUST_LOG override was unset.

No schema changes or historical data rewrites.

DRAFT: required local model self-review is incomplete because provider authentication failed. No clean local model review is claimed. Keep as draft until the required review completes and findings are addressed.

The single-level directory shortcut treats everything before the final
star as a literal prefix. Patterns such as arch/*/net/* and
arch/*/events/* therefore never match source files, omitting the BPF and
performance-events sections from subsystem attribution.

Use the shortcut only for literal parent directories. Let the existing
glob matcher handle wildcard parents while preserving component boundaries
and the existing specificity ranking.

Add offline regressions for matching, exclusions and parsed subsystem
attribution. Both tests fail against the previous compiler.

Fixes: d0889e3 ("ai: add localized vector space search and subsystem detection")
Signed-off-by: Ollie Slapinski <oliverslapinski@hotmail.com>
@sashiko-bot

sashiko-bot Bot commented Oct 25, 2026

Copy link
Copy Markdown

Sashiko review

✓ No issues found across 1 commit.

Full review log on sashiko.sashiko.dev

@OllieinCanada
OllieinCanada marked this pull request as draft October 25, 2026 20:21
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.

maintainers: wildcard parents in single-level globs fail to match

1 participant