@proletariat/cli 0.3.26 → 0.3.28

package/dist/lib/mcp/tools/work.js

@@ -2,9 +2,9 @@
  * MCP Work Tools (Agent workflow)
  */
 import { z } from 'zod';
-import { formatTicket, errorResponse } from '../helpers.js';
+import { formatTicket, errorResponse, strictTool } from '../helpers.js';
 export function registerWorkTools(server, ctx) {
-    server.tool('work_status', 'Get current work status (in-progress tickets)', {}, async () => {
+    strictTool(server, 'work_status', 'Get current work status (in-progress tickets)', {}, async () => {
         try {
             const tickets = await ctx.storage.listTickets(undefined, { allProjects: true });
             const inProgress = tickets.filter((t) => t.statusCategory === 'started' && t.assignee);
@@ -31,7 +31,7 @@ export function registerWorkTools(server, ctx) {
             return errorResponse(error);
         }
     });
-    server.tool('work_start', 'Start working on a ticket (moves to In Progress)', {
+    strictTool(server, 'work_start', 'Start working on a ticket (moves to In Progress)', {
         ticket_id: z.string().describe('Ticket ID'),
         assignee: z.string().optional().describe('Who is working'),
     }, async (params) => {
@@ -60,7 +60,7 @@ export function registerWorkTools(server, ctx) {
             return errorResponse(error);
         }
     });
-    server.tool('work_complete', 'Mark ticket complete (moves to Done)', { ticket_id: z.string().describe('Ticket ID') }, async (params) => {
+    strictTool(server, 'work_complete', 'Mark ticket complete (moves to Done)', { ticket_id: z.string().describe('Ticket ID') }, async (params) => {
         try {
             const ticket = await ctx.storage.getTicket(params.ticket_id);
             if (!ticket)
@@ -83,7 +83,7 @@ export function registerWorkTools(server, ctx) {
             return errorResponse(error);
         }
     });
-    server.tool('work_ready', 'Mark ticket ready for review', { ticket_id: z.string().describe('Ticket ID') }, async (params) => {
+    strictTool(server, 'work_ready', 'Mark ticket ready for review', { ticket_id: z.string().describe('Ticket ID') }, async (params) => {
         try {
             const ticket = await ctx.storage.getTicket(params.ticket_id);
             if (!ticket)
@@ -110,7 +110,7 @@ export function registerWorkTools(server, ctx) {
             return errorResponse(error);
         }
     });
-    server.tool('work_revise', 'Send ticket back for revision', { ticket_id: z.string().describe('Ticket ID') }, async (params) => {
+    strictTool(server, 'work_revise', 'Send ticket back for revision', { ticket_id: z.string().describe('Ticket ID') }, async (params) => {
         try {
             const ticket = await ctx.storage.getTicket(params.ticket_id);
             if (!ticket)

package/dist/lib/mcp/tools/workflow.js

@@ -2,9 +2,9 @@
  * MCP Workflow Tools
  */
 import { z } from 'zod';
-import { errorResponse } from '../helpers.js';
+import { errorResponse, strictTool } from '../helpers.js';
 export function registerWorkflowTools(server, ctx) {
-    server.tool('workflow_list', 'List all workflows', { include_builtin: z.boolean().optional() }, async (params) => {
+    strictTool(server, 'workflow_list', 'List all workflows', { include_builtin: z.boolean().optional() }, async (params) => {
         try {
             const workflows = await ctx.storage.listWorkflows({
                 isBuiltin: params.include_builtin ? undefined : false,
@@ -28,7 +28,7 @@ export function registerWorkflowTools(server, ctx) {
             return errorResponse(error);
         }
     });
-    server.tool('workflow_show', 'Get workflow details with statuses', { id: z.string().describe('Workflow ID') }, async (params) => {
+    strictTool(server, 'workflow_show', 'Get workflow details with statuses', { id: z.string().describe('Workflow ID') }, async (params) => {
         try {
             const workflow = await ctx.storage.getWorkflow(params.id);
             if (!workflow)
@@ -57,7 +57,7 @@ export function registerWorkflowTools(server, ctx) {
             return errorResponse(error);
         }
     });
-    server.tool('workflow_create', 'Create a new workflow', {
+    strictTool(server, 'workflow_create', 'Create a new workflow', {
         name: z.string().describe('Workflow name'),
         description: z.string().optional(),
         statuses: z.array(z.string()).optional().describe('Status names'),
@@ -78,7 +78,7 @@ export function registerWorkflowTools(server, ctx) {
             return errorResponse(error);
         }
     });
-    server.tool('workflow_delete', 'Delete a workflow', { id: z.string().describe('Workflow ID') }, async (params) => {
+    strictTool(server, 'workflow_delete', 'Delete a workflow', { id: z.string().describe('Workflow ID') }, async (params) => {
         try {
             await ctx.storage.deleteWorkflow(params.id);
             return {

package/dist/lib/pmo/index.js

@@ -6,6 +6,7 @@ import { SQLiteStorage } from './storage-sqlite.js';
 import { createSpecFolders } from './create-spec-folders.js';
 import { slugify } from './utils.js';
 import { createDevcontainerConfig } from '../execution/devcontainer.js';
+import { getGitIdentity } from '../pr/index.js';
 // Re-export new PMO modules
 export * from './types.js';
 export * from './utils.js';
@@ -383,10 +384,13 @@ ${columns.join(', ')}
     // Create devcontainer for separate PMO (it's its own repo)
     if (location === 'separate') {
         console.log(chalk.blue('Creating devcontainer config for PMO...'));
+        const gitIdentity = getGitIdentity();
         createDevcontainerConfig({
             agentName: 'pmo',
             agentDir: pmoPath,
             repoWorktrees: [], // PMO is the repo itself, no nested worktrees
+            gitUserName: gitIdentity.name || undefined,
+            gitUserEmail: gitIdentity.email || undefined,
         });
         console.log(chalk.green('  ✓ Devcontainer config created'));
     }

package/dist/lib/pmo/storage/base.js

@@ -218,6 +218,55 @@ export function runMigrations(db) {
             // Table may already be dropped
         }
     }
+    // Migration: Reassign orphaned tickets (TKT-940)
+    // Tickets with project_id that doesn't match any existing project are "orphaned".
+    // This can happen when a 'default' project never existed or was deleted.
+    // NOTE: This runs on every init but is idempotent — if no orphaned tickets exist, it's a no-op.
+    // Kept as a runtime check rather than a one-time migration since orphaned tickets could
+    // reappear if projects are deleted in the future.
+    if (tableExists(T.tickets) && tableExists(T.projects)) {
+        try {
+            // Find orphaned tickets (project_id doesn't match any project)
+            const orphanedTickets = db.prepare(`
+        SELECT t.id, t.project_id
+        FROM ${T.tickets} t
+        LEFT JOIN ${T.projects} p ON t.project_id = p.id
+        WHERE p.id IS NULL
+      `).all();
+            if (orphanedTickets.length > 0) {
+                // Get the first available project to reassign to
+                const firstProject = db.prepare(`
+          SELECT id FROM ${T.projects} ORDER BY created_at ASC LIMIT 1
+        `).get();
+                if (firstProject) {
+                    // Get the default status for the target project's workflow
+                    const project = db.prepare(`
+            SELECT workflow_id FROM ${T.projects} WHERE id = ?
+          `).get(firstProject.id);
+                    const workflowId = project?.workflow_id || 'default';
+                    const defaultStatus = db.prepare(`
+            SELECT id FROM ${T.workflow_statuses}
+            WHERE workflow_id = ? AND is_default = 1
+          `).get(workflowId);
+                    // Reassign orphaned tickets to the first project
+                    const updateStmt = defaultStatus
+                        ? db.prepare(`UPDATE ${T.tickets} SET project_id = ?, status_id = COALESCE(status_id, ?) WHERE id = ?`)
+                        : db.prepare(`UPDATE ${T.tickets} SET project_id = ? WHERE id = ?`);
+                    for (const ticket of orphanedTickets) {
+                        if (defaultStatus) {
+                            updateStmt.run(firstProject.id, defaultStatus.id, ticket.id);
+                        }
+                        else {
+                            updateStmt.run(firstProject.id, ticket.id);
+                        }
+                    }
+                }
+            }
+        }
+        catch {
+            // Non-critical migration - don't fail initialization
+        }
+    }
 }
 /**
  * Seed built-in workflows from BUILTIN_TEMPLATES (single source of truth).

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

@@ -81,7 +81,7 @@ export declare const LEGACY_PRIORITY_MAP: Record<LegacyPriority, Priority>;
 /**
  * Ticket categories for classification.
  */
-export declare const TICKET_CATEGORIES: readonly ["feature", "bug", "refactor", "docs", "test", "chore", "performance", "ci", "build", "security", "database", "release"];
+export declare const TICKET_CATEGORIES: readonly ["feature", "bug", "refactor", "docs", "test", "chore", "performance", "ci", "build", "security", "database", "release", "adhoc"];
 export type TicketCategory = typeof TICKET_CATEGORIES[number];
 /**
  * Category types (ticket vs status).

package/dist/lib/pmo/types.js

@@ -49,6 +49,7 @@ export const TICKET_CATEGORIES = [
     'security',
     'database',
     'release',
+    'adhoc',
 ];
 /**
  * Check if a string is a valid Priority value.

package/dist/lib/pr/index.d.ts

@@ -44,6 +44,11 @@ export declare function getGHUsername(): string | null;
  * Check if GH_TOKEN or GITHUB_TOKEN is set in environment.
  */
 export declare function isGHTokenInEnv(): boolean;
+export interface GitIdentity {
+    name: string | null;
+    email: string | null;
+}
+export declare function getGitIdentity(cwd?: string): GitIdentity;
 /**
  * Get the GitHub repository from git remote.
  * Returns format: owner/repo

package/dist/lib/pr/index.js

@@ -53,6 +53,75 @@ export function getGHUsername() {
 export function isGHTokenInEnv() {
     return !!(process.env.GH_TOKEN || process.env.GITHUB_TOKEN);
 }
+/**
+ * Detect the user's git identity for commit attribution.
+ * Tries GitHub CLI first (gh api user), falls back to git config.
+ * Results are memoized so subprocess calls only run once per process.
+ */
+let _cachedGitIdentity;
+export function getGitIdentity(cwd) {
+    if (_cachedGitIdentity)
+        return _cachedGitIdentity;
+    let name = null;
+    let email = null;
+    // Method 1: Try gh api user (most reliable for GitHub identity)
+    try {
+        const userJson = execSync('gh api user', {
+            encoding: 'utf-8',
+            stdio: ['pipe', 'pipe', 'pipe'],
+        });
+        const user = JSON.parse(userJson);
+        name = user.name || user.login || null;
+        email = user.email || null;
+        // If no public email, try GitHub emails API for primary email
+        if (!email) {
+            try {
+                const emailsJson = execSync('gh api user/emails', {
+                    encoding: 'utf-8',
+                    stdio: ['pipe', 'pipe', 'pipe'],
+                });
+                const emails = JSON.parse(emailsJson);
+                const primary = emails.find(e => e.primary);
+                if (primary) {
+                    email = primary.email;
+                }
+            }
+            catch {
+                // emails API may not be accessible with current token scope
+            }
+        }
+    }
+    catch {
+        // gh not available or not authenticated
+    }
+    // Method 2: Fall back to git config
+    if (!name) {
+        try {
+            name = execSync('git config user.name', {
+                cwd,
+                encoding: 'utf-8',
+                stdio: ['pipe', 'pipe', 'pipe'],
+            }).trim() || null;
+        }
+        catch {
+            // git config not set
+        }
+    }
+    if (!email) {
+        try {
+            email = execSync('git config user.email', {
+                cwd,
+                encoding: 'utf-8',
+                stdio: ['pipe', 'pipe', 'pipe'],
+            }).trim() || null;
+        }
+        catch {
+            // git config not set
+        }
+    }
+    _cachedGitIdentity = { name, email };
+    return _cachedGitIdentity;
+}
 // =============================================================================
 // Git Remote Detection
 // =============================================================================

package/dist/lib/repos/index.js

@@ -7,6 +7,7 @@ import { styles } from '../styles.js';
 import { colors } from '../colors.js';
 import { openWorkspaceDatabase, addRepositoriesToDatabase, getWorkspaceRepositories } from '../database/index.js';
 import { createDevcontainerConfig } from '../execution/devcontainer.js';
+import { getGitIdentity } from '../pr/index.js';
 import { findHQRoot } from '../workspace.js';
 /**
  * Check if we're currently in a git repository
@@ -452,10 +453,13 @@ export async function addRepository(hqPath, repoPath, action) {
         await createWorktreesForRepo(hqPath, repoName, targetPath);
         // Create devcontainer config for sandboxed execution in the central repo
         console.log(styles.muted(`Creating devcontainer config for ${repoName}...`));
+        const gitIdentity = getGitIdentity();
         createDevcontainerConfig({
             agentName: repoName, // Use repo name as identifier
             agentDir: targetPath,
             repoWorktrees: [], // No nested worktrees - this is the repo itself
+            gitUserName: gitIdentity.name || undefined,
+            gitUserEmail: gitIdentity.email || undefined,
         });
         return { success: true, name: repoName };
     }

package/dist/lib/string-utils.d.ts

@@ -0,0 +1,10 @@
+/**
+ * String utilities for CLI table formatting.
+ *
+ * Handles emoji and wide characters that occupy more than one terminal column.
+ */
+/**
+ * Pad a string to the given visual width, accounting for emoji and wide characters.
+ * Unlike String.padEnd(), this measures visual column width rather than character count.
+ */
+export declare function visualPadEnd(str: string, width: number): string;

package/dist/lib/string-utils.js

@@ -0,0 +1,16 @@
+/**
+ * String utilities for CLI table formatting.
+ *
+ * Handles emoji and wide characters that occupy more than one terminal column.
+ */
+import stringWidth from 'string-width';
+/**
+ * Pad a string to the given visual width, accounting for emoji and wide characters.
+ * Unlike String.padEnd(), this measures visual column width rather than character count.
+ */
+export function visualPadEnd(str, width) {
+    const visWidth = stringWidth(str);
+    if (visWidth >= width)
+        return str;
+    return str + ' '.repeat(width - visWidth);
+}