Skip to content

Deduplicate action pin resolution warnings across workflows#10886

Merged
pelikhan merged 4 commits intomainfrom
copilot/fix-action-pin-resolution
Jan 20, 2026
Merged

Deduplicate action pin resolution warnings across workflows#10886
pelikhan merged 4 commits intomainfrom
copilot/fix-action-pin-resolution

Conversation

Copy link
Contributor

Copilot AI commented Jan 20, 2026

Action pin resolution warnings were emitted once per workflow, causing the same failure message to repeat dozens of times during batch compilation.

Changes

  • Added actionPinWarnings map[string]bool to Compiler struct to track warned action@version combinations across all workflows in a compilation session
  • Modified GetActionPinWithData to check cache before emitting warnings, using "repo@version" as cache key
  • Ensured cache is initialized in NewCompiler and passed to each WorkflowData instance via the existing shared resource pattern (alongside ActionCache and ActionResolver)

Example

Before:

⚠ Unable to pin action githubnext/gh-aw/actions/setup@v0.37.0: resolution failed
⚠ Unable to pin action githubnext/gh-aw/actions/setup@v0.37.0: resolution failed
⚠ Unable to pin action githubnext/gh-aw/actions/setup@v0.37.0: resolution failed
⚠ Unable to pin action githubnext/gh-aw/actions/setup@v0.37.0: resolution failed
⚠ Unable to pin action githubnext/gh-aw/actions/setup@v0.37.0: resolution failed
⚠ Unable to pin action githubnext/gh-aw/actions/setup@v0.37.0: resolution failed

After:

⚠ Unable to pin action githubnext/gh-aw/actions/setup@v0.37.0: resolution failed

Warnings for different versions of the same action remain independent (e.g., action@v1 and action@v2 each warn once).

Original prompt

Do not repeat failed resolution of action pins. See this output:

@pelikhan ➜ /workspaces/agent_sandbox (main) $ gh aw compile
✗ failed to parse workflow file /workspaces/agent_sandbox/.github/workflows/README.md: no frontmatter found
⚠ Unable to pin action githubnext/gh-aw/actions/setup@v0.37.0: resolution failed
⚠ Unable to pin action githubnext/gh-aw/actions/setup@v0.37.0: resolution failed
⚠ Unable to pin action githubnext/gh-aw/actions/setup@v0.37.0: resolution failed
⚠ Unable to pin action githubnext/gh-aw/actions/setup@v0.37.0: resolution failed
⚠ Unable to pin action githubnext/gh-aw/actions/setup@v0.37.0: resolution failed
⚠ Unable to pin action githubnext/gh-aw/actions/setup@v0.37.0: resolution failed
✓ .github/workflows/daily-repo-report.md (55.2 KB)
⚠ Unable to pin action githubnext/gh-aw/actions/setup@v0.37.0: resolution failed
⚠ Unable to pin action githubnext/gh-aw/actions/setup@v0.37.0: resolution failed
⚠ Unable to pin action githubnext/gh-aw/actions/setup@v0.37.0: resolution failed
⚠ Unable to pin action githubnext/gh-aw/actions/setup@v0.37.0: resolution failed
⚠ Unable to pin action githubnext/gh-aw/actions/setup@v0.37.0: resolution failed
⚠ Unable to pin action githubnext/gh-aw/actions/setup@v0.37.0: resolution failed
✓ .github/workflows/microsoft-research-daily-report.md (57.3 KB)
✗ Compiled 3 workflow(s): 1 error(s), 0 warning(s)
✗ Failed workflows:
  - /workspaces/agent_sandbox/.github/workflows/README.md (1 error)

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 3 commits January 20, 2026 18:10
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix repeated failure in action pin resolution Deduplicate action pin resolution warnings across workflows Jan 20, 2026
Copilot AI requested a review from pelikhan January 20, 2026 18:25
@pelikhan pelikhan marked this pull request as ready for review January 20, 2026 18:26
@pelikhan pelikhan merged commit dca781d into main Jan 20, 2026
116 checks passed
@pelikhan pelikhan deleted the copilot/fix-action-pin-resolution branch January 20, 2026 18:30
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