fix: correct readiness hint, detect corrupted config, preserve CRLF, dedupe computation

- Fix getPlatformReadinessHint pointing to /architect instead of
  /implementation-readiness (guidance.ts, artifact-scan.ts)
- Distinguish corrupted config from missing config in status, run,
  and watch commands — show "bmalph doctor" hint instead of misleading
  "bmalph init" (status.ts, run.ts, watch.ts, config.ts)
- Catch broken JSON in readConfig so parse errors don't bypass the
  friendly corrupted-config message
- Hoist duplicate buildCompletedTitleMap/newTitleMap computation in
  fix-plan-sync.ts to avoid redundant work
- Preserve CRLF line endings in removeGitignoreLines (reset.ts)
- Remove exports field from package.json that triggered CLI side
  effects on import("bmalph")

Author: LarsCowe

package.json

@@ -52,9 +52,6 @@
   "engines": {
     "node": ">=20.0.0"
   },
-  "exports": {
-    ".": "./dist/cli.js"
-  },
   "files": [
     "bin/",
     "dist/",

src/commands/run.ts

@@ -1,6 +1,7 @@
 import chalk from "chalk";
 import { readConfig } from "../utils/config.js";
 import { withErrorHandling } from "../utils/errors.js";
+import { isInitialized } from "../installer/project-files.js";
 import { isPlatformId, getPlatform, getFullTierPlatformNames } from "../platform/registry.js";
 import { validateCursorRuntime } from "../platform/cursor-runtime-checks.js";
 import {
@@ -32,6 +33,9 @@ async function executeRun(options: RunCommandOptions): Promise<void> {
 
   const config = await readConfig(projectDir);
   if (!config) {
+    if (await isInitialized(projectDir)) {
+      throw new Error("Config file is corrupted. Run: bmalph doctor");
+    }
     throw new Error("Project not initialized. Run: bmalph init");
   }
 

src/commands/status.ts

@@ -2,6 +2,7 @@ import chalk from "chalk";
 import { readConfig } from "../utils/config.js";
 import { readState, readRalphStatus, getPhaseLabel, getPhaseInfo } from "../utils/state.js";
 import { withErrorHandling } from "../utils/errors.js";
+import { isInitialized } from "../installer/project-files.js";
 import { formatStatus } from "../utils/format-status.js";
 import { ARTIFACT_DEFINITIONS } from "../utils/artifact-definitions.js";
 import { resolveProjectPlatform } from "../platform/resolve.js";
@@ -46,7 +47,11 @@ export async function runStatus(options: StatusOptions): Promise<void> {
   // Check if project is initialized
   const config = await readConfig(projectDir);
   if (!config) {
-    console.log(chalk.red("Project not initialized. Run: bmalph init"));
+    if (await isInitialized(projectDir)) {
+      console.log(chalk.red("Config file is corrupted. Run: bmalph doctor"));
+    } else {
+      console.log(chalk.red("Project not initialized. Run: bmalph init"));
+    }
     return;
   }
 

src/commands/watch.ts

@@ -1,6 +1,7 @@
 import chalk from "chalk";
 import { readConfig } from "../utils/config.js";
 import { withErrorHandling } from "../utils/errors.js";
+import { isInitialized } from "../installer/project-files.js";
 import { parseInterval } from "../utils/validate.js";
 import { startDashboard } from "../watch/dashboard.js";
 import { getDashboardTerminalSupport } from "../watch/frame-writer.js";
@@ -21,6 +22,9 @@ async function runWatch(options: WatchCommandOptions): Promise<void> {
 
   const config = await readConfig(projectDir);
   if (!config) {
+    if (await isInitialized(projectDir)) {
+      throw new Error("Config file is corrupted. Run: bmalph doctor");
+    }
     throw new Error("Project not initialized. Run: bmalph init");
   }
 

src/platform/guidance.ts

@@ -62,11 +62,11 @@ export function getPlatformEpicsStoriesHint(platform: Platform): string {
 
 export function getPlatformReadinessHint(platform: Platform): string {
   if (platform.commandDelivery.kind === "directory") {
-    return "Run /architect to generate readiness report";
+    return "Run /implementation-readiness to generate readiness report";
   }
 
   if (platform.commandDelivery.kind === "skills") {
-    return getSkillHint(platform, "architect", platform.commandDelivery.dir);
+    return getSkillHint(platform, "check-implementation-readiness", platform.commandDelivery.dir);
   }
 
   return getCommandIndexHint(platform);

src/reset.ts

@@ -178,11 +178,12 @@ export async function executeResetPlan(projectDir: string, plan: ResetPlan): Pro
   }
 }
 
-function removeGitignoreLines(content: string, linesToRemove: string[]): string {
+export function removeGitignoreLines(content: string, linesToRemove: string[]): string {
   const removeSet = new Set(linesToRemove);
+  const eol = content.includes("\r\n") ? "\r\n" : "\n";
   const lines = content.split(/\r?\n/);
   const filtered = lines.filter((line) => !removeSet.has(line.trim()));
-  return filtered.join("\n");
+  return filtered.join(eol);
 }
 
 export function planToDryRunActions(plan: ResetPlan): DryRunAction[] {

src/transition/artifact-scan.ts

@@ -126,7 +126,7 @@ export function suggestNext(
   if (!foundNames.has("Readiness Report")) {
     return platform
       ? getPlatformReadinessHint(platform)
-      : "Run /architect to generate readiness report";
+      : "Run /implementation-readiness to generate readiness report";
   }
 
   return "Run: bmalph implement";

src/transition/fix-plan-sync.ts

@@ -57,6 +57,9 @@ export async function syncFixPlan(
     }
   }
 
+  const completedTitles = buildCompletedTitleMap(existingItems);
+  const newTitleMap = new Map(inputs.stories.map((story) => [story.id, story.title]));
+
   const sprintStatusSource = await resolveSprintStatusSource(projectDir, inputs);
   if (sprintStatusSource) {
     useTitleBasedMerge = false;
@@ -96,8 +99,6 @@ export async function syncFixPlan(
     const newStoryIds = new Set(inputs.stories.map((story) => story.id));
     orphanWarnings = detectOrphanedCompletedStories(existingItems, newStoryIds);
 
-    const completedTitles = buildCompletedTitleMap(existingItems);
-    const newTitleMap = new Map(inputs.stories.map((story) => [story.id, story.title]));
     const preservedIds = new Set<string>();
 
     for (const [id, title] of newTitleMap) {
@@ -109,9 +110,6 @@ export async function syncFixPlan(
     renumberWarnings = detectRenumberedStories(existingItems, inputs.stories, preservedIds);
   }
 
-  const completedTitles = buildCompletedTitleMap(existingItems);
-  const newTitleMap = new Map(inputs.stories.map((story) => [story.id, story.title]));
-
   info(`Generating fix plan for ${inputs.stories.length} stories...`);
   const newFixPlan = generateFixPlan(inputs.stories, undefined, inputs.planningSpecsSubpath);
   const mergedFixPlan = mergeFixPlanProgress(

src/utils/config.ts

@@ -21,7 +21,13 @@ export interface BmalphConfig {
 }
 
 export async function readConfig(projectDir: string): Promise<BmalphConfig | null> {
-  const data = await readJsonFile<unknown>(join(projectDir, CONFIG_FILE));
+  let data: unknown;
+  try {
+    data = await readJsonFile<unknown>(join(projectDir, CONFIG_FILE));
+  } catch (err) {
+    warn(`Config file is corrupted, treating as missing: ${formatError(err)}`);
+    return null;
+  }
   if (data === null) return null;
   try {
     return validateConfig(data);

tests/commands/status.test.ts

@@ -79,6 +79,32 @@ describe("status command", () => {
       expect(output).toContain("not initialized");
     });
 
+    it("shows corrupted config message when config exists but has wrong schema", async () => {
+      await mkdir(join(testDir, "bmalph"), { recursive: true });
+      await writeFile(join(testDir, "bmalph/config.json"), '{"invalid": true}');
+
+      const { runStatus } = await import("../../src/commands/status.js");
+      await runStatus({ projectDir: testDir });
+
+      const output = consoleSpy.mock.calls.map((c) => c[0]).join("\n");
+      expect(output).toContain("corrupted");
+      expect(output).toContain("bmalph doctor");
+      expect(output).not.toContain("bmalph init");
+    });
+
+    it("shows corrupted config message when config contains broken JSON", async () => {
+      await mkdir(join(testDir, "bmalph"), { recursive: true });
+      await writeFile(join(testDir, "bmalph/config.json"), "{corrupt");
+
+      const { runStatus } = await import("../../src/commands/status.js");
+      await runStatus({ projectDir: testDir });
+
+      const output = consoleSpy.mock.calls.map((c) => c[0]).join("\n");
+      expect(output).toContain("corrupted");
+      expect(output).toContain("bmalph doctor");
+      expect(output).not.toContain("bmalph init");
+    });
+
     it("shows phase 1 status when in planning", async () => {
       await setupProject();
       await setupState({ currentPhase: 1, status: "planning" });