Skip to content

chore(event formatter): add new event formatter to mcp - #1162

Open
shayna-ch wants to merge 4 commits into
mainfrom
shayna-ch/format-mcp
Open

chore(event formatter): add new event formatter to mcp #1162
shayna-ch wants to merge 4 commits into
mainfrom
shayna-ch/format-mcp

Conversation

@shayna-ch

@shayna-ch shayna-ch commented Jul 16, 2026

Copy link
Copy Markdown
Member

to be merged after endpoint pr

Switches the MCP issue/event and autofix output to consume Sentry's shared-formatter output when it's available:

  • getEventForIssue and getAutofixState now request ?llmFormat=markdown.
  • BaseEventSchema and AutofixRunStateSchema gain an optional formatted: {format, content} field.
  • formatIssueOutput uses event.formatted.content for error / default / generic / csp events (re-adding the replay note so output still matches formatEventOutput), falling back to formatEventOutput otherwise.
  • analyze-issue-with-seer uses autofixState.formatted.content at each terminal/progress output point, falling back to getOutputForAutofixRun.

No behavioral change on merge. behind the issues.standardized-markdown-for-llm option. Rollout will happen gradually via feature flag.

Comment thread packages/mcp-core/src/internal/formatting.ts Outdated
@shayna-ch shayna-ch changed the title mcp chore(event formatter): add new event formatter to mcp Jul 20, 2026
@shayna-ch

Copy link
Copy Markdown
Member Author

bugbot run

Comment thread packages/mcp-core/src/internal/formatting.ts Outdated
@shayna-ch
shayna-ch marked this pull request as ready for review July 20, 2026 19:44

@cursor cursor Bot left a comment

Copy link
Copy Markdown

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.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit e7f534f. Configure here.

autofixState.autofix.run_id,
);
return `## Seer Analysis\n\n${wrapped}\n`;
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Seer status omitted with formatted content

Medium Severity

When autofixState.formatted.content is present, formatSeerSummary returns early and skips the existing status handling for in-progress, failed, and awaiting_user_input runs. analyze_issue_with_seer still appends that status after using formatted content, so get_issue_details can hide that Seer failed or needs input once the shared formatter rolls out.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit e7f534f. Configure here.

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.

1 participant