@chriskealley/openroad 0.1.1 → 0.1.2

package/README.md

@@ -48,7 +48,7 @@ If npm reports a permissions error during global installation, configure an npm-
 
 ## What installation changes
 
-If `--tools` is omitted, the CLI detects existing `.codex`, `.pi`, `.claude`, and `.cursor` directories. It then:
+If `--tools` is omitted, the CLI detects existing `.agents`, `.pi`, `.claude`, and `.cursor` directories. Codex uses the shared `.agents/skills` location. It then:
 
 - Creates `openspec/roadmap.md` if it does not already exist.
 - Installs the `openroad` and `openroad-next` skills for each selected tool.

package/dist/src/installer.js

@@ -4,11 +4,15 @@ import { fileURLToPath } from "node:url";
 import { mergeConfig, removeConfig } from "./config.js";
 import { MANIFEST_NAME } from "./types.js";
 const CONSUMER_DIRS = {
-    codex: ".codex/skills",
+    codex: ".agents/skills",
     pi: ".pi/skills",
     claude: ".claude/skills",
     cursor: ".cursor/skills"
 };
+const LEGACY_CODEX_SKILLS = new Set([
+    ".codex/skills/openroad/SKILL.md",
+    ".codex/skills/openroad-next/SKILL.md"
+]);
 async function exists(path) { try {
     await access(path);
     return true;
@@ -22,6 +26,9 @@ const ROADMAP_FILE = "openspec/roadmap.md";
 function manifestPath(root, target) {
     return relative(root, target).split(sep).join("/");
 }
+function toPosix(file) {
+    return file.split("\\").join("/");
+}
 // Exported for tests: the guard that keeps remove() from deleting the roadmap.
 export function isRoadmapEntry(file) {
     return file.split("\\").join("/") === ROADMAP_FILE;
@@ -67,6 +74,17 @@ export async function install(root, requested) {
             await copyFile(join(assetRoot(), "skills", skill, "SKILL.md"), destination);
             files.add(manifestPath(root, destination));
         }
+    // Current Codex discovers project skills through .agents/skills. Remove only
+    // legacy files that an earlier OpenRoad manifest explicitly claimed, and do
+    // so only after their current replacements have been written successfully.
+    for (const file of prior?.files ?? []) {
+        const normalized = toPosix(file);
+        if (!LEGACY_CODEX_SKILLS.has(normalized))
+            continue;
+        await rm(join(root, ...normalized.split("/")), { force: true });
+        files.delete(file);
+        files.delete(normalized);
+    }
     await mergeConfig(config);
     const manifest = { schemaVersion: 1, packageVersion: await packageVersion(), installedAt: prior?.installedAt ?? new Date().toISOString(), consumers, files: [...files].sort() };
     await writeFile(join(root, MANIFEST_NAME), JSON.stringify(manifest, null, 2) + "\n");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@chriskealley/openroad",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "description": "A roadmap companion for OpenSpec projects",
   "author": "Chris Kealley",
   "type": "module",
@@ -24,6 +24,7 @@
   "engines": { "node": ">=22" },
   "dependencies": { "yaml": "^2.8.1" },
   "devDependencies": {
+    "@fission-ai/openspec": "1.13.1",
     "@types/node": "^24.3.0",
     "typescript": "^5.9.2"
   },