autocrew 0.3.8 → 0.3.10

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "autocrew",
-  "version": "0.3.8",
+  "version": "0.3.10",
   "description": "One-person content studio powered by AI — from trending topics to published posts",
   "type": "module",
   "license": "MIT",

package/src/cli/commands/review.ts

@@ -4,7 +4,7 @@ import { getOption, resolveProjectText } from "./index.js";
 export const cmd: CommandDef = {
   name: "review",
   description: "Run full content review or auto-fix",
-  usage: "autocrew review <content-id-or-project-slug> [--platform <p>] [--fix] [--file <path>]",
+  usage: "autocrew review <id-or-slug> [--platform <p>] [--fix] [--dry-run] [--file <path>]",
   action: async (args, runner) => {
     const id = args[0];
     if (!id && !args.includes("--file")) {
@@ -15,6 +15,7 @@ export const cmd: CommandDef = {
 
     const platform = getOption(args, "--platform");
     const isFix = args.includes("--fix");
+    const isDryRun = args.includes("--dry-run");
 
     const resolved = await resolveProjectText(id, args);
     if (!resolved) {
@@ -23,7 +24,7 @@ export const cmd: CommandDef = {
       return;
     }
 
-    if (isFix) {
+    if (isFix || isDryRun) {
       const result = await runner.execute("autocrew_review", {
         action: "auto_fix",
         text: resolved.text,
@@ -36,13 +37,47 @@ export const cmd: CommandDef = {
         return;
       }
 
-      // Write fixed text back to source file
       const fixedText = (result.autoFixedText || result.fixedText || "") as string;
+      console.log(`  Sensitive words fixed: ${result.sensitiveWordsFixed || 0}`);
+      console.log(`  AI traces fixed: ${result.aiFixesApplied || 0}`);
+
+      // Show diff preview
+      if (fixedText && fixedText !== resolved.text) {
+        // Simple line-by-line diff
+        const origLines = resolved.text.split("\n");
+        const fixedLines = fixedText.split("\n");
+        const diffLines: string[] = [];
+        const maxLen = Math.max(origLines.length, fixedLines.length);
+        for (let i = 0; i < maxLen; i++) {
+          const orig = origLines[i] ?? "";
+          const fixed = fixedLines[i] ?? "";
+          if (orig !== fixed) {
+            if (orig) diffLines.push(`  - ${orig}`);
+            if (fixed) diffLines.push(`  + ${fixed}`);
+          }
+        }
+        if (diffLines.length > 0) {
+          console.log(`\n  Changes preview:`);
+          for (const line of diffLines.slice(0, 20)) {
+            console.log(line);
+          }
+          if (diffLines.length > 20) {
+            console.log(`  ... and ${diffLines.length - 20} more lines`);
+          }
+        }
+      }
+
+      if (isDryRun) {
+        console.log(`\n  (dry-run mode — no files modified)`);
+        return;
+      }
+
+      // Write fixed text back to source file
       if (fixedText && resolved.source.startsWith("file:")) {
         const fs = await import("node:fs/promises");
         const filePath = resolved.source.replace("file:", "");
         await fs.writeFile(filePath, fixedText, "utf-8");
-        console.log(`Auto-fix complete for "${resolved.title}" — saved back to ${filePath}`);
+        console.log(`\n  Saved back to ${filePath}`);
       } else if (fixedText && resolved.source.startsWith("pipeline:")) {
         const fs = await import("node:fs/promises");
         const { findProject } = await import("../../storage/pipeline-store.js");
@@ -51,13 +86,9 @@ export const cmd: CommandDef = {
         if (found) {
           const path = await import("node:path");
           await fs.writeFile(path.join(found.dir, "draft.md"), fixedText, "utf-8");
-          console.log(`Auto-fix complete for "${resolved.title}" — saved back to draft.md`);
+          console.log(`\n  Saved back to draft.md`);
         }
-      } else {
-        console.log(`Auto-fix complete for "${resolved.title}" (${resolved.source})`);
       }
-      console.log(`  Sensitive words fixed: ${result.sensitiveWordsFixed || 0}`);
-      console.log(`  AI traces fixed: ${result.aiFixesApplied || 0}`);
       return;
     }
 

package/src/tools/review.ts

@@ -212,10 +212,14 @@ export async function executeReview(params: Record<string, unknown>) {
     const scanResult = await scanText(fullText, platform, dataDir);
     let fixedText = scanResult.autoFixedText || fullText;
 
-    // 2. Humanizer pass
+    // 2. Humanizer pass (substitution + deletion only, never insertion)
     const humanResult = humanizeZh({ text: fixedText });
     fixedText = humanResult.humanizedText;
 
+    // 3. Safety guard — strip any hardcoded phrases that old humanizer versions
+    // may have inserted. This is a belt-and-suspenders defense.
+    fixedText = fixedText.replace(/\n*说白了，这件事拼的不是工具数量，而是表达和执行。\n*/g, "\n\n");
+
     // Save back if content_id provided
     if (contentId) {
       await updateContent(contentId, { body: fixedText }, dataDir);
@@ -250,13 +254,21 @@ export async function executeReview(params: Record<string, unknown>) {
   const fixes: string[] = [];
   if (sensitiveWords.hitCount > 0) {
     fixes.push(`修复 ${sensitiveWords.hitCount} 个敏感词`);
-    for (const hit of sensitiveWords.hits.slice(0, 5)) {
+    for (const hit of sensitiveWords.hits.slice(0, 10)) {
       const fix = hit.suggestion ? `"${hit.word}" → "${hit.suggestion}"` : `删除"${hit.word}"`;
-      fixes.push(`  - ${fix} (${hit.category})`);
+      // Show surrounding context for each hit
+      const pos = hit.positions[0];
+      const contextStart = Math.max(0, pos - 10);
+      const contextEnd = Math.min(fullText.length, pos + hit.word.length + 10);
+      const context = fullText.slice(contextStart, contextEnd).replace(/\n/g, " ");
+      fixes.push(`  - ${fix} (${hit.category}) — "...${context}..."`);
     }
   }
   if (aiCheck.hasAiTraces) {
     fixes.push(`去 AI 味：${aiCheck.changeCount} 处需要修改`);
+    for (const change of aiCheck.changes.slice(0, 5)) {
+      fixes.push(`  - ${change}`);
+    }
   }
   for (const note of qualityScore.notes) {
     fixes.push(note);