@probelabs/visor 0.1.42 → 0.1.44

package/README.md

@@ -366,7 +366,8 @@ Learn more: [docs/http.md](docs/http.md)
 
 Mix providers (`ai`, `http`, `http_client`, `log`, `command`, `claude-code`) or add your own.
 
-Learn more: [docs/pluggable.md](docs/pluggable.md)
+- **Command Provider**: Execute shell commands with templating and security - [docs/command-provider.md](docs/command-provider.md)
+- **Custom Providers**: Build your own providers - [docs/pluggable.md](docs/pluggable.md)
 
 ## 🎯 GitHub Action Reference
 

package/dist/cli-main.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"","sourceRoot":"","sources":["file:///home/runner/work/visor/visor/src/cli-main.ts"],"names":[],"mappings":"AAOA;;GAEG;AACH,wBAAsB,IAAI,IAAI,OAAO,CAAC,IAAI,CAAC,CA0Q1C"}
+{"version":3,"file":"","sourceRoot":"","sources":["file:///home/runner/work/visor/visor/src/cli-main.ts"],"names":[],"mappings":"AAOA;;GAEG;AACH,wBAAsB,IAAI,IAAI,OAAO,CAAC,IAAI,CAAC,CAmS1C"}

package/dist/cli.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"","sourceRoot":"","sources":["file:///home/runner/work/visor/visor/src/cli.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,UAAU,EAA2B,MAAM,aAAa,CAAC;AAIlE;;GAEG;AACH,qBAAa,GAAG;IACd,OAAO,CAAC,OAAO,CAAU;IACzB,OAAO,CAAC,WAAW,CAA4E;IAC/F,OAAO,CAAC,YAAY,CAA0D;;IAO9E;;OAEG;IACH,OAAO,CAAC,YAAY;IAwCpB;;OAEG;IACH,OAAO,CAAC,aAAa,CAEnB;IAEF;;OAEG;IACI,SAAS,CAAC,IAAI,EAAE,MAAM,EAAE,GAAG,UAAU;IAkI5C;;OAEG;IACH,OAAO,CAAC,eAAe;IAkCvB;;OAEG;IACI,WAAW,IAAI,MAAM;IAsC5B;;OAEG;IACI,UAAU,IAAI,MAAM;IAa3B;;OAEG;IACI,eAAe,IAAI,MAAM;IAiBhC;;OAEG;IACI,QAAQ,IAAI,IAAI;IAIvB;;OAEG;IACI,WAAW,IAAI,IAAI;CAG3B"}
+{"version":3,"file":"","sourceRoot":"","sources":["file:///home/runner/work/visor/visor/src/cli.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,UAAU,EAA2B,MAAM,aAAa,CAAC;AAIlE;;GAEG;AACH,qBAAa,GAAG;IACd,OAAO,CAAC,OAAO,CAAU;IACzB,OAAO,CAAC,WAAW,CAA4E;IAC/F,OAAO,CAAC,YAAY,CAA0D;;IAO9E;;OAEG;IACH,OAAO,CAAC,YAAY;IAwCpB;;OAEG;IACH,OAAO,CAAC,aAAa,CAEnB;IAEF;;OAEG;IACI,SAAS,CAAC,IAAI,EAAE,MAAM,EAAE,GAAG,UAAU;IAkI5C;;OAEG;IACH,OAAO,CAAC,eAAe;IAkCvB;;OAEG;IACI,WAAW,IAAI,MAAM;IAsC5B;;OAEG;IACI,UAAU,IAAI,MAAM;IAoC3B;;OAEG;IACI,eAAe,IAAI,MAAM;IAiBhC;;OAEG;IACI,QAAQ,IAAI,IAAI;IAIvB;;OAEG;IACI,WAAW,IAAI,IAAI;CAG3B"}

package/dist/config.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"","sourceRoot":"","sources":["file:///home/runner/work/visor/visor/src/config.ts"],"names":[],"mappings":"AAIA,OAAO,EACL,WAAW,EAOX,oBAAoB,EACpB,YAAY,EACZ,iBAAiB,EAClB,MAAM,gBAAgB,CAAC;AACxB,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAC;AAIzC;;GAEG;AACH,qBAAa,aAAa;IACxB,OAAO,CAAC,eAAe,CASrB;IACF,OAAO,CAAC,kBAAkB,CASxB;IACF,OAAO,CAAC,kBAAkB,CAAgE;IAC1F,OAAO,CAAC,mBAAmB,CAAkD;IAE7E;;OAEG;IACU,UAAU,CACrB,UAAU,EAAE,MAAM,EAClB,OAAO,GAAE,iBAAsB,GAC9B,OAAO,CAAC,WAAW,CAAC;IA6EvB;;OAEG;IACU,iBAAiB,CAAC,OAAO,GAAE,iBAAsB,GAAG,OAAO,CAAC,WAAW,CAAC;IAyBrF;;OAEG;YACW,qBAAqB;IAiBnC;;OAEG;IACU,gBAAgB,IAAI,OAAO,CAAC,WAAW,CAAC;IAerD;;OAEG;IACI,wBAAwB,IAAI,WAAW,GAAG,IAAI;IAoDrD;;OAEG;IACH,OAAO,CAAC,eAAe;IAuBvB;;OAEG;IACI,mBAAmB,CAAC,MAAM,EAAE,OAAO,CAAC,WAAW,CAAC,EAAE,UAAU,EAAE,UAAU,GAAG,YAAY;IAqB9F;;OAEG;IACU,0BAA0B,IAAI,OAAO,CAAC;QACjD,MAAM,CAAC,EAAE,WAAW,CAAC;QACrB,oBAAoB,EAAE,oBAAoB,CAAC;KAC5C,CAAC;IA2BF;;OAEG;IACH,OAAO,CAAC,cAAc;IAqEtB;;OAEG;IACH,OAAO,CAAC,mBAAmB;IAuJ3B;;OAEG;IACH,OAAO,CAAC,iBAAiB;IA6DzB;;OAEG;IACH,OAAO,CAAC,wBAAwB;IA6EhC;;OAEG;IACH,OAAO,CAAC,oBAAoB;IA+B5B;;OAEG;IACH,OAAO,CAAC,sBAAsB;IAY9B;;OAEG;IACH,OAAO,CAAC,iBAAiB;CA6B1B"}
+{"version":3,"file":"","sourceRoot":"","sources":["file:///home/runner/work/visor/visor/src/config.ts"],"names":[],"mappings":"AAIA,OAAO,EACL,WAAW,EAOX,oBAAoB,EACpB,YAAY,EACZ,iBAAiB,EAClB,MAAM,gBAAgB,CAAC;AACxB,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAC;AAIzC;;GAEG;AACH,qBAAa,aAAa;IACxB,OAAO,CAAC,eAAe,CASrB;IACF,OAAO,CAAC,kBAAkB,CASxB;IACF,OAAO,CAAC,kBAAkB,CAAgE;IAC1F,OAAO,CAAC,mBAAmB,CAAkD;IAE7E;;OAEG;IACU,UAAU,CACrB,UAAU,EAAE,MAAM,EAClB,OAAO,GAAE,iBAAsB,GAC9B,OAAO,CAAC,WAAW,CAAC;IA2FvB;;OAEG;IACU,iBAAiB,CAAC,OAAO,GAAE,iBAAsB,GAAG,OAAO,CAAC,WAAW,CAAC;IAyBrF;;OAEG;YACW,qBAAqB;IAiBnC;;OAEG;IACU,gBAAgB,IAAI,OAAO,CAAC,WAAW,CAAC;IAerD;;OAEG;IACI,wBAAwB,IAAI,WAAW,GAAG,IAAI;IAoDrD;;OAEG;IACH,OAAO,CAAC,eAAe;IAuBvB;;OAEG;IACI,mBAAmB,CAAC,MAAM,EAAE,OAAO,CAAC,WAAW,CAAC,EAAE,UAAU,EAAE,UAAU,GAAG,YAAY;IAqB9F;;OAEG;IACU,0BAA0B,IAAI,OAAO,CAAC;QACjD,MAAM,CAAC,EAAE,WAAW,CAAC;QACrB,oBAAoB,EAAE,oBAAoB,CAAC;KAC5C,CAAC;IA2BF;;OAEG;IACH,OAAO,CAAC,cAAc;IAkEtB;;OAEG;IACH,OAAO,CAAC,mBAAmB;IA0J3B;;OAEG;IACH,OAAO,CAAC,iBAAiB;IA6DzB;;OAEG;IACH,OAAO,CAAC,wBAAwB;IA6EhC;;OAEG;IACH,OAAO,CAAC,oBAAoB;IA+B5B;;OAEG;IACH,OAAO,CAAC,sBAAsB;IAY9B;;OAEG;IACH,OAAO,CAAC,iBAAiB;CA6B1B"}

package/dist/index.js

@@ -1,4 +1,5 @@
 #!/usr/bin/env node
+process.env.VISOR_VERSION = '0.1.44';
 /******/ (() => { // webpackBootstrap
 /******/ 	var __webpack_modules__ = ({
 
@@ -96907,15 +96908,36 @@ async function main() {
             try {
                 config = await configManager.loadConfig(options.configPath);
             }
-            catch {
-                console.error(`⚠️ Warning: Configuration file not found: ${options.configPath}`);
-                console.error('Falling back to default configuration');
-                config = await configManager
-                    .findAndLoadConfig()
-                    .catch(() => configManager.getDefaultConfig());
+            catch (error) {
+                // Show the actual error message, not just assume "file not found"
+                if (error instanceof Error) {
+                    console.error(`❌ Error loading configuration from ${options.configPath}:`);
+                    console.error(`   ${error.message}`);
+                    // Provide helpful hints based on the error type
+                    if (error.message.includes('not found')) {
+                        console.error('\n💡 Hint: Check that the file path is correct and the file exists.');
+                        console.error('   You can use an absolute path: --config $(pwd)/.visor.yaml');
+                    }
+                    else if (error.message.includes('Invalid YAML')) {
+                        console.error('\n💡 Hint: Check your YAML syntax. You can validate it at https://www.yamllint.com/');
+                    }
+                    else if (error.message.includes('extends')) {
+                        console.error('\n💡 Hint: Check that extended configuration files exist and are accessible.');
+                    }
+                    else if (error.message.includes('permission')) {
+                        console.error('\n💡 Hint: Check file permissions. The file must be readable.');
+                    }
+                }
+                else {
+                    console.error(`❌ Error loading configuration: ${error}`);
+                }
+                // Exit with error when explicit config path fails
+                console.error('\n🛑 Exiting: Cannot proceed when specified configuration file fails to load.');
+                process.exit(1);
             }
         }
         else {
+            // Auto-discovery mode - fallback to defaults is OK
             config = await configManager
                 .findAndLoadConfig()
                 .catch(() => configManager.getDefaultConfig());
@@ -97367,17 +97389,38 @@ class CLI {
      * Get version from package.json
      */
     getVersion() {
+        // First, try environment variable (set during build/publish)
+        if (process.env.VISOR_VERSION) {
+            return process.env.VISOR_VERSION;
+        }
         try {
-            const packageJsonPath = path.join(__dirname, '../../package.json');
-            if (fs.existsSync(packageJsonPath)) {
-                const packageJson = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
-                return packageJson.version || '1.0.0';
+            // Try multiple possible locations for package.json
+            const possiblePaths = [
+                path.join(__dirname, '../../package.json'), // Development
+                path.join(__dirname, '../package.json'), // Alternate bundled
+                path.join(process.cwd(), 'package.json'), // Current directory
+                '/snapshot/visor/package.json', // Vercel pkg snapshot
+            ];
+            for (const packageJsonPath of possiblePaths) {
+                if (fs.existsSync(packageJsonPath)) {
+                    try {
+                        const packageJson = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
+                        // Only use if it's the visor package
+                        if (packageJson.name === '@probelabs/visor' && packageJson.version) {
+                            return packageJson.version;
+                        }
+                    }
+                    catch {
+                        // Try next path
+                    }
+                }
             }
         }
         catch {
-            // Fallback to default version
+            // Continue to fallback
         }
-        return '1.0.0';
+        // Fallback to the actual current version in package.json
+        return '0.1.42';
     }
     /**
      * Get examples text for help
@@ -97611,10 +97654,22 @@ class ConfigManager {
         }
         catch (error) {
             if (error instanceof Error) {
-                if (error.message.includes('not found') || error.message.includes('Invalid YAML')) {
+                // Pass through detailed error messages unchanged
+                if (error.message.includes('not found') ||
+                    error.message.includes('Invalid YAML') ||
+                    error.message.includes('extends') ||
+                    error.message.includes('EACCES') ||
+                    error.message.includes('EISDIR')) {
                     throw error;
                 }
-                throw new Error(`Failed to read configuration file: ${error.message}`);
+                // Add more context for generic errors
+                if (error.message.includes('ENOENT')) {
+                    throw new Error(`Configuration file not found: ${configPath}`);
+                }
+                if (error.message.includes('EPERM')) {
+                    throw new Error(`Permission denied reading configuration file: ${configPath}`);
+                }
+                throw new Error(`Failed to read configuration file ${configPath}: ${error.message}`);
             }
             throw error;
         }
@@ -97821,10 +97876,7 @@ class ConfigManager {
                 if (!checkConfig.type) {
                     checkConfig.type = 'ai';
                 }
-                // Default 'on' to ['manual'] if not specified
-                if (!checkConfig.on) {
-                    checkConfig.on = ['manual'];
-                }
+                // 'on' field is optional - if not specified, check can run on any event
                 this.validateCheckConfig(checkName, checkConfig, errors);
             }
         }
@@ -97926,21 +97978,24 @@ class ConfigManager {
                 });
             }
         }
-        if (!checkConfig.on || !Array.isArray(checkConfig.on)) {
-            errors.push({
-                field: `checks.${checkName}.on`,
-                message: `Invalid check configuration for "${checkName}": missing or invalid 'on' field`,
-            });
-        }
-        else {
-            // Validate event triggers
-            for (const event of checkConfig.on) {
-                if (!this.validEventTriggers.includes(event)) {
-                    errors.push({
-                        field: `checks.${checkName}.on`,
-                        message: `Invalid event "${event}". Must be one of: ${this.validEventTriggers.join(', ')}`,
-                        value: event,
-                    });
+        // 'on' field is optional - if not specified, check can be triggered by any event
+        if (checkConfig.on) {
+            if (!Array.isArray(checkConfig.on)) {
+                errors.push({
+                    field: `checks.${checkName}.on`,
+                    message: `Invalid check configuration for "${checkName}": 'on' field must be an array`,
+                });
+            }
+            else {
+                // Validate event triggers
+                for (const event of checkConfig.on) {
+                    if (!this.validEventTriggers.includes(event)) {
+                        errors.push({
+                            field: `checks.${checkName}.on`,
+                            message: `Invalid event "${event}". Must be one of: ${this.validEventTriggers.join(', ')}`,
+                            value: event,
+                        });
+                    }
                 }
             }
         }
@@ -100183,8 +100238,8 @@ async function handleEvent(octokit, inputs, eventName, context, config) {
     const eventChecks = [];
     for (const [checkName, checkConfig] of Object.entries(config.checks || {})) {
         // Check if this check should run for this event
-        const checkEvents = checkConfig.on || ['pr_opened', 'pr_updated'];
-        if (checkEvents.includes(eventType)) {
+        // If 'on' is not specified, the check can run on any event
+        if (!checkConfig.on || checkConfig.on.includes(eventType)) {
             eventChecks.push(checkName);
         }
     }
@@ -100553,13 +100608,16 @@ async function handleIssueComment(octokit, owner, repo, context, inputs, actionC
                     if (!config?.checks?.[checkId])
                         return false;
                     const checkConfig = config.checks[checkId];
-                    const checkEvents = checkConfig.on || ['pr_opened', 'pr_updated'];
+                    // If 'on' is not specified, the check can run on any event
+                    if (!checkConfig.on) {
+                        return true;
+                    }
                     // For issue comments, only run checks that are configured for issue_comment events
                     if (!isPullRequest) {
-                        return checkEvents.includes('issue_comment');
+                        return checkConfig.on.includes('issue_comment');
                     }
                     // For PR comments, run checks configured for PR events or issue_comment
-                    return checkEvents.includes('pr_updated') || checkEvents.includes('issue_comment');
+                    return checkConfig.on.includes('pr_updated') || checkConfig.on.includes('issue_comment');
                 });
                 if (filteredCheckIds.length === 0) {
                     console.log(`No checks configured to run for ${isPullRequest ? 'PR' : 'issue'} comments`);
@@ -103583,9 +103641,12 @@ class CommandCheckProvider extends check_provider_interface_1.CheckProvider {
             const { exec } = await Promise.resolve().then(() => __importStar(__nccwpck_require__(35317)));
             const { promisify } = await Promise.resolve().then(() => __importStar(__nccwpck_require__(39023)));
             const execAsync = promisify(exec);
+            // Get timeout from config (in seconds) or use default (60 seconds)
+            const timeoutSeconds = config.timeout || 60;
+            const timeoutMs = timeoutSeconds * 1000;
             const { stdout, stderr } = await execAsync(renderedCommand, {
                 env: scriptEnv,
-                timeout: 60000, // 60 second timeout
+                timeout: timeoutMs,
                 maxBuffer: 10 * 1024 * 1024, // 10MB buffer
             });
             if (stderr && process.env.DEBUG) {
@@ -103688,7 +103749,18 @@ class CommandCheckProvider extends check_provider_interface_1.CheckProvider {
         return safeVars;
     }
     getSupportedConfigKeys() {
-        return ['type', 'exec', 'transform', 'env', 'depends_on', 'on', 'if', 'group', 'forEach'];
+        return [
+            'type',
+            'exec',
+            'transform',
+            'env',
+            'timeout',
+            'depends_on',
+            'on',
+            'if',
+            'group',
+            'forEach',
+        ];
     }
     async isAvailable() {
         // Command provider is always available as long as we can execute commands

package/dist/providers/command-check-provider.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"command-check-provider.d.ts","sourceRoot":"","sources":["file:///home/runner/work/visor/visor/src/providers/command-check-provider.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,aAAa,EAAE,mBAAmB,EAAE,MAAM,4BAA4B,CAAC;AAChF,OAAO,EAAE,MAAM,EAAE,MAAM,gBAAgB,CAAC;AACxC,OAAO,EAAE,aAAa,EAAE,MAAM,aAAa,CAAC;AAG5C;;;GAGG;AACH,qBAAa,oBAAqB,SAAQ,aAAa;IACrD,OAAO,CAAC,MAAM,CAAS;;IAWvB,OAAO,IAAI,MAAM;IAIjB,cAAc,IAAI,MAAM;IAIlB,cAAc,CAAC,MAAM,EAAE,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC;IAejD,OAAO,CACX,MAAM,EAAE,MAAM,EACd,MAAM,EAAE,mBAAmB,EAC3B,iBAAiB,CAAC,EAAE,GAAG,CAAC,MAAM,EAAE,aAAa,CAAC,GAC7C,OAAO,CAAC,aAAa,CAAC;IA0HzB,OAAO,CAAC,kBAAkB;IAqB1B,OAAO,CAAC,2BAA2B;IAgBnC,sBAAsB,IAAI,MAAM,EAAE;IAI5B,WAAW,IAAI,OAAO,CAAC,OAAO,CAAC;IAKrC,eAAe,IAAI,MAAM,EAAE;CAO5B"}
+{"version":3,"file":"command-check-provider.d.ts","sourceRoot":"","sources":["file:///home/runner/work/visor/visor/src/providers/command-check-provider.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,aAAa,EAAE,mBAAmB,EAAE,MAAM,4BAA4B,CAAC;AAChF,OAAO,EAAE,MAAM,EAAE,MAAM,gBAAgB,CAAC;AACxC,OAAO,EAAE,aAAa,EAAE,MAAM,aAAa,CAAC;AAG5C;;;GAGG;AACH,qBAAa,oBAAqB,SAAQ,aAAa;IACrD,OAAO,CAAC,MAAM,CAAS;;IAWvB,OAAO,IAAI,MAAM;IAIjB,cAAc,IAAI,MAAM;IAIlB,cAAc,CAAC,MAAM,EAAE,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC;IAejD,OAAO,CACX,MAAM,EAAE,MAAM,EACd,MAAM,EAAE,mBAAmB,EAC3B,iBAAiB,CAAC,EAAE,GAAG,CAAC,MAAM,EAAE,aAAa,CAAC,GAC7C,OAAO,CAAC,aAAa,CAAC;IA8HzB,OAAO,CAAC,kBAAkB;IAqB1B,OAAO,CAAC,2BAA2B;IAgBnC,sBAAsB,IAAI,MAAM,EAAE;IAe5B,WAAW,IAAI,OAAO,CAAC,OAAO,CAAC;IAKrC,eAAe,IAAI,MAAM,EAAE;CAO5B"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@probelabs/visor",
-  "version": "0.1.42",
+  "version": "0.1.44",
   "main": "dist/index.js",
   "bin": {
     "visor": "./dist/index.js"
@@ -17,7 +17,7 @@
     "registry": "https://registry.npmjs.org/"
   },
   "scripts": {
-    "build": "ncc build src/index.ts -o dist && cp -r defaults dist/ && echo '#!/usr/bin/env node' | cat - dist/index.js > temp && mv temp dist/index.js && chmod +x dist/index.js",
+    "build": "ncc build src/index.ts -o dist && cp -r defaults dist/ && node scripts/inject-version.js && echo '#!/usr/bin/env node' | cat - dist/index.js > temp && mv temp dist/index.js && chmod +x dist/index.js",
     "test": "jest",
     "prepublishOnly": "npm run build",
     "test:watch": "jest --watch",