Skip to content

fix(shared): register live runtime error codes - #58

Merged
vastsa merged 2 commits into
vastsa:mainfrom
1sdv:error-code-registry-20260907
Sep 7, 2026
Merged

fix(shared): register live runtime error codes#58
vastsa merged 2 commits into
vastsa:mainfrom
1sdv:error-code-registry-20260907

Conversation

@1sdv

@1sdv 1sdv commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Summary

Register live runtime and Edit error codes that were emitted but missing from the shared ErrorCodes registry.

Changes

  • Add CONTEXT_COMPACTION_FAILED and EMPTY_MODEL_RESPONSE.
  • Add all live EDIT_* error codes.
  • Add regression coverage in packages/shared/src/errors.test.ts.
  • Update the error-code specification and E2E test plan.

Validation

  • git diff --check passed.
  • pnpm --filter @pi-desktop/shared test --run passed.
  • pnpm --filter @pi-desktop/shared typecheck passed.
  • Tested with Node.js >=22.19.0 and pnpm 11.18.0.

Scope

Reserved plugin and host-internal codes remain unchanged because they are not part of the shared live AppError
registry.

@vercel

vercel Bot commented Sep 7, 2026

Copy link
Copy Markdown

@1sdv is attempting to deploy a commit to the vastsa's projects Team on Vercel.

A member of the Team first needs to authorize it.

@vastsa
vastsa merged commit 41b7998 into vastsa:main Sep 7, 2026
1 check failed
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