mattermost-claude-code 0.10.4 → 0.10.6

package/CHANGELOG.md

@@ -7,6 +7,24 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.10.6] - 2025-12-28
+
+### Fixed
+- **Fixed diff display** - Removed misleading line numbers and noise from diffs
+  - No more fake `@@ -1,1 +1,1 @@` headers (we don't have real line numbers)
+  - No more `` noise
+  - Uses `diffLines()` for proper line-by-line change detection
+  - Shows context lines (unchanged parts) naturally
+
+## [0.10.5] - 2025-12-28
+
+### Changed
+- **Improved diff display** - Edit operations now show unified diffs with context
+  - Uses standard unified diff format (like `git diff`)
+  - Shows 3 lines of context around changes
+  - More compact: changed lines shown once, not duplicated
+  - Line numbers in `@@ -X,Y +X,Y @@` format
+
 ## [0.10.4] - 2025-12-28
 
 ### Added

package/dist/claude/session.js

@@ -7,6 +7,7 @@ import { randomUUID } from 'crypto';
 import { readFileSync } from 'fs';
 import { dirname, resolve } from 'path';
 import { fileURLToPath } from 'url';
+import * as Diff from 'diff';
 const __dirname = dirname(fileURLToPath(import.meta.url));
 const pkg = JSON.parse(readFileSync(resolve(__dirname, '..', '..', 'package.json'), 'utf-8'));
 const REACTION_EMOJIS = ['one', 'two', 'three', 'four'];
@@ -850,25 +851,44 @@ export class SessionManager {
             case 'Read': return `📄 **Read** \`${short(input.file_path)}\``;
             case 'Edit': {
                 const filePath = short(input.file_path);
-                const oldStr = (input.old_string || '').trim();
-                const newStr = (input.new_string || '').trim();
+                const oldStr = (input.old_string || '');
+                const newStr = (input.new_string || '');
                 // Show diff if we have old/new strings
                 if (oldStr || newStr) {
-                    const maxLines = 8;
-                    const oldLines = oldStr.split('\n').slice(0, maxLines);
-                    const newLines = newStr.split('\n').slice(0, maxLines);
-                    let diff = `✏️ **Edit** \`${filePath}\`\n\`\`\`diff\n`;
-                    for (const line of oldLines) {
-                        diff += `- ${line}\n`;
+                    // Use diff library to compute actual line-by-line changes
+                    const changes = Diff.diffLines(oldStr, newStr);
+                    const maxLines = 20;
+                    let lineCount = 0;
+                    const diffLines = [];
+                    for (const change of changes) {
+                        const lines = change.value.replace(/\n$/, '').split('\n');
+                        for (const line of lines) {
+                            if (lineCount >= maxLines)
+                                break;
+                            if (change.added) {
+                                diffLines.push(`+ ${line}`);
+                                lineCount++;
+                            }
+                            else if (change.removed) {
+                                diffLines.push(`- ${line}`);
+                                lineCount++;
+                            }
+                            else {
+                                // Context line (unchanged)
+                                diffLines.push(`  ${line}`);
+                                lineCount++;
+                            }
+                        }
+                        if (lineCount >= maxLines)
+                            break;
                     }
-                    if (oldStr.split('\n').length > maxLines)
-                        diff += `- ... (${oldStr.split('\n').length - maxLines} more lines)\n`;
-                    for (const line of newLines) {
-                        diff += `+ ${line}\n`;
+                    const totalLines = changes.reduce((sum, c) => sum + c.value.split('\n').length - 1, 0);
+                    let diff = `✏️ **Edit** \`${filePath}\`\n\`\`\`diff\n`;
+                    diff += diffLines.join('\n');
+                    if (totalLines > maxLines) {
+                        diff += `\n... (+${totalLines - maxLines} more lines)`;
                     }
-                    if (newStr.split('\n').length > maxLines)
-                        diff += `+ ... (${newStr.split('\n').length - maxLines} more lines)\n`;
-                    diff += '```';
+                    diff += '\n```';
                     return diff;
                 }
                 return `✏️ **Edit** \`${filePath}\``;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mattermost-claude-code",
-  "version": "0.10.4",
+  "version": "0.10.6",
   "description": "Share Claude Code sessions live in a Mattermost channel with interactive features",
   "main": "dist/index.js",
   "type": "module",
@@ -43,6 +43,7 @@
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.25.1",
     "commander": "^14.0.2",
+    "diff": "^8.0.2",
     "dotenv": "^16.4.7",
     "prompts": "^2.4.2",
     "update-notifier": "^7.3.1",
@@ -50,6 +51,7 @@
     "zod": "^4.2.1"
   },
   "devDependencies": {
+    "@types/diff": "^7.0.2",
     "@types/node": "^22.10.2",
     "@types/prompts": "^2.4.9",
     "@types/update-notifier": "^6.0.8",