jira-ai 0.4.10 → 0.4.13

package/dist/cli.js

@@ -28,7 +28,7 @@ import { hasCredentials } from './lib/auth-storage.js';
 import { CliError } from './types/errors.js';
 import { CommandError } from './lib/errors.js';
 import { ui } from './lib/ui.js';
-import { validateOptions, CreateTaskSchema, AddCommentSchema, UpdateDescriptionSchema, RunJqlSchema, GetPersonWorklogSchema, TimeframeSchema, IssueKeySchema, ProjectKeySchema } from './lib/validation.js';
+import { CreateTaskSchema, AddCommentSchema, UpdateDescriptionSchema, RunJqlSchema, GetPersonWorklogSchema, GetIssueStatisticsSchema, validateOptions, IssueKeySchema, ProjectKeySchema, TimeframeSchema } from './lib/validation.js';
 import { realpathSync } from 'fs';
 // Load environment variables
 // @ts-ignore - quiet option exists but is not in types
@@ -223,7 +223,10 @@ program
 program
     .command('get-issue-statistics <task-ids>')
     .description('Calculate and display time-based metrics for one or more issues (comma-separated). Returns a table containing key, summary, total time logged, original estimate, and a detailed breakdown of duration spent in each status.')
-    .action(withPermission('get-issue-statistics', getIssueStatisticsCommand));
+    .option('--full-breakdown', 'Display each status in its own column')
+    .action(withPermission('get-issue-statistics', getIssueStatisticsCommand, {
+    schema: GetIssueStatisticsSchema
+}));
 // Get person worklog command
 program
     .command('get-person-worklog <person> <timeframe>')

package/dist/commands/get-issue-statistics.js

@@ -2,7 +2,7 @@ import chalk from 'chalk';
 import { getIssueStatistics, validateIssuePermissions } from '../lib/jira-client.js';
 import { formatIssueStatistics } from '../lib/formatters.js';
 import { ui } from '../lib/ui.js';
