Merge pull request #20 from MiniMax-AI-Dev/refactor/json-only

refactor: JSON-only config, remove yaml dependency

Author: RyanLee-Dev

bun.lock

@@ -22,8 +22,7 @@
     "prepublishOnly": "bun run build:npm"
   },
   "dependencies": {
-    "@clack/prompts": "^0.7.0",
-    "yaml": "^2.7.1"
+    "@clack/prompts": "^0.7.0"
   },
   "devDependencies": {
     "typescript": "^5.8.3",

package.json

@@ -20,7 +20,7 @@ export async function resolveCredential(config: Config): Promise<ResolvedCredent
 
   // 3. API key from config file
   if (config.fileApiKey) {
-    return { token: config.fileApiKey, method: 'api-key', source: 'config.yaml' };
+    return { token: config.fileApiKey, method: 'api-key', source: 'config.json' };
   }
 
   // 4. Environment variable

src/auth/resolver.ts

@@ -42,7 +42,7 @@ export const GLOBAL_OPTIONS: OptionDef[] = [
   { flag: '--api-key <key>',     description: 'API key' },
   { flag: '--region <region>',   description: 'API region: global, cn' },
   { flag: '--base-url <url>',    description: 'API base URL' },
-  { flag: '--output <format>',   description: 'Output format: text, json, yaml' },
+  { flag: '--output <format>',   description: 'Output format: text, json' },
   { flag: '--timeout <seconds>', description: 'Request timeout', type: 'number' },
   { flag: '--quiet',             description: 'Suppress non-essential output' },
   { flag: '--verbose',           description: 'Print HTTP request/response details' },

src/command.ts

@@ -77,7 +77,7 @@ export default defineCommand({
           );
         }
 
-        // Store key in config.yaml
+        // Store key in config.json
         const existing = readConfigFile() as Record<string, unknown>;
         existing.api_key = key;
         await writeConfigFile(existing);

src/commands/auth/login.ts

@@ -23,7 +23,7 @@ export default defineCommand({
 
     if (config.dryRun) {
       if (creds) console.log('Would remove ~/.minimax/credentials.json');
-      if (hasConfigKey) console.log('Would clear api_key from ~/.minimax/config.yaml');
+      if (hasConfigKey) console.log('Would clear api_key from ~/.minimax/config.json');
       if (!creds && !hasConfigKey) console.log('No credentials to clear.');
       console.log('No changes made.');
       return;
@@ -39,7 +39,7 @@ export default defineCommand({
         const updated = fileConfig as Record<string, unknown>;
         delete updated.api_key;
         await writeConfigFile(updated);
-        process.stderr.write('Cleared api_key from ~/.minimax/config.yaml\n');
+        process.stderr.write('Cleared api_key from ~/.minimax/config.json\n');
       } catch { /* ignore */ }
     }
 

src/commands/auth/logout.ts

@@ -48,9 +48,9 @@ export default defineCommand({
       );
     }
 
-    if (key === 'output' && !['text', 'json', 'yaml'].includes(value)) {
+    if (key === 'output' && !['text', 'json'].includes(value)) {
       throw new CLIError(
-        `Invalid output format "${value}". Valid values: text, json, yaml`,
+        `Invalid output format "${value}". Valid values: text, json`,
         ExitCode.USAGE,
       );
     }

src/commands/config/set.ts

@@ -1,5 +1,4 @@
 import { readFileSync, writeFileSync, existsSync } from 'fs';
-import { parse as parseYaml, stringify as yamlStringify } from 'yaml';
 import { parseConfigFile, REGIONS, type Config, type ConfigFile, type Region } from './schema';
 import { ensureConfigDir, getConfigPath } from './paths';
 import { detectOutputFormat, type OutputFormat } from '../output/formatter';
@@ -9,15 +8,15 @@ export function readConfigFile(): ConfigFile {
   const path = getConfigPath();
   if (!existsSync(path)) return {};
   try {
-    return parseConfigFile(parseYaml(readFileSync(path, 'utf-8')));
+    return parseConfigFile(JSON.parse(readFileSync(path, 'utf-8')));
   } catch {
     return {};
   }
 }
 
 export async function writeConfigFile(data: Record<string, unknown>): Promise<void> {
   await ensureConfigDir();
-  writeFileSync(getConfigPath(), yamlStringify(data), { mode: 0o600 });
+  writeFileSync(getConfigPath(), JSON.stringify(data, null, 2) + '\n', { mode: 0o600 });
 }
 
 export function loadConfig(flags: GlobalFlags): Config {
@@ -31,7 +30,6 @@ export function loadConfig(flags: GlobalFlags): Config {
   const cachedRegion = file.region;
   const region = (explicitRegion || cachedRegion || 'global') as Region;
 
-  // Re-detect if: no explicit region AND (no cached region OR key fingerprint changed)
   const activeKey = apiKey || fileApiKey || envApiKey;
   const keyFingerprint = activeKey ? activeKey.slice(0, 8) : undefined;
   const needsRegionDetection = !explicitRegion

src/config/loader.ts

@@ -8,7 +8,7 @@ export function getConfigDir(): string {
 }
 
 export function getConfigPath(): string {
-  return join(getConfigDir(), 'config.yaml');
+  return join(getConfigDir(), 'config.json');
 }
 
 export function getCredentialsPath(): string {

src/config/paths.ts

@@ -10,12 +10,12 @@ export interface ConfigFile {
   region?: Region;
   region_key_fingerprint?: string;
   base_url?: string;
-  output?: 'text' | 'json' | 'yaml';
+  output?: 'text' | 'json';
   timeout?: number;
 }
 
 const VALID_REGIONS = new Set<string>(['global', 'cn']);
-const VALID_OUTPUTS = new Set<string>(['text', 'json', 'yaml']);
+const VALID_OUTPUTS = new Set<string>(['text', 'json']);
 
 export function parseConfigFile(raw: unknown): ConfigFile {
   if (!raw || typeof raw !== 'object' || Array.isArray(raw)) return {};
@@ -38,7 +38,7 @@ export interface Config {
   fileApiKey?: string;
   region: Region;
   baseUrl: string;
-  output: 'text' | 'json' | 'yaml';
+  output: 'text' | 'json';
   timeout: number;
   verbose: boolean;
   quiet: boolean;