@vfarcic/dot-ai 1.16.1 → 1.16.3

package/README.md

@@ -85,3 +85,4 @@ DevOps AI Toolkit is built on:
 ---
 
 **DevOps AI Toolkit** - Making cloud native operations accessible through AI-powered intelligence.
+

package/dist/core/ai-provider-factory.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"ai-provider-factory.d.ts","sourceRoot":"","sources":["../../src/core/ai-provider-factory.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,EACL,UAAU,EACV,gBAAgB,EACjB,MAAM,yBAAyB,CAAC;AA8BjC;;;;;;;;;;;;;;GAcG;AACH,qBAAa,iBAAiB;IAC5B;;;;;;OAMG;IACH,MAAM,CAAC,MAAM,CAAC,MAAM,EAAE,gBAAgB,GAAG,UAAU;IA2BnD;;;;;;;;;;OAUG;IACH,MAAM,CAAC,aAAa,IAAI,UAAU;IAqGlC;;;;;OAKG;IACH,MAAM,CAAC,mBAAmB,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO;IAWrD;;;;OAIG;IACH,MAAM,CAAC,qBAAqB,IAAI,MAAM,EAAE;IAMxC;;;;;OAKG;IACH,MAAM,CAAC,qBAAqB,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO;CAGxD;AAED;;;GAGG;AACH,wBAAgB,gBAAgB,IAAI,UAAU,CAE7C"}
+{"version":3,"file":"ai-provider-factory.d.ts","sourceRoot":"","sources":["../../src/core/ai-provider-factory.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,EAAE,UAAU,EAAE,gBAAgB,EAAE,MAAM,yBAAyB,CAAC;AAiCvE;;;;;;;;;;;;;;GAcG;AACH,qBAAa,iBAAiB;IAC5B;;;;;;OAMG;IACH,MAAM,CAAC,MAAM,CAAC,MAAM,EAAE,gBAAgB,GAAG,UAAU;IA+BnD;;;;;;;;;;OAUG;IACH,MAAM,CAAC,aAAa,IAAI,UAAU;IAsGlC;;;;;OAKG;IACH,MAAM,CAAC,mBAAmB,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO;IAWrD;;;;OAIG;IACH,MAAM,CAAC,qBAAqB,IAAI,MAAM,EAAE;IAMxC;;;;;OAKG;IACH,MAAM,CAAC,qBAAqB,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO;CAGxD;AAED;;;GAGG;AACH,wBAAgB,gBAAgB,IAAI,UAAU,CAE7C"}

package/dist/core/ai-provider-factory.js

@@ -30,6 +30,7 @@ const PROVIDER_ENV_KEYS = {
     kimi: 'MOONSHOT_API_KEY', // PRD #353: Moonshot AI Kimi K2.5
     alibaba: 'ALIBABA_API_KEY', // PRD #382: Alibaba Qwen 3.5 Plus
     xai: 'XAI_API_KEY',
+    custom: 'CUSTOM_LLM_API_KEY', // PRD #194 / Issue #474: Explicit opt-in to custom OpenAI-compatible endpoint
 };
 const IMPLEMENTED_PROVIDERS = Object.keys(model_config_1.CURRENT_MODELS);
 /**

package/dist/core/internal-tools.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"internal-tools.d.ts","sourceRoot":"","sources":["../../src/core/internal-tools.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;GAeG;AAIH,OAAO,KAAK,EAAE,MAAM,EAAE,YAAY,EAAE,MAAM,4BAA4B,CAAC;AAsBvE;;;;GAIG;AACH,wBAAgB,wBAAwB,CAAC,SAAS,EAAE,MAAM,GAAG,MAAM,CAUlE;AAkBD;;;GAGG;AACH,wBAAgB,gBAAgB,IAAI,MAAM,EAAE,CAkD3C;AAiHD,MAAM,WAAW,gBAAgB;IAC/B,QAAQ,EAAE,MAAM,CAAC;IACjB,KAAK,EAAE,KAAK,CAAC;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,OAAO,EAAE,MAAM,CAAA;KAAE,CAAC,CAAC;IAChD,KAAK,EAAE,MAAM,CAAC;IACd,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,UAAU,EAAE,MAAM,CAAC;IACnB,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB;AAED,MAAM,MAAM,iBAAiB,GACzB;IAAE,OAAO,EAAE,IAAI,CAAC;IAAC,KAAK,EAAE,MAAM,CAAC;IAAC,QAAQ,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,MAAM,CAAC;IAAC,UAAU,EAAE,MAAM,CAAC;IAAC,YAAY,EAAE,MAAM,EAAE,CAAA;CAAE,GAC9G;IAAE,OAAO,EAAE,IAAI,CAAC;IAAC,MAAM,EAAE,MAAM,CAAC;IAAC,UAAU,EAAE,MAAM,CAAC;IAAC,YAAY,EAAE,MAAM,EAAE,CAAC;IAAC,KAAK,EAAE,MAAM,CAAA;CAAE,GAC5F;IAAE,OAAO,EAAE,KAAK,CAAC;IAAC,KAAK,EAAE,MAAM,CAAA;CAAE,CAAC;AAgItC;;;GAGG;AACH,wBAAgB,0BAA0B,CAAC,SAAS,EAAE,MAAM,GAAG,YAAY,CAqB1E;AAID;;;;GAIG;AACH,wBAAgB,gBAAgB,CAAC,QAAQ,GAAE,MAAuB,GAAG,IAAI,CAoBxE"}
+{"version":3,"file":"internal-tools.d.ts","sourceRoot":"","sources":["../../src/core/internal-tools.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;GAeG;AAIH,OAAO,KAAK,EAAE,MAAM,EAAE,YAAY,EAAE,MAAM,4BAA4B,CAAC;AAsBvE;;;;GAIG;AACH,wBAAgB,wBAAwB,CAAC,SAAS,EAAE,MAAM,GAAG,MAAM,CAUlE;AAgBD;;;GAGG;AACH,wBAAgB,gBAAgB,IAAI,MAAM,EAAE,CAkD3C;AAiHD,MAAM,WAAW,gBAAgB;IAC/B,QAAQ,EAAE,MAAM,CAAC;IACjB,KAAK,EAAE,KAAK,CAAC;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,OAAO,EAAE,MAAM,CAAA;KAAE,CAAC,CAAC;IAChD,KAAK,EAAE,MAAM,CAAC;IACd,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,UAAU,EAAE,MAAM,CAAC;IACnB,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB;AAED,MAAM,MAAM,iBAAiB,GACzB;IACE,OAAO,EAAE,IAAI,CAAC;IACd,KAAK,EAAE,MAAM,CAAC;IACd,QAAQ,EAAE,MAAM,CAAC;IACjB,MAAM,EAAE,MAAM,CAAC;IACf,UAAU,EAAE,MAAM,CAAC;IACnB,YAAY,EAAE,MAAM,EAAE,CAAC;CACxB,GACD;IACE,OAAO,EAAE,IAAI,CAAC;IACd,MAAM,EAAE,MAAM,CAAC;IACf,UAAU,EAAE,MAAM,CAAC;IACnB,YAAY,EAAE,MAAM,EAAE,CAAC;IACvB,KAAK,EAAE,MAAM,CAAC;CACf,GACD;IAAE,OAAO,EAAE,KAAK,CAAC;IAAC,KAAK,EAAE,MAAM,CAAA;CAAE,CAAC;AA4ItC;;;GAGG;AACH,wBAAgB,0BAA0B,CAAC,SAAS,EAAE,MAAM,GAAG,YAAY,CAqB1E;AAID;;;;GAIG;AACH,wBAAgB,gBAAgB,CAAC,QAAQ,GAAE,MAAuB,GAAG,IAAI,CAuBxE"}

package/dist/core/internal-tools.js

@@ -89,9 +89,7 @@ function validatePathWithinClones(inputPath) {
 function repoUrlToDirectoryName(repoUrl) {
     try {
         const url = new URL(repoUrl);
-        const repoPath = url.pathname
-            .replace(/^\//, '')
-            .replace(/\.git$/, '');
+        const repoPath = url.pathname.replace(/^\//, '').replace(/\.git$/, '');
         return (0, solution_utils_js_1.sanitizeIntentForLabel)(repoPath);
     }
     catch {
@@ -196,7 +194,7 @@ function handleFsList(args) {
         return `Error: path is not a directory: ${inputPath}`;
     }
     const entries = fs.readdirSync(resolved, { withFileTypes: true });
-    return entries.map((entry) => ({
+    return entries.map(entry => ({
         name: entry.name,
         type: entry.isDirectory() ? 'directory' : 'file',
     }));
@@ -257,7 +255,10 @@ async function handleGitCreatePr(args) {
         return { success: false, error: 'repoPath is required' };
     }
     if (!files || !Array.isArray(files) || files.length === 0) {
-        return { success: false, error: 'files array is required and must not be empty' };
+        return {
+            success: false,
+            error: 'files array is required and must not be empty',
+        };
     }
     if (!title) {
         return { success: false, error: 'title is required' };
@@ -324,6 +325,12 @@ async function handleGitCreatePr(args) {
                 body,
                 head: branchName,
                 base: baseBranch,
+                // Test-only switch: integration tests set this so PRs they create
+                // don't trigger CodeRabbit (which has drafts: false in .coderabbit.yaml).
+                // Production never sets this env var.
+                ...(process.env.DOT_AI_GIT_CREATE_DRAFT_PRS === 'true' && {
+                    draft: true,
+                }),
             }),
             signal: AbortSignal.timeout(30000),
         });
@@ -356,7 +363,7 @@ async function handleGitCreatePr(args) {
  */
 function createInternalToolExecutor(sessionId) {
     const handlers = {
-        git_clone: (args) => handleGitClone(args, sessionId),
+        git_clone: args => handleGitClone(args, sessionId),
         fs_list: handleFsList,
         fs_read: handleFsRead,
         git_create_pr: handleGitCreatePr,
@@ -380,7 +387,9 @@ function createInternalToolExecutor(sessionId) {
  */
 function cleanupOldClones(maxAgeMs = DEFAULT_TTL_MS) {
     const clonesDir = getClonesDir();
-    fs.promises.access(clonesDir).then(async () => {
+    fs.promises
+        .access(clonesDir)
+        .then(async () => {
         const now = Date.now();
         const entries = await fs.promises.readdir(clonesDir);
         for (const entry of entries) {
@@ -395,7 +404,8 @@ function cleanupOldClones(maxAgeMs = DEFAULT_TTL_MS) {
                 // Ignore errors during cleanup (e.g., concurrent deletion)
             }
         }
-    }).catch(() => {
+    })
+        .catch(() => {
         // Directory doesn't exist yet, nothing to clean
     });
 }

package/dist/tools/answer-question.d.ts

@@ -4,6 +4,22 @@
 import { z } from 'zod';
 import { DotAI } from '../core/index';
 import { Logger } from '../core/error-handling';
+interface QuestionValidation {
+    required?: boolean;
+    message?: string;
+    min?: number;
+    max?: number;
+    pattern?: string;
+    options?: string[];
+}
+interface Question {
+    id?: string;
+    question: string;
+    type?: 'text' | 'number' | 'boolean' | 'select' | 'multiSelect';
+    validation?: QuestionValidation;
+    answer?: unknown;
+    options?: string[];
+}
 export declare const ANSWERQUESTION_TOOL_NAME = "answerQuestion";
 export declare const ANSWERQUESTION_TOOL_DESCRIPTION = "Process user answers and return remaining questions or completion status. For open stage, use \"open\" as the answer key.";
 export declare const ANSWERQUESTION_TOOL_INPUT_SCHEMA: {
@@ -17,6 +33,10 @@ export declare const ANSWERQUESTION_TOOL_INPUT_SCHEMA: {
     answers: z.ZodRecord<z.ZodString, z.ZodAny>;
     interaction_id: z.ZodOptional<z.ZodString>;
 };
+/**
+ * Validate answer against question schema
+ */
+export declare function validateAnswer(answer: unknown, question: Question): string | null;
 /**
  * Direct MCP tool handler for answerQuestion functionality
  */
@@ -31,4 +51,5 @@ export declare function handleAnswerQuestionTool(args: {
         text: string;
     }[];
 }>;
+export {};
 //# sourceMappingURL=answer-question.d.ts.map

package/dist/tools/answer-question.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"answer-question.d.ts","sourceRoot":"","sources":["../../src/tools/answer-question.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAOxB,OAAO,EAAE,KAAK,EAAE,MAAM,eAAe,CAAC;AACtC,OAAO,EAAE,MAAM,EAAE,MAAM,wBAAwB,CAAC;AA0EhD,eAAO,MAAM,wBAAwB,mBAAmB,CAAC;AACzD,eAAO,MAAM,+BAA+B,8HAC+E,CAAC;AAG5H,eAAO,MAAM,gCAAgC;;;;;;;;;;CAmB5C,CAAC;AAinBF;;GAEG;AACH,wBAAsB,wBAAwB,CAC5C,IAAI,EAAE;IACJ,UAAU,EAAE,MAAM,CAAC;IACnB,KAAK,EAAE,UAAU,GAAG,OAAO,GAAG,UAAU,GAAG,MAAM,CAAC;IAClD,OAAO,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;IACjC,cAAc,CAAC,EAAE,MAAM,CAAC;CACzB,EACD,KAAK,EAAE,KAAK,EACZ,MAAM,EAAE,MAAM,EACd,SAAS,EAAE,MAAM,GAChB,OAAO,CAAC;IAAE,OAAO,EAAE;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,MAAM,CAAA;KAAE,EAAE,CAAA;CAAE,CAAC,CAgZxD"}
+{"version":3,"file":"answer-question.d.ts","sourceRoot":"","sources":["../../src/tools/answer-question.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAOxB,OAAO,EAAE,KAAK,EAAE,MAAM,eAAe,CAAC;AACtC,OAAO,EAAE,MAAM,EAAE,MAAM,wBAAwB,CAAC;AAUhD,UAAU,kBAAkB;IAC1B,QAAQ,CAAC,EAAE,OAAO,CAAC;IACnB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,GAAG,CAAC,EAAE,MAAM,CAAC;IACb,GAAG,CAAC,EAAE,MAAM,CAAC;IACb,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,OAAO,CAAC,EAAE,MAAM,EAAE,CAAC;CACpB;AAED,UAAU,QAAQ;IAChB,EAAE,CAAC,EAAE,MAAM,CAAC;IACZ,QAAQ,EAAE,MAAM,CAAC;IACjB,IAAI,CAAC,EAAE,MAAM,GAAG,QAAQ,GAAG,SAAS,GAAG,QAAQ,GAAG,aAAa,CAAC;IAChE,UAAU,CAAC,EAAE,kBAAkB,CAAC;IAChC,MAAM,CAAC,EAAE,OAAO,CAAC;IACjB,OAAO,CAAC,EAAE,MAAM,EAAE,CAAC;CACpB;AAgDD,eAAO,MAAM,wBAAwB,mBAAmB,CAAC;AACzD,eAAO,MAAM,+BAA+B,8HAC+E,CAAC;AAG5H,eAAO,MAAM,gCAAgC;;;;;;;;;;CAmB5C,CAAC;AAIF;;GAEG;AACH,wBAAgB,cAAc,CAC5B,MAAM,EAAE,OAAO,EACf,QAAQ,EAAE,QAAQ,GACjB,MAAM,GAAG,IAAI,CA4Ef;AAyiBD;;GAEG;AACH,wBAAsB,wBAAwB,CAC5C,IAAI,EAAE;IACJ,UAAU,EAAE,MAAM,CAAC;IACnB,KAAK,EAAE,UAAU,GAAG,OAAO,GAAG,UAAU,GAAG,MAAM,CAAC;IAClD,OAAO,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;IACjC,cAAc,CAAC,EAAE,MAAM,CAAC;CACzB,EACD,KAAK,EAAE,KAAK,EACZ,MAAM,EAAE,MAAM,EACd,SAAS,EAAE,MAAM,GAChB,OAAO,CAAC;IAAE,OAAO,EAAE;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,MAAM,CAAA;KAAE,EAAE,CAAA;CAAE,CAAC,CAgZxD"}

package/dist/tools/answer-question.js

@@ -4,6 +4,7 @@
  */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.ANSWERQUESTION_TOOL_INPUT_SCHEMA = exports.ANSWERQUESTION_TOOL_DESCRIPTION = exports.ANSWERQUESTION_TOOL_NAME = void 0;
+exports.validateAnswer = validateAnswer;
 exports.handleAnswerQuestionTool = handleAnswerQuestionTool;
 const zod_1 = require("zod");
 const error_handling_1 = require("../core/error-handling");
@@ -36,9 +37,19 @@ exports.ANSWERQUESTION_TOOL_INPUT_SCHEMA = {
  * Validate answer against question schema
  */
 function validateAnswer(answer, question) {
-    // Check required validation
+    // Check required validation.
+    // Issue #474: For 'select' questions, empty string is a valid answer when it
+    // is explicitly listed in `options` (e.g., options=["", "soft", "hard"] where
+    // "" represents the "none/disabled" choice). Skip the empty-string rejection
+    // in that case so the question's own option list is the source of truth.
+    const emptyAnswer = answer === undefined || answer === null || answer === '';
+    const isExplicitEmptySelectOption = question.type === 'select' &&
+        answer === '' &&
+        Array.isArray(question.options) &&
+        question.options.includes('');
     if (question.validation?.required &&
-        (answer === undefined || answer === null || answer === '')) {
+        emptyAnswer &&
+        !isExplicitEmptySelectOption) {
         return question.validation.message || `${question.question} is required`;
     }
     // Skip validation if answer is empty and not required

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vfarcic/dot-ai",
-  "version": "1.16.1",
+  "version": "1.16.3",
   "description": "AI-powered development productivity platform that enhances software development workflows through intelligent automation and AI-driven assistance",
   "mcpName": "io.github.vfarcic/dot-ai",
   "main": "dist/index.js",
@@ -103,7 +103,7 @@
     "ts-node": "^10.9.0",
     "typescript": "^5.0.0",
     "typescript-eslint": "^8.54.0",
-    "uuid": "^13.0.0",
+    "uuid": "^14.0.0",
     "vitest": "^3.2.4"
   },
   "dependencies": {