bunosh 0.4.6 → 0.4.8

package/README.md

@@ -398,41 +398,6 @@ bunosh --bunoshfile Bunoshfile.dev.js -mcp
 
 </details>
 
-<details>
-
-#### Best Practices for MCP
-
-### Command Design for AI
-
-Design your commands to work well with AI assistants:
-
-* Add comments to all commands
-* Describe use cases for each Command
-* Describe each param using JSDoc style comments
-* Include types when describing parameters
-
-```javascript
-/**
- * Builds the project for specified environment
- * @param {string} environment - Target environment: 'development', 'staging', or 'production'
- * @param {Object} options - Build options
- * @param {boolean} options.verbose - Enable verbose logging
- * @param {boolean} options.analyze - Run bundle analysis after build
- */
-export async function build(environment = 'development', options = {}) {
-  const config = getBuildConfig(environment);
-  if (options.verbose) say(`🔨 Building for ${environment}...`);
-
-  await exec`npm run build:${environment}`;
-
-  if (options.analyze) {
-    await exec`npm run analyze`;
-  }
-
-  say('✅ Build complete!');
-}
-```
-
 ## Tasks
 
 Bunosh provides built-in tasks which are available via `global.bunosh`:
@@ -738,7 +703,7 @@ Gracefully handle operations that might fail:
  * Check service availability
  */
 export async function checkServices() {
-  const dbConnected = await task.try(shell`nc -z localhost 5432`);
+  const dbConnected = await task.try(() => shell`nc -z localhost 5432`);
 
   if (dbConnected) {
     say('✅ Database connected');

package/bunosh.js

@@ -4,6 +4,7 @@
 globalThis._bunoshStartTime = Date.now();
 globalThis._bunoshCommandCompleted = false;
 globalThis._bunoshGlobalTasksExecuted = [];
+globalThis._bunoshIgnoreFailuresMode = false;
 globalThis._bunoshGlobalTaskStatus = {
   RUNNING: 'running',
   FAIL: 'fail',
@@ -310,39 +311,60 @@ process.on('uncaughtException', (error) => {
   process.exit(1);
 });
 
-// Handle exit for task summary
 process.on('exit', (code) => {
   if (!process.env.BUNOSH_COMMAND_STARTED) return;
-  
-  // Don't print summary if exit was due to stdin closing during an ask operation
-  // This prevents duplicate output when ask commands don't receive all required input
+
   if (globalThis._bunoshInAskOperation && code === 0) {
     return;
   }
-  
-  // Access global values directly
+
   const tasksExecuted = globalThis._bunoshGlobalTasksExecuted || [];
   const TaskStatus = globalThis._bunoshGlobalTaskStatus || { FAIL: 'fail', WARNING: 'warning' };
-  
-  // Calculate total time from when the process started
+
   const totalTime = Date.now() - globalThis._bunoshStartTime || 0;
   const tasksFailed = tasksExecuted.filter(ti => ti.result?.status === TaskStatus.FAIL).length;
   const tasksWarning = tasksExecuted.filter(ti => ti.result?.status === TaskStatus.WARNING).length;
-  
-  // Check if we're in test environment
+
+  const commandArgs = process.argv.slice(2);
   const isTestEnvironment = process.env.NODE_ENV === 'test' ||
                             (typeof Bun !== 'undefined' && typeof Bun?.jest !== 'undefined') ||
-                            process.argv.some(arg => arg.includes('vitest') || arg.includes('jest') || arg.includes('--test') || arg.includes('test:'));
-  
-  // Set exit code to 1 if any tasks failed AND we're not in test environment
-  if (tasksFailed > 0 && !isTestEnvironment) {
+                            commandArgs.some(arg => {
+                              const lowerArg = arg.toLowerCase();
+                              return lowerArg.includes('vitest') ||
+                                     lowerArg.includes('jest') ||
+                                     lowerArg === '--test' ||
+                                     lowerArg.startsWith('test:');
+                            });
+
+  const ignoreFailuresMode = globalThis._bunoshIgnoreFailuresMode || false;
+
+  if (process.env.BUNOSH_DEBUG) {
+    console.log('\n[DEBUG] Exit handler:');
+    console.log('  tasksFailed:', tasksFailed);
+    console.log('  isTestEnvironment:', isTestEnvironment);
+    console.log('  ignoreFailuresMode:', ignoreFailuresMode);
+    console.log('  NODE_ENV:', process.env.NODE_ENV);
+    console.log('  commandArgs:', commandArgs);
+    console.log('  full process.argv:', process.argv);
+    if (isTestEnvironment) {
+      const matchingArg = commandArgs.find(arg => {
+        const lowerArg = arg.toLowerCase();
+        return lowerArg.includes('vitest') ||
+               lowerArg.includes('jest') ||
+               lowerArg === '--test' ||
+               lowerArg.startsWith('test:');
+      });
+      console.log('  Matched command arg:', matchingArg);
+    }
+  }
+
+  if (tasksFailed > 0 && !isTestEnvironment && !ignoreFailuresMode) {
     process.exitCode = 1;
   }
-  
-  const finalExitCode = (tasksFailed > 0 && !isTestEnvironment) ? 1 : code;
+
+  const finalExitCode = (tasksFailed > 0 && !isTestEnvironment && !ignoreFailuresMode) ? 1 : code;
   const success = finalExitCode === 0;
 
-  // Debug: Check if this handler has already run
   if (globalThis._bunoshExitHandlerCalled) {
     console.log('\n[DEBUG] Exit handler already called, skipping duplicate');
     return;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bunosh",
-  "version": "0.4.6",
+  "version": "0.4.8",
   "type": "module",
   "module": "index.js",
   "bin": {

package/src/formatters/console.js

@@ -56,9 +56,8 @@ export class ConsoleFormatter extends BaseFormatter {
     let line = leftContent + padding + rightContent;
 
     if (icon.trim()) {
-      // Apply underline to the task name/type only, not the status in parentheses
-      const underlineContent = taskTypeFormatted + taskNameFormatted + padding;
-      line = icon + ' ' + chalk.underline(underlineContent) + rightContent;
+      const underlineContent = taskTypeFormatted + taskNameFormatted;
+      line = icon + ' ' + chalk.underline(underlineContent) + padding + rightContent;
     }
 
     let result = line;
@@ -74,7 +73,40 @@ export class ConsoleFormatter extends BaseFormatter {
 
   formatOutput(line, isError = false) {
     if (!line.trim()) return '';
-    return isError ? chalk.red(line) : line;
+
+    const indent = '   ';
+    const terminalWidth = process.stdout.columns || 100;
+    const maxLineWidth = terminalWidth - indent.length;
+
+    let formattedLine = line;
+    const plainLine = this._stripAnsi(line);
+
+    if (plainLine.length > maxLineWidth) {
+      const truncateAt = maxLineWidth - 3;
+      let charCount = 0;
+      let truncatedLine = '';
+
+      for (let i = 0; i < line.length; i++) {
+        if (line[i] === '\u001b') {
+          let j = i;
+          while (j < line.length && line[j] !== 'm') j++;
+          truncatedLine += line.substring(i, j + 1);
+          i = j;
+          continue;
+        }
+
+        if (charCount >= truncateAt) break;
+
+        truncatedLine += line[i];
+        charCount++;
+      }
+
+      formattedLine = truncatedLine + '...';
+    }
+
+    formattedLine = indent + formattedLine;
+
+    return isError ? chalk.red(formattedLine) : formattedLine;
   }
 
   _stripAnsi(str) {

package/src/task.js

@@ -28,15 +28,23 @@ export function ignoreFail(enable = true) {
 }
 
 // Global failure mode control
-// true = stop on failures (exit with code 1), false = continue on failures
 let stopOnFailuresMode = false;
+let ignoreFailuresMode = false;
 
 export function ignoreFailures() {
   stopOnFailuresMode = false;
+  ignoreFailuresMode = true;
+  globalThis._bunoshIgnoreFailuresMode = true;
 }
 
 export function stopOnFailures() {
   stopOnFailuresMode = true;
+  ignoreFailuresMode = false;
+  globalThis._bunoshIgnoreFailuresMode = false;
+}
+
+export function getIgnoreFailuresMode() {
+  return ignoreFailuresMode;
 }
 
 export function silence() {
@@ -110,7 +118,7 @@ export class TaskInfo {
   }
 }
 
-export async function tryTask(name, fn, isSilent = false) {
+export async function tryTask(name, fn, isSilent = true) {
   if (!fn) {
     fn = name;
     name = fn.toString().slice(0, 50).replace(/\s+/g, ' ').trim();
@@ -198,9 +206,16 @@ export async function task(name, fn, isSilent = false) {
     runningTasks.delete(taskInfo.id);
 
     // Don't exit during testing
+    const commandArgs = process.argv.slice(2);
     const isTestEnvironment = process.env.NODE_ENV === 'test' ||
                               typeof Bun?.jest !== 'undefined' ||
-                              process.argv.some(arg => arg.includes('vitest') || arg.includes('jest') || arg.includes('--test') || arg.includes('test:'));
+                              commandArgs.some(arg => {
+                                const lowerArg = arg.toLowerCase();
+                                return lowerArg.includes('vitest') ||
+                                       lowerArg.includes('jest') ||
+                                       lowerArg === '--test' ||
+                                       lowerArg.startsWith('test:');
+                              });
 
     // Exit immediately if stopOnFailures mode is enabled
     if (stopOnFailuresMode && !isTestEnvironment) {