Skip to content

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

Merged
Zylphrex merged 1 commit intomasterfrom
txiao/feat/render-diff-in-explorer-autofix-slack-integration
Feb 26, 2026
Merged

feat(autofix): Render diff in explorer autofix slack integration#109396
Zylphrex merged 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

@Zylphrex Zylphrex merged commit 1919afe into master Feb 26, 2026
80 checks passed
@Zylphrex Zylphrex deleted the txiao/feat/render-diff-in-explorer-autofix-slack-integration branch February 26, 2026 13:04
@github-actions github-actions bot locked and limited conversation to collaborators Mar 13, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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