@proletariat/cli 0.3.19 → 0.3.20

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

@@ -2,6 +2,7 @@ import { Flags, Args } from '@oclif/core';
 import inquirer from 'inquirer';
 import { PMOCommand, pmoBaseFlags } from '../../../lib/pmo/index.js';
 import { styles } from '../../../lib/styles.js';
+import { shouldOutputJson, outputPromptAsJson, outputSuccessAsJson, outputErrorAsJson, createMetadata, buildPromptConfig, } from '../../../lib/prompt-json.js';
 export default class TicketTemplateSave extends PMOCommand {
     static description = 'Create a template from an existing ticket';
     static examples = [
@@ -20,20 +21,45 @@ export default class TicketTemplateSave extends PMOCommand {
     };
     static flags = {
         ...pmoBaseFlags,
+        'template-name': Flags.string({
+            char: 'n',
+            description: 'Template name (alternative to positional arg, required in non-TTY/JSON mode)',
+        }),
         description: Flags.string({
             char: 'd',
             description: 'Template description',
         }),
+        json: Flags.boolean({
+            description: 'Output prompt configuration as JSON (for AI agents/scripts)',
+            default: false,
+        }),
     };
     async execute() {
         const { args, flags } = await this.parse(TicketTemplateSave);
+        // Check if JSON output mode is active
+        const jsonMode = shouldOutputJson(flags);
+        // Helper to handle errors in JSON mode
+        const handleError = (code, message) => {
+            if (jsonMode) {
+                outputErrorAsJson(code, message, createMetadata('ticket template save', flags));
+            }
+            this.error(message);
+        };
         // Get ticket ID - prompt with picker if not provided
         let ticketId = args.ticket;
         if (!ticketId) {
             const projectId = await this.requireProject();
             const tickets = await this.storage.listTickets(projectId);
             if (tickets.length === 0) {
-                this.error('No tickets found in this project.\nCreate a ticket first: prlt ticket create');
+                return handleError('NO_TICKETS', 'No tickets found in this project.\nCreate a ticket first: prlt ticket create');
+            }
+            // In JSON mode, output prompt config for ticket selection
+            if (jsonMode) {
+                const choices = tickets.slice(0, 20).map(t => ({
+                    name: `${t.id} - ${t.title}`,
+                    value: t.id,
+                }));
+                outputPromptAsJson(buildPromptConfig('list', 'ticket', 'Select a ticket to save as template:', choices), createMetadata('ticket template save', flags));
             }
             const { selectedTicket } = await inquirer.prompt([{
                     type: 'list',
@@ -49,23 +75,28 @@ export default class TicketTemplateSave extends PMOCommand {
         // Verify ticket exists
         const ticket = await this.storage.getTicket(ticketId);
         if (!ticket) {
-            this.error(`Ticket not found: ${ticketId}\nRun 'prlt ticket list' to see available tickets.`);
+            return handleError('TICKET_NOT_FOUND', `Ticket not found: ${ticketId}\nRun 'prlt ticket list' to see available tickets.`);
         }
-        // Get template name - prompt if not provided
-        let templateName = args.name;
+        // Get template name - prefer flag over positional arg
+        let templateName = flags['template-name'] || args.name;
         if (!templateName) {
+            const defaultName = ticket.category || ticket.title.split(' ')[0];
+            // In JSON mode, output prompt config for template name
+            if (jsonMode) {
+                outputPromptAsJson(buildPromptConfig('input', 'name', 'Template name:', undefined, defaultName), createMetadata('ticket template save', flags));
+            }
             const { name } = await inquirer.prompt([{
                     type: 'input',
                     name: 'name',
                     message: 'Template name:',
-                    default: ticket.category || ticket.title.split(' ')[0],
+                    default: defaultName,
                     validate: (input) => input.length > 0 || 'Name is required',
                 }]);
             templateName = name;
         }
-        // Get description if not provided
+        // Get description if not provided - only prompt interactively (not in JSON mode)
         let description = flags.description;
-        if (description === undefined) {
+        if (description === undefined && !jsonMode) {
             const { desc } = await inquirer.prompt([{
                     type: 'input',
                     name: 'desc',
@@ -75,6 +106,26 @@ export default class TicketTemplateSave extends PMOCommand {
         }
         // Create template from ticket
         const template = await this.storage.createTicketTemplateFromTicket(ticketId, templateName, description);
+        // In JSON mode, output success as JSON
+        if (jsonMode) {
+            outputSuccessAsJson({
+                template: {
+                    id: template.id,
+                    name: template.name,
+                    description: template.description,
+                    titlePattern: template.titlePattern,
+                    defaultPriority: template.defaultPriority,
+                    defaultCategory: template.defaultCategory,
+                    defaultStatusId: template.defaultStatusId,
+                    defaultAssignee: template.defaultAssignee,
+                    defaultOwner: template.defaultOwner,
+                    defaultLabels: template.defaultLabels,
+                    suggestedSubtasks: template.suggestedSubtasks,
+                },
+                sourceTicketId: ticketId,
+            }, createMetadata('ticket template save', flags));
+            return;
+        }
         this.log(styles.success(`\nCreated template "${styles.emphasis(template.name)}" from ticket ${ticketId}`));
         this.log(styles.muted(`  ID: ${template.id}`));
         if (template.description) {

package/dist/commands/work/ready.js

@@ -106,18 +106,18 @@ export default class WorkReady extends PMOCommand {
                 this.error(`Ticket "${ticketId}" not found.`);
             }
             // Get configured column name (from pmo_settings or default)
-            // In Linear-style workflow, "ready" moves ticket to Done (review is implicit via PR)
-            const targetColumnName = getWorkColumnSetting(db, 'done');
+            // "ready" moves ticket to Review column (work complete moves to Done)
+            const targetColumnName = getWorkColumnSetting(db, 'review');
             const board = await this.storage.getBoard(ticket.projectId);
             const columnNames = board.columns.map(col => col.name);
-            const doneColumn = findColumnByName(columnNames, targetColumnName);
-            if (!doneColumn) {
+            const reviewColumn = findColumnByName(columnNames, targetColumnName);
+            if (!reviewColumn) {
                 db.close();
-                this.error(`No "${targetColumnName}" column found in board configuration. Configure with: prlt config set column_done <column-name>`);
+                this.error(`No "${targetColumnName}" column found in board configuration. Configure with: prlt config set column_review <column-name>`);
             }
             const previousColumn = ticket.statusName;
-            // Move to Done column (moveTicket also updates status_id)
-            await this.storage.moveTicket(ticket.projectId, ticketId, doneColumn);
+            // Move to Review column (moveTicket also updates status_id)
+            await this.storage.moveTicket(ticket.projectId, ticketId, reviewColumn);
             // Auto-export to board.md if configured
             await autoExportToBoard(this.pmoPath, this.storage);
             // Mark any running executions for this ticket as completed
@@ -172,7 +172,7 @@ export default class WorkReady extends PMOCommand {
             this.log(styles.success(`Work ready: ${ticketId}`));
             this.log(styles.muted(`   Title: ${ticket.title}`));
             this.log(styles.muted(`   From: ${previousColumn}`));
-            this.log(styles.muted(`   To: ${doneColumn}`));
+            this.log(styles.muted(`   To: ${reviewColumn}`));
             if (prUrl) {
                 this.log(styles.muted(`   PR: ${prUrl}`));
             }

package/dist/lib/agents/index.js

@@ -217,14 +217,15 @@ export async function createAgentWorktrees(workspacePath, agents, hqPath, option
                             }
                         }
                     }
-                    // Create devcontainer config for sandboxed execution (only for repos that were created)
+                    // Create devcontainer config for sandboxed execution
                     // Note: Agent metadata is stored in SQLite (agents table), not in config files
-                    if (!options?.skipDevcontainer && createdRepos.length > 0) {
+                    // Always create devcontainer config (even if no repos were created) so agent rebuild works
+                    if (!options?.skipDevcontainer) {
                         console.log(styles.muted(`  Creating devcontainer config...`));
                         createDevcontainerConfig({
                             agentName: agent,
                             agentDir,
-                            repoWorktrees: mountMode === 'worktree' ? createdRepos : undefined, // Only pass repos for worktree mode
+                            repoWorktrees: mountMode === 'worktree' && createdRepos.length > 0 ? createdRepos : undefined,
                             mountMode,
                         });
                     }
@@ -237,10 +238,19 @@ export async function createAgentWorktrees(workspacePath, agents, hqPath, option
         }
         else {
             console.log(chalk.yellow('No repositories found in HQ. Creating placeholder agent directories.'));
-            // Create placeholder directories for now
+            // Create placeholder directories with devcontainer configs
             for (const agent of agents) {
                 const agentDir = path.join(workspacePath, agent);
                 fs.mkdirSync(agentDir, { recursive: true });
+                // Create devcontainer config even without repos so agent rebuild works
+                if (!options?.skipDevcontainer) {
+                    console.log(styles.muted(`  Creating devcontainer config...`));
+                    createDevcontainerConfig({
+                        agentName: agent,
+                        agentDir,
+                        mountMode: mountMode,
+                    });
+                }
                 console.log(chalk.green(`✅ Placeholder agent ${agent} created`));
             }
         }

package/dist/lib/branch/index.js

@@ -129,6 +129,14 @@ export function validateBranchName(name) {
     if (parts.length === 2) {
         // {type}/{description}
         const description = parts[1];
+        // Check if description looks like a ticket ID (user put ticket in wrong position)
+        if (isTicketId(description)) {
+            return {
+                valid: false,
+                error: `Segment "${description}" looks like a ticket ID, but ticket IDs must be the first segment. ` +
+                    `Expected format: {ticketId}/{type}/{description}`,
+            };
+        }
         if (!isKebabCase(description)) {
             return {
                 valid: false,
@@ -143,12 +151,28 @@ export function validateBranchName(name) {
     // {type}/{owner}/{description}
     const owner = parts[1];
     const description = parts[2];
+    // Check if owner looks like a ticket ID (user put ticket in wrong position)
+    if (isTicketId(owner)) {
+        return {
+            valid: false,
+            error: `Segment "${owner}" looks like a ticket ID, but it's in the owner position (segment 2). ` +
+                `Ticket IDs must be the first segment. Expected format: {ticketId}/{type}/{owner}/{description}`,
+        };
+    }
     if (!isKebabCase(owner)) {
         return {
             valid: false,
             error: `Owner name must be kebab-case: "${owner}"`,
         };
     }
+    // Check if description looks like a ticket ID (user put ticket in wrong position)
+    if (isTicketId(description)) {
+        return {
+            valid: false,
+            error: `Segment "${description}" looks like a ticket ID, but it's in the description position (segment 3). ` +
+                `Ticket IDs must be the first segment.`,
+        };
+    }
     if (!isKebabCase(description)) {
         return {
             valid: false,

package/dist/lib/execution/config.d.ts

@@ -14,6 +14,8 @@ declare const CONFIG_KEYS: {
     defaultMode: string;
     defaultExecutor: string;
     autoExecute: string;
+    outputMode: string;
+    sandboxed: string;
     tmuxSession: string;
     tmuxLayout: string;
     tmuxControlMode: string;

package/dist/lib/execution/config.js

@@ -18,6 +18,8 @@ const CONFIG_KEYS = {
     defaultMode: 'execution.default_mode',
     defaultExecutor: 'execution.default_executor',
     autoExecute: 'execution.auto_execute',
+    outputMode: 'execution.output_mode',
+    sandboxed: 'execution.sandboxed',
     tmuxSession: 'execution.tmux.session',
     tmuxLayout: 'execution.tmux.layout',
     tmuxControlMode: 'execution.tmux.control_mode',
@@ -85,6 +87,16 @@ export function loadExecutionConfig(db) {
     if (autoExecute !== null) {
         config.autoExecute = autoExecute === 'true';
     }
+    // Load output mode
+    const outputMode = getSetting(db, CONFIG_KEYS.outputMode);
+    if (outputMode) {
+        config.outputMode = outputMode;
+    }
+    // Load sandboxed preference
+    const sandboxed = getSetting(db, CONFIG_KEYS.sandboxed);
+    if (sandboxed !== null) {
+        config.sandboxed = sandboxed === 'true';
+    }
     // Load tmux settings
     const tmuxSession = getSetting(db, CONFIG_KEYS.tmuxSession);
     if (tmuxSession) {

package/dist/lib/pmo/utils.d.ts

@@ -75,14 +75,16 @@ export declare function deepClone<T>(obj: T): T;
 /**
  * Default column names for work commands (Linear-style workflow)
  *
- * Linear-style: Backlog → Planned → In Progress → Done
+ * Linear-style: Backlog → Planned → In Progress → Review → Done
  * - planned: Move tickets here when scheduled/assigned
  * - in_progress: Move tickets here when work starts
- * - done: Move tickets here when work is complete (includes review/merged)
+ * - review: Move tickets here when work is ready for review
+ * - done: Move tickets here when work is complete (reviewed/merged)
  */
 export declare const DEFAULT_WORK_COLUMNS: {
     readonly planned: "Planned";
     readonly in_progress: "In Progress";
+    readonly review: "Review";
     readonly done: "Done";
 };
 export type WorkColumnType = keyof typeof DEFAULT_WORK_COLUMNS;

package/dist/lib/pmo/utils.js

@@ -122,14 +122,16 @@ export function deepClone(obj) {
 /**
  * Default column names for work commands (Linear-style workflow)
  *
- * Linear-style: Backlog → Planned → In Progress → Done
+ * Linear-style: Backlog → Planned → In Progress → Review → Done
  * - planned: Move tickets here when scheduled/assigned
  * - in_progress: Move tickets here when work starts
- * - done: Move tickets here when work is complete (includes review/merged)
+ * - review: Move tickets here when work is ready for review
+ * - done: Move tickets here when work is complete (reviewed/merged)
  */
 export const DEFAULT_WORK_COLUMNS = {
     planned: 'Planned',
     in_progress: 'In Progress',
+    review: 'Review',
     done: 'Done',
 };
 /**