jira-ai 0.2.0 → 0.2.5

package/dist/commands/about.js

@@ -76,7 +76,7 @@ async function aboutCommand() {
     console.log(chalk_1.default.bold('For detailed help on any command, run:'));
     console.log(chalk_1.default.green('  jira-ai <command> --help\n'));
     console.log(chalk_1.default.bold('Configuration:'));
-    console.log('  Settings are managed in settings.yaml');
+    console.log(`  Settings file: ${chalk_1.default.cyan((0, settings_1.getSettingsPath)())}`);
     const allowedProjects = (0, settings_1.getAllowedProjects)();
     console.log(`  - Projects: ${allowedProjects.includes('all') ? 'All allowed' : allowedProjects.join(', ')}`);
     console.log(`  - Commands: ${isAllAllowed ? 'All allowed' : allowedCommandsList.join(', ')}\n`);

package/dist/lib/settings.js

@@ -3,6 +3,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.getSettingsPath = getSettingsPath;
 exports.loadSettings = loadSettings;
 exports.isProjectAllowed = isProjectAllowed;
 exports.isCommandAllowed = isCommandAllowed;
@@ -11,23 +12,55 @@ exports.getAllowedCommands = getAllowedCommands;
 exports.__resetCache__ = __resetCache__;
 const fs_1 = __importDefault(require("fs"));
 const path_1 = __importDefault(require("path"));
+const os_1 = __importDefault(require("os"));
 const js_yaml_1 = __importDefault(require("js-yaml"));
+const chalk_1 = __importDefault(require("chalk"));
+const CONFIG_DIR = path_1.default.join(os_1.default.homedir(), '.jira-ai');
+const SETTINGS_FILE = path_1.default.join(CONFIG_DIR, 'settings.yaml');
 let cachedSettings = null;
+function getSettingsPath() {
+    return SETTINGS_FILE;
+}
 function loadSettings() {
     if (cachedSettings) {
         return cachedSettings;
     }
-    const settingsPath = path_1.default.join(process.cwd(), 'settings.yaml');
-    if (!fs_1.default.existsSync(settingsPath)) {
-        console.warn('Warning: settings.yaml not found. Using default settings (all allowed).');
-        cachedSettings = {
-            projects: ['all'],
-            commands: ['all']
-        };
-        return cachedSettings;
+    // Ensure config directory exists
+    if (!fs_1.default.existsSync(CONFIG_DIR)) {
+        fs_1.default.mkdirSync(CONFIG_DIR, { recursive: true });
+    }
+    if (!fs_1.default.existsSync(SETTINGS_FILE)) {
+        // Check if settings.yaml exists in current working directory (migration/backward compatibility)
+        const localSettingsPath = path_1.default.join(process.cwd(), 'settings.yaml');
+        if (fs_1.default.existsSync(localSettingsPath)) {
+            try {
+                const fileContents = fs_1.default.readFileSync(localSettingsPath, 'utf8');
+                fs_1.default.writeFileSync(SETTINGS_FILE, fileContents);
+                console.log(chalk_1.default?.cyan ? chalk_1.default.cyan(`Migrated settings.yaml to ${SETTINGS_FILE}`) : `Migrated settings.yaml to ${SETTINGS_FILE}`);
+            }
+            catch (error) {
+                console.error('Error migrating settings.yaml:', error);
+            }
+        }
+        else {
+            // Create default settings.yaml if it doesn't exist anywhere
+            const defaultSettings = {
+                projects: ['all'],
+                commands: ['all']
+            };
+            try {
+                const yamlStr = js_yaml_1.default.dump(defaultSettings);
+                fs_1.default.writeFileSync(SETTINGS_FILE, yamlStr);
+            }
+            catch (error) {
+                console.error('Error creating default settings.yaml:', error);
+            }
+            cachedSettings = defaultSettings;
+            return cachedSettings;
+        }
     }
     try {
-        const fileContents = fs_1.default.readFileSync(settingsPath, 'utf8');
+        const fileContents = fs_1.default.readFileSync(SETTINGS_FILE, 'utf8');
         const settings = js_yaml_1.default.load(fileContents);
         cachedSettings = {
             projects: settings.projects || ['all'],
@@ -36,7 +69,7 @@ function loadSettings() {
         return cachedSettings;
     }
     catch (error) {
-        console.error('Error loading settings.yaml:', error);
+        console.error(`Error loading ${SETTINGS_FILE}:`, error);
         process.exit(1);
     }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jira-ai",
-  "version": "0.2.0",
+  "version": "0.2.5",
   "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/commands/about.ts

@@ -1,5 +1,5 @@
 import chalk from 'chalk';
-import { getAllowedCommands, getAllowedProjects, isCommandAllowed } from '../lib/settings';
+import { getAllowedCommands, getAllowedProjects, isCommandAllowed, getSettingsPath } from '../lib/settings';
 
 interface CommandInfo {
   name: string;
@@ -91,7 +91,7 @@ export async function aboutCommand() {
   console.log(chalk.green('  jira-ai <command> --help\n'));
 
   console.log(chalk.bold('Configuration:'));
-  console.log('  Settings are managed in settings.yaml');
+  console.log(`  Settings file: ${chalk.cyan(getSettingsPath())}`);
   const allowedProjects = getAllowedProjects();
   console.log(`  - Projects: ${allowedProjects.includes('all') ? 'All allowed' : allowedProjects.join(', ')}`);
   console.log(`  - Commands: ${isAllAllowed ? 'All allowed' : allowedCommandsList.join(', ')}\n`);

package/src/lib/settings.ts

@@ -1,32 +1,64 @@
 import fs from 'fs';
 import path from 'path';
+import os from 'os';
 import yaml from 'js-yaml';
+import chalk from 'chalk';
 
 export interface Settings {
   projects: string[];
   commands: string[];
 }
 
+const CONFIG_DIR = path.join(os.homedir(), '.jira-ai');
+const SETTINGS_FILE = path.join(CONFIG_DIR, 'settings.yaml');
+
 let cachedSettings: Settings | null = null;
 
+export function getSettingsPath(): string {
+  return SETTINGS_FILE;
+}
+
 export function loadSettings(): Settings {
   if (cachedSettings) {
     return cachedSettings;
   }
 
-  const settingsPath = path.join(process.cwd(), 'settings.yaml');
+  // Ensure config directory exists
+  if (!fs.existsSync(CONFIG_DIR)) {
+    fs.mkdirSync(CONFIG_DIR, { recursive: true });
+  }
 
-  if (!fs.existsSync(settingsPath)) {
-    console.warn('Warning: settings.yaml not found. Using default settings (all allowed).');
-    cachedSettings = {
-      projects: ['all'],
-      commands: ['all']
-    };
-    return cachedSettings;
+  if (!fs.existsSync(SETTINGS_FILE)) {
+    // Check if settings.yaml exists in current working directory (migration/backward compatibility)
+    const localSettingsPath = path.join(process.cwd(), 'settings.yaml');
+    if (fs.existsSync(localSettingsPath)) {
+      try {
+        const fileContents = fs.readFileSync(localSettingsPath, 'utf8');
+        fs.writeFileSync(SETTINGS_FILE, fileContents);
+        console.log(chalk?.cyan ? chalk.cyan(`Migrated settings.yaml to ${SETTINGS_FILE}`) : `Migrated settings.yaml to ${SETTINGS_FILE}`);
+      } catch (error) {
+        console.error('Error migrating settings.yaml:', error);
+      }
+    } else {
+      // Create default settings.yaml if it doesn't exist anywhere
+      const defaultSettings: Settings = {
+        projects: ['all'],
+        commands: ['all']
+      };
+      try {
+        const yamlStr = yaml.dump(defaultSettings);
+        fs.writeFileSync(SETTINGS_FILE, yamlStr);
+      } catch (error) {
+        console.error('Error creating default settings.yaml:', error);
+      }
+      
+      cachedSettings = defaultSettings;
+      return cachedSettings;
+    }
   }
 
   try {
-    const fileContents = fs.readFileSync(settingsPath, 'utf8');
+    const fileContents = fs.readFileSync(SETTINGS_FILE, 'utf8');
     const settings = yaml.load(fileContents) as Settings;
 
     cachedSettings = {
@@ -36,7 +68,7 @@ export function loadSettings(): Settings {
 
     return cachedSettings;
   } catch (error) {
-    console.error('Error loading settings.yaml:', error);
+    console.error(`Error loading ${SETTINGS_FILE}:`, error);
     process.exit(1);
   }
 }

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