Skip to content

chore: update OpenAPI to c4fbef6545cc - #440

Merged
petebacondarwin merged 1 commit into
mainfrom
automation/update-forge
Oct 28, 2026
Merged

petebacondarwin merged 1 commit into
mainfrom
automation/update-forge

Conversation

@ant-release-bot

@ant-release-bot ant-release-bot commented Oct 28, 2026 •

Copy link
Copy Markdown
Collaborator

Updates cf to openapi@c4fbef6545cc3ae3ef7724398dc12e1fecbd9183.

  • pins OpenAPI revision c4fbef6545cc3ae3ef7724398dc12e1fecbd9183
  • regenerates the committed command surface with the pinned Forge package

This PR is maintained automatically by the Update OpenAPI workflow.

@ant-release-bot
ant-release-bot requested a review from a team as a code owner October 28, 2026 15:29
@ask-bonk

ask-bonk Bot commented Oct 28, 2026

Copy link
Copy Markdown
Contributor

LGTM

github run

@pkg-pr-new

pkg-pr-new Bot commented Oct 28, 2026 •

Copy link
Copy Markdown
npm i https://pkg.pr.new/cf@440

commit: be889fd

@clanky-the-clanker

clanky-the-clanker Bot commented Oct 28, 2026 •

Copy link
Copy Markdown

✅ Approved — The pin bump now comes with consistent regenerated output for the new accounts entitlements get command and a matching minor changeset. No issues found.

The decision is advisory: this comment never approves, blocks or requests changes on the pull request.


✅ All checks passed

🚀 Release: ✅ LGTM

Risk: Low. The minor changeset fits a new generated command.


🔒 Security: ✅ LGTM

The new generated command builds its request path from fixed values and the resolved account ID. Nothing to flag.


📖 AGENTS.md: ✅ LGTM

The change follows the repository conventions: generated output is regenerated rather than hand-edited, and a changeset comes with the new command.


📋 Code Quality: ✅ LGTM

The pin bump at packages/cli/generate.ts:20 now comes with regenerated _generated/ output for cf accounts entitlements get. The operation path, operationId and group description are the same in the operation file, the command file, commands.json and schemas.json. The minor changeset describes a real new command.


⚡ Performance: ✅ LGTM

No performance concerns.


📝 Documentation: ✅ LGTM

No documentation concerns.


Reviewed commit: be889fdd

@github-actions

github-actions Bot commented Oct 28, 2026 •

Copy link
Copy Markdown
Contributor

cf --help startup

mean stddev min max
PR 291.0 ms 3.9 ms 284.7 ms 298.9 ms
base 288.6 ms 4.3 ms 280.2 ms 296.3 ms

Δ (PR vs base): ↑ +2.5 ms (+0.9%)

Measured with hyperfine on ubuntu-latest (3 warmup runs, 20 timed runs). CI numbers carry ±a few ms of runner jitter; treat small deltas as noise.

@ant-release-bot ant-release-bot changed the title chore: update OpenAPI to b9ac7dba95f7 chore: update OpenAPI to d068e3174dc0 Oct 28, 2026
@ant-release-bot
ant-release-bot force-pushed the automation/update-forge branch from d6d4974 to 20aaf15 Compare October 28, 2026 17:28
@ask-bonk

ask-bonk Bot commented Oct 28, 2026

Copy link
Copy Markdown
Contributor

LGTM!

github run

@ant-release-bot ant-release-bot changed the title chore: update OpenAPI to d068e3174dc0 chore: update OpenAPI to c4fbef6545cc Oct 28, 2026
@ant-release-bot
ant-release-bot force-pushed the automation/update-forge branch from 20aaf15 to be889fd Compare October 28, 2026 18:32
@ask-bonk

ask-bonk Bot commented Oct 28, 2026

Copy link
Copy Markdown
Contributor

LGTM!

github run

@petebacondarwin
petebacondarwin added this pull request to the merge queue Oct 28, 2026
Merged via the queue into main with commit d11af6b Oct 28, 2026
12 checks passed
@petebacondarwin
petebacondarwin deleted the automation/update-forge branch October 28, 2026 20:48
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