-export async function getIssueStatisticsCommand(taskIds) {
+export async function getIssueStatisticsCommand(taskIds, options = {}) {
     const ids = taskIds.split(',').map(id => id.trim()).filter(id => id !== '');
     if (ids.length === 0) {
         console.error(chalk.red('Please provide at least one issue ID.'));
@@ -34,7 +34,7 @@ Failed to fetch statistics for ${id}: ${error.message}`));
     }
     if (results.length > 0) {
         ui.succeedSpinner(chalk.green('Statistics retrieved'));
-        console.log(formatIssueStatistics(results));
+        console.log(formatIssueStatistics(results, options.fullBreakdown));
     }
     else {
         ui.failSpinner('Failed to retrieve statistics or all issues were filtered out');

package/dist/commands/task-with-details.js

@@ -1,37 +1,16 @@
 import chalk from 'chalk';
-import { getTaskWithDetails, getCurrentUser } from '../lib/jira-client.js';
+import { validateIssuePermissions } from '../lib/jira-client.js';
 import { formatTaskDetails } from '../lib/formatters.js';
 import { CommandError } from '../lib/errors.js';
 import { ui } from '../lib/ui.js';
-import { isCommandAllowed, validateIssueAgainstFilters } from '../lib/settings.js';
 export async function taskWithDetailsCommand(taskId, options = {}) {
     ui.startSpinner(`Fetching details for ${taskId}...`);
     try {
-        const task = await getTaskWithDetails(taskId, {
+        const task = await validateIssuePermissions(taskId, 'task-with-details', {
             includeHistory: options.includeDetailedHistory,
             historyLimit: options.historyLimit ? parseInt(options.historyLimit, 10) : undefined,
             historyOffset: options.historyOffset ? parseInt(options.historyOffset, 10) : undefined,
         });
-        const projectKey = task.key.split('-')[0];
-        // Check if command is allowed for this project
-        if (!isCommandAllowed('task-with-details', projectKey)) {
-            ui.failSpinner(chalk.red('Command not allowed for this project'));
-            throw new CommandError(`Command 'task-with-details' is not allowed for project ${projectKey}.`, {
-                hints: [
-                    `Update settings.yaml to enable this command for this project.`
-                ]
-            });
-        }
-        // Check granular filters
-        const currentUser = await getCurrentUser();
-        if (!validateIssueAgainstFilters(task, currentUser.accountId)) {
-            ui.failSpinner(chalk.red('Access denied by project filters'));
-            throw new CommandError(`Access to issue ${taskId} is restricted by project filters.`, {
-                hints: [
-                    `This project has filters that you do not meet (e.g., participated roles).`
-                ]
-            });
-        }
         ui.succeedSpinner(chalk.green('Task details retrieved'));
         console.log(formatTaskDetails(task));
     }

package/dist/lib/formatters.js

@@ -1,5 +1,6 @@
 import chalk from 'chalk';
 import Table from 'cli-table3';
+import { decode } from 'html-entities';
 import { formatTimestamp, truncate, formatDuration } from './utils.js';
 /**
  * Create a styled table
@@ -47,7 +48,7 @@ export function formatProjects(projects) {
  * Format task details with comments
  */
 export function formatTaskDetails(task) {
-    let output = '\n' + chalk.bold.cyan(`${task.key}: ${task.summary}`) + '\n\n';
+    let output = '\n' + chalk.bold.cyan(`${task.key}: ${decode(task.summary)}`) + '\n\n';
     // Basic info table
     const infoTable = createTable(['Property', 'Value'], [15, 65]);
     infoTable.push(['Status', chalk.green(task.status.name)], ['Assignee', task.assignee?.displayName || chalk.gray('Unassigned')], ['Reporter', task.reporter?.displayName || chalk.gray('N/A')], ['Created', formatTimestamp(task.created)], ['Updated', formatTimestamp(task.updated)]);
@@ -81,7 +82,7 @@ export function formatTaskDetails(task) {
         const parentTable = createTable(['Key', 'Summary', 'Status'], [12, 50, 18]);
         parentTable.push([
             chalk.cyan(task.parent.key),
-            truncate(task.parent.summary, 50),
+            truncate(decode(task.parent.summary), 50),
             task.parent.status.name,
         ]);
         output += parentTable.toString() + '\n\n';
@@ -93,7 +94,7 @@ export function formatTaskDetails(task) {
         task.subtasks.forEach((subtask) => {
             subtasksTable.push([
                 chalk.cyan(subtask.key),
-                truncate(subtask.summary, 50),
+                truncate(decode(subtask.summary), 50),
                 subtask.status.name,
             ]);
         });
@@ -103,7 +104,7 @@ export function formatTaskDetails(task) {
     if (task.description) {
         output += chalk.bold('Description:') + '\n';
         output += chalk.dim('─'.repeat(80)) + '\n';
-        output += task.description + '\n';
+        output += decode(task.description) + '\n';
         output += chalk.dim('─'.repeat(80)) + '\n\n';
     }
     // Comments
@@ -113,7 +114,7 @@ export function formatTaskDetails(task) {
             output += chalk.cyan(`${index + 1}. ${comment.author.displayName}`) +
                 chalk.gray(` - ${formatTimestamp(comment.created)}`) + '\n';
             output += chalk.dim('─'.repeat(80)) + '\n';
-            output += comment.body + '\n';
+            output += decode(comment.body) + '\n';
             output += chalk.dim('─'.repeat(80)) + '\n\n';
         });
     }
@@ -221,7 +222,7 @@ export function formatJqlResults(issues) {
         }
         table.push([
             chalk.cyan(issue.key),
-            truncate(issue.summary, 40),
+            truncate(decode(issue.summary), 40),
             statusColor(issue.status.name),
             issue.assignee ? truncate(issue.assignee.displayName, 20) : chalk.gray('Unassigned'),
             typeof priorityName === 'string' ? priorityColor(priorityName) : priorityName,
@@ -277,38 +278,137 @@ export function formatProjectIssueTypes(projectKey, issueTypes) {
 /**
  * Format issue statistics
  */
-export function formatIssueStatistics(statsList) {
+export function formatIssueStatistics(statsList, fullBreakdown = false) {
     if (statsList.length === 0) {
         return chalk.yellow('No statistics to display.');
     }
-    const table = createTable([
-        'Key',
-        'Summary',
-        'Time Logged',
-        'Estimate',
-        'Status Breakdown'
-    ], [12, 30, 15, 15, 40]);
+    if (!fullBreakdown) {
+        const table = createTable([
+            'Key',
+            'Summary',
+            'Time Logged',
+            'Estimate',
+            'Status Breakdown'
+        ], [12, 30, 15, 15, 40]);
+        statsList.forEach((stats) => {
+            // Breakdown of time spent in each unique status
+            const breakdown = Object.entries(stats.statusDurations)
+                .map(([status, seconds]) => `${status}: ${formatDuration(seconds, 24)}`)
+                .join('\n');
+            const timeSpentStr = formatDuration(stats.timeSpentSeconds, 8);
+            const estimateStr = formatDuration(stats.originalEstimateSeconds, 8);
+            // Highlight if time spent exceeds estimate
+            const timeSpentFormatted = stats.timeSpentSeconds > stats.originalEstimateSeconds && stats.originalEstimateSeconds > 0
+                ? chalk.red(timeSpentStr)
+                : chalk.green(timeSpentStr);
+            table.push([
+                chalk.cyan(stats.key),
+                truncate(decode(stats.summary), 30),
+                timeSpentFormatted,
+                estimateStr,
+                breakdown
+            ]);
+        });
+        let output = '\n' + chalk.bold('Issue Statistics') + '\n\n';
+        output += table.toString() + '\n';
+        return output;
+    }
+    // Full breakdown logic
+    // 1. Identify all unique statuses and their total durations
+    const statusTotals = {};
+    statsList.forEach(stats => {
+        Object.entries(stats.statusDurations).forEach(([status, duration]) => {
+            statusTotals[status] = (statusTotals[status] || 0) + duration;
+        });
+    });
+    // 2. Filter out statuses with 0 total time
+    const activeStatuses = Object.keys(statusTotals)
+        .filter(status => statusTotals[status] > 0)
+        .sort(); // Sort for consistent column order
+    const omittedStatuses = Object.keys(statusTotals)
+        .filter(status => statusTotals[status] === 0)
+        .sort();
+    // 3. Define headers and column widths
+    const baseHeaders = ['Key', 'Summary', 'Time Logged', 'Estimate'];
+    const headers = [...baseHeaders, ...activeStatuses];
+    // Calculate column widths - 12 for Key, 25 for Summary, 12 for others
+    const colWidths = [12, 25, 12, 12, ...activeStatuses.map(() => 15)];
+    const table = createTable(headers, colWidths);
+    // 4. Helper for median calculation
+    const calculateMedian = (values) => {
+        if (values.length === 0)
+            return 0;
+        const sorted = [...values].sort((a, b) => a - b);
+        const mid = Math.floor(sorted.length / 2);
+        return sorted.length % 2 !== 0
+            ? sorted[mid]
+            : (sorted[mid - 1] + sorted[mid]) / 2;
+    };
+    // 5. Track values for summary rows
+    const columnValues = {
+        'Time Logged': [],
+        'Estimate': []
+    };
+    activeStatuses.forEach(status => { columnValues[status] = []; });
+    // 6. Add rows for each issue
     statsList.forEach((stats) => {
-        // Breakdown of time spent in each unique status
-        const breakdown = Object.entries(stats.statusDurations)
-            .map(([status, seconds]) => `${status}: ${formatDuration(seconds, 24)}`)
-            .join('\n');
-        const timeSpentStr = formatDuration(stats.timeSpentSeconds, 8);
-        const estimateStr = formatDuration(stats.originalEstimateSeconds, 8);
-        // Highlight if time spent exceeds estimate
-        const timeSpentFormatted = stats.timeSpentSeconds > stats.originalEstimateSeconds && stats.originalEstimateSeconds > 0
+        const timeSpentSeconds = stats.timeSpentSeconds;
+        const originalEstimateSeconds = stats.originalEstimateSeconds;
+        columnValues['Time Logged'].push(timeSpentSeconds);
+        columnValues['Estimate'].push(originalEstimateSeconds);
+        const timeSpentStr = formatDuration(timeSpentSeconds, 8);
+        const estimateStr = formatDuration(originalEstimateSeconds, 8);
+        const timeSpentFormatted = timeSpentSeconds > originalEstimateSeconds && originalEstimateSeconds > 0
             ? chalk.red(timeSpentStr)
             : chalk.green(timeSpentStr);
-        table.push([
+        const row = [
             chalk.cyan(stats.key),
-            truncate(stats.summary, 30),
+            truncate(decode(stats.summary), 25),
             timeSpentFormatted,
-            estimateStr,
-            breakdown
-        ]);
+            estimateStr
+        ];
+        activeStatuses.forEach(status => {
+            const duration = stats.statusDurations[status] || 0;
+            columnValues[status].push(duration);
+            row.push(formatDuration(duration, 8));
+        });
+        table.push(row);
     });
-    let output = '\n' + chalk.bold('Issue Statistics') + '\n\n';
+    // 7. Add summary rows
+    const summaryTypes = ['Total', 'Mean', 'Median'];
+    summaryTypes.forEach(type => {
+        const row = [chalk.bold(type), '', '', ''];
+        // Fill basic metrics
+        ['Time Logged', 'Estimate'].forEach((col, idx) => {
+            const values = columnValues[col];
+            let val = 0;
+            if (type === 'Total')
+                val = values.reduce((a, b) => a + b, 0);
+            else if (type === 'Mean')
+                val = values.reduce((a, b) => a + b, 0) / values.length;
+            else if (type === 'Median')
+                val = calculateMedian(values);
+            row[idx + 2] = chalk.bold(formatDuration(val, 8));
+        });
+        // Fill status columns
+        activeStatuses.forEach(status => {
+            const values = columnValues[status];
+            let val = 0;
+            if (type === 'Total')
+                val = values.reduce((a, b) => a + b, 0);
+            else if (type === 'Mean')
+                val = values.reduce((a, b) => a + b, 0) / values.length;
+            else if (type === 'Median')
+                val = calculateMedian(values);
+            row.push(chalk.bold(formatDuration(val, 8)));
+        });
+        table.push(row);
+    });
+    let output = '\n' + chalk.bold('Issue Statistics (Full Breakdown)') + '\n\n';
     output += table.toString() + '\n';
+    if (omittedStatuses.length > 0) {
+        output += chalk.gray(`Note: The following statuses were omitted because they had 0 total time: ${omittedStatuses.join(', ')}`) + '\n';
+    }
     return output;
 }
 /**
@@ -349,9 +449,9 @@ export function formatWorklogs(worklogs, groupByIssue = false) {
         table.push([
             w.started.split('T')[0],
             chalk.cyan(w.issueKey),
-            truncate(w.summary, 30),
+            truncate(decode(w.summary), 30),
             w.timeSpent,
-            truncate(w.comment || '', 45),
+            truncate(decode(w.comment || ''), 45),
         ]);
     });
     const totalHours = (totalSeconds / 3600).toFixed(2);

package/dist/lib/jira-client.js

@@ -1,7 +1,7 @@
 import { Version3Client } from 'jira.js';
 import { calculateStatusStatistics, convertADFToMarkdown } from './utils.js';
 import { loadCredentials } from './auth-storage.js';
-import { applyGlobalFilters, isProjectAllowed, isCommandAllowed, validateIssueAgainstFilters } from './settings.js';
+import { applyGlobalFilters, isProjectAllowed, isCommandAllowed, validateIssueAgainstFilters, loadSettings } from './settings.js';
 import { CommandError } from './errors.js';
 let jiraClient = null;
 let organizationOverride = undefined;
@@ -356,8 +356,8 @@ export async function createIssue(projectKey, summary, issueTypeName, parentKey)
 /**
  * Validate that the current user has permission to perform a command on an issue
  */
-export async function validateIssuePermissions(issueKey, commandName) {
-    const task = await getTaskWithDetails(issueKey);
+export async function validateIssuePermissions(issueKey, commandName, options = {}) {
+    const task = await getTaskWithDetails(issueKey, options);
     const projectKey = task.key.split('-')[0];
     if (!isProjectAllowed(projectKey)) {
         throw new CommandError(`Project '${projectKey}' is not allowed by your settings.`);
@@ -373,6 +373,28 @@ export async function validateIssuePermissions(issueKey, commandName) {
             hints: [`This project has filters that you do not meet (e.g., participated roles).`]
         });
     }
+    // Check JQL filters
+    const settings = loadSettings();
+    let project = settings.projects.find(p => typeof p !== 'string' && p.key === projectKey);
+    if (!project) {
+        project = settings.projects.find(p => typeof p === 'string' && (p === 'all' || p === projectKey));
+    }
+    if (project && typeof project !== 'string' && project.filters?.jql) {
+        const client = getJiraClient();
+        const jql = `key = "${issueKey}" AND (${project.filters.jql})`;
+        const response = await client.issueSearch.searchForIssuesUsingJqlEnhancedSearch({
+            jql,
+            maxResults: 1,
+            fields: ['key'],
+        });
+        if (!response.issues || response.issues.length === 0) {
+            throw new CommandError(`Access to issue ${issueKey} is restricted by project filters.`, {
+                hints: [
+                    `This project has a JQL filter that this issue does not meet: ${project.filters.jql}`,
+                ]
+            });
+        }
+    }
     return task;
 }
 /**

package/dist/lib/settings.js

@@ -32,7 +32,7 @@ export function loadSettings() {
                 console.error('Error migrating settings.yaml:', error);
                 const defaultSettings = {
                     projects: ['all'],
-                    commands: ['me', 'projects', 'task-with-details', 'run-jql', 'list-issue-types', 'project-statuses', 'create-task', 'list-colleagues', 'add-comment', 'add-label', 'delete-label', 'get-issue-statistics', 'get-person-worklog', 'organization', 'transition', 'update-description']
+                    commands: ['me', 'projects', 'task-with-details', 'run-jql', 'list-issue-types', 'project-statuses', 'create-task', 'list-colleagues', 'add-comment', 'add-label-to-issue', 'delete-label-from-issue', 'get-issue-statistics', 'get-person-worklog', 'organization', 'transition', 'update-description']
                 };
                 cachedSettings = defaultSettings;
                 return cachedSettings;
@@ -42,7 +42,7 @@ export function loadSettings() {
             // Create default settings.yaml if it doesn't exist anywhere
             const defaultSettings = {
                 projects: ['all'],
-                commands: ['me', 'projects', 'task-with-details', 'run-jql', 'list-issue-types', 'project-statuses', 'create-task', 'list-colleagues', 'add-comment', 'add-label', 'delete-label', 'get-issue-statistics', 'get-person-worklog', 'organization', 'transition', 'update-description']
+                commands: ['me', 'projects', 'task-with-details', 'run-jql', 'list-issue-types', 'project-statuses', 'create-task', 'list-colleagues', 'add-comment', 'add-label-to-issue', 'delete-label-from-issue', 'get-issue-statistics', 'get-person-worklog', 'organization', 'transition', 'update-description']
             };
             try {
                 const yamlStr = yaml.dump(defaultSettings);
@@ -68,7 +68,7 @@ export function loadSettings() {
             const settings = rawSettings;
             cachedSettings = {
                 projects: settings?.projects || ['all'],
-                commands: settings?.commands || ['me', 'projects', 'task-with-details', 'run-jql', 'list-issue-types', 'project-statuses', 'create-task', 'list-colleagues', 'add-comment', 'add-label', 'delete-label', 'get-issue-statistics', 'get-person-worklog', 'organization', 'transition', 'update-description']
+                commands: settings?.commands || ['me', 'projects', 'task-with-details', 'run-jql', 'list-issue-types', 'project-statuses', 'create-task', 'list-colleagues', 'add-comment', 'add-label-to-issue', 'delete-label-from-issue', 'get-issue-statistics', 'get-person-worklog', 'organization', 'transition', 'update-description']
             };
         }
         else {
@@ -111,7 +111,10 @@ export function isCommandAllowed(commandName, projectKey) {
         return true;
     }
     if (projectKey) {
-        const project = settings.projects.find(p => (typeof p === 'string' ? p : p.key) === projectKey);
+        let project = settings.projects.find(p => typeof p !== 'string' && p.key === projectKey);
+        if (!project) {
+            project = settings.projects.find(p => typeof p === 'string' && (p === 'all' || p === projectKey));
+        }
         if (project && typeof project !== 'string' && project.commands) {
             return project.commands.includes(commandName);
         }
@@ -166,11 +169,12 @@ export function applyGlobalFilters(jql) {
 export function validateIssueAgainstFilters(issue, currentUserId) {
     const settings = loadSettings();
     const projectKey = issue.key.split('-')[0];
-    const project = settings.projects.find(p => {
-        if (typeof p === 'string')
-            return p === 'all' || p === projectKey;
-        return p.key === projectKey;
-    });
+    // Find specific project config first
+    let project = settings.projects.find(p => typeof p !== 'string' && p.key === projectKey);
+    // If not found, look for string match (exact project key or 'all')
+    if (!project) {
+        project = settings.projects.find(p => typeof p === 'string' && (p === 'all' || p === projectKey));
+    }
     if (!project) {
         return false;
     }

package/dist/lib/validation.js

@@ -72,6 +72,9 @@ export const TimeframeSchema = z.string().regex(/^\d+d$/, 'Timeframe must be in
 export const GetPersonWorklogSchema = z.object({
     groupByIssue: z.boolean().optional(),
 });
+export const GetIssueStatisticsSchema = z.object({
+    fullBreakdown: z.boolean().optional(),
+});
 export const ProjectFiltersSchema = z.object({
     participated: z.object({
         was_assignee: z.boolean().optional(),
@@ -92,5 +95,5 @@ export const ProjectSettingSchema = z.union([
 ]);
 export const SettingsSchema = z.object({
     projects: z.array(ProjectSettingSchema).nullish().transform(val => val || ['all']),
-    commands: z.array(z.string()).nullish().transform(val => val || ['me', 'projects', 'task-with-details', 'run-jql', 'list-issue-types', 'project-statuses', 'create-task', 'list-colleagues', 'add-comment', 'add-label', 'delete-label', 'get-issue-statistics', 'get-person-worklog', 'organization', 'transition', 'update-description']),
+    commands: z.array(z.string()).nullish().transform(val => val || ['me', 'projects', 'task-with-details', 'run-jql', 'list-issue-types', 'project-statuses', 'create-task', 'list-colleagues', 'add-comment', 'add-label-to-issue', 'delete-label-from-issue', 'get-issue-statistics', 'get-person-worklog', 'organization', 'transition', 'update-description']),
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jira-ai",
-  "version": "0.4.10",
+  "version": "0.4.13",
   "description": "AI friendly Jira CLI to save context",
   "type": "module",
   "main": "dist/cli.js",
@@ -52,6 +52,7 @@
     "cli-table3": "^0.6.5",
     "commander": "^11.0.0",
     "dotenv": "^17.2.3",
+    "html-entities": "^2.6.0",
     "inquirer": "^9.3.8",
     "jira.js": "^5.2.2",
     "js-yaml": "^4.1.1",