@proletariat/cli 0.3.9 → 0.3.11

package/dist/commands/ticket/link/index.js

@@ -82,9 +82,10 @@ export default class TicketLink extends PMOCommand {
             await this.addDependency(this.storage, this.pmoPath, ticketId, targetId, dependencyType, ticket.title);
             return;
         }
-        // Interactive mode: show menu in a loop
+        // Interactive mode: show menu in a loop - user interaction requires sequential processing
         let continueLoop = true;
         while (continueLoop) {
+            // eslint-disable-next-line no-await-in-loop
             const allTickets = await this.storage.listTickets(projectId);
             const otherTickets = allTickets.filter(t => t.id !== ticketId);
             const menuChoices = [
@@ -95,6 +96,7 @@ export default class TicketLink extends PMOCommand {
                 { id: 'remove', name: 'Remove dependency' },
                 { id: 'done', name: 'Done' },
             ];
+            // eslint-disable-next-line no-await-in-loop
             const action = await this.selectFromList({
                 message: `Dependencies for ${ticket.id}:`,
                 items: menuChoices,
@@ -117,15 +119,18 @@ export default class TicketLink extends PMOCommand {
                 continue;
             }
             if (action === 'view') {
+                // eslint-disable-next-line no-await-in-loop
                 await this.viewDependencies(this.storage, ticketId, ticket, flags.all);
                 continue;
             }
             if (action === 'remove') {
+                // eslint-disable-next-line no-await-in-loop
                 const dependencies = await this.storage.listTicketDependencies(ticketId);
                 if (dependencies.length === 0) {
                     this.log(styles.muted('\nNo dependencies to remove.'));
                     continue;
                 }
+                // eslint-disable-next-line no-await-in-loop
                 const depChoices = await Promise.all(dependencies.map(async (dep) => {
                     const depTicket = await this.storage.getTicket(dep.dependsOnTicketId);
                     return {
@@ -134,6 +139,7 @@ export default class TicketLink extends PMOCommand {
                         type: dep.dependencyType
                     };
                 }));
+                // eslint-disable-next-line no-await-in-loop
                 const selected = await this.selectFromList({
                     message: 'Select dependency to remove:',
                     items: depChoices,
@@ -147,7 +153,9 @@ export default class TicketLink extends PMOCommand {
                 }
                 const selectedDep = depChoices.find(d => d.id === selected);
                 if (selectedDep) {
+                    // eslint-disable-next-line no-await-in-loop
                     await this.storage.deleteTicketDependency(ticketId, selected, selectedDep.type);
+                    // eslint-disable-next-line no-await-in-loop
                     await autoExportToBoard(this.pmoPath, this.storage, (msg) => this.log(styles.muted(msg)));
                     this.log(styles.success(`\n✅ Removed dependency: ${ticketId} → ${selected}`));
                 }
@@ -158,6 +166,7 @@ export default class TicketLink extends PMOCommand {
                 this.log(styles.muted('\nNo other tickets to link to.'));
                 continue;
             }
+            // eslint-disable-next-line no-await-in-loop
             const targetId = await this.selectFromList({
                 message: `Select ticket that ${ticketId} ${action === 'blocks' ? 'is blocked by' : action === 'relates_to' ? 'relates to' : 'duplicates'}:`,
                 items: otherTickets,
@@ -167,6 +176,7 @@ export default class TicketLink extends PMOCommand {
                 jsonMode: jsonMode ? { flags, commandName: 'ticket link' } : null,
             });
             if (targetId) {
+                // eslint-disable-next-line no-await-in-loop
                 await this.addDependency(this.storage, this.pmoPath, ticketId, targetId, action, ticket.title);
             }
         }
@@ -215,8 +225,12 @@ export default class TicketLink extends PMOCommand {
         const otherDeps = dependencies.filter(d => d.dependencyType !== 'blocks');
         if (otherDeps.length > 0) {
             this.log(styles.muted('\n  Related:'));
-            for (const dep of otherDeps) {
-                const relatedTicket = await this.storage.getTicket(dep.dependsOnTicketId);
+            // Fetch all related tickets in parallel
+            const relatedTickets = await Promise.all(otherDeps.map(async (dep) => ({
+                dep,
+                ticket: await this.storage.getTicket(dep.dependsOnTicketId)
+            })));
+            for (const { dep, ticket: relatedTicket } of relatedTickets) {
                 if (relatedTicket) {
                     this.log(`    - ${dep.dependencyType}: ${relatedTicket.id} - ${relatedTicket.title}`);
                 }

package/dist/commands/ticket/link/remove.js

@@ -80,7 +80,9 @@ export default class TicketLinkRemove extends PMOCommand {
                 this.log(styles.muted('\nCancelled.'));
                 return;
             }
+            // Delete sequentially for data integrity
             for (const dep of dependencies) {
+                // eslint-disable-next-line no-await-in-loop
                 await this.storage.deleteTicketDependency(args.id, dep.dependsOnTicketId, dep.dependencyType);
             }
             await autoExportToBoard(this.pmoPath, this.storage, (msg) => this.log(styles.muted(msg)));

package/dist/commands/ticket/list.js

@@ -58,9 +58,8 @@ export default class TicketList extends Command {
         const { flags } = await this.parse(TicketList);
         // When --all is set, we don't need to select a specific project
         // Otherwise, use the normal project selection flow
-        let pmoContext;
         // Get PMO context - no project selection needed
-        pmoContext = await getPMOContext({
+        const pmoContext = await getPMOContext({
             logger: (msg) => this.log(styles.muted(msg)),
         });
         try {

package/dist/commands/ticket/move.js

@@ -187,8 +187,10 @@ export default class TicketMove extends PMOCommand {
         // Move each ticket
         let successCount = 0;
         let failCount = 0;
+        // Process sequentially for clear success/failure logging
         for (const ticketId of selectedTickets) {
             try {
+                // eslint-disable-next-line no-await-in-loop
                 await this.storage.moveTicket(projectId, ticketId, targetColumn);
                 this.log(styles.success(`Moved ${ticketId} to ${targetColumn}`));
                 successCount++;

package/dist/commands/ticket/project.js

@@ -235,15 +235,17 @@ export default class TicketProject extends PMOCommand {
                 return;
             }
         }
-        // Move each ticket using the storage method
+        // Move each ticket using the storage method - sequential for clear logging
         let lastMovedTicket;
         for (const ticketId of selectedTickets) {
             const ticket = tickets.find(t => t.id === ticketId);
             // Unlink from epic if needed
             if (ticket?.epicId) {
+                // eslint-disable-next-line no-await-in-loop
                 await this.storage.updateTicket(ticketId, { epicId: undefined });
             }
             // Move ticket to new project
+            // eslint-disable-next-line no-await-in-loop
             lastMovedTicket = await this.storage.moveTicketToProject(ticketId, targetProjectId);
             this.log(styles.success(`  Moved ${ticketId} to ${targetProjectId}`));
         }

package/dist/commands/ticket/reassign.js

@@ -252,8 +252,10 @@ export default class TicketReassign extends PMOCommand {
         // Reassign each ticket
         let successCount = 0;
         let failCount = 0;
+        // Process sequentially for clear success/failure logging
         for (const ticketId of selectedTickets) {
             try {
+                // eslint-disable-next-line no-await-in-loop
                 await this.storage.updateTicket(ticketId, { assignee: targetAssignee || undefined });
                 const action = targetAssignee ? `Reassigned to ${targetAssignee}` : 'Unassigned';
                 this.log(styles.success(`${ticketId}: ${action}`));

package/dist/commands/ticket/spec.js

@@ -214,9 +214,10 @@ export default class TicketSpec extends PMOCommand {
             this.log(styles.muted('No tickets selected.'));
             return;
         }
-        // Handle unlink
+        // Handle unlink - sequential for clear logging
         if (flags.unlink) {
             for (const ticketId of selectedTickets) {
+                // eslint-disable-next-line no-await-in-loop
                 await this.storage.updateTicket(ticketId, { specId: undefined });
                 this.log(styles.success(`  Unlinked spec from ${ticketId}`));
             }
@@ -248,8 +249,9 @@ export default class TicketSpec extends PMOCommand {
         if (!spec) {
             this.error(`Spec not found: ${specId}`);
         }
-        // Assign spec to all selected tickets
+        // Assign spec to all selected tickets - sequential for clear logging
         for (const ticketId of selectedTickets) {
+            // eslint-disable-next-line no-await-in-loop
             await this.storage.updateTicket(ticketId, { specId });
             this.log(styles.success(`  Linked ${ticketId} to ${specId}`));
         }

package/dist/commands/ticket/template/apply.js

@@ -127,8 +127,8 @@ export default class TicketTemplateApply extends PMOCommand {
         let category = flags.category || template.defaultCategory;
         let assignee = flags.assignee || template.defaultAssignee;
         let owner = flags.owner || template.defaultOwner;
-        let statusId = flags.status || template.defaultStatusId;
-        let labels = flags.labels ? flags.labels.split(',').map(l => l.trim()).filter(l => l) : template.defaultLabels;
+        const statusId = flags.status || template.defaultStatusId;
+        const labels = flags.labels ? flags.labels.split(',').map(l => l.trim()).filter(Boolean) : template.defaultLabels;
         let description = flags.description || template.descriptionTemplate;
         // Interactive mode - prompt for values
         if (flags.interactive || !title) {
@@ -207,9 +207,10 @@ export default class TicketTemplateApply extends PMOCommand {
             description,
             epicId: flags.epic,
         });
-        // Add subtasks from template (unless disabled)
+        // Add subtasks from template (unless disabled) - sequential for ordering
         if (!flags['no-subtasks'] && template.suggestedSubtasks.length > 0) {
             for (const subtask of template.suggestedSubtasks) {
+                // eslint-disable-next-line no-await-in-loop
                 await this.storage.addSubtask(ticket.id, subtask.title);
             }
         }

package/dist/commands/ticket/template/create.js

@@ -134,6 +134,7 @@ export default class TicketTemplateCreate extends PMOCommand {
         if (wantSubtasks) {
             let addMore = true;
             while (addMore) {
+                // eslint-disable-next-line no-await-in-loop -- Interactive loop for subtask creation
                 const { subtaskTitle } = await inquirer.prompt([{
                         type: 'input',
                         name: 'subtaskTitle',
@@ -141,6 +142,7 @@ export default class TicketTemplateCreate extends PMOCommand {
                         validate: (input) => input.length > 0 || 'Title is required',
                     }]);
                 subtasks.push(subtaskTitle);
+                // eslint-disable-next-line no-await-in-loop -- Interactive loop continuation
                 const { another } = await inquirer.prompt([{
                         type: 'list',
                         name: 'another',

package/dist/commands/ticket/template/index.js

@@ -2,7 +2,7 @@ import { Flags } from '@oclif/core';
 import inquirer from 'inquirer';
 import { PMOCommand, pmoBaseFlags } from '../../../lib/pmo/index.js';
 import { styles } from '../../../lib/styles.js';
-import { shouldOutputJson, } from '../../../lib/prompt-json.js';
+import { shouldOutputJson } from '../../../lib/prompt-json.js';
 export default class TicketTemplateIndex extends PMOCommand {
     static description = 'Interactive menu for ticket template operations';
     static aliases = ['ticket:templates'];

package/dist/commands/ticket/update.js

@@ -288,6 +288,7 @@ export default class TicketUpdate extends PMOCommand {
         // Update each ticket
         let successCount = 0;
         let failCount = 0;
+        // Process sequentially for clear success/failure logging
         for (const ticketId of selectedTickets) {
             try {
                 const changes = {};
@@ -297,6 +298,7 @@ export default class TicketUpdate extends PMOCommand {
                 if (updateCategory !== undefined) {
                     changes.category = updateCategory || undefined;
                 }
+                // eslint-disable-next-line no-await-in-loop
                 await this.storage.updateTicket(ticketId, changes);
                 const updates = [];
                 if (updatePriority !== undefined)

package/dist/commands/work/index.js

@@ -1,6 +1,6 @@
 import { Flags } from '@oclif/core';
 import { PMOCommand, pmoBaseFlags } from '../../lib/pmo/index.js';
-import { shouldOutputJson, } from '../../lib/prompt-json.js';
+import { shouldOutputJson } from '../../lib/prompt-json.js';
 export default class Work extends PMOCommand {
     static description = 'Interactive menu for work operations (ownership, assignment, execution)';
     static examples = [

package/dist/commands/work/revise.js

@@ -9,7 +9,7 @@ import { getWorkColumnSetting, findColumnByName } from '../../lib/pmo/utils.js';
 import { styles } from '../../lib/styles.js';
 import { getWorkspaceInfo } from '../../lib/agents/commands.js';
 import { DEFAULT_EXECUTION_CONFIG, } from '../../lib/execution/types.js';
-import { runExecution, isDockerRunning } from '../../lib/execution/runners.js';
+import { runExecution, isDockerRunning, isDevcontainerCliInstalled } from '../../lib/execution/runners.js';
 import { ExecutionStorage } from '../../lib/execution/storage.js';
 import { loadExecutionConfig, getTerminalApp, getShell, hasTerminalPreference, hasShellPreference } from '../../lib/execution/config.js';
 import { hasDevcontainerConfig } from '../../lib/execution/devcontainer.js';
@@ -84,6 +84,12 @@ export default class WorkRevise extends PMOCommand {
                 'Please start Docker Desktop and try again.\n\n' +
                 'Alternatively, use --run-on-host to run directly on your machine (bypasses sandbox).');
         }
+        // Early devcontainer CLI check
+        if (!flags['run-on-host'] && !isDevcontainerCliInstalled()) {
+            return handleError('DEVCONTAINER_CLI_NOT_INSTALLED', 'devcontainer CLI is not installed.\n\n' +
+                'Install with: npm install -g @devcontainers/cli\n\n' +
+                'Alternatively, use --run-on-host to run directly on your machine (bypasses sandbox).');
+        }
         // Get workspace info
         let workspaceInfo;
         try {

package/dist/commands/work/spawn.d.ts

@@ -10,7 +10,7 @@ export default class WorkSpawn extends PMOCommand {
         column: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
         strategy: import("@oclif/core/interfaces").OptionFlag<string, import("@oclif/core/interfaces").CustomOptions>;
         'dry-run': import("@oclif/core/interfaces").BooleanFlag<boolean>;
-        mode: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        display: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
         executor: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
         force: import("@oclif/core/interfaces").BooleanFlag<boolean>;
         'run-on-host': import("@oclif/core/interfaces").BooleanFlag<boolean>;
@@ -23,6 +23,7 @@ export default class WorkSpawn extends PMOCommand {
         'no-pr': import("@oclif/core/interfaces").BooleanFlag<boolean>;
         action: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
         session: import("@oclif/core/interfaces").OptionFlag<string, import("@oclif/core/interfaces").CustomOptions>;
+        focus: import("@oclif/core/interfaces").BooleanFlag<boolean>;
         project: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
     };
     execute(): Promise<void>;

package/dist/commands/work/spawn.js

@@ -5,8 +5,7 @@ import Database from 'better-sqlite3';
 import { PMOCommand, pmoBaseFlags, autoExportToBoard } from '../../lib/pmo/index.js';
 import { styles } from '../../lib/styles.js';
 import { getWorkspaceInfo, getTicketTmuxSession, killTmuxSession } from '../../lib/agents/commands.js';
-import { ExecutionStorage } from '../../lib/execution/storage.js';
-import { isDockerRunning } from '../../lib/execution/runners.js';
+import { isDockerRunning, isGitHubTokenAvailable, isDevcontainerCliInstalled } from '../../lib/execution/runners.js';
 import { shouldOutputJson, outputPromptAsJson, outputSuccessAsJson, outputErrorAsJson, createMetadata, buildPromptConfig, } from '../../lib/prompt-json.js';
 export default class WorkSpawn extends PMOCommand {
     static description = 'Spawn work for multiple tickets by column (batch mode)';
@@ -49,10 +48,10 @@ export default class WorkSpawn extends PMOCommand {
             description: 'Show what would be spawned without executing',
             default: false,
         }),
-        mode: Flags.string({
-            char: 'm',
-            description: 'Runtime mode for spawned agents',
-            options: ['foreground', 'background', 'tmux', 'terminal', 'devcontainer', 'docker', 'vm'],
+        display: Flags.string({
+            char: 'd',
+            description: 'Display mode for spawned agents (foreground not available for batch)',
+            options: ['terminal', 'background'],
         }),
         executor: Flags.string({
             char: 'e',
@@ -106,6 +105,10 @@ export default class WorkSpawn extends PMOCommand {
             options: ['tmux', 'direct'],
             default: 'tmux',
         }),
+        focus: Flags.boolean({
+            description: 'Bring terminal to foreground when opening new tabs (default: opens in background)',
+            default: false,
+        }),
     };
     async execute() {
         const { flags, argv } = await this.parse(WorkSpawn);
@@ -133,10 +136,9 @@ export default class WorkSpawn extends PMOCommand {
         catch {
             return handleError('NOT_IN_WORKSPACE', 'Not in a workspace. Run "prlt init" first.');
         }
-        // Open database for execution storage
+        // Open database
         const dbPath = path.join(workspaceInfo.path, '.proletariat', 'workspace.db');
         const db = new Database(dbPath);
-        const executionStorage = new ExecutionStorage(db);
         try {
             // Get board to list available columns
             const board = await this.storage.getBoard(projectId);
@@ -446,9 +448,10 @@ export default class WorkSpawn extends PMOCommand {
                 return;
             }
             // Batch mode settings - prompt once for all tickets
-            let batchMode = flags.mode;
+            let batchDisplay = flags.display;
             let batchOutput = flags.output;
-            let batchSkipPermissions = flags['skip-permissions'];
+            // Track permission mode - default to 'safe', check flag to determine if prompting needed
+            let batchPermissionMode = flags['skip-permissions'] ? 'danger' : 'safe';
             let batchCreatePr = flags['create-pr'];
             let batchNoPr = flags['no-pr'];
             let batchRunOnHost = flags['run-on-host'];
@@ -473,6 +476,11 @@ export default class WorkSpawn extends PMOCommand {
                         name: `${a.id.padEnd(12)} - ${a.description || a.name}`,
                         value: a.id,
                     }));
+                    // Add adhoc option at the end
+                    actionChoices.push({
+                        name: 'adhoc        - Unstructured exploration/debugging',
+                        value: '__adhoc__',
+                    });
                     const { selectedAction } = await inquirer.prompt([
                         {
                             type: 'list',
@@ -482,12 +490,27 @@ export default class WorkSpawn extends PMOCommand {
                             default: 'implement',
                         },
                     ]);
-                    batchAction = selectedAction;
+                    batchAction = selectedAction === '__adhoc__' ? 'adhoc' : selectedAction;
                 }
                 // Now fetch action details after selection is made
-                selectedActionDetails = await this.storage.getAction(batchAction || 'implement');
+                if (batchAction === 'adhoc') {
+                    // Adhoc is a synthetic action, not stored in database
+                    selectedActionDetails = {
+                        id: 'adhoc',
+                        name: 'Ad-hoc',
+                        description: 'Unstructured exploration and debugging',
+                        prompt: 'You are working on an ad-hoc session for exploration and debugging. Help the user with whatever they need.',
+                        modifiesCode: false,
+                        defaultMoveToCategory: 'started',
+                        isBuiltin: false,
+                        createdAt: new Date(),
+                    };
+                }
+                else {
+                    selectedActionDetails = await this.storage.getAction(batchAction || 'implement');
+                }
                 // Check if any explicit settings were provided via flags
-                const hasExplicitSettings = flags.mode || flags.output || flags['skip-permissions'] ||
+                const hasExplicitSettings = flags.display || flags.output || flags['skip-permissions'] ||
                     flags['create-pr'] || flags['no-pr'] || flags['run-on-host'];
                 // Offer to use default settings if no explicit flags provided
                 if (!hasExplicitSettings) {
@@ -511,14 +534,14 @@ export default class WorkSpawn extends PMOCommand {
                     if (useDefaults) {
                         // Apply defaults
                         if (hasDevcontainer) {
-                            batchMode = 'devcontainer';
+                            batchDisplay = 'devcontainer';
                             batchDisplayMode = 'terminal';
                         }
                         else {
-                            batchMode = 'terminal';
+                            batchDisplay = 'terminal';
                         }
                         batchOutput = 'interactive';
-                        batchSkipPermissions = false;
+                        batchPermissionMode = 'safe';
                         // For non-code-modifying actions, don't create PRs
                         if (modifiesCode) {
                             batchCreatePr = true;
@@ -532,20 +555,35 @@ export default class WorkSpawn extends PMOCommand {
                     }
                 }
                 // Prompt for environment (devcontainer vs host) if devcontainer available and not already set
-                if (hasDevcontainer && !batchRunOnHost && !batchMode) {
+                if (hasDevcontainer && !batchRunOnHost && !batchDisplay) {
+                    // Check devcontainer prerequisites upfront
+                    const dockerRunning = isDockerRunning();
+                    const devcontainerCliInstalled = isDevcontainerCliInstalled();
+                    const devcontainerReady = dockerRunning && devcontainerCliInstalled;
+                    // Build missing requirements message for devcontainer option
+                    let devcontainerLabel = '🐳 devcontainer (sandboxed, recommended)';
+                    if (!devcontainerReady) {
+                        const missing = [];
+                        if (!dockerRunning)
+                            missing.push('Docker');
+                        if (!devcontainerCliInstalled)
+                            missing.push('devcontainer CLI');
+                        devcontainerLabel = `🐳 devcontainer (requires: ${missing.join(', ')})`;
+                    }
                     let environmentSelected = false;
                     while (!environmentSelected) {
+                        // eslint-disable-next-line no-await-in-loop -- Interactive loop with retry on Docker check
                         const { selectedEnvironment } = await inquirer.prompt([
                             {
                                 type: 'list',
                                 name: 'selectedEnvironment',
                                 message: 'Where should agents run?',
                                 choices: [
-                                    { name: '🐳 devcontainer (sandboxed, recommended)', value: 'devcontainer' },
+                                    { name: devcontainerLabel, value: 'devcontainer', disabled: !devcontainerReady },
                                     { name: '💻 host (runs directly on your machine)', value: 'host' },
                                     { name: '✗  cancel', value: 'cancel' },
                                 ],
-                                default: 'devcontainer',
+                                default: devcontainerReady ? 'devcontainer' : 'host',
                             },
                         ]);
                         if (selectedEnvironment === 'cancel') {
@@ -554,6 +592,7 @@ export default class WorkSpawn extends PMOCommand {
                             return;
                         }
                         if (selectedEnvironment === 'devcontainer') {
+                            // Double-check prerequisites (in case user retried after starting Docker)
                             if (!isDockerRunning()) {
                                 this.log('');
                                 this.warn('Docker is not running.\n' +
@@ -562,10 +601,59 @@ export default class WorkSpawn extends PMOCommand {
                                 this.log('');
                                 continue;
                             }
-                            batchMode = 'devcontainer';
+                            if (!isDevcontainerCliInstalled()) {
+                                this.log('');
+                                this.warn('devcontainer CLI is not installed.\n' +
+                                    'Install with: npm install -g @devcontainers/cli\n' +
+                                    'Or select "host" to run directly on your machine.');
+                                this.log('');
+                                continue;
+                            }
+                            // Check GitHub token is available for git push operations
+                            if (!isGitHubTokenAvailable()) {
+                                const tokenChoices = [
+                                    { name: 'Yes, continue anyway (git push may fail)', value: 'continue' },
+                                    { name: 'No, let me run gh auth login first', value: 'cancel' },
+                                    { name: 'Switch to host mode instead', value: 'host' },
+                                ];
+                                const tokenMessage = 'GitHub token not found. Git push may fail. Continue without token?';
+                                if (jsonMode) {
+                                    outputPromptAsJson(buildPromptConfig('list', 'tokenAction', tokenMessage, tokenChoices), createMetadata('work spawn', flags));
+                                    db.close();
+                                    return;
+                                }
+                                this.log('');
+                                this.warn('GitHub token not found.\n' +
+                                    'Git push operations may fail inside containers.\n' +
+                                    'Run `gh auth login` to authenticate, or continue without token.');
+                                this.log('');
+                                // eslint-disable-next-line no-await-in-loop -- Interactive user prompt in loop
+                                const { tokenAction } = await inquirer.prompt([
+                                    {
+                                        type: 'list',
+                                        name: 'tokenAction',
+                                        message: tokenMessage,
+                                        choices: tokenChoices,
+                                        default: 'continue',
+                                    },
+                                ]);
+                                if (tokenAction === 'cancel') {
+                                    db.close();
+                                    this.log(styles.muted('Run `gh auth login` and try again.'));
+                                    return;
+                                }
+                                if (tokenAction === 'host') {
+                                    batchRunOnHost = true;
+                                    environmentSelected = true;
+                                    continue;
+                                }
+                                // tokenAction === 'continue' - fall through to devcontainer setup
+                            }
+                            batchDisplay = 'devcontainer';
                             environmentSelected = true;
                             // For devcontainer, prompt for display mode
                             // Simplified: tmux is always used inside container for session persistence
+                            // eslint-disable-next-line no-await-in-loop -- Follow-up prompt after selection
                             const { selectedDisplay } = await inquirer.prompt([
                                 {
                                     type: 'list',
@@ -589,7 +677,7 @@ export default class WorkSpawn extends PMOCommand {
                     }
                 }
                 // Prompt for display mode if not already set (for host mode without devcontainer)
-                if (!batchMode) {
+                if (!batchDisplay) {
                     const { selectedMode } = await inquirer.prompt([
                         {
                             type: 'list',
@@ -601,15 +689,15 @@ export default class WorkSpawn extends PMOCommand {
                             ],
                         },
                     ]);
-                    batchMode = selectedMode;
+                    batchDisplay = selectedMode;
                 }
                 // Default to interactive output mode (streaming UI)
                 // Can be overridden via --output flag if needed
                 if (!batchOutput) {
                     batchOutput = 'interactive';
                 }
-                // Prompt for permissions mode if not provided
-                if (!batchSkipPermissions) {
+                // Prompt for permissions mode if not explicitly set via --skip-permissions flag
+                if (!flags['skip-permissions']) {
                     const { permissionMode } = await inquirer.prompt([
                         {
                             type: 'list',
@@ -622,7 +710,7 @@ export default class WorkSpawn extends PMOCommand {
                             default: 'danger',
                         },
                     ]);
-                    batchSkipPermissions = permissionMode === 'danger';
+                    batchPermissionMode = permissionMode;
                 }
                 // Prompt for PR creation if not provided AND action modifies code
                 // Skip this prompt entirely for non-code-modifying actions (like groom)
@@ -661,6 +749,7 @@ export default class WorkSpawn extends PMOCommand {
             // Spawn each ticket - work:start will create ephemeral agents on-demand
             let successCount = 0;
             let failCount = 0;
+            // Process sequentially for clear logging and resource management
             for (const ticket of ticketsToSpawn) {
                 try {
                     this.log(styles.muted(`Starting ${ticket.id} with ephemeral agent...`));
@@ -669,24 +758,26 @@ export default class WorkSpawn extends PMOCommand {
                     // Pass --ephemeral to signal work:start should create an ephemeral agent
                     const startArgs = [ticket.id, '--project', projectId, '--ephemeral'];
                     if (flags['per-ticket']) {
-                        // Per-ticket mode: only pass mode flag, let start prompt for the rest
-                        if (batchMode)
-                            startArgs.push('--mode', batchMode);
-                        if (batchDisplayMode)
-                            startArgs.push('--display', batchDisplayMode);
+                        // Per-ticket mode: only pass display flag, let start prompt for the rest
+                        // batchDisplayMode is for devcontainer, batchDisplay is for host
+                        const displayToUse = batchDisplayMode || batchDisplay;
+                        if (displayToUse && displayToUse !== 'devcontainer')
+                            startArgs.push('--display', displayToUse);
                         if (flags.executor)
                             startArgs.push('--executor', flags.executor);
                         if (batchRunOnHost)
                             startArgs.push('--run-on-host');
                         if (flags.force)
                             startArgs.push('--force');
+                        if (flags.focus)
+                            startArgs.push('--focus');
                     }
                     else {
                         // Batch mode: pass all settings to skip prompts
-                        if (batchMode)
-                            startArgs.push('--mode', batchMode);
-                        if (batchDisplayMode)
-                            startArgs.push('--display', batchDisplayMode);
+                        // batchDisplayMode is for devcontainer, batchDisplay is for host
+                        const displayToUse = batchDisplayMode || batchDisplay;
+                        if (displayToUse && displayToUse !== 'devcontainer')
+                            startArgs.push('--display', displayToUse);
                         if (flags.executor)
                             startArgs.push('--executor', flags.executor);
                         if (batchRunOnHost)
@@ -695,8 +786,8 @@ export default class WorkSpawn extends PMOCommand {
                             startArgs.push('--force');
                         if (batchOutput)
                             startArgs.push('--output', batchOutput);
-                        if (batchSkipPermissions)
-                            startArgs.push('--skip-permissions');
+                        // Always pass permission mode to skip the prompt in work:start
+                        startArgs.push('--permission-mode', batchPermissionMode);
                         if (batchCreatePr)
                             startArgs.push('--create-pr');
                         if (batchNoPr)
@@ -706,7 +797,11 @@ export default class WorkSpawn extends PMOCommand {
                         // Pass session manager (tmux inside container by default)
                         if (flags.session)
                             startArgs.push('--session', flags.session);
+                        // Pass focus flag (brings terminal to foreground)
+                        if (flags.focus)
+                            startArgs.push('--focus');
                     }
+                    // eslint-disable-next-line no-await-in-loop
                     await this.config.runCommand('work:start', startArgs);
                     successCount++;
                 }