fix(cli): package install/publish and plugin sign let the exit signal through their catch

A `this.exit(1)` inside a `try` throws oclif's exit signal, and these
commands' catches reported it as a second error line (`✗ EEXIT: 1`).
Each affected catch now opens with the `isExitSignal` rethrow, and the
exit-signal pin's population is widened from JSON-capable commands to
every command.

Co-Authored-By: Claude <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_016GiHYRmLSNWTfbX9gVQkpz

Author: claude

packages/cli/src/commands/package/install.ts

@@ -26,7 +26,7 @@ import { readFile } from 'node:fs/promises';
 import { existsSync } from 'node:fs';
 import { resolve as resolvePath } from 'node:path';
 import { Args, Command, Flags } from '@oclif/core';
-import { printHeader, printKV, printSuccess, printError, printStep } from '../../utils/format.js';
+import { printHeader, printKV, printSuccess, printError, printStep, isExitSignal } from '../../utils/format.js';
 
 export default class PackageInstall extends Command {
   static override description =
@@ -246,6 +246,7 @@ export default class PackageInstall extends Command {
         console.log(`    ${storageDir}`);
       }
     } catch (error) {
+      if (isExitSignal(error)) throw error;
       printError((error as Error).message);
       this.exit(1);
     }

packages/cli/src/commands/package/publish.ts

@@ -32,7 +32,7 @@ import { readFile } from 'node:fs/promises';
 import { resolve as resolvePath, basename, dirname, isAbsolute } from 'node:path';
 import { Args, Command, Flags } from '@oclif/core';
 import { PackageSchema } from '@objectstack/spec/marketplace';
-import { printHeader, printKV, printSuccess, printError, printStep } from '../../utils/format.js';
+import { printHeader, printKV, printSuccess, printError, printStep, isExitSignal } from '../../utils/format.js';
 import { DEFAULT_CLOUD_URL, tryReadCloudConfig } from '../../utils/cloud-config.js';
 import { resolveCloudActiveEnvironmentId } from '../../utils/active-environment.js';
 import { readErrorMessage } from '../../utils/response-envelope.js';
@@ -665,6 +665,7 @@ export default class PackagePublish extends Command {
           const iconUrl = iconRes.body?.data?.icon_url ?? iconRes.body?.icon_url;
           if (iconUrl) printKV('  Icon URL', String(iconUrl));
         } catch (err: any) {
+          if (isExitSignal(err)) throw err;
           printError(`Cannot read --icon-file '${iconPath}': ${err.message}`);
           this.exit(1);
           return;
@@ -794,6 +795,7 @@ export default class PackagePublish extends Command {
         for (const v of violations) console.log(`    • ${v}`);
       }
     } catch (error) {
+      if (isExitSignal(error)) throw error;
       printError((error as Error).message);
       this.exit(1);
     }

packages/cli/src/commands/plugin/sign.ts

@@ -19,7 +19,7 @@ import { createPublicKey } from 'node:crypto';
 import { resolve as resolvePath } from 'node:path';
 import { Args, Command, Flags } from '@oclif/core';
 import { parseSignature, signPayload, verifyPayload } from '@objectstack/core';
-import { printError, printHeader, printKV, printStep, printSuccess } from '../../utils/format.js';
+import { isExitSignal, printError, printHeader, printKV, printStep, printSuccess } from '../../utils/format.js';
 import { OSPLUGIN_EXT } from '../../utils/osplugin.js';
 
 export default class PluginSign extends Command {
@@ -99,6 +99,7 @@ export default class PluginSign extends Command {
         return;
       }
     } catch (err) {
+      if (isExitSignal(err)) throw err;
       printError(`Self-verification error: ${(err as Error).message}`);
       this.exit(1);
       return;

…es/cli/test/json-exit-signal.pin.test.ts packages/cli/test/exit-signal.pin.test.tspackages/cli/test/json-exit-signal.pin.test.ts renamed to packages/cli/test/exit-signal.pin.test.ts packages/cli/test/json-exit-signal.pin.test.ts renamed to packages/cli/test/exit-signal.pin.test.ts

@@ -369,7 +369,6 @@ for (const file of COMMAND_FILES) {
 const POPULATION: JsonCommand[] = [];
 for (const [cls, file] of classFile) {
   const faces = jsonFaces(cls as CommandClass);
-  if (faces.length === 0) continue;
   const chain: string[] = [];
   for (let k: unknown = cls; k && classFile.has(k); k = Object.getPrototypeOf(k)) chain.push(classFile.get(k)!);
   POPULATION.push({ id: commandId(file), faces, chain });