@warpgogol/forge 2.21.1 → 2.21.3

package/README.md

@@ -16,6 +16,18 @@ Install https://npmjs.com/package/@warpgogol/forge in this folder and set up my
 
 Replace `[describe your project]` with your idea — a game, a library, a knowledge base, anything. The AI agent installs Forge, scaffolds the project, and sets up a live preview. From there on, you just talk: describe what you want, and the agent builds it. No programming, no terminal, no commands.
 
+### After the prompt completes — the Bootstrap skill
+
+Once the AI agent has installed Forge and scaffolded your project, it will run the `/forge-bootstrap` skill. This is a required step that configures your project interactively:
+
+- The language the AI uses to communicate with you
+- The language for project documentation (RFCs, ADRs, READMEs) — defaults to English
+- Your preferred working style — business or creative
+- Your name and how you want to be addressed
+- Your stack configuration (or migration of an existing project)
+
+The skill asks a few simple questions in the chat, then sets everything up. After it completes, you're ready to create — just tell the AI agent what you want to build.
+
 ---
 
 ## What you can build with Forge

package/README.uk.md

@@ -16,6 +16,18 @@ Install https://npmjs.com/package/@warpgogol/forge in this folder and set up my
 
 Замініть `[опишіть ваш проєкт]` на вашу ідею — гра, бібліотека, база знань, будь-що. ШІ-агент встановить Forge, згенерує каркас проєкту та налаштує живий перегляд. Далі ви просто розмовляєте: описуєте, що хочете, і агент будує це. Жодного програмування, жодного терміналу, жодних команд.
 
+### Після виконання промпту — навичка Bootstrap
+
+Коли ШІ-агент встановить Forge і згенерує каркас проєкту, він запустить навичку `/forge-bootstrap`. Це обов'язковий крок, який інтерактивно налаштовує ваш проєкт:
+
+- Мову, якою ШІ спілкується з вами
+- Мову документації проєкту (RFC, ADR, README) — за замовчуванням англійська
+- Ваш стиль роботи — діловий або творчий
+- Ваше ім'я та форму звертання
+- Конфігурацію стеку (або міграцію наявного проєкту)
+
+Навичка ставить кілька простих запитань у чаті, а потім налаштовує все. Після завершення ви готові творити — просто скажіть ШІ-агенту, що ви хочете створити.
+
 ---
 
 ## Що можна створити за допомогою Forge

package/os/mission/handlers/archive.test.ts

@@ -147,16 +147,16 @@ describe("mission.archive", () => {
     expect(data.skipped.some((s) => s.reason === "destination exists")).toBe(true);
   });
 
-  test("unreadable manifest → skipped with 'unreadable manifest' reason", async () => {
+  test("unreadable manifest with empty dir → skipped with 'empty remnant' reason", async () => {
     const missionDir = path.join(missionsDir, "test-m008");
     await fs.mkdir(missionDir, { recursive: true });
-    // No mission.yaml — unreadable
+    // No mission.yaml, no workpiece — empty remnant
 
     const data = unwrap(await runMissionArchive(makeInput(), makeContext(tmpDir)));
 
     expect(data.moved).toHaveLength(0);
     expect(data.skipped).toHaveLength(1);
-    expect(data.skipped[0].reason).toBe("unreadable manifest");
+    expect(data.skipped[0].reason).toBe("empty remnant — use --clean-orphans to remove");
   });
 
   test("open mission in archive/ → moved back to missions/ (bidirectional)", async () => {
@@ -388,4 +388,134 @@ describe("mission.archive", () => {
     );
     expect(installCall).toBeUndefined();
   });
+
+  // RFC-0982: Fallback state detection and --clean-orphans tests
+
+  test("RFC-0982: orphaned workpiece with only .astro/ cache → state detected as closed, archived", async () => {
+    const missionDir = path.join(missionsDir, "test-r982-01");
+    const workpieceDir = path.join(missionDir, "workpiece");
+    await fs.mkdir(path.join(workpieceDir, ".astro"), { recursive: true });
+    await fs.writeFile(path.join(workpieceDir, ".astro", "cache.txt"), "cache\n");
+    // No mission.yaml — orphaned remnant
+
+    const data = unwrap(await runMissionArchive(makeInput(), makeContext(tmpDir)));
+
+    expect(data.moved).toHaveLength(1);
+    expect(data.moved[0].missionId).toBe("test-r982-01");
+    expect(data.moved[0].state).toBe("closed");
+    expect(data.moved[0].direction).toBe("into-archive");
+    expect(existsSync(path.join(missionsDir, "archive", "closed", "test-r982-01"))).toBe(true);
+  });
+
+  test("RFC-0982: .closed marker fallback → state detected as closed", async () => {
+    const missionDir = path.join(missionsDir, "test-r982-02");
+    const workpieceDir = path.join(missionDir, "workpiece");
+    await fs.mkdir(workpieceDir, { recursive: true });
+    await fs.writeFile(path.join(workpieceDir, ".closed"), "2026-08-29T12:00:00Z\n");
+    // Also add a source file so it's not cache-only
+    await fs.writeFile(path.join(workpieceDir, "src.ts"), "// source\n");
+    // No mission.yaml — but .closed marker present
+
+    const data = unwrap(await runMissionArchive(makeInput(), makeContext(tmpDir)));
+
+    expect(data.moved).toHaveLength(1);
+    expect(data.moved[0].missionId).toBe("test-r982-02");
+    expect(data.moved[0].state).toBe("closed");
+    expect(existsSync(path.join(missionsDir, "archive", "closed", "test-r982-02"))).toBe(true);
+  });
+
+  test("RFC-0982: --clean-orphans trashes orphaned remnant dir", async () => {
+    const missionDir = path.join(missionsDir, "test-r982-03");
+    const workpieceDir = path.join(missionDir, "workpiece");
+    await fs.mkdir(path.join(workpieceDir, ".astro"), { recursive: true });
+    await fs.writeFile(path.join(workpieceDir, ".astro", "cache.txt"), "cache\n");
+    // No mission.yaml — orphaned remnant with only cache
+
+    const data = unwrap(
+      await runMissionArchive(makeInput({ "clean-orphans": true }), makeContext(tmpDir)),
+    );
+
+    expect(data.moved).toHaveLength(1);
+    expect(data.moved[0].missionId).toBe("test-r982-03");
+    expect(data.moved[0].direction).toBe("trashed-orphan");
+    expect(data.moved[0].to).toBe("(trashed)");
+    expect(existsSync(missionDir)).toBe(false);
+  });
+
+  test("RFC-0982: --clean-orphans --dry-run → reported but not trashed", async () => {
+    const missionDir = path.join(missionsDir, "test-r982-04");
+    const workpieceDir = path.join(missionDir, "workpiece");
+    await fs.mkdir(path.join(workpieceDir, ".astro"), { recursive: true });
+    await fs.writeFile(path.join(workpieceDir, ".astro", "cache.txt"), "cache\n");
+
+    const data = unwrap(
+      await runMissionArchive(
+        makeInput({ "clean-orphans": true, "dry-run": true }),
+        makeContext(tmpDir),
+      ),
+    );
+
+    expect(data.moved).toHaveLength(1);
+    expect(data.moved[0].direction).toBe("trashed-orphan");
+    expect(data.dryRun).toBe(true);
+    // Dir should still exist in dry-run
+    expect(existsSync(missionDir)).toBe(true);
+  });
+
+  test("RFC-0982: --clean-orphans skips non-orphaned dir with mission.yaml", async () => {
+    await writeMissionManifest(missionsDir, "test-r982-05", "closed");
+    const workpieceDir = path.join(missionsDir, "test-r982-05", "workpiece");
+    await fs.mkdir(path.join(workpieceDir, ".astro"), { recursive: true });
+
+    const data = unwrap(
+      await runMissionArchive(makeInput({ "clean-orphans": true }), makeContext(tmpDir)),
+    );
+
+    // Should be archived normally, not trashed
+    expect(data.moved).toHaveLength(1);
+    expect(data.moved[0].direction).toBe("into-archive");
+    expect(existsSync(path.join(missionsDir, "archive", "closed", "test-r982-05"))).toBe(true);
+  });
+
+  test("RFC-0982: skip reason 'manually inspect' for dir with non-cache content but no mission.yaml", async () => {
+    const missionDir = path.join(missionsDir, "test-r982-06");
+    const workpieceDir = path.join(missionDir, "workpiece");
+    await fs.mkdir(workpieceDir, { recursive: true });
+    // Source file in workpiece — non-cache content, no .closed marker
+    await fs.writeFile(path.join(workpieceDir, "src.ts"), "// source code\n");
+    // No mission.yaml
+
+    const data = unwrap(await runMissionArchive(makeInput(), makeContext(tmpDir)));
+
+    expect(data.moved).toHaveLength(0);
+    expect(data.skipped).toHaveLength(1);
+    expect(data.skipped[0].reason).toContain("manually inspect");
+  });
+
+  test("RFC-0982: --clean-orphans trashes empty remnant dir (no workpiece, no mission.yaml)", async () => {
+    const missionDir = path.join(missionsDir, "test-r982-08");
+    await fs.mkdir(missionDir, { recursive: true });
+    // No mission.yaml, no workpiece — empty remnant, state would be null
+
+    const data = unwrap(
+      await runMissionArchive(makeInput({ "clean-orphans": true }), makeContext(tmpDir)),
+    );
+
+    expect(data.moved).toHaveLength(1);
+    expect(data.moved[0].missionId).toBe("test-r982-08");
+    expect(data.moved[0].direction).toBe("trashed-orphan");
+    expect(existsSync(missionDir)).toBe(false);
+  });
+
+  test("RFC-0982: skip reason 'use --clean-orphans' for empty remnant dir", async () => {
+    const missionDir = path.join(missionsDir, "test-r982-07");
+    await fs.mkdir(missionDir, { recursive: true });
+    // No mission.yaml, no workpiece, no content — empty remnant
+
+    const data = unwrap(await runMissionArchive(makeInput(), makeContext(tmpDir)));
+
+    expect(data.moved).toHaveLength(0);
+    expect(data.skipped).toHaveLength(1);
+    expect(data.skipped[0].reason).toContain("use --clean-orphans");
+  });
 });

package/os/mission/handlers/archive.ts

@@ -18,6 +18,7 @@ archive subdirectories back to missions/.
   <item>RFC-0801: add service-folder cleanup (node_modules, dist, .astro, .wrangler, .cache, .turbo) before archive move.</item>
   <item>RFC-0733: add pinned-files pre-check — skip pinned mission directories with warning instead of moving them.</item>
   <item>RFC-0804: auto-refresh pnpm-lock.yaml after directory moves.</item>
+  <item>RFC-0982: fallback state detection for orphaned workpiece dirs (no mission.yaml) via .closed marker and cache-only heuristic; --clean-orphans flag; improved skip reasons.</item>
 </CHANGE_SUMMARY>
 */
 
@@ -63,6 +64,11 @@ async function cleanServiceFolders(workpieceDir: string): Promise<string[]> {
   return removed;
 }
 
+// RFC-0982: Shared cache-entry predicate — prevents filter logic divergence.
+function isCacheEntry(name: string): boolean {
+  return name.startsWith(".") || name === "node_modules" || name === "dist";
+}
+
 async function readMissionState(missionDir: string): Promise<string | null> {
   const manifestPath = path.join(missionDir, "mission.yaml");
   try {
@@ -70,10 +76,64 @@ async function readMissionState(missionDir: string): Promise<string | null> {
     const parsed = parseYaml(raw) as Record<string, unknown>;
     const state = parsed?.state;
     if (typeof state === "string") return state.trim();
-    return null;
   } catch {
-    return null;
+    // Fall through to secondary checks
+  }
+
+  // RFC-0982: Secondary — check for workpiece/.closed marker
+  const closedMarker = path.join(missionDir, "workpiece", ".closed");
+  if (existsSync(closedMarker)) return "closed";
+
+  // RFC-0982: Tertiary — if workpiece/ exists and contains only cache entries,
+  // treat as closed (post-close remnant pattern)
+  const workpieceDir = path.join(missionDir, "workpiece");
+  if (existsSync(workpieceDir)) {
+    const entries = await fs.readdir(workpieceDir);
+    const nonCacheEntries = entries.filter((e) => !isCacheEntry(e));
+    if (nonCacheEntries.length === 0) {
+      return "closed";
+    }
   }
+
+  return null;
+}
+
+// RFC-0982: Check if a mission directory is an orphaned remnant — no mission.yaml
+// and workpiece/ contains only cache entries (or no workpiece at all).
+async function isOrphanedRemnant(missionDir: string): Promise<boolean> {
+  const manifestPath = path.join(missionDir, "mission.yaml");
+  if (existsSync(manifestPath)) return false;
+
+  const workpieceDir = path.join(missionDir, "workpiece");
+  if (!existsSync(workpieceDir)) return true;
+
+  const entries = await fs.readdir(workpieceDir);
+  const nonCacheEntries = entries.filter((e) => !isCacheEntry(e));
+  return nonCacheEntries.length === 0;
+}
+
+// RFC-0982: Check if a mission directory has non-cache content (source files,
+// configs, etc.) that warrants manual inspection rather than auto-cleanup.
+async function hasNonCacheContent(missionDir: string): Promise<boolean> {
+  const entries = await fs.readdir(missionDir, { withFileTypes: true });
+  for (const entry of entries) {
+    if (entry.name === "workpiece") continue;
+    if (entry.isDirectory() && isCacheEntry(entry.name)) {
+      continue;
+    }
+    // Any file or non-cache directory at mission root is content
+    return true;
+  }
+
+  // Check workpiece/ for non-cache content
+  const workpieceDir = path.join(missionDir, "workpiece");
+  if (existsSync(workpieceDir)) {
+    const wpEntries = await fs.readdir(workpieceDir);
+    const nonCacheEntries = wpEntries.filter((e) => !isCacheEntry(e));
+    if (nonCacheEntries.length > 0) return true;
+  }
+
+  return false;
 }
 
 interface MoveAttempt {
@@ -158,6 +218,7 @@ export async function runMissionArchive(
 
   const dryRun = context.dryRun || input.flags["dry-run"] === true;
   const statusFilter = input.flags["status"] as string | undefined;
+  const cleanOrphans = input.flags["clean-orphans"] === true;
 
   if (statusFilter && !MISSION_TERMINAL_STATUSES.includes(statusFilter as never)) {
     throw new Error(
@@ -223,13 +284,35 @@ export async function runMissionArchive(
 
   for (const missionId of rootDirs) {
     const missionDir = path.join(missionsPath, missionId);
+    const sourceRel = `${MISSIONS_DIR}/${missionId}`;
+
+    // RFC-0982: --clean-orphans — trash orphaned remnant directories before state
+    // detection. This catches dirs where mission.yaml is missing and only cache
+    // entries remain, regardless of whether readMissionState returns "closed" or null.
+    if (cleanOrphans && (await isOrphanedRemnant(missionDir))) {
+      if (!dryRun) {
+        await trashPath(missionDir);
+      }
+      moved.push({
+        missionId,
+        state: "closed",
+        from: sourceRel,
+        to: "(trashed)",
+        direction: "trashed-orphan",
+      });
+      continue;
+    }
+
     const state = await readMissionState(missionDir);
 
     if (state === null) {
+      const hasContent = await hasNonCacheContent(missionDir);
       skipped.push({
         missionId,
         dir: `${MISSIONS_DIR}/${missionId}`,
-        reason: "unreadable manifest",
+        reason: hasContent
+          ? "unreadable manifest — manually inspect or add mission.yaml"
+          : "empty remnant — use --clean-orphans to remove",
       });
       continue;
     }
@@ -257,7 +340,6 @@ export async function runMissionArchive(
     const targetDir = path.join(missionsPath, ARCHIVE_DIR_NAME, state);
     const targetPath = path.join(targetDir, missionId);
     const targetRel = `${MISSIONS_DIR}/${ARCHIVE_DIR_NAME}/${state}/${missionId}`;
-    const sourceRel = `${MISSIONS_DIR}/${missionId}`;
 
     // RFC-0733: Check if mission directory is pinned before moving
     // Gap fix: exempt intra-directory moves (dir stays within the same pinned parent)

package/os/mission/mission.module.ts

@@ -7,6 +7,7 @@
 </MODULE_CONTRACT>
 <CHANGE_SUMMARY>
   <item>RFC-0573: initial forgeMissionModule registering mission.archive command.</item>
+  <item>RFC-0982: add --clean-orphans flag for trashing orphaned remnant directories.</item>
 </CHANGE_SUMMARY>
 */
 
@@ -42,6 +43,11 @@ export const forgeMissionModule: ForgeModule = {
           kind: "string",
           description: "Filter to a single terminal status (closed, aborted).",
         },
+        "clean-orphans": {
+          kind: "boolean",
+          description:
+            "Trash orphaned directories (no mission.yaml, only cache files) instead of archiving them.",
+        },
       },
       execute: runMissionArchive,
     });

package/os/mission/types.ts

@@ -10,6 +10,7 @@ manifest state extraction, and archive result shapes.
 </MODULE_CONTRACT>
 <CHANGE_SUMMARY>
   <item>RFC-0573: initial mission archive types.</item>
+  <item>RFC-0982: add "trashed-orphan" to MissionArchiveMove.direction.</item>
 </CHANGE_SUMMARY>
 */
 
@@ -23,7 +24,7 @@ export interface MissionArchiveMove {
   state: string;
   from: string;
   to: string;
-  direction: "into-archive" | "out-of-archive";
+  direction: "into-archive" | "out-of-archive" | "trashed-orphan";
 }
 
 export interface MissionArchiveSkip {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@warpgogol/forge",
-  "version": "2.21.1",
+  "version": "2.21.3",
   "private": false,
   "type": "module",
   "license": "Apache-2.0",

package/skills/meta/forge-bootstrap/SKILL.md

@@ -66,12 +66,20 @@ Before any operator interaction, silently check whether the installed `@warpgogo
 
 Read `PREFERENCES.md` at the project root. `forge create` writes a placeholder with `aiLanguage: en` and `documentationLanguage: en`.
 
-Ask the operator:
+Ask the operator two questions, one at a time:
+
+**Question 1 — AI communication language:**
 
-> In which language should the AI communicate with you? (e.g. en, ru, uk, de, es) In which language should project documentation be written? (RFCs, ADRs, READMEs)
+> In which language should the AI communicate with you? (e.g. en, ru, uk, de, es)
 
 Accept free-form answers like "Russian", "русский", "uk" or "English". Prefer IETF BCP 47 language tags when the operator provides them.
 
+**Question 2 — Documentation language:**
+
+> In which language should project documentation be written? (RFCs, ADRs, READMEs) Press Enter to use English (default).
+
+Accept free-form answers like "Russian", "русский", "uk" or "English". If the operator presses Enter or says "default" / "English", use `en`. Prefer IETF BCP 47 language tags when the operator provides them.
+
 Write or merge the values into `PREFERENCES.md`. **All subsequent communication in this skill session uses the operator's chosen `aiLanguage`.**
 
 If `PREFERENCES.md` already has non-default `aiLanguage` set (re-run of the skill), confirm the existing values with the operator instead of asking again. The operator may change them if desired.

package/src/onboarding/create.ts

@@ -397,7 +397,7 @@ Your Forge project is ready. The next step is mandatory: run \`/forge-bootstrap\
 Run \`/forge-bootstrap\` now. It will ask you:
 
 - Which language the AI should communicate with you in
-- Which language project documentation should be written in
+- Which language project documentation should be written in (defaults to English)
 - Whether you prefer a business or creative working style
 - Your name and how you want to be addressed
 - Your stack configuration (or migrate an existing project)