Skip to content

chore: update OpenAPI to 59bd82aa6c34 - #369

Merged
dario-piotrowicz merged 1 commit into
mainfrom
automation/update-forge
Oct 20, 2026
Merged

dario-piotrowicz merged 1 commit into
mainfrom
automation/update-forge

Conversation

@ant-release-bot

Copy link
Copy Markdown
Collaborator

Updates cf to openapi@59bd82aa6c349b9097e34621bd15c4ddf46d6dd1.

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

This PR is maintained automatically by the Update OpenAPI workflow.

@ask-bonk

ask-bonk Bot commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

LGTM

github run

@pkg-pr-new

pkg-pr-new Bot commented Oct 20, 2026

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

commit: 6a5068b

@clanky-the-clanker

Copy link
Copy Markdown

✅ Approved — This automated OpenAPI pin bump moves to the new revision, and the regenerated output is consistent with it. The only command change is that cf zaraz is no longer hidden. No issues found.

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


✅ All checks passed

📋 Code Quality: ✅ LGTM

This pull request is an automated OpenAPI bump. FORGE_OPENAPI_VERSION in packages/cli/generate.ts moves to 59bd82aa6c349b9097e34621bd15c4ddf46d6dd1. The regenerated output now shows the zaraz root: hideCommand goes from true to false in _generated/index.ts, and the same visibility flag changes on the 10 zaraz leaves in _meta/commands.json.

  • The root flag in index.ts and the leaf flags in the catalogue diff agree with each other.
  • A minor changeset (.changeset/update-openapi-59bd82aa6c34.md) is included, as the repository's conventions require for OpenAPI updates.
  • No unit tests under src/__tests__ check whether zaraz is visible.
  • This review did not compare the e2e fixture e2e/zaraz.json against the new visibility. That suite runs against a real account outside normal CI.

📝 Documentation: ✅ LGTM

The only documentation in this pull request is the generated changeset. It is well formed: it has "cf": minor frontmatter and names the pinned OpenAPI revision. No other documentation needs updating for a pin bump.


Reviewed commit: 6a5068b2

@github-actions

Copy link
Copy Markdown
Contributor

cf --help startup

mean stddev min max
PR 216.7 ms 3.4 ms 211.6 ms 221.5 ms
base 216.6 ms 7.8 ms 210.0 ms 244.2 ms

Δ (PR vs base): — +0.1 ms (+0.1%)

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.

@dario-piotrowicz
dario-piotrowicz added this pull request to the merge queue Oct 20, 2026
Merged via the queue into main with commit 38091b1 Oct 20, 2026
12 checks passed
@dario-piotrowicz
dario-piotrowicz deleted the automation/update-forge branch October 20, 2026 13:57
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