Skip to content

docs(pagerduty): add the PagerDuty integration guide sections - #1248

Merged
EricGustin merged 3 commits into
mainfrom
ericgustin/pagerduty-integration-guide
Oct 18, 2026
Merged

EricGustin merged 3 commits into
mainfrom
ericgustin/pagerduty-integration-guide

Conversation

@EricGustin

@EricGustin EricGustin commented Oct 16, 2026 •

Copy link
Copy Markdown
Member

Closes TOO-2603

Adds the sections from PagerDuty's integration guide template to the PagerDuty toolkit page, so PagerDuty can review the integration for its Integration Directory and publish Arcade's OAuth app.

  • New curation sections, shown below the tools table: benefits, how it works, requirements, support, an integration walkthrough (install in PagerDuty, set up in Arcade), and how to uninstall.
  • The top callout now sends Arcade Cloud users to the walkthrough. The register-your-own-app guidance stays for self-hosted deployments and agents built for people outside the project.
  • Removes the auth chunk that repeated the auth provider page.
  • Re-syncs pagerduty.json (documentationChunks and curationSourceHash only), so this preview shows the page, and raises the curation corpus test's chunk count to match.

Verified: validate-curation passes for all 82 toolkits, the 529 toolkit-docs-generator tests pass, and the page renders locally with the guide after the tools table and before each tool's details.

Before merging: confirm the install link (https://<subdomain>.pagerduty.com/oauth-apps/PHRMXL1) in the arcadeai PagerDuty account.


Note

Low Risk
Documentation and generated toolkit JSON curation only; no product or auth runtime changes.

Overview
Expands the PagerDuty toolkit docs with integration-guide sections (benefits, how it works, requirements, support, walkthrough, uninstall) as curated MDX chunks placed after_available_tools, synced into pagerduty.json.

The description callout is consolidated into a single info callout: Arcade Cloud users are directed to install Arcade’s OAuth app via the walkthrough, while self-hosted or multi-tenant setups still point to registering a Scoped OAuth app. The separate warning callout and the auth chunk (Bearer/OAuth boilerplate) are removed.

The curation corpus test expectation for total chunk count is updated 89 → 93.

Reviewed by Cursor Bugbot for commit f5cc85a. Bugbot is set up for automated code reviews on this repo. Configure here.

PagerDuty's Integration Directory review asks for an integration guide
that follows its template and has a support section. Add curation chunks
that give the PagerDuty toolkit page those sections: benefits, how it
works, requirements, support, an integration walkthrough (install in
PagerDuty, set up in Arcade), and how to uninstall.

- Replace the two top callouts, which told every reader to register
  their own Scoped OAuth app, with one that sends Arcade Cloud users to
  the walkthrough and keeps the own-app guidance for self-hosted
  deployments and agents built for people outside the project.
- Drop the auth chunk, which repeated the auth provider page and pointed
  at scopes "shown above".
- Write the support address with a <wbr> before the @ so Cloudflare's
  email obfuscation doesn't turn it into a link that 404s for crawlers.
- Re-sync pagerduty.json (documentationChunks and curationSourceHash
  only) so the preview deploy shows the page.
@vercel

vercel Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
docs Ready Ready Preview Oct 16, 2026 10:45pm UTC

Request Review

@EricGustin
EricGustin marked this pull request as ready for review October 16, 2026 22:28
… curation corpus

The PagerDuty guide adds six chunks and removes two, so the checked-in corpus now holds 93 chunks.
… halve it

The guide sat between the overview and the tools table, so readers scrolled past about 1,050 words before reaching the tools. Render it in the after_available_tools slot instead, and cut it to about 540 words while keeping every section from PagerDuty's template.

@samkujovich samkujovich left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm - seems like what they were looking for

@EricGustin
EricGustin merged commit faf89cd into main Oct 18, 2026
8 checks passed

This branch was successfully deployed

1 active deployment
Preview — f5cc85af Deployed Oct 16, 2026 by vercel[bot]
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.

2 participants