Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 19, 2025

The update_project safe output was generating 10+ error annotations in GitHub Actions UI for successful workflow runs. The logGraphQLError() function used core.error() for diagnostic information, creating false error annotations despite job success.

Changes

  • Changed logGraphQLError() in pkg/workflow/js/update_project.cjs to use core.info() for diagnostic output
  • Preserved core.error() for actual operation failures
  • Recompiled workflows to propagate the fix

Impact

Diagnostic information (error details, request/response data, permission troubleshooting) now appears in job logs without creating error annotations. Actual failures that prevent operation completion still generate appropriate error annotations.

Before:

function logGraphQLError(error, operation) {
  core.error(`GraphQL Error during: ${operation}`);
  core.error(`Message: ${error.message}`);
  // ... more core.error() calls for diagnostics
}
// Result: 10+ error annotations in UI

After:

function logGraphQLError(error, operation) {
  core.info(`GraphQL Error during: ${operation}`);
  core.info(`Message: ${error.message}`);
  // ... diagnostic details logged as info
}
// Result: Clean UI, diagnostics still visible in logs
Original prompt

Analyze safe output erros in run: https://github.com/githubnext/gh-aw/actions/runs/20365622919/job/58520092414


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@mnkiefer mnkiefer marked this pull request as ready for review December 19, 2025 09:40
…iagnostics

Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
Copilot AI changed the title [WIP] Analyze safe output errors in GitHub Actions run Fix spurious error annotations in safe_outputs jobs from GraphQL diagnostic logging Dec 19, 2025
Copilot AI requested a review from mnkiefer December 19, 2025 09:47
@mnkiefer mnkiefer merged commit 4a50b62 into main Dec 19, 2025
154 checks passed
@mnkiefer mnkiefer deleted the copilot/analyze-safe-output-errors branch December 19, 2025 09:57
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