@thinremote/thinr-cli 1.2.0-alpha.1 → 1.3.0

package/commands/profile.js

@@ -1,22 +1,30 @@
 import chalk from 'chalk';
+import { input } from '@inquirer/prompts';
 import {
     listProfiles,
     getActiveProfile,
     getProfile,
+    setActiveProfile,
     setDefaultProfile,
     deleteProfile,
 } from '../lib/config.js';
+import { authenticate } from '../lib/auth.js';
 import { setJsonMode, isJsonMode, printOk, printErr } from '../lib/output.js';
+import { hint, success, label } from '../lib/format.js';
 
 function applyJsonFlag(opts) {
     if (opts.json) setJsonMode(true);
 }
 
 export function profileCommand(program) {
-    const profile = program.command('profile').description('Manage configuration profiles');
+    const profile = program
+        .command('profile')
+        .helpGroup('Configuration:')
+        .description('Manage configuration profiles');
 
     profile
         .command('list')
+        .helpGroup('Profile:')
         .description('List configured profiles')
         .option('-j, --json', 'Output as JSON')
         .action((opts) => {
@@ -37,19 +45,20 @@ export function profileCommand(program) {
                 return;
             }
             if (!names.length) {
-                console.log(chalk.gray('No profiles configured. Run thinr to set one up.'));
+                console.log(hint('No profiles configured. Run thinr to set one up.'));
                 return;
             }
             for (const name of names) {
                 const data = getProfile(name) || {};
-                const marker = name === active ? chalk.green('* ') : '  ';
-                const detail = chalk.gray(`${data.username || ''}@${data.server || name}`);
-                console.log(`${marker}${chalk.bold(name)}  ${detail}`);
+                const marker = name === active ? success('* ') : '  ';
+                const detail = hint(`${data.username || ''}@${data.server || name}`);
+                console.log(`${marker}${label(name)}  ${detail}`);
             }
         });
 
     profile
         .command('current')
+        .helpGroup('Profile:')
         .description('Print the active profile')
         .option('-j, --json', 'Output as JSON')
         .action((opts) => {
@@ -60,7 +69,7 @@ export function profileCommand(program) {
                 return;
             }
             if (!name) {
-                console.log(chalk.gray('No active profile.'));
+                console.log(hint('No active profile.'));
                 return;
             }
             console.log(name);
@@ -68,6 +77,7 @@ export function profileCommand(program) {
 
     profile
         .command('use <name>')
+        .helpGroup('Profile:')
         .description('Set the default profile')
         .option('-j, --json', 'Output as JSON')
         .action((name, opts) => {
@@ -82,11 +92,69 @@ export function profileCommand(program) {
                 printOk({ active: name });
                 return;
             }
-            console.log(chalk.green(`Default profile set to ${name}`));
+            console.log(success(`Default profile set to ${name}`));
+        });
+
+    profile
+        .command('add [name]')
+        .description('Add a new profile via interactive authentication')
+        .option('--no-activate', 'Do not set the new profile as default')
+        .option('-j, --json', 'Output as JSON')
+        .action(async (nameArg, opts) => {
+            applyJsonFlag(opts);
+            let name = nameArg;
+            if (!name) {
+                try {
+                    name = await input({
+                        message: 'Profile name:',
+                        validate: (v) => {
+                            const trimmed = (v || '').trim();
+                            if (!trimmed) return 'Profile name is required';
+                            if (/\s/.test(trimmed)) return 'Profile name cannot contain whitespace';
+                            return true;
+                        },
+                    });
+                } catch (error) {
+                    printErr(error.message || 'Prompt cancelled');
+                    return;
+                }
+            }
+            name = name.trim();
+            if (getProfile(name)) {
+                printErr(`Profile already exists: ${name}`, { code: 'already_exists' });
+                return;
+            }
+            setActiveProfile(name);
+            try {
+                await authenticate();
+                const saved = getProfile(name);
+                if (!saved || !saved.token || !saved.username) {
+                    throw new Error('Authentication did not complete; profile discarded.');
+                }
+            } catch (error) {
+                setActiveProfile(null);
+                try {
+                    deleteProfile(name);
+                } catch {
+                    // Best-effort cleanup — ignore secondary failures
+                }
+                printErr(error.message || 'Authentication failed');
+                return;
+            }
+            const activate = opts.activate !== false;
+            if (activate) setDefaultProfile(name);
+            setActiveProfile(null);
+            if (isJsonMode()) {
+                printOk({ added: name, activated: activate, active: getActiveProfile() });
+                return;
+            }
+            const suffix = activate ? ' and set as default' : '';
+            console.log(chalk.green(`Profile ${name} added${suffix}`));
         });
 
     profile
         .command('delete <name>')
+        .helpGroup('Profile:')
         .description('Remove a profile')
         .option('-j, --json', 'Output as JSON')
         .action((name, opts) => {
@@ -100,6 +168,6 @@ export function profileCommand(program) {
                 printOk({ removed: name });
                 return;
             }
-            console.log(chalk.green(`Removed profile ${name}`));
+            console.log(success(`Removed profile ${name}`));
         });
 }