Skip to content

Fix jq compile error in cache/resolve keys unknown-key binding - #1105

Draft
phlax with Copilot wants to merge 4 commits into
ci-cachefrom
copilot/fix-jq-parse-failure
Draft

Fix jq compile error in cache/resolve keys unknown-key binding#1105
phlax with Copilot wants to merge 4 commits into
ci-cachefrom
copilot/fix-jq-parse-failure

Conversation

Copilot AI commented Aug 16, 2026

Copy link
Copy Markdown

The entries jq filter in actions/bazel/cache/resolve/action.yml failed to compile (Child process exited with code 5) due to an unparenthesized unique as $unknown binding. This change applies the minimal one-line fix so the filter parses and executes as intended for keys selection and unknown-key validation.

  • Root-cause fix (one-line jq syntax correction)

    • Parenthesize the pipeline feeding as $unknown so jq binds the computed array result, not a malformed expression.
    • No changes to lock-hash, resolved, modules, or caches steps; no workflow/test rewrites.
  • Behavior preserved

    • Existing keys handling remains intact, including the string-splitting path used for multi-line YAML scalar input.
    • Unknown key detection continues to fail resolution via the existing error path.
- | [$keys[] | . as $key | select($entries | has($key) | not)] | unique as $unknown
+ | ([$keys[] | . as $key | select($entries | has($key) | not)] | unique) as $unknown

phlax and others added 3 commits August 16, 2026 20:16
Signed-off-by: Ryan Northey <ryan@synca.io>
Signed-off-by: Ryan Northey <ryan@synca.io>
Co-authored-by: phlax <454682+phlax@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix jq parse failure in keys filter of action.yml Fix jq compile error in cache/resolve keys unknown-key binding Aug 16, 2026
Copilot AI requested a review from phlax August 16, 2026 19:55
@phlax
phlax force-pushed the ci-cache branch 9 times, most recently from 4247b4b to 73d8732 Compare August 16, 2026 22:12
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.

2 participants