Auto-expand diff files and dedupe effect installs

- Collapse removal now resets when switching a diff file to full context
- Add Bun hoisted linker config and dedupe nested effect node_modules on prepare

Author: BunsDev

apps/web/src/lib/diffFileReviewState.test.ts

@@ -76,6 +76,20 @@ describe("setDiffFileContextMode", () => {
       "src/a.ts": { accepted: true, collapsed: false, contextMode: "full" },
     });
   });
+
+  it("auto-expands a file when switching to full context", () => {
+    expect(
+      setDiffFileContextMode(
+        {
+          "src/a.ts": { accepted: false, collapsed: true, contextMode: "patch" },
+        },
+        "src/a.ts",
+        "full",
+      ),
+    ).toEqual({
+      "src/a.ts": { accepted: false, collapsed: false, contextMode: "full" },
+    });
+  });
 });
 
 describe("expandDiffFile", () => {

apps/web/src/lib/diffFileReviewState.ts

@@ -66,6 +66,7 @@ export function setDiffFileContextMode(
     ...current,
     [path]: {
       ...previous,
+      collapsed: contextMode === "full" ? false : previous.collapsed,
       contextMode,
     },
   };

bunfig.toml

@@ -0,0 +1,2 @@
+[install]
+linker = "hoisted"

package.json

@@ -54,7 +54,7 @@
     "clean": "rm -rf node_modules apps/*/node_modules packages/*/node_modules apps/*/dist apps/*/dist-electron packages/*/dist .turbo apps/*/.turbo packages/*/.turbo",
     "sync:vscode-icons": "node scripts/sync-vscode-icons.mjs",
     "regenerate:brand-assets": "python3 scripts/generate-brand-assets.py",
-    "prepare": "husky && node scripts/patch-effect-language-service.ts"
+    "prepare": "husky && node scripts/patch-effect-language-service.ts && node scripts/dedupe-effect-node-modules.mjs"
   },
   "devDependencies": {
     "@types/node": "catalog:",

scripts/dedupe-effect-node-modules.mjs

@@ -0,0 +1,62 @@
+import fs from "node:fs/promises";
+import path from "node:path";
+import { fileURLToPath } from "node:url";
+
+const scriptDir = path.dirname(fileURLToPath(import.meta.url));
+const repoRoot = path.resolve(scriptDir, "..");
+const effectRoot = path.join(repoRoot, "node_modules", "effect");
+const effectScopeDir = path.join(repoRoot, "node_modules", "@effect");
+
+async function exists(targetPath) {
+  try {
+    await fs.lstat(targetPath);
+    return true;
+  } catch {
+    return false;
+  }
+}
+
+async function dedupeNestedEffect(packageDir) {
+  const nestedEffectDir = path.join(packageDir, "node_modules", "effect");
+  if (!(await exists(nestedEffectDir))) {
+    return false;
+  }
+
+  const nestedStat = await fs.lstat(nestedEffectDir);
+  if (nestedStat.isSymbolicLink()) {
+    const linkTarget = await fs.readlink(nestedEffectDir);
+    const resolvedTarget = path.resolve(path.dirname(nestedEffectDir), linkTarget);
+    if (resolvedTarget === effectRoot) {
+      return false;
+    }
+  }
+
+  await fs.rm(nestedEffectDir, { recursive: true, force: true });
+  const relativeTarget = path.relative(path.dirname(nestedEffectDir), effectRoot);
+  await fs.symlink(relativeTarget, nestedEffectDir, "dir");
+  return true;
+}
+
+async function main() {
+  if (!(await exists(effectRoot)) || !(await exists(effectScopeDir))) {
+    return;
+  }
+
+  const entries = await fs.readdir(effectScopeDir, { withFileTypes: true });
+  let dedupedCount = 0;
+  for (const entry of entries) {
+    if (!entry.isDirectory()) {
+      continue;
+    }
+    const changed = await dedupeNestedEffect(path.join(effectScopeDir, entry.name));
+    if (changed) {
+      dedupedCount += 1;
+    }
+  }
+
+  if (dedupedCount > 0) {
+    console.log(`[dedupe-effect-node-modules] linked root effect into ${dedupedCount} package(s)`);
+  }
+}
+
+await main();