Skip to content

fix(ci): pass the action-pins path to the self-heal own-repo reconcile - #486

Merged
joshua-temple merged 1 commit into
mainfrom
fix/reconcile-companion-action-pins
Jul 6, 2026
Merged

joshua-temple merged 1 commit into
mainfrom
fix/reconcile-companion-action-pins

Conversation

@joshua-temple

Copy link
Copy Markdown
Collaborator

Problem

The self-heal companion (.github/workflows/pin-reconcile.yaml) invokes
cascade reconcile --own-repo without --action-pins. That flag has no
default (internal/pinreconcile/command.go), so own-repo mode fails with
reading action pins : open : no such file or directory and never adopts
the pin change.

Fix

Pass the canonical manifest path explicitly:
cascade reconcile --own-repo --action-pins internal/generate/action_pins.yaml.
--own-repo is cascade-only, so this is a workflow-only change; the companion
already runs the released binary that has the flag, so no new release is
needed.

Verification

Built the CLI from this branch and reproduced the trip locally: cloned a
fresh copy of main, bumped actions/create-github-app-token's sha in
.github/workflows/action-pins.yml (leaving the manifest untouched), then
ran the exact companion invocation from that copy.

  • Exit code: 0
  • internal/generate/action_pins.yaml picked up the new sha/version for
    actions/create-github-app-token
  • cascade verify reports 3 files, no drift afterward
  • Confirmed no other flag is missing: --root defaults to . and
    --config/--manifest-key default safely for own-repo mode; only
    --action-pins had no fallback.

actionlint on the modified workflow: no findings. go build ./...: clean.

Signed-off-by: Joshua Temple <joshua.temple@stablekernel.com>
@joshua-temple
joshua-temple merged commit c252e31 into main Jul 6, 2026
15 checks passed
@joshua-temple
joshua-temple deleted the fix/reconcile-companion-action-pins branch July 6, 2026 08:56
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