Skip to content

Conversation

Aias00
Copy link
Contributor

@Aias00 Aias00 commented Sep 5, 2025

Describe what this PR does / why we need it

Does this pull request fix one issue?

Describe how you did it

Describe how to verify it

Special notes for reviews

@Aias00 Aias00 merged commit 0264cf5 into main Sep 5, 2025
8 checks passed
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

This is the final PR Bugbot will review for you during this billing cycle.

Your free Bugbot reviews will reset on October 7

Details

You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.

To receive Bugbot reviews on all of your PRs, please upgrade to Bugbot Pro by visiting the Cursor dashboard. Your first 14 days will be free!

console.log('Posted clarity request comment to issue #' + issue_number);
} else {
console.log('Skipped posting comment - not an issue event');
console.log('Issue has sufficient information, no comment needed');
Copy link

Choose a reason for hiding this comment

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

Bug: Workflow Fails on Discussion Events

The workflow triggers on both issue and discussion events, but the new logic only handles issues. For discussion events, the workflow fails by attempting to access issue-specific github.event.issue properties and calling github.rest.issues.createComment(), which isn't for discussions. The previous version correctly distinguished event types.

Fix in Cursor Fix in Web

Issue 标题: ${{ github.event.issue.title }}
Issue 内容: ${{ github.event.issue.body }}
Issue 标签: ${{ join(github.event.issue.labels.*.name, ', ') }}
Copy link

Choose a reason for hiding this comment

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

Bug: Claude Prompt Fails with Discussion Events

The workflow triggers on both issue and discussion events, but the Claude prompt uses issue-specific context variables. For discussion events, these variables are undefined, which can cause the analysis to fail or produce incorrect results.

Fix in Cursor Fix in Web

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant