Repository navigation
chore: update OpenAPI to d35bb4402f38 - #430
Conversation
commit: |
|
LGTM! |
|
💬 Approved with comments — A clean regenerated pin bump with a correct changeset, and the earlier bulk-delete array-flag issue is fixed. The only note is an INFO that most generated files were not checked individually. Findings: 1 info The decision is advisory: this comment never approves, blocks or requests changes on the pull request. 📋 Code Quality: 💬 1 infoThe earlier bulk-delete array-flag warning is resolved; most generated files were not checked individually. Recommendation: 💬 Optional Nits
View full reviewThe earlier WARNING about
Most of the other regenerated files were not checked individually. ✅ All other checks passed🚀 Release: ✅ LGTMRisk: Low. The 🔒 Security: ✅ LGTMNo new auth handling, secret handling or ad-hoc fetches. The pin bump and the removed modules only change generated output. 📖 AGENTS.md: ✅ LGTMThe output was regenerated rather than hand-edited, and a changeset is included. ⚡ Performance: ✅ LGTMNew roots are registered through the existing lazy generated index. 📝 Documentation: ✅ LGTMCommand descriptions come from the upstream OpenAPI document. Reviewed commit: |
|
| mean | stddev | min | max | |
|---|---|---|---|---|
| PR | 289.6 ms | 3.6 ms | 282.3 ms | 295.9 ms |
| base | 291.5 ms | 4.6 ms | 279.5 ms | 299.9 ms |
Δ (PR vs base): ↓ -1.9 ms (-0.6%)
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.
a624dcc to
4ae252b
Compare
|
LGTM |
4ae252b to
d65d0ae
Compare
|
Review: 1 findings. |
d65d0ae to
08b6f48
Compare
|
🤖 I've done a thorough review of your PR. Updates the pinned Forge OpenAPI revision and regenerates the CLI command surface.
|
08b6f48 to
a4b9d95
Compare
|
Review: 1 findings. Updates the Forge OpenAPI revision and regenerated command surface.
|
a4b9d95 to
77d4574
Compare
|
Review: 1 findings. Updates the OpenAPI pin and regenerated CLI surface. Updated 1 existing inline suggestion. |
77d4574 to
bc55574
Compare
|
🤖 I've done a thorough review of your PR. Updates the Forge OpenAPI pin and regenerates the CLI surface.
|
bc55574 to
4ff1b59
Compare
|
🤖 I've done a thorough review of your PR. Updates the pinned Forge OpenAPI revision and regenerated CLI surface.
|
4ff1b59 to
ba2b19e
Compare
|
Review: 1 findings. Posted 1 inline suggestion. |
ba2b19e to
7d97c0e
Compare
|
No new concerns since the last review |
7d97c0e to
64cd952
Compare
|
No new concerns since the last review |
64cd952 to
cc21d08
Compare
|
🤖 I've done a thorough review of your PR. Updates the Forge OpenAPI pin and regenerates the CLI command surface.
|
Updates cf to
openapi@d35bb4402f3864b83e802be14ca7f58eb920528b.d35bb4402f3864b83e802be14ca7f58eb920528bThis PR is maintained automatically by the Update OpenAPI workflow.