Skip to content

feat(integrations): Add Holmes third-party integration - #19860

Merged
souredoutlook merged 1 commit into
masterfrom
cursor/add-holmes-integration-b068
Oct 8, 2026
Merged

souredoutlook merged 1 commit into
masterfrom
cursor/add-holmes-integration-b068

Conversation

@souredoutlook

Copy link
Copy Markdown
Contributor

DESCRIBE YOUR PR

This PR adds the Holmes Sentry Integration Platform app to the third-party integrations documentation.

Changes:

  • Added Holmes to the alphabetically-ordered list in /docs/integrations/third-party-integrations/index.mdx
  • Added redirect from /organization/integrations/holmes/ to the consolidated integrations page
  • Added redirect from /integrations/holmes/ to the consolidated integrations page

Integration details:

IS YOUR CHANGE URGENT?

  • Urgent deadline (GA date, etc.): YYYY-MM-DD
  • Other deadline: YYYY-MM-DD
  • No deadline: Not urgent, can wait up to 1 week+

PRE-MERGE CHECKLIST

  • Checked Vercel preview for correctness, including links
  • PR was reviewed and approved by any necessary SMEs (subject matter experts)
  • PR was reviewed and approved by a member of the Sentry docs team
Open in Web Open in Cursor 

- Add Holmes to third-party integrations list
- Add redirects for /organization/integrations/holmes/
- Add redirects for /integrations/holmes/

Co-authored-by: Nicholas Meisenheimer <souredoutlook@users.noreply.github.com>
@vercel

vercel Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
develop-docs Ready Ready Preview Oct 8, 2026 4:19pm UTC
sentry-docs Ready Ready Preview Oct 8, 2026 4:19pm UTC

Request Review

@cursor

cursor Bot commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

The change adds a Holmes listing on the third-party integrations index and two redirects onto that page. The plan checks that listing, those redirect responses, integrations 404s, and client errors. An issue escalates when integrations 404s rise or client errors rise.

Rollout complete

Services: sentry-docs.

Plan

What changed

Requests to /integrations/holmes/ and /organization/integrations/holmes/ now send a permanent redirect to /integrations/third-party-integrations/.
The third-party integrations index now lists Holmes.
sentry-docs must deploy before this change is live.
The Vercel production report maps Production to sentry-docs, and the catalog names that same id.
develop-docs does not ship this change. The new routes sit in userDocsRedirects only.

Risk

A wrong redirect source can steal another integrations URL and send readers to the index.
A broken line on that index can fail the page render.
The new sources are exact paths, so sibling integration routes should hold.
redirects.js feeds every user-docs redirect. A syntax error fails the build and does not ship.

Intended effect

The new listing and the two redirects are the intended effect. Absent means the index still loads while Holmes is missing, or the Holmes paths still 404.

Signal Baseline Rule Source
Holmes row on the third-party integrations index Live index at 2026-10-08T16:21:27Z lists FML then Kawach.ai and has no Holmes row Confirmed when Holmes appears on that index. Absent if Holmes is still missing GET https://docs.sentry.io/integrations/third-party-integrations/
Holmes paths redirect GET https://docs.sentry.io/integrations/holmes/ returned Page Not Found at 2026-10-08T16:21:27Z. Sentry url.path counts for /integrations/holmes/ and /organization/integrations/holmes/ were 0 from 2026-10-07T16:21:27Z to 2026-10-08T16:21:27Z Confirmed when both paths return 301 to /integrations/third-party-integrations/. Absent if they still 404 or do not redirect GET https://docs.sentry.io/integrations/holmes/ and GET https://docs.sentry.io/organization/integrations/holmes/. The org path was not fetched live before deploy

Regression watch

This diff can break the integrations index or other integrations URLs if a redirect source is too broad. Watch client errors and 5xx as the symptom class. Watch integrations 404s and index traffic on the same path family.

Signal Baseline Rule Source
Client errors 98 events from 2026-10-07T16:21:27Z to 2026-10-08T16:21:27Z. Share 0.49% of 20124 docs.page.load counts Hold this 24h share. Escalate if the share rises above 0.49% Sentry org sentry, project docs. Dataset errors. Query environment:production. Compare count() to docs.page.load
http.server 5xx 0 spans from 2026-10-07T16:21:27Z to 2026-10-08T16:21:27Z Hold at 0 Sentry org sentry, project docs. Dataset spans. Query span.op:http.server AND http.status_code:>=500 AND environment:production
Integrations 404s 14 docs.page.not_found counts on requested_path integrations* from 2026-10-07T16:21:27Z to 2026-10-08T16:21:27Z. Path integrations/holmes was 0 Hold near 14 per 24h. Escalate if integrations 404s rise, or if integrations/holmes appears after the redirects ship Sentry org sentry, project docs. Metric docs.page.not_found. Query metric.name:docs.page.not_found AND metric.type:counter AND environment:production AND requested_path:integrations*
Third-party integrations index volume 2010 spans with url.path /integrations/third-party-integrations/ from 2026-10-07T16:21:27Z to 2026-10-08T16:21:27Z. Integrations docs.page.load was 775 counts, 3.85% of 20124 total loads Hold index span volume and the 3.85% page-load share. Escalate if either falls while site-wide loads hold Sentry org sentry, project docs. Spans query url.path:/integrations/third-party-integrations/ environment:production. Metric query metric.name:docs.page.load AND metric.type:counter AND page_type:integrations AND environment:production

Not observable

Next.js permanent redirects do not emit Sentry spans. Sibling path /integrations/fml/ was also 0 in the same 24h window, so a 301 cannot be confirmed from traces.
docs.page.not_found runs in client JS. The live Holmes URL 404s while that metric stayed 0, so crawler 404s are missed.
No hourly series was read. Share bands use one 24h sample from 2026-10-07T16:21:27Z to 2026-10-08T16:21:27Z.
Traces sample at 30%. Pageload transaction names collapse to /:path*?, so path checks must use url.path.

Get Slack DMs about this change: cursor.com/automations/rollouts

@souredoutlook
souredoutlook enabled auto-merge (squash) October 8, 2026 16:21
@souredoutlook
souredoutlook merged commit ce52261 into master Oct 8, 2026
29 of 30 checks passed
@souredoutlook
souredoutlook deleted the cursor/add-holmes-integration-b068 branch October 8, 2026 18:27
@cursor

cursor Bot commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

Verified in production

  • Holmes still lists on the live third-party index.

This branch was successfully deployed

2 active deployments
Preview – sentry-docs — 7adc06c7 Deployed Oct 8, 2026 by vercel[bot]
Preview – develop-docs — 7adc06c7 Deployed Oct 8, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Priority: Normal Docs review has no urgent deadline

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants