Skip to content

feat(autofix): Render diff in explorer autofix slack integration#109396

Open
Zylphrex wants to merge 1 commit intomasterfrom
txiao/feat/render-diff-in-explorer-autofix-slack-integration
Open

feat(autofix): Render diff in explorer autofix slack integration#109396
Zylphrex wants to merge 1 commit intomasterfrom
txiao/feat/render-diff-in-explorer-autofix-slack-integration

Conversation

@Zylphrex
Copy link
Member

Following up from getsentry/seer#4965 to render the computed diff in slack.

Following up from getsentry/seer#4965 to render the computed diff in slack.
@Zylphrex Zylphrex requested a review from a team as a code owner February 25, 2026 23:22
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Feb 25, 2026
Copy link
Contributor

@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.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

webhook_payload["code_changes"] = {
repo: [
{
"diff": p.diff,
Copy link
Contributor

Choose a reason for hiding this comment

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

Unbounded diffs can break Slack updates

Medium Severity

on_completion_hook.py now forwards raw p.diff into event_payload["code_changes"], and entrypoint.py passes it through unchanged to Slack rendering. Large unified diffs can exceed Slack block text limits, causing message update failures (invalid_blocks) and dropping the code-changes update for that run.

Additional Locations (1)

Fix in Cursor Fix in Web

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

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants