newo 2.0.3 → 2.0.4

package/dist/sync/skill-files.js

@@ -102,44 +102,65 @@ export async function askForOverwrite(skillIdn, existingContent, newContent, fil
         output: process.stdout
     });
     console.log(`\n⚠️  Local changes will be replaced by remote content for skill ${skillIdn} (${fileName}):`);
-    // ANSI color codes with background colors
-    const redBg = '\x1b[41m\x1b[37m'; // Red background, white text
-    const greenBg = '\x1b[42m\x1b[30m'; // Green background, black text
+    // ANSI color codes matching GitHub diff colors from screenshot
+    const redBg = '\x1b[101m\x1b[97m'; // Light red background, white text (like GitHub)
+    const greenBg = '\x1b[102m\x1b[30m'; // Light green background, black text (like GitHub)
     const gray = '\x1b[90m';
     const reset = '\x1b[0m';
-    // Show a GitHub-style colored diff with line numbers
+    // Show a GitHub-style colored diff with line numbers and context
     const localLines = existingContent.trim().split('\n');
     const remoteLines = newContent.trim().split('\n');
-    // Create a unified diff view
-    const maxLines = Math.max(localLines.length, remoteLines.length);
-    let diffShown = 0;
-    const maxDiffLines = 10;
-    console.log(`${gray}   Local (-) vs Remote (+):${reset}`);
-    for (let i = 0; i < maxLines && diffShown < maxDiffLines; i++) {
-        const localLine = localLines[i];
-        const remoteLine = remoteLines[i];
-        const lineNum = String(i + 1).padStart(3);
-        // Show context line if both exist and are the same
-        if (localLine !== undefined && remoteLine !== undefined && localLine === remoteLine) {
-            console.log(`${gray}    ${lineNum}      ${localLine}${reset}`);
-        }
-        else {
-            // Show differences
-            if (localLine !== undefined) {
-                console.log(`${redBg} -  ${lineNum}      ${localLine} ${reset}`);
-                diffShown++;
+    // Find differences and show with 2 lines context before/after
+    const diffs = [];
+    // Simple diff algorithm to find changes
+    let localIdx = 0;
+    let remoteIdx = 0;
+    while (localIdx < localLines.length || remoteIdx < remoteLines.length) {
+        const localLine = localLines[localIdx];
+        const remoteLine = remoteLines[remoteIdx];
+        if (localLine !== remoteLine) {
+            if (localLine !== undefined && remoteLine !== undefined) {
+                diffs.push({ localIndex: localIdx, remoteIndex: remoteIdx, type: 'change' });
+            }
+            else if (localLine !== undefined) {
+                diffs.push({ localIndex: localIdx, remoteIndex: -1, type: 'remove' });
             }
-            if (remoteLine !== undefined) {
-                console.log(`${greenBg} +  ${lineNum}      ${remoteLine} ${reset}`);
-                diffShown++;
+            else if (remoteLine !== undefined) {
+                diffs.push({ localIndex: -1, remoteIndex: remoteIdx, type: 'add' });
             }
         }
+        if (localLine !== undefined)
+            localIdx++;
+        if (remoteLine !== undefined)
+            remoteIdx++;
     }
-    if (diffShown === maxDiffLines) {
-        const remainingDiffs = Math.abs(localLines.length - remoteLines.length);
-        if (remainingDiffs > 0) {
-            console.log(`${gray}   ... (${remainingDiffs} more line differences)${reset}`);
+    // Show diffs with context (2 lines before/after each change)
+    let shown = 0;
+    const maxDiffGroups = 3;
+    for (const diff of diffs.slice(0, maxDiffGroups)) {
+        if (shown > 0)
+            console.log(''); // Separator between diff groups
+        // Show context before
+        const contextStart = Math.max(0, diff.localIndex - 2);
+        for (let i = contextStart; i < diff.localIndex && i < localLines.length; i++) {
+            console.log(`    ${String(i + 1).padStart(3)}      ${localLines[i]}`);
+        }
+        // Show the actual diff
+        if (diff.type === 'change' || diff.type === 'remove') {
+            console.log(`${redBg} -  ${String(diff.localIndex + 1).padStart(3)}      ${localLines[diff.localIndex]} ${reset}`);
         }
+        if (diff.type === 'change' || diff.type === 'add') {
+            console.log(`${greenBg} +  ${String(diff.remoteIndex + 1).padStart(3)}      ${remoteLines[diff.remoteIndex]} ${reset}`);
+        }
+        // Show context after
+        const contextEnd = Math.min(localLines.length, diff.localIndex + 3);
+        for (let i = diff.localIndex + 1; i < contextEnd; i++) {
+            console.log(`    ${String(i + 1).padStart(3)}      ${localLines[i]}`);
+        }
+        shown++;
+    }
+    if (diffs.length > maxDiffGroups) {
+        console.log(`${gray}... (${diffs.length - maxDiffGroups} more diff groups)${reset}`);
     }
     const answer = await new Promise((resolve) => {
         rl.question(`\nReplace local with remote? (y)es/(n)o/(a)ll/(q)uit: `, resolve);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "newo",
-  "version": "2.0.3",
+  "version": "2.0.4",
   "description": "NEWO CLI: Professional command-line tool with modular architecture for NEWO AI Agent development. Features IDN-based file management, real-time progress tracking, intelligent sync operations, and comprehensive multi-customer support.",
   "type": "module",
   "bin": {

package/src/sync/skill-files.ts

@@ -156,49 +156,73 @@ export async function askForOverwrite(skillIdn: string, existingContent: string,
 
   console.log(`\n⚠️  Local changes will be replaced by remote content for skill ${skillIdn} (${fileName}):`);
 
-  // ANSI color codes with background colors
-  const redBg = '\x1b[41m\x1b[37m';     // Red background, white text
-  const greenBg = '\x1b[42m\x1b[30m';   // Green background, black text
+  // ANSI color codes matching GitHub diff colors from screenshot
+  const redBg = '\x1b[101m\x1b[97m';    // Light red background, white text (like GitHub)
+  const greenBg = '\x1b[102m\x1b[30m';  // Light green background, black text (like GitHub)
   const gray = '\x1b[90m';
   const reset = '\x1b[0m';
 
-  // Show a GitHub-style colored diff with line numbers
+  // Show a GitHub-style colored diff with line numbers and context
   const localLines = existingContent.trim().split('\n');
   const remoteLines = newContent.trim().split('\n');
 
-  // Create a unified diff view
-  const maxLines = Math.max(localLines.length, remoteLines.length);
-  let diffShown = 0;
-  const maxDiffLines = 10;
-
-  console.log(`${gray}   Local (-) vs Remote (+):${reset}`);
-
-  for (let i = 0; i < maxLines && diffShown < maxDiffLines; i++) {
-    const localLine = localLines[i];
-    const remoteLine = remoteLines[i];
-    const lineNum = String(i + 1).padStart(3);
-
-    // Show context line if both exist and are the same
-    if (localLine !== undefined && remoteLine !== undefined && localLine === remoteLine) {
-      console.log(`${gray}    ${lineNum}      ${localLine}${reset}`);
-    } else {
-      // Show differences
-      if (localLine !== undefined) {
-        console.log(`${redBg} -  ${lineNum}      ${localLine} ${reset}`);
-        diffShown++;
-      }
-      if (remoteLine !== undefined) {
-        console.log(`${greenBg} +  ${lineNum}      ${remoteLine} ${reset}`);
-        diffShown++;
+  // Find differences and show with 2 lines context before/after
+  const diffs: Array<{localIndex: number, remoteIndex: number, type: 'change' | 'add' | 'remove'}> = [];
+
+  // Simple diff algorithm to find changes
+  let localIdx = 0;
+  let remoteIdx = 0;
+
+  while (localIdx < localLines.length || remoteIdx < remoteLines.length) {
+    const localLine = localLines[localIdx];
+    const remoteLine = remoteLines[remoteIdx];
+
+    if (localLine !== remoteLine) {
+      if (localLine !== undefined && remoteLine !== undefined) {
+        diffs.push({localIndex: localIdx, remoteIndex: remoteIdx, type: 'change'});
+      } else if (localLine !== undefined) {
+        diffs.push({localIndex: localIdx, remoteIndex: -1, type: 'remove'});
+      } else if (remoteLine !== undefined) {
+        diffs.push({localIndex: -1, remoteIndex: remoteIdx, type: 'add'});
       }
     }
+
+    if (localLine !== undefined) localIdx++;
+    if (remoteLine !== undefined) remoteIdx++;
   }
 
-  if (diffShown === maxDiffLines) {
-    const remainingDiffs = Math.abs(localLines.length - remoteLines.length);
-    if (remainingDiffs > 0) {
-      console.log(`${gray}   ... (${remainingDiffs} more line differences)${reset}`);
+  // Show diffs with context (2 lines before/after each change)
+  let shown = 0;
+  const maxDiffGroups = 3;
+
+  for (const diff of diffs.slice(0, maxDiffGroups)) {
+    if (shown > 0) console.log(''); // Separator between diff groups
+
+    // Show context before
+    const contextStart = Math.max(0, diff.localIndex - 2);
+    for (let i = contextStart; i < diff.localIndex && i < localLines.length; i++) {
+      console.log(`    ${String(i + 1).padStart(3)}      ${localLines[i]}`);
     }
+
+    // Show the actual diff
+    if (diff.type === 'change' || diff.type === 'remove') {
+      console.log(`${redBg} -  ${String(diff.localIndex + 1).padStart(3)}      ${localLines[diff.localIndex]} ${reset}`);
+    }
+    if (diff.type === 'change' || diff.type === 'add') {
+      console.log(`${greenBg} +  ${String(diff.remoteIndex + 1).padStart(3)}      ${remoteLines[diff.remoteIndex]} ${reset}`);
+    }
+
+    // Show context after
+    const contextEnd = Math.min(localLines.length, diff.localIndex + 3);
+    for (let i = diff.localIndex + 1; i < contextEnd; i++) {
+      console.log(`    ${String(i + 1).padStart(3)}      ${localLines[i]}`);
+    }
+
+    shown++;
+  }
+
+  if (diffs.length > maxDiffGroups) {
+    console.log(`${gray}... (${diffs.length - maxDiffGroups} more diff groups)${reset}`);
   }
 
   const answer = await new Promise<string>((resolve) => {