Skip to content

chore(cli): migrate @fern-api/openapi-ir-parser to use CliError#15007

Open
FedeZara wants to merge 1 commit intomainfrom
FedeZara/refactor/cli-error-openapi-ir-parser
Open

chore(cli): migrate @fern-api/openapi-ir-parser to use CliError#15007
FedeZara wants to merge 1 commit intomainfrom
FedeZara/refactor/cli-error-openapi-ir-parser

Conversation

@FedeZara
Copy link
Copy Markdown
Contributor

@FedeZara FedeZara commented Apr 14, 2026

Description

Migrates @fern-api/openapi-ir-parser to use explicit CliError error codes on every failAndThrow / failWithoutThrowing call site.

This is one of the package migration PRs that follow the error classification system introduced in #14749.

Changes Made

Assigned typed error codes across call sites in the OpenAPI IR parser package.

Error codes used

Code Usage
PARSE_ERROR Malformed OpenAPI specs, invalid schema structures
VALIDATION_ERROR Schema validation failures, unsupported OpenAPI features
REFERENCE_ERROR Unresolvable $ref references in OpenAPI specs

Files touched (grouped by area)

  • OpenAPI parsing: Various parser files in the openapi-ir-parser package

Testing

  • Existing tests pass (pnpm test, excluding e2e and pre-existing environment failures)

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@FedeZara FedeZara changed the title refactor(cli): rename FernCliError to TaskAbortSignal chore(cli): migrate @fern-api/openapi-ir-parser to use CliError Apr 14, 2026
@FedeZara FedeZara force-pushed the FedeZara/refactor/cli-error-openapi-ir-parser branch 2 times, most recently from 0a6ff29 to 6eff7bf Compare April 15, 2026 12:51
@FedeZara FedeZara force-pushed the FedeZara/refactor/cli-error-openapi-ir-parser branch from 6eff7bf to ee9ca3c Compare April 17, 2026 15:11
@FedeZara FedeZara marked this pull request as ready for review April 17, 2026 15:12
Copy link
Copy Markdown

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

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.

Tip: disable this comment in your organization's Code Review settings.

@github-actions
Copy link
Copy Markdown
Contributor

SDK Generation Benchmark Results

Comparing PR branch against latest nightly baseline on main (2026-04-17T04:58:39Z).

Full benchmark table (click to expand)
Generator Spec main (generator) main (E2E) PR (generator) Delta
csharp-sdk square 98s 141s 42s -56s (-57.1%)
go-sdk square 114s 148s 55s -59s (-51.8%)
java-sdk square 170s 187s 120s -50s (-29.4%)
php-sdk square 99s 127s 43s -56s (-56.6%)
python-sdk square 114s 152s 62s -52s (-45.6%)
ruby-sdk-v2 square 102s 156s 67s -35s (-34.3%)
rust-sdk square 98s 94s 51s -47s (-48.0%)
swift-sdk square 84s 130s 40s -44s (-52.4%)
ts-sdk square 100s 138s 51s -49s (-49.0%)

main (generator): generator-only time via --skip-scripts (includes Docker image build, container startup, IR parsing, and code generation — this is the same Docker-based flow customers use via fern generate). main (E2E): full customer-observable time including build/test scripts (nightly baseline, informational). Delta is computed against generator-only baseline.
⚠️ = generation exited with a non-zero exit code (timing may not reflect a successful run).
Baseline from nightly runs on main (latest: 2026-04-17T04:58:39Z). Trigger benchmark-baseline to refresh.
Last updated: 2026-04-17 15:29 UTC

@github-actions
Copy link
Copy Markdown
Contributor

Docs Generation Benchmark Results

Comparing PR branch against latest nightly baseline on main (2026-04-17T04:58:39Z).

Fixture main PR Delta
docs 289.7s 332.5s (35 versions) +42.8s (+14.8%)

Docs generation runs fern generate --docs --preview end-to-end against the benchmark fixture with 35 API versions (each version: markdown processing + OpenAPI-to-IR + FDR upload).
Delta is computed against the nightly baseline on main.
Baseline from nightly run(s) on main (latest: 2026-04-17T04:58:39Z). Trigger benchmark-baseline to refresh.
Last updated: 2026-04-17 15:32 UTC

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant