chore(cli): re-enable Sentry reporting for unclassified errors

[FedeZara]

Description

Removes the temporary UNCLASSIFIED error code and reverts catch-all handlers back to INTERNAL_ERROR (which is Sentry-reportable).

This is the final PR in the error classification effort started in #14749. It should be merged last, after all ~35 package migration PRs have landed, so that any remaining unclassified new Error(...) calls are treated as genuine internal errors worth investigating in Sentry.

Changes Made

• Removed UNCLASSIFIED from CliError.Code
• Changed default error code fallback from UNCLASSIFIED → INTERNAL_ERROR in:
  • cli.ts (CLI v1 top-level catch)
  • CliContext.ts (CLI v1 task runner)
  • withContext.ts → reportError (CLI v2)

Testing

• Existing tests pass (only changes default string literals)

[github-actions]

🌱 Seed Test Selector

Select languages to run seed tests for:

• Python
• TypeScript
• Java
• Go
• Ruby
• C#
• PHP
• Swift
• Rust
• OpenAPI
• Postman

---

How to use: Click the ⋯ menu above → "Edit" → check the boxes you want → click "Update comment". Tests will run automatically and snapshots will be committed to this PR.

[claude]

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

[Swimburger]

use your code map

[graphite-app]

Invalid syntax: export const CliError.Code = {...} is not valid TypeScript/JavaScript. Dot notation cannot be used in a const declaration outside of a namespace or class.

The original code had this inside a namespace CliError block. This refactor breaks the syntax. The code will fail to compile.

Fix: Either keep the namespace structure or use a different approach:

export namespace CliError {
    export const Code = {
        InternalError: "INTERNAL_ERROR",
        // ... rest of codes
    } as const;
    
    export type Code = (typeof Code)[keyof typeof Code];
}

Suggested change

	export const CliError.Code = {
	export namespace CliError {
	export const Code = {

Spotted by Graphite



Is this helpful? React 👍 or 👎 to let us know.

[FedeZara]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• chore(cli): re-enable Sentry reporting for unclassified errors #14752 👈 (View in Graphite)
• chore(cli): temporarily disable Sentry reporting for unclassified errors #14750 : 1 other dependent PR (#14753 )
• feat(cli): introduce structured CliError with typed error codes and Sentry routing #14749
• chore(cli): eagerly resolve TelemetryClient distinctId via async factory #14748
• chore(cli): make instrumentPostHogEvent synchronous #14747
• chore(cli): rename FernCliError to TaskAbortSignal #14746
• main

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