Skip to content

📝 Add schema view and drift comparison to API endpoints docs#37785

Merged
mauricesvay merged 23 commits into
masterfrom
msvay/api-endpoints-schema-view
Jul 3, 2026
Merged

📝 Add schema view and drift comparison to API endpoints docs#37785
mauricesvay merged 23 commits into
masterfrom
msvay/api-endpoints-schema-view

Conversation

@mauricesvay

Copy link
Copy Markdown
Contributor

What does this PR do? What is the motivation?

Adds a new "View and compare API schemas" section to the API endpoints page, covering:

  • How to read an endpoint's inferred OpenAPI schema (Preview and YAML views)
  • How to compare declared vs inferred schemas to detect drift
  • Setup requirements (AAP enabled, Catalog registration)

Merge readiness

  • Ready for merge

AI assistance

Used Claude Code for initial draft.

Additional notes

@github-actions

Copy link
Copy Markdown
Contributor

Preview links (active after the build_preview check completes)

Modified Files

@datadog-prod-us1-5

This comment has been minimized.

mauricesvay and others added 5 commits July 1, 2026 13:47
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Expanded the comparison of declared and inferred schemas by adding severity levels and detailed descriptions of potential differences. Included specific areas of schema changes that can occur and clarified which differences are excluded to reduce noise. This improves the clarity and usability of the documentation for users comparing API schemas.
Explain that the inferred schema filters fields by observation count and recency, reducing false-positive drift from one-off or attacker traffic.
@mauricesvay mauricesvay force-pushed the msvay/api-endpoints-schema-view branch from 592c75a to a59cc1c Compare July 1, 2026 11:49
@mauricesvay

Copy link
Copy Markdown
Contributor Author

/review

@mauricesvay mauricesvay marked this pull request as ready for review July 1, 2026 11:57
@mauricesvay mauricesvay requested a review from a team as a code owner July 1, 2026 11:57

@github-actions github-actions Bot 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.

🤖 Automated review by Claude. AI-generated; verify before acting.

Nice cleanup — pass on the terminology and style changes, plus a solid new section on schema comparison. A few small style/consistency nits below.

Reviewed a59cc1c0cc76a067bcd85d17fd8b65e31851c4cdworkflow run

mauricesvay and others added 4 commits July 1, 2026 14:09
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…tory/api_endpoints.md

Co-authored-by: domalessi <111786334+domalessi@users.noreply.github.com>

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

Left some feedback. Let me know if you have any Qs, and hit me up when ready for another look!

@mauricesvay mauricesvay requested a review from domalessi July 2, 2026 07:28
mauricesvay and others added 10 commits July 2, 2026 09:40
…tory/api_endpoints.md

Co-authored-by: domalessi <111786334+domalessi@users.noreply.github.com>
…tory/api_endpoints.md

Co-authored-by: domalessi <111786334+domalessi@users.noreply.github.com>
…tory/api_endpoints.md

Co-authored-by: domalessi <111786334+domalessi@users.noreply.github.com>
…tory/api_endpoints.md

Co-authored-by: domalessi <111786334+domalessi@users.noreply.github.com>
…tory/api_endpoints.md

Co-authored-by: domalessi <111786334+domalessi@users.noreply.github.com>
…tory/api_endpoints.md

Co-authored-by: domalessi <111786334+domalessi@users.noreply.github.com>
…tory/api_endpoints.md

Co-authored-by: domalessi <111786334+domalessi@users.noreply.github.com>
…tory/api_endpoints.md

Co-authored-by: domalessi <111786334+domalessi@users.noreply.github.com>
…tory/api_endpoints.md

Co-authored-by: domalessi <111786334+domalessi@users.noreply.github.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@github-actions github-actions Bot added the Images Images are added/removed with this PR label Jul 2, 2026

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

Two v minor editorial suggestions, and then a note about the screenshot. Otherwise, this is good to go! I'll approve so you can proceed once the screenshot is cropped.

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.

Let's crop this photo to cut out the lefthand edge with the product symbols. That left-hand nav tends to change frequently, and we also want to keep out that little face pic lol

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sorry, I merged before doing the update. I'll open another PR for the edits.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Screenshot is being cropped in #37946

Co-authored-by: domalessi <111786334+domalessi@users.noreply.github.com>
@mauricesvay mauricesvay merged commit 0fec1e6 into master Jul 3, 2026
17 checks passed
@mauricesvay mauricesvay deleted the msvay/api-endpoints-schema-view branch July 3, 2026 08:15
mauricesvay added a commit that referenced this pull request Jul 6, 2026
* 📝 Crop API endpoint schema screenshot

Removes the left-hand nav from the screenshot per review feedback on #37785.

* fix: use new filename for cropped API endpoint schema screenshot

Avoids cache invalidation issues by keeping the original image
filename intact and referencing the cropped version under a new name.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Images Images are added/removed with this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants