Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
82353b8 to
00ddca9
Compare
…_ERROR wrappers at call sites
00ddca9 to
b0e4181
Compare
There was a problem hiding this comment.
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.
SDK Generation Benchmark ResultsComparing PR branch against latest nightly baseline on Full benchmark table (click to expand)
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 |
Docs Generation Benchmark ResultsComparing PR branch against latest nightly baseline on
Docs generation runs |

Description
Keeps
@fern-api/docker-utilsas a plain Error utility package and addsCONTAINER_ERRORwrappers at the call sites that invoke docker-utils functions.This is one of the package migration PRs that follow the error classification system introduced in #14749.
Changes Made
Since
docker-utilsis a low-level utility that doesn't depend on@fern-api/task-context, the package itself is left unchanged. Instead, call sites in higher-level packages now wrap docker-utils errors withCliError.Code.ContainerError.Error codes used
CONTAINER_ERRORFiles touched (grouped by area)
CONTAINER_ERRORcodeTesting
pnpm test, excluding e2e and docker-utils tests which fail due to workspace path containing spaces — pre-existing environment issue)