Skip to content

fix(ci): remove unused Setup Scripts step from update_cache_memory jobs#1252

Merged
lpcox merged 2 commits intoclaude/fix-issue-20590from
copilot/66732385167-fix-cache-memory
Mar 12, 2026
Merged

fix(ci): remove unused Setup Scripts step from update_cache_memory jobs#1252
lpcox merged 2 commits intoclaude/fix-issue-20590from
copilot/66732385167-fix-cache-memory

Conversation

Copy link
Contributor

Copilot AI commented Mar 12, 2026

update_cache_memory jobs fail with 401 Unauthorized because they have permissions: {} but attempt to download the private github/gh-aw/actions/setup action — which the empty token cannot access. The Setup Scripts step is entirely unused in these jobs; all subsequent steps (actions/download-artifact, actions/cache/save) use the runner token, not GITHUB_TOKEN.

Changes

  • scripts/ci/postprocess-smoke-workflows.ts: Added a transformation that strips the Setup Scripts step from update_cache_memory jobs when it is immediately followed by Download cache-memory artifact, ensuring future recompiled lock files are also patched automatically
  • 10 lock files (smoke-claude, smoke-copilot, smoke-codex, security-review, ci-doctor, issue-duplication-detector, pelis-agent-factory-advisor, secret-digger-{claude,codex,copilot}): Applied the transformation — Setup Scripts step removed from each update_cache_memory job

Before (broken):

update_cache_memory:
  permissions: {}
  steps:
    - name: Setup Scripts                         # 401 — token has no perms
      uses: github/gh-aw/actions/setup@v0.47.0
      with:
        destination: /opt/gh-aw/actions
    - name: Download cache-memory artifact (default)
      uses: actions/download-artifact@...

After (fixed):

update_cache_memory:
  permissions: {}
  steps:
    - name: Download cache-memory artifact (default)
      uses: actions/download-artifact@...
Original prompt

Fix the failing GitHub Actions workflow update_cache_memory
Analyze the workflow logs, identify the root cause of the failure, and implement a fix.
Job ID: 66732385167
Job URL: https://github.com/github/gh-aw-firewall/actions/runs/22984713099/job/66732385167


💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.

Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>
Copilot AI changed the title [WIP] [66732385167] Fix failing GitHub Actions workflow update_cache_memory fix(ci): remove unused Setup Scripts step from update_cache_memory jobs Mar 12, 2026
@lpcox lpcox marked this pull request as ready for review March 12, 2026 03:30
Copilot AI review requested due to automatic review settings March 12, 2026 03:30
@lpcox lpcox merged commit 255b68f into claude/fix-issue-20590 Mar 12, 2026
@lpcox lpcox deleted the copilot/66732385167-fix-cache-memory branch March 12, 2026 03:30
lpcox added a commit that referenced this pull request Mar 12, 2026
…#1249)

* Initial plan

* feat: add openai/anthropic api target flags for custom endpoints

* Update src/cli.ts

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix(proxy): warn when custom api targets are not in --allow-domains

Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>

* fix(ci): skip agentic workflow activation for bot-triggered pull requests (#1250)

* Initial plan

* fix(ci): skip agentic workflow activation for bot-triggered pull requests

Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>

* fix: resolve pid-tracker integration test failure and add missing test coverage for API target validation (#1251)

* Initial plan

* fix: resolve failing Build and Lint (Node 22) CI workflow

Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>

* fix(ci): remove unused Setup Scripts step from update_cache_memory jobs (#1252)

* Initial plan

* fix(ci): remove unused Setup Scripts step from update_cache_memory jobs

Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>

* docs(proxy): document --openai-api-target and --anthropic-api-target in api-proxy-sidecar.md

Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>

* fix: resolve test coverage regression and locked PR comment failure (#1254)

* Initial plan

* fix: resolve coverage regression and locked PR comment failure

Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>

---------

Co-authored-by: anthropic-code-agent[bot] <242468646+Claude@users.noreply.github.com>
Co-authored-by: Landon Cox <landon.cox@microsoft.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>
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