Skip to content

Add SDK failure reason as a comment from SDK generation agent workflow#40897

Merged
praveenkuttappan merged 13 commits intomainfrom
sdk_agent_pipeline_status
Mar 2, 2026
Merged

Add SDK failure reason as a comment from SDK generation agent workflow#40897
praveenkuttappan merged 13 commits intomainfrom
sdk_agent_pipeline_status

Conversation

@praveenkuttappan
Copy link
Member

This PR adds an instruction to run pipeline analysis if SDK generation fails and add the failure summary as a comment in GitHub issue to make it easier for user to find the failure reason.

@github-actions
Copy link

github-actions bot commented Mar 1, 2026

Next Steps to Merge

Next steps that must be taken to merge this PR:
  • ❌ The required check named Protected Files has failed. Refer to the check in the PR's 'Checks' tab for details on how to fix it and consult the aka.ms/ci-fix guide


Comment generated by summarize-checks workflow run.

@praveenkuttappan praveenkuttappan changed the title Sdk agent pipeline status Add SDK failure reason as a comment from SDK generation agent workflow Mar 1, 2026
@praveenkuttappan
Copy link
Member Author

Fixes Azure/azure-sdk-tools#14245

@praveenkuttappan praveenkuttappan merged commit c760c69 into main Mar 2, 2026
44 of 46 checks passed
@praveenkuttappan praveenkuttappan deleted the sdk_agent_pipeline_status branch March 2, 2026 20:00
- Review the analysis output and extract a concise summary of the core error (e.g. compilation error, missing dependency, TypeSpec validation failure).
- Include the error summary under each failed language in the issue comment so users can understand the failure without inspecting pipeline logs.

4. Include the following troubleshooting guidance in the comment:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ideally we'd update existing comments instead of posting new, we have JS code to do this from code-based workflows, but it sounds like agentic workflows are currently blocked from updating comments (though they can add new ones)

https://github.com/Azure/azure-rest-api-specs/blob/main/.github/workflows/src/comment.js

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.

3 participants