fix: resolve TypeScript CI errors from victor's branch merge

- registry.ts: remove duplicate Command/OptionDef/CommandSpec/defineCommand definitions, import from command.ts; change printHelp param type to NodeJS.WriteStream to accept process.stderr
- utils/schema.ts: fix unknown type access on input_schema properties
- test files: add missing nonInteractive/async fields to all Config and GlobalFlags objects

Made-with: Cursor

Author: RyanLee-Dev

bun.lock

@@ -1,4 +1,4 @@
-import { defineCommand } from './command';
+import type { Command } from './command';
 import { CLIError } from './errors/base';
 import { ExitCode } from './errors/codes';
 
@@ -25,44 +25,7 @@ import fileList from './commands/file/list';
 import fileDelete from './commands/file/delete';
 import update from './commands/update';
 
-import type { Config } from './config/schema';
-import type { GlobalFlags } from './types/flags';
-
-export interface OptionDef {
-  flag: string;
-  description: string;
-  type?: 'string' | 'number' | 'boolean' | 'array';
-  required?: boolean;
-}
-
-export interface Command {
-  name: string;
-  description: string;
-  usage?: string;
-  options?: OptionDef[];
-  examples?: string[];
-  execute(config: Config, flags: GlobalFlags): Promise<void>;
-}
-
-export interface CommandSpec {
-  name: string;
-  description: string;
-  usage?: string;
-  options?: OptionDef[];
-  examples?: string[];
-  run(config: Config, flags: GlobalFlags): Promise<void>;
-}
-
-export function defineCommand(spec: CommandSpec): Command {
-  return {
-    name: spec.name,
-    description: spec.description,
-    usage: spec.usage,
-    options: spec.options,
-    examples: spec.examples,
-    execute: spec.run,
-  };
-}
+export type { Command, OptionDef } from './command';
 
 interface CommandNode {
   command?: Command;
@@ -145,7 +108,7 @@ class CommandRegistry {
    * Defaults to stdout; pass stderr (or a non-TTY stream) to keep stdout
    * clean for piped / JSON output.
    */
-  printHelp(commandPath: string[], out: typeof process.stdout = process.stdout): void {
+  printHelp(commandPath: string[], out: NodeJS.WriteStream = process.stdout): void {
     if (commandPath.length === 0) {
       this.printRootHelp(out);
       return;
@@ -173,7 +136,7 @@ class CommandRegistry {
     out.write('\n');
   }
 
-  private printRootHelp(out: typeof process.stdout): void {
+  private printRootHelp(out: NodeJS.WriteStream): void {
     out.write(`
   __  __ ___ _   _ ___ __  __    _   __  __
  |  \\/  |_ _| \\ | |_ _|  \\/  |  / \\ \\ \\/ /
@@ -219,7 +182,7 @@ Getting Help:
 `);
   }
 
-  private printCommandHelp(cmd: Command, out: typeof process.stdout): void {
+  private printCommandHelp(cmd: Command, out: NodeJS.WriteStream): void {
     out.write(`\n${cmd.description}\n`);
     if (cmd.usage) out.write(`Usage: ${cmd.usage}\n`);
     if (cmd.options && cmd.options.length > 0) {
@@ -241,7 +204,7 @@ Getting Help:
     out.write(`Run 'minimax --help' for the full list.\n`);
   }
 
-  private printChildren(node: CommandNode, prefix: string, out: typeof process.stdout): void {
+  private printChildren(node: CommandNode, prefix: string, out: NodeJS.WriteStream): void {
     for (const [name, child] of node.children) {
       if (child.command) {
         out.write(`  ${prefix} ${name.padEnd(12)} ${child.command.description}\n`);

src/registry.ts

@@ -66,10 +66,11 @@ export function generateToolSchema(cmd: Command): Record<string, unknown> {
         propSchema.type = effectiveType;
       }
 
-      (schema.input_schema as Record<string, unknown>).properties[name] = propSchema;
+      const inputSchema = schema.input_schema as Record<string, unknown>;
+      (inputSchema.properties as Record<string, unknown>)[name] = propSchema;
 
       if (opt.required) {
-        (schema.input_schema.required as string[]).push(name);
+        (inputSchema.required as string[]).push(name);
       }
     }
   }

src/utils/schema.ts

@@ -13,6 +13,8 @@ function makeConfig(overrides: Partial<Config> = {}): Config {
     noColor: false,
     yes: false,
     dryRun: false,
+    nonInteractive: false,
+    async: false,
     ...overrides,
   };
 }

test/auth/resolver.test.ts

@@ -15,6 +15,8 @@ function makeConfig(baseUrl: string): Config {
     noColor: false,
     yes: false,
     dryRun: false,
+    nonInteractive: false,
+    async: false,
   };
 }
 

test/client/http.test.ts

@@ -18,6 +18,8 @@ describe('auth login command', () => {
       noColor: true,
       yes: false,
       dryRun: false,
+      nonInteractive: false,
+      async: false,
     };
 
     await expect(
@@ -29,6 +31,8 @@ describe('auth login command', () => {
         yes: false,
         dryRun: false,
         help: false,
+        nonInteractive: false,
+        async: false,
       }),
     ).rejects.toThrow('--api-key is required');
   });

test/commands/auth/login.test.ts

@@ -17,6 +17,8 @@ describe('auth logout command', () => {
       noColor: true,
       yes: false,
       dryRun: true,
+      nonInteractive: false,
+      async: false,
     };
 
     const originalLog = console.log;
@@ -31,6 +33,8 @@ describe('auth logout command', () => {
         yes: false,
         dryRun: true,
         help: false,
+        nonInteractive: false,
+        async: false,
       });
 
       expect(output).toContain('No changes made');

test/commands/auth/logout.test.ts

@@ -17,6 +17,8 @@ describe('auth refresh command', () => {
       noColor: true,
       yes: false,
       dryRun: false,
+      nonInteractive: false,
+      async: false,
     };
 
     await expect(
@@ -27,6 +29,8 @@ describe('auth refresh command', () => {
         yes: false,
         dryRun: false,
         help: false,
+        nonInteractive: false,
+        async: false,
       }),
     ).rejects.toThrow('not authenticated via OAuth');
   });

test/commands/auth/refresh.test.ts

@@ -17,6 +17,8 @@ describe('auth status command', () => {
       noColor: true,
       yes: false,
       dryRun: false,
+      nonInteractive: false,
+      async: false,
     };
 
     const originalLog = console.log;
@@ -31,6 +33,8 @@ describe('auth status command', () => {
         yes: false,
         dryRun: false,
         help: false,
+        nonInteractive: false,
+        async: false,
       });
 
       const parsed = JSON.parse(output);

test/commands/auth/status.test.ts

@@ -17,6 +17,8 @@ describe('config set command', () => {
       noColor: true,
       yes: false,
       dryRun: false,
+      nonInteractive: false,
+      async: false,
     };
 
     await expect(
@@ -27,6 +29,8 @@ describe('config set command', () => {
         yes: false,
         dryRun: false,
         help: false,
+        nonInteractive: false,
+        async: false,
       }),
     ).rejects.toThrow('--key and --value are required');
   });
@@ -42,6 +46,8 @@ describe('config set command', () => {
       noColor: true,
       yes: false,
       dryRun: false,
+      nonInteractive: false,
+      async: false,
     };
 
     await expect(
@@ -54,6 +60,8 @@ describe('config set command', () => {
         yes: false,
         dryRun: false,
         help: false,
+        nonInteractive: false,
+        async: false,
       }),
     ).rejects.toThrow('Invalid config key');
   });