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

Remove the temporary UNCLASSIFIED error code and revert catch-all
handlers back to INTERNAL_ERROR (which is Sentry-reportable). This
should be merged last, after all CliError classification branches.

Made-with: Cursor

Author: FedeZara

packages/cli/task-context/src/CliError.ts

@@ -10,8 +10,7 @@ export const CliErrorCode = {
     ValidationError: "VALIDATION_ERROR",
     NetworkError: "NETWORK_ERROR",
     AuthError: "AUTH_ERROR",
-    ConfigError: "CONFIG_ERROR",
-    Unclassified: "UNCLASSIFIED"
+    ConfigError: "CONFIG_ERROR"
 } as const;
 
 export type CliErrorCode = (typeof CliErrorCode)[keyof typeof CliErrorCode];
@@ -28,8 +27,7 @@ const SENTRY_REPORTABLE: Record<CliErrorCode, boolean> = {
     VALIDATION_ERROR: false,
     NETWORK_ERROR: false,
     AUTH_ERROR: false,
-    CONFIG_ERROR: false,
-    UNCLASSIFIED: false
+    CONFIG_ERROR: false
 };
 
 export function shouldReportToSentry(code: CliErrorCode): boolean {
@@ -49,8 +47,7 @@ function isNodeVersionError(error: unknown): boolean {
 /**
  * Resolves the effective error code: explicit override wins,
  * then auto-detects from known error types,
- * and falls back to UNCLASSIFIED for unknown errors until all packages
- * are migrated to the new error system.
+ * and falls back to INTERNAL_ERROR for truly unknown errors.
  */
 export function resolveErrorCode(error: unknown, explicitCode?: CliErrorCode): CliErrorCode {
     if (explicitCode != null) {
@@ -65,7 +62,7 @@ export function resolveErrorCode(error: unknown, explicitCode?: CliErrorCode): C
     if (isNodeVersionError(error)) {
         return "ENVIRONMENT_ERROR";
     }
-    return "UNCLASSIFIED";
+    return "INTERNAL_ERROR";
 }
 
 export class CliError extends Error {