jira-pilot 2.0.0 → 2.0.2

package/src/utils/config.js

@@ -1,60 +1,97 @@
-import Conf from 'conf';
-
-const schema = {
-    jiraUrl: {
-        type: 'string',
-        format: 'url'
-    },
-    email: {
-        type: 'string',
-        format: 'email'
-    },
-    apiToken: {
-        type: 'string'
-    },
-    aiKey: {
-        type: 'string'
-    },
-    aiProvider: {
-        type: 'string',
-        default: 'openai'
-    },
-    aiEnabled: {
-        type: 'boolean',
-        default: false
-    }
-};
-
-const config = new Conf({
-    projectName: 'jira-pilot',
-    schema
-});
-
-export const getCredentials = () => {
-    return {
-        jiraUrl: config.get('jiraUrl'),
-        email: config.get('email'),
-        apiToken: config.get('apiToken'),
-        aiKey: config.get('aiKey'),
-        aiProvider: config.get('aiProvider'),
-        aiEnabled: config.get('aiEnabled')
-    };
-};
-
-export const setCredentials = ({ jiraUrl, email, apiToken, aiKey, aiProvider, aiEnabled }) => {
-    if (jiraUrl) config.set('jiraUrl', jiraUrl);
-    if (email) config.set('email', email);
-    if (apiToken) config.set('apiToken', apiToken);
-    if (aiKey) config.set('aiKey', aiKey);
-    if (aiProvider) config.set('aiProvider', aiProvider);
-    if (typeof aiEnabled !== 'undefined') config.set('aiEnabled', aiEnabled);
-};
-
-export const clearCredentials = () => {
-    config.clear();
-};
-
-export const hasCredentials = () => {
-    const creds = getCredentials();
-    return !!(creds.jiraUrl && creds.email && creds.apiToken);
-};
+import Conf from 'conf';
+
+const schema = {
+    jiraUrl: {
+        type: 'string',
+        format: 'url'
+    },
+    email: {
+        type: 'string',
+        format: 'email'
+    },
+    apiToken: {
+        type: 'string'
+    },
+    aiKey: {
+        type: 'string'
+    },
+    aiProvider: {
+        type: 'string',
+        default: 'openai'
+    },
+    aiEnabled: {
+        type: 'boolean',
+        default: false
+    },
+    githubToken: {
+        type: 'string'
+    }
+};
+
+const config = new Conf({
+    projectName: 'jira-pilot',
+    schema
+});
+
+export const getCredentials = () => {
+    return {
+        jiraUrl: config.get('jiraUrl'),
+        email: config.get('email'),
+        apiToken: config.get('apiToken'),
+        aiKey: config.get('aiKey'),
+        aiProvider: config.get('aiProvider'),
+        aiEnabled: config.get('aiEnabled'),
+        githubToken: config.get('githubToken')
+    };
+};
+
+export const setCredentials = ({ jiraUrl, email, apiToken, aiKey, aiProvider, aiEnabled, githubToken }) => {
+    if (jiraUrl) config.set('jiraUrl', jiraUrl);
+    if (email) config.set('email', email);
+    if (apiToken) config.set('apiToken', apiToken);
+    if (aiKey) config.set('aiKey', aiKey);
+    if (aiProvider) config.set('aiProvider', aiProvider);
+    if (typeof aiEnabled !== 'undefined') config.set('aiEnabled', aiEnabled);
+    if (githubToken) config.set('githubToken', githubToken);
+};
+
+export const clearCredentials = () => {
+    config.clear();
+};
+
+export const hasCredentials = () => {
+    const creds = getCredentials();
+    return !!(creds.jiraUrl && creds.email && creds.apiToken);
+};
+
+// ── Profile Management ──────────────────────────────────────────────
+
+export const saveProfile = (name) => {
+    const creds = getCredentials();
+    config.set(`profiles.${name}`, creds);
+    config.set('activeProfile', name);
+};
+
+export const loadProfile = (name) => {
+    const profile = config.get(`profiles.${name}`);
+    if (!profile) return false;
+    setCredentials(profile);
+    config.set('activeProfile', name);
+    return true;
+};
+
+export const deleteProfile = (name) => {
+    config.delete(`profiles.${name}`);
+    if (config.get('activeProfile') === name) {
+        config.delete('activeProfile');
+    }
+};
+
+export const listProfiles = () => {
+    const profiles = config.get('profiles') || {};
+    return Object.keys(profiles);
+};
+
+export const getActiveProfile = () => {
+    return config.get('activeProfile') || null;
+};

package/src/utils/error-handler.js

@@ -1,41 +1,41 @@
-import chalk from 'chalk';
-
-/**
- * Standardized error handler for CLI commands.
- * Stops the spinner and prints a formatted error message.
- *
- * @param {object|null} spinner - Ora spinner instance (will be stopped/failed)
- * @param {Error} error - The error object
- * @param {string} [context] - Optional context (e.g., "Failed to list issues")
- */
-export function handleCommandError(spinner, error, context = 'Operation failed') {
-    // Handle user cancellation (Ctrl+C in enquirer)
-    if (error === '' || (error && error.message === '')) {
-        if (spinner) spinner.stop();
-        console.log(chalk.yellow('\nCancelled.'));
-        return;
-    }
-
-    if (spinner) {
-        spinner.fail(context);
-    } else {
-        console.error(chalk.red(`\n${context}:`));
-    }
-
-    if (error.response) {
-        const status = error.response.status;
-        if (status === 404) {
-            console.error(chalk.red('Resource not found. Check the ID or key.'));
-        } else if (status === 400) {
-            const data = error.response.data;
-            const messages = data?.errorMessages?.join(', ') || data?.errors
-                ? Object.entries(data.errors).map(([k, v]) => `${k}: ${v}`).join(', ')
-                : JSON.stringify(data);
-            console.error(chalk.red(`Bad Request: ${messages}`));
-        } else {
-            console.error(chalk.red(`Error ${status}: `), error.response.data);
-        }
-    } else {
-        console.error(chalk.red(error.message));
-    }
-}
+import chalk from 'chalk';
+
+/**
+ * Standardized error handler for CLI commands.
+ * Stops the spinner and prints a formatted error message.
+ *
+ * @param {object|null} spinner - Ora spinner instance (will be stopped/failed)
+ * @param {Error} error - The error object
+ * @param {string} [context] - Optional context (e.g., "Failed to list issues")
+ */
+export function handleCommandError(spinner, error, context = 'Operation failed') {
+    // Handle user cancellation (Ctrl+C in enquirer)
+    if (error === '' || (error && error.message === '')) {
+        if (spinner) spinner.stop();
+        console.log(chalk.yellow('\nCancelled.'));
+        return;
+    }
+
+    if (spinner) {
+        spinner.fail(context);
+    } else {
+        console.error(chalk.red(`\n${context}:`));
+    }
+
+    if (error.response) {
+        const status = error.response.status;
+        if (status === 404) {
+            console.error(chalk.red('Resource not found. Check the ID or key.'));
+        } else if (status === 400) {
+            const data = error.response.data;
+            const messages = data?.errorMessages?.join(', ') || (data?.errors
+                ? Object.entries(data.errors).map(([k, v]) => `${k}: ${v}`).join(', ')
+                : JSON.stringify(data));
+            console.error(chalk.red(`Bad Request: ${messages}`));
+        } else {
+            console.error(chalk.red(`Error ${status}: `), error.response.data);
+        }
+    } else {
+        console.error(chalk.red(error.message));
+    }
+}

package/src/utils/text-to-adf.js

@@ -1,34 +1,34 @@
-/**
- * Converts plain text to Atlassian Document Format (ADF).
- * Splits on newlines to create separate paragraphs.
- *
- * @param {string} text - Plain text string
- * @returns {object} ADF document node
- */
-export function textToADF(text) {
-    if (!text || typeof text !== 'string') {
-        return {
-            type: 'doc',
-            version: 1,
-            content: [
-                {
-                    type: 'paragraph',
-                    content: []
-                }
-            ]
-        };
-    }
-
-    const paragraphs = text.split('\n').map(line => ({
-        type: 'paragraph',
-        content: line.trim().length > 0
-            ? [{ type: 'text', text: line }]
-            : []
-    }));
-
-    return {
-        type: 'doc',
-        version: 1,
-        content: paragraphs
-    };
-}
+/**
+ * Converts plain text to Atlassian Document Format (ADF).
+ * Splits on newlines to create separate paragraphs.
+ *
+ * @param {string} text - Plain text string
+ * @returns {object} ADF document node
+ */
+export function textToADF(text) {
+    if (!text || typeof text !== 'string') {
+        return {
+            type: 'doc',
+            version: 1,
+            content: [
+                {
+                    type: 'paragraph',
+                    content: []
+                }
+            ]
+        };
+    }
+
+    const paragraphs = text.split('\n').map(line => ({
+        type: 'paragraph',
+        content: line.trim().length > 0
+            ? [{ type: 'text', text: line }]
+            : []
+    }));
+
+    return {
+        type: 'doc',
+        version: 1,
+        content: paragraphs
+    };
+}

package/src/utils/validators.js

@@ -0,0 +1,88 @@
+/**
+ * Input Validators for CLI commands.
+ * Validates user input before API calls to catch errors early.
+ */
+
+/**
+ * Validates a Jira issue key format (e.g., PROJ-123).
+ * @param {string} key - The issue key to validate.
+ * @returns {{ valid: boolean, message?: string }}
+ */
+export function validateIssueKey(key) {
+    if (!key || typeof key !== 'string') {
+        return { valid: false, message: 'Issue key is required.' };
+    }
+
+    const trimmed = key.trim().toUpperCase();
+    const pattern = /^[A-Z][A-Z0-9_]+-\d+$/;
+
+    if (!pattern.test(trimmed)) {
+        return {
+            valid: false,
+            message: `Invalid issue key "${key}". Expected format: PROJ-123 (letters/numbers, dash, digits).`
+        };
+    }
+
+    return { valid: true };
+}
+
+/**
+ * Validates a Jira project key (e.g., PROJ).
+ * @param {string} key - The project key to validate.
+ * @returns {{ valid: boolean, message?: string }}
+ */
+export function validateProjectKey(key) {
+    if (!key || typeof key !== 'string') {
+        return { valid: false, message: 'Project key is required.' };
+    }
+
+    const trimmed = key.trim().toUpperCase();
+    const pattern = /^[A-Z][A-Z0-9_]+$/;
+
+    if (!pattern.test(trimmed)) {
+        return {
+            valid: false,
+            message: `Invalid project key "${key}". Must start with a letter and contain only uppercase letters, digits, or underscores.`
+        };
+    }
+
+    return { valid: true };
+}
+
+/**
+ * Validates a Jira site URL.
+ * @param {string} url - The URL to validate.
+ * @returns {{ valid: boolean, message?: string }}
+ */
+export function validateUrl(url) {
+    if (!url || typeof url !== 'string') {
+        return { valid: false, message: 'URL is required.' };
+    }
+
+    const trimmed = url.trim();
+
+    try {
+        const parsed = new URL(trimmed);
+        if (!['http:', 'https:'].includes(parsed.protocol)) {
+            return { valid: false, message: 'URL must use http or https protocol.' };
+        }
+        return { valid: true };
+    } catch {
+        return { valid: false, message: `Invalid URL: "${url}". Example: https://your-company.atlassian.net` };
+    }
+}
+
+/**
+ * Sanitizes a JQL string by escaping potentially dangerous characters.
+ * This is a basic sanitization — Jira's API does its own validation too.
+ * @param {string} jql - The JQL query string.
+ * @returns {string} The sanitized JQL string.
+ */
+export function sanitizeJql(jql) {
+    if (!jql || typeof jql !== 'string') {
+        return '';
+    }
+
+    // Remove null bytes and control characters
+    return jql.replace(/[\x00-\x08\x0B\x0C\x0E-\x1F\x7F]/g, '').trim();
+}