Skip to content

docs: add OpenAPI snapshot auto-update to CI documentation#2557

Closed
inkeep[bot] wants to merge 2 commits intomainfrom
docs-writer-ai-update-2026-03-05T23-44-14-402Z-dja90i
Closed

docs: add OpenAPI snapshot auto-update to CI documentation#2557
inkeep[bot] wants to merge 2 commits intomainfrom
docs-writer-ai-update-2026-03-05T23-44-14-402Z-dja90i

Conversation

@inkeep
Copy link
Copy Markdown
Contributor

@inkeep inkeep Bot commented Mar 5, 2026

Summary

Updates the Contributing documentation to reflect the new CI automation for OpenAPI snapshot updates introduced in PR #2521.

Changes

  • Added a bullet point to the Continuous Integration section documenting the OpenAPI snapshot auto-update feature

Why

PR #2521 added CI automation that auto-regenerates and commits OpenAPI snapshots when they're out of date on a PR branch. This documentation update helps developers understand that:

  1. They no longer need to manually regenerate OpenAPI snapshots
  2. If they miss the pre-commit hook, CI will catch and fix it automatically
  3. The snapshot diff remains visible in the PR for code review

Triggered by #2521

cc @vnv-varun

@inkeep inkeep Bot requested a review from vnv-varun March 5, 2026 23:45
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Mar 5, 2026

⚠️ No Changeset found

Latest commit: d389c01

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 5, 2026

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

Project Deployment Actions Updated (UTC)
agents-api Ready Ready Preview, Comment Mar 6, 2026 1:58pm
agents-docs Ready Ready Preview, Comment Mar 6, 2026 1:58pm
agents-manage-ui Ready Ready Preview, Comment Mar 6, 2026 1:58pm

Request Review

@pullfrog
Copy link
Copy Markdown
Contributor

pullfrog Bot commented Mar 5, 2026

Documents the new OpenAPI snapshot auto-update CI step so contributors know the pipeline will regenerate and commit a fresh snapshot when it drifts.

  • agents-docs/content/community/contributing/overview.mdx — adds a bullet to the CI checks list describing the OpenAPI snapshot auto-update behavior

Pullfrog  | View workflow run | Using Claude Code | Triggered by Pullfrogpullfrog.com𝕏

Copy link
Copy Markdown
Contributor

@pullfrog pullfrog Bot left a comment

Choose a reason for hiding this comment

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

Single-line docs addition accurately reflects the OpenAPI snapshot auto-update CI job added in #2521. The wording is clear and consistent with the surrounding bullet points.

Pullfrog  | View workflow runpullfrog.com𝕏

@itoqa itoqa Bot added the model-sync Automated model sync from provider APIs label Mar 5, 2026
@itoqa itoqa Bot closed this Mar 6, 2026
@itoqa itoqa Bot reopened this Mar 6, 2026
@itoqa itoqa Bot closed this Mar 6, 2026
@itoqa itoqa Bot reopened this Mar 6, 2026
@itoqa itoqa Bot closed this Mar 6, 2026
@itoqa itoqa Bot reopened this Mar 6, 2026
@tim-inkeep tim-inkeep removed the model-sync Automated model sync from provider APIs label Mar 6, 2026
Copy link
Copy Markdown
Contributor

@claude claude Bot left a comment

Choose a reason for hiding this comment

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

PR Review Summary

(0) Total Issues | Risk: Low

This is a minimal, clean documentation update. All three reviewers (pr-review-docs, pr-review-product, pr-review-consistency) returned no findings.

Review Notes

The single-line addition accurately documents the OpenAPI snapshot auto-update CI feature introduced in PR #2521:

- OpenAPI snapshot auto-update — if the snapshot is out of date, CI regenerates it and commits the fix back to your PR branch

Quality assessment:

  • Accuracy: Correctly describes the CI behavior
  • Placement: Appropriately located in the "Continuous Integration" section
  • Style: Uses active voice, concise language, and em-dash formatting consistent with documentation conventions
  • No issues: No stale framing, no anti-patterns, consistent with surrounding content

✅ APPROVE

Summary: Clean docs update that helps contributors understand they no longer need to manually regenerate OpenAPI snapshots. The wording is clear, accurate, and well-placed. Ship it! 🚀

Reviewers (3)
Reviewer Returned Main Findings Consider While You're Here Inline Comments Pending Recs Discarded
pr-review-docs 0 0 0 0 0 0 0
pr-review-product 0 0 0 0 0 0 0
pr-review-consistency 0 0 0 0 0 0 0
Total 0 0 0 0 0 0 0

@github-actions github-actions Bot deleted a comment from claude Bot Mar 6, 2026
@amikofalvy
Copy link
Copy Markdown
Collaborator

I don't want to pollute our docs with something this small.

@amikofalvy amikofalvy closed this Mar 6, 2026
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.

4 participants