fix(builtins): match root action metadata files in pinact_update glob - #1269
fix(builtins): match root action metadata files in pinact_update glob#1269risu729 wants to merge 1 commit into
Conversation
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ Run configurationConfiguration used: Repository YAML (base), Central YAML (inherited), Organization UI (inherited) Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Consolidated into #1268 — one logical change across both builtins, so a single PR is simpler to review. AI-assisted — Tool: Claude Code; model: anthropic/claude-fable-5; version: 2.1.241. |
The
pinact_updatebuiltin's globList(".github/workflows/*", "*/action.*")never matches a root-levelaction.yml/action.yaml(standalone action repositories), because the pattern requires a/beforeaction..pinact's own default discovery does include root action metadata (see
defaultWorkflowPatternsinpkg/controller/run/list_workflows.go:action.yml,action.yaml, and one-to-three-level-deep variants). Under hk,{{files}}replaces pinact's own discovery, so the step glob is the gate — this change restores parity.Notes:
literal_separator(src/glob.rs), so*already crosses/and nested files like.github/actions/setup/action.ymlmatched before this change; root was the only gap.**/action.*covers root and all depths.types = List("yaml")composes with AND, so the widened glob still only selects YAML.StepTestpassesfilesexplicitly, so glob selection is not exercised byhk test.Sibling PR #1268 applies the same fix to the
pinactbuiltin, which defines the same glob independently.AI-assisted — Tool: Claude Code; model: anthropic/claude-fable-5; version: 2.1.241.