refactor: Decompose 6 mega-files into 13 focused modules, remove legacy Logger

Optimization sweep executing all 5 recommendations from /frg-optimize:

1. Decompose large files (>800 lines each):
   - init-service.ts: 865→425 lines (extract types, templates, detection)
   - diagnostics.ts: 852→226 lines (extract tests, formatters)
   - vision-service.ts: 825→567 lines (extract parsers, formatters)
   - automation-service.ts: 824→612 lines (extract rules, safety)
   - vision.ts: 812→478 lines (extract parser, serializer)
   - state.ts: 749→695 lines (extract context-graph)

2. Remove legacy Logger class from utils.ts, migrate sole consumer (forge.ts)
   to proper winston-based getLogger() from utils/logger.ts

3. Upgrade concurrently 8→9 (safe dev-tool upgrade; express 5, eslint 9+,
   tailwind v4 deferred as each requires dedicated migration)

4. Minor dep patches via npm update (12 packages)

All backward-compatible — existing imports still work via re-exports.
Verified: tsc clean, 4105 tests passing, server smoke test OK.

Co-Authored-By: Claude Opus 4.6 <[email protected]>

Author: awaliuddin

package-lock.json

@@ -93,7 +93,7 @@
     "autoprefixer": "^10.4.23",
     "class-variance-authority": "^0.7.1",
     "clsx": "^2.1.1",
-    "concurrently": "^8.2.2",
+    "concurrently": "^9.2.1",
     "cors": "^2.8.5",
     "express": "^4.19.2",
     "framer-motion": "^12.31.0",

package.json

@@ -25,6 +25,7 @@ import {
 import { CanonicalVision } from "../types/vision";
 import { CheckpointManager, TaskCheckpoint } from "./checkpoint-manager";
 import { Result, Ok, Err } from "../utils/result";
+import { buildContextGraph } from "./state/context-graph";
 
 const logger = new Logger("StateManager");
 
@@ -293,62 +294,7 @@ export class StateManager extends EventEmitter {
       throw new Error("No state available");
     }
 
-    const nodes: ContextNode[] = [];
-    const edges: ContextEdge[] = [];
-
-    // Add vision as root node
-    nodes.push({
-      id: "vision",
-      type: "vision",
-      title: "Canonical Vision",
-      data: this.currentState.vision,
-    });
-
-    // Add goals
-    for (const goal of this.currentState.vision.strategicGoals) {
-      nodes.push({
-        id: `goal-${goal.id}`,
-        type: "goal",
-        title: goal.title,
-        data: goal,
-      });
-
-      edges.push({
-        from: "vision",
-        to: `goal-${goal.id}`,
-        type: "implements",
-      });
-    }
-
-    // Add tasks
-    for (const task of this.currentState.currentTasks) {
-      nodes.push({
-        id: `task-${task.id}`,
-        type: "task",
-        title: task.title,
-        data: task,
-      });
-
-      // Link tasks to goals (simplified)
-      if (this.currentState.vision.strategicGoals.length > 0) {
-        edges.push({
-          from: `goal-${this.currentState.vision.strategicGoals[0].id}`,
-          to: `task-${task.id}`,
-          type: "implements",
-        });
-      }
-
-      // Add task dependencies
-      for (const depId of task.dependencies) {
-        edges.push({
-          from: `task-${depId}`,
-          to: `task-${task.id}`,
-          type: "depends-on",
-        });
-      }
-    }
-
-    this.contextGraph = { nodes, edges };
+    this.contextGraph = await buildContextGraph(this.currentState);
 
     // Save graph
     await fs.writeFile(
@@ -358,7 +304,7 @@ export class StateManager extends EventEmitter {
     );
 
     logger.info(
-      `Context graph built with ${nodes.length} nodes and ${edges.length} edges`,
+      `Context graph built with ${this.contextGraph.nodes.length} nodes and ${this.contextGraph.edges.length} edges`,
     );
 
     return this.contextGraph;

src/core/state.ts

@@ -0,0 +1,75 @@
+/**
+ * Context Graph Builder
+ * Builds the context graph from system state
+ */
+
+import {
+  SystemState,
+  ContextGraph,
+  ContextNode,
+  ContextEdge,
+} from "../../types/state";
+
+/**
+ * Build context graph from system state
+ */
+export async function buildContextGraph(
+  state: SystemState,
+): Promise<ContextGraph> {
+  const nodes: ContextNode[] = [];
+  const edges: ContextEdge[] = [];
+
+  // Add vision as root node
+  nodes.push({
+    id: "vision",
+    type: "vision",
+    title: "Canonical Vision",
+    data: state.vision,
+  });
+
+  // Add goals
+  for (const goal of state.vision.strategicGoals) {
+    nodes.push({
+      id: `goal-${goal.id}`,
+      type: "goal",
+      title: goal.title,
+      data: goal,
+    });
+
+    edges.push({
+      from: "vision",
+      to: `goal-${goal.id}`,
+      type: "implements",
+    });
+  }
+
+  // Add tasks
+  for (const task of state.currentTasks) {
+    nodes.push({
+      id: `task-${task.id}`,
+      type: "task",
+      title: task.title,
+      data: task,
+    });
+
+    // Link tasks to goals (simplified)
+    if (state.vision.strategicGoals.length > 0) {
+      edges.push({
+        from: `goal-${state.vision.strategicGoals[0].id}`,
+        to: `task-${task.id}`,
+        type: "implements",
+      });
+    }
+
+    // Add task dependencies
+    for (const depId of task.dependencies) {
+      edges.push({
+        from: `task-${depId}`,
+        to: `task-${task.id}`,
+        type: "depends-on",
+      });
+    }
+  }
+
+  return { nodes, edges };
+}