jira-ai 0.2.2 → 0.2.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jira-ai",
-  "version": "0.2.2",
+  "version": "0.2.6",
   "description": "CLI tool for interacting with Atlassian Jira",
   "main": "dist/cli.js",
   "bin": {
@@ -43,5 +43,9 @@
     "ts-jest": "^29.4.6",
     "ts-node": "^10.9.2",
     "typescript": "^5.9.3"
+  },
+  "overrides": {
+    "adf-builder": "npm:@atlaskit/adf-utils@^19.26.4",
+    "uuid": "^10.0.0"
   }
 }

package/src/cli.ts

@@ -8,6 +8,7 @@ import { meCommand } from './commands/me';
 import { projectsCommand } from './commands/projects';
 import { taskWithDetailsCommand } from './commands/task-with-details';
 import { projectStatusesCommand } from './commands/project-statuses';
+import { listIssueTypesCommand } from './commands/list-issue-types';
 import { runJqlCommand } from './commands/run-jql';
 import { updateDescriptionCommand } from './commands/update-description';
 import { addCommentCommand } from './commands/add-comment';
@@ -78,6 +79,12 @@ program
   .description('Show all possible statuses for a project')
   .action(withPermission('project-statuses', projectStatusesCommand));
 
+// List issue types command
+program
+  .command('list-issue-types <project-key>')
+  .description('Show all issue types for a project')
+  .action(withPermission('list-issue-types', listIssueTypesCommand));
+
 // Run JQL command
 program
   .command('run-jql <jql-query>')

package/src/commands/about.ts

@@ -28,6 +28,11 @@ const ALL_COMMANDS: CommandInfo[] = [
     description: 'Show all possible statuses for a project',
     usage: 'jira-ai project-statuses --help'
   },
+  {
+    name: 'list-issue-types',
+    description: 'Show all issue types for a project (e.g., Epic, Task, Subtask)',
+    usage: 'jira-ai list-issue-types <project-key>'
+  },
   {
     name: 'run-jql',
     description: 'Execute JQL query and display results',

package/src/commands/list-issue-types.ts

@@ -0,0 +1,18 @@
+import chalk from 'chalk';
+import ora from 'ora';
+import { getProjectIssueTypes } from '../lib/jira-client';
+import { formatProjectIssueTypes } from '../lib/formatters';
+
+export async function listIssueTypesCommand(projectKey: string): Promise<void> {
+  const spinner = ora(`Fetching issue types for project ${projectKey}...`).start();
+
+  try {
+    const issueTypes = await getProjectIssueTypes(projectKey);
+    spinner.succeed(chalk.green('Issue types retrieved'));
+    console.log(formatProjectIssueTypes(projectKey, issueTypes));
+  } catch (error) {
+    spinner.fail(chalk.red('Failed to fetch issue types'));
+    console.error(chalk.red('\nError: ' + (error instanceof Error ? error.message : 'Unknown error')));
+    process.exit(1);
+  }
+}

package/src/lib/formatters.ts

@@ -1,6 +1,6 @@
 import chalk from 'chalk';
 import Table from 'cli-table3';
-import { UserInfo, Project, TaskDetails, Status, JqlIssue } from './jira-client';
+import { UserInfo, Project, TaskDetails, Status, JqlIssue, IssueType } from './jira-client';
 import { formatTimestamp, truncate } from './utils';
 
 /**
@@ -222,3 +222,52 @@ export function formatJqlResults(issues: JqlIssue[]): string {
 
   return output;
 }
+
+/**
+ * Format project issue types list
+ */
+export function formatProjectIssueTypes(projectKey: string, issueTypes: IssueType[]): string {
+  if (issueTypes.length === 0) {
+    return chalk.yellow('No issue types found for this project.');
+  }
+
+  // Separate standard issue types and subtasks
+  const standardTypes = issueTypes.filter(type => !type.subtask);
+  const subtaskTypes = issueTypes.filter(type => type.subtask);
+
+  let output = '\n' + chalk.bold(`Project ${projectKey} - Issue Types (${issueTypes.length} total)`) + '\n\n';
+
+  // Display standard issue types
+  if (standardTypes.length > 0) {
+    output += chalk.bold('Standard Issue Types:') + '\n';
+    const table = createTable(['Name', 'Type', 'Description'], [20, 15, 55]);
+
+    standardTypes.forEach((issueType) => {
+      table.push([
+        chalk.cyan(issueType.name),
+        issueType.subtask ? chalk.yellow('Subtask') : chalk.green('Standard'),
+        truncate(issueType.description || chalk.gray('No description'), 55),
+      ]);
+    });
+
+    output += table.toString() + '\n';
+  }
+
+  // Display subtask types separately if they exist
+  if (subtaskTypes.length > 0) {
+    output += '\n' + chalk.bold('Subtask Types:') + '\n';
+    const subtaskTable = createTable(['Name', 'Type', 'Description'], [20, 15, 55]);
+
+    subtaskTypes.forEach((issueType) => {
+      subtaskTable.push([
+        chalk.cyan(issueType.name),
+        chalk.yellow('Subtask'),
+        truncate(issueType.description || chalk.gray('No description'), 55),
+      ]);
+    });
+
+    output += subtaskTable.toString() + '\n';
+  }
+
+  return output;
+}

package/src/lib/jira-client.ts

@@ -87,6 +87,14 @@ export interface JqlIssue {
   } | null;
 }
 
+export interface IssueType {
+  id: string;
+  name: string;
+  description?: string;
+  subtask: boolean;
+  hierarchyLevel: number;
+}
+
 let jiraClient: Version3Client | null = null;
 
 /**
@@ -355,3 +363,23 @@ export async function addIssueComment(
     comment: adfContent,
   });
 }
+
+/**
+ * Get all issue types for a project
+ */
+export async function getProjectIssueTypes(projectIdOrKey: string): Promise<IssueType[]> {
+  const client = getJiraClient();
+
+  const project = await client.projects.getProject({
+    projectIdOrKey,
+    expand: 'issueTypes',
+  });
+
+  return project.issueTypes?.map((issueType: any) => ({
+    id: issueType.id || '',
+    name: issueType.name || '',
+    description: issueType.description,
+    subtask: issueType.subtask || false,
+    hierarchyLevel: issueType.hierarchyLevel || 0,
+  })) || [];
+}

package/tests/settings.test.ts

@@ -1,5 +1,6 @@
 import fs from 'fs';
 import path from 'path';
+import os from 'os';
 import {
   loadSettings,
   isProjectAllowed,
@@ -14,7 +15,9 @@ jest.mock('fs');
 const mockFs = fs as jest.Mocked<typeof fs>;
 
 describe('Settings Module', () => {
-  const mockSettingsPath = path.join(process.cwd(), 'settings.yaml');
+  const mockConfigDir = path.join(os.homedir(), '.jira-ai');
+  const mockSettingsPath = path.join(mockConfigDir, 'settings.yaml');
+  const mockLocalSettingsPath = path.join(process.cwd(), 'settings.yaml');
 
   beforeEach(() => {
     jest.clearAllMocks();
@@ -33,28 +36,38 @@ commands:
   - me
   - projects
 `;
-      mockFs.existsSync.mockReturnValue(true);
+      // Mock that config dir exists and settings file exists
+      mockFs.existsSync.mockImplementation((path) => {
+        if (path === mockConfigDir) return true;
+        if (path === mockSettingsPath) return true;
+        return false;
+      });
       mockFs.readFileSync.mockReturnValue(mockYaml);
 
       const settings = loadSettings();
 
       expect(settings.projects).toEqual(['BP', 'PM', 'PS']);
       expect(settings.commands).toEqual(['me', 'projects']);
-      expect(mockFs.existsSync).toHaveBeenCalledWith(mockSettingsPath);
       expect(mockFs.readFileSync).toHaveBeenCalledWith(mockSettingsPath, 'utf8');
     });
 
     it('should return default settings when file does not exist', () => {
+      // Mock that neither config dir nor settings files exist
       mockFs.existsSync.mockReturnValue(false);
+      mockFs.mkdirSync.mockReturnValue(undefined);
+      mockFs.writeFileSync.mockReturnValue(undefined);
 
-      const consoleWarnSpy = jest.spyOn(console, 'warn').mockImplementation();
       const settings = loadSettings();
 
       expect(settings.projects).toEqual(['all']);
       expect(settings.commands).toEqual(['all']);
-      expect(consoleWarnSpy).toHaveBeenCalled();
-
-      consoleWarnSpy.mockRestore();
+      // Should create the config directory
+      expect(mockFs.mkdirSync).toHaveBeenCalledWith(mockConfigDir, { recursive: true });
+      // Should create default settings file
+      expect(mockFs.writeFileSync).toHaveBeenCalledWith(
+        mockSettingsPath,
+        expect.stringContaining('projects')
+      );
     });
 
     it('should handle null/undefined projects/commands by defaulting to all', () => {
@@ -62,7 +75,11 @@ commands:
 projects:
 commands:
 `;
-      mockFs.existsSync.mockReturnValue(true);
+      mockFs.existsSync.mockImplementation((path) => {
+        if (path === mockConfigDir) return true;
+        if (path === mockSettingsPath) return true;
+        return false;
+      });
       mockFs.readFileSync.mockReturnValue(mockYaml);
 
       const settings = loadSettings();
@@ -72,7 +89,11 @@ commands:
     });
 
     it('should exit process on invalid YAML', () => {
-      mockFs.existsSync.mockReturnValue(true);
+      mockFs.existsSync.mockImplementation((path) => {
+        if (path === mockConfigDir) return true;
+        if (path === mockSettingsPath) return true;
+        return false;
+      });
       mockFs.readFileSync.mockReturnValue('invalid: yaml: content:');
 
       const consoleErrorSpy = jest.spyOn(console, 'error').mockImplementation();
@@ -97,7 +118,11 @@ projects:
 commands:
   - me
 `;
-      mockFs.existsSync.mockReturnValue(true);
+      mockFs.existsSync.mockImplementation((path) => {
+        if (path === mockConfigDir) return true;
+        if (path === mockSettingsPath) return true;
+        return false;
+      });
       mockFs.readFileSync.mockReturnValue(mockYaml);
 
       expect(isProjectAllowed('BP')).toBe(true);
@@ -114,7 +139,11 @@ projects:
 commands:
   - me
 `;
-      mockFs.existsSync.mockReturnValue(true);
+      mockFs.existsSync.mockImplementation((path) => {
+        if (path === mockConfigDir) return true;
+        if (path === mockSettingsPath) return true;
+        return false;
+      });
       mockFs.readFileSync.mockReturnValue(mockYaml);
 
       expect(isProjectAllowed('BP')).toBe(true);
@@ -131,7 +160,11 @@ projects:
 commands:
   - me
 `;
-      mockFs.existsSync.mockReturnValue(true);
+      mockFs.existsSync.mockImplementation((path) => {
+        if (path === mockConfigDir) return true;
+        if (path === mockSettingsPath) return true;
+        return false;
+      });
       mockFs.readFileSync.mockReturnValue(mockYaml);
 
       expect(isProjectAllowed('XYZ')).toBe(false);
@@ -146,7 +179,11 @@ projects:
 commands:
   - me
 `;
-      mockFs.existsSync.mockReturnValue(true);
+      mockFs.existsSync.mockImplementation((path) => {
+        if (path === mockConfigDir) return true;
+        if (path === mockSettingsPath) return true;
+        return false;
+      });
       mockFs.readFileSync.mockReturnValue(mockYaml);
 
       expect(isProjectAllowed('BP')).toBe(true);
@@ -163,7 +200,11 @@ projects:
 commands:
   - all
 `;
-      mockFs.existsSync.mockReturnValue(true);
+      mockFs.existsSync.mockImplementation((path) => {
+        if (path === mockConfigDir) return true;
+        if (path === mockSettingsPath) return true;
+        return false;
+      });
       mockFs.readFileSync.mockReturnValue(mockYaml);
 
       expect(isCommandAllowed('me')).toBe(true);
@@ -179,7 +220,11 @@ commands:
   - me
   - projects
 `;
-      mockFs.existsSync.mockReturnValue(true);
+      mockFs.existsSync.mockImplementation((path) => {
+        if (path === mockConfigDir) return true;
+        if (path === mockSettingsPath) return true;
+        return false;
+      });
       mockFs.readFileSync.mockReturnValue(mockYaml);
 
       expect(isCommandAllowed('me')).toBe(true);
@@ -194,7 +239,11 @@ commands:
   - me
   - projects
 `;
-      mockFs.existsSync.mockReturnValue(true);
+      mockFs.existsSync.mockImplementation((path) => {
+        if (path === mockConfigDir) return true;
+        if (path === mockSettingsPath) return true;
+        return false;
+      });
       mockFs.readFileSync.mockReturnValue(mockYaml);
 
       expect(isCommandAllowed('task-with-details')).toBe(false);
@@ -212,7 +261,11 @@ projects:
 commands:
   - me
 `;
-      mockFs.existsSync.mockReturnValue(true);
+      mockFs.existsSync.mockImplementation((path) => {
+        if (path === mockConfigDir) return true;
+        if (path === mockSettingsPath) return true;
+        return false;
+      });
       mockFs.readFileSync.mockReturnValue(mockYaml);
 
       const projects = getAllowedProjects();
@@ -226,7 +279,11 @@ projects:
 commands:
   - me
 `;
-      mockFs.existsSync.mockReturnValue(true);
+      mockFs.existsSync.mockImplementation((path) => {
+        if (path === mockConfigDir) return true;
+        if (path === mockSettingsPath) return true;
+        return false;
+      });
       mockFs.readFileSync.mockReturnValue(mockYaml);
 
       const projects = getAllowedProjects();
@@ -243,7 +300,11 @@ commands:
   - me
   - projects
 `;
-      mockFs.existsSync.mockReturnValue(true);
+      mockFs.existsSync.mockImplementation((path) => {
+        if (path === mockConfigDir) return true;
+        if (path === mockSettingsPath) return true;
+        return false;
+      });
       mockFs.readFileSync.mockReturnValue(mockYaml);
 
       const commands = getAllowedCommands();
@@ -257,7 +318,11 @@ projects:
 commands:
   - all
 `;
-      mockFs.existsSync.mockReturnValue(true);
+      mockFs.existsSync.mockImplementation((path) => {
+        if (path === mockConfigDir) return true;
+        if (path === mockSettingsPath) return true;
+        return false;
+      });
       mockFs.readFileSync.mockReturnValue(mockYaml);
 
       const commands = getAllowedCommands();
@@ -273,7 +338,11 @@ projects:
 commands:
   - me
 `;
-      mockFs.existsSync.mockReturnValue(true);
+      mockFs.existsSync.mockImplementation((path) => {
+        if (path === mockConfigDir) return true;
+        if (path === mockSettingsPath) return true;
+        return false;
+      });
       mockFs.readFileSync.mockReturnValue(mockYaml);
 
       // Call multiple times