...

Author: rogeralsing

src/commands/context.md

@@ -23,6 +23,7 @@
 - `replace.js` enforces `g` flag and supports dry-run reporting.
 - `escapeString.js` centralises string coercion, enabling new built-ins without touching `loop.js` internals.
 - `preapproval.js` validation now guards against risky shells/flags with dedicated unit coverage for new heuristics.
+- `edit.js` and `replace.js` now return the full updated file contents (with headings) in their stdout payloads, letting the LLM consume changes without issuing a follow-up read.
 
 ## Risks / Gaps
 

src/commands/replace.js

@@ -44,21 +44,37 @@ function validateFiles(files) {
   });
 }
 
-function buildResultSummary(results, dryRun) {
+function buildResultOutput(results, dryRun) {
   const totalMatches = results.reduce((acc, item) => acc + item.matches, 0);
   const touchedFiles = results.filter((item) => item.matches > 0).length;
 
-  const lines = [`Total matches: ${totalMatches}`, `Files with changes: ${touchedFiles}`];
+  const summaryLines = [
+    `Total matches: ${totalMatches}`,
+    `Files with changes: ${touchedFiles}`,
+  ];
 
   results.forEach((item) => {
-    lines.push(`${item.path}: ${item.matches} matches${dryRun ? ' (dry-run)' : ''}`);
+    summaryLines.push(`${item.path}: ${item.matches} matches${dryRun ? ' (dry-run)' : ''}`);
   });
 
   if (dryRun) {
-    lines.push('Dry-run: no files were modified.');
+    summaryLines.push('Dry-run: no files were modified.');
   }
 
-  return lines.join('\n');
+  const detailSections = results.map((item) => {
+    const status = dryRun
+      ? item.matches > 0
+        ? `Preview ${item.path}`
+        : `Preview (no matches) ${item.path}`
+      : item.changed
+        ? `Updated ${item.path}`
+        : `Unchanged ${item.path}`;
+
+    const header = `--- ${item.path}`;
+    return `${status}\n\n${header}\n${item.content}`;
+  });
+
+  return summaryLines.concat('', detailSections).join('\n');
 }
 
 export function runReplace(spec, cwd = '.') {
@@ -101,22 +117,29 @@ export function runReplace(spec, cwd = '.') {
         return replacement;
       });
 
-      if (!dryRun && matches > 0 && replaced !== original) {
+      const relOutputPath = path.relative(process.cwd(), absPath);
+      const wouldChange = matches > 0 && replaced !== original;
+
+      if (!dryRun && wouldChange) {
         try {
           fs.writeFileSync(absPath, replaced, { encoding });
         } catch (err) {
           throw new Error(`Unable to write file: ${absPath} — ${err.message}`);
         }
       }
 
+      const finalContent = dryRun ? replaced : wouldChange ? replaced : original;
+
       results.push({
-        path: path.relative(process.cwd(), absPath),
+        path: relOutputPath,
         matches,
+        content: finalContent,
+        changed: !dryRun && wouldChange,
       });
     }
 
     return {
-      stdout: buildResultSummary(results, dryRun),
+      stdout: buildResultOutput(results, dryRun),
       stderr: '',
       exit_code: 0,
       killed: false,

tests/unit/replaceCommand.test.js

@@ -23,8 +23,16 @@ describe('runReplace', () => {
       dir,
     );
 
+    const relOutput = path.relative(process.cwd(), file);
+
     expect(result.exit_code).toBe(0);
     expect(fs.readFileSync(file, 'utf8')).toBe('bar and bar again');
+    expect(result.stdout).toContain('Total matches: 2');
+    expect(result.stdout).toContain('Files with changes: 1');
+    expect(result.stdout).toContain(`${relOutput}: 2 matches`);
+    expect(result.stdout).toContain(`Updated ${relOutput}`);
+    expect(result.stdout).toContain(`--- ${relOutput}`);
+    expect(result.stdout).toContain('bar and bar again');
   });
 
   test('supports dry-run without modifying files', async () => {
@@ -42,8 +50,14 @@ describe('runReplace', () => {
       dir,
     );
 
+    const relOutput = path.relative(process.cwd(), file);
+
     expect(result.exit_code).toBe(0);
-    expect(result.stdout).toMatch(/Dry-run/);
+    expect(result.stdout).toContain('Dry-run: no files were modified.');
+    expect(result.stdout).toContain(`${relOutput}: 1 matches (dry-run)`);
+    expect(result.stdout).toContain(`Preview ${relOutput}`);
+    expect(result.stdout).toContain(`--- ${relOutput}`);
+    expect(result.stdout).toContain('hello universe');
     expect(fs.readFileSync(file, 'utf8')).toBe('hello world');
   });