@overscore/cli 0.11.10 → 0.11.12

package/dist/index.js

@@ -1262,16 +1262,6 @@ async function uploadKnowledgeBase(targetDir, baseUrl, projectSlug, apiKey) {
         // Non-fatal
     }
 }
-/**
- * Fetch the latest platform-owned Claude Code rules from the Hub and write
- * them into the project's .claude/ directory. This keeps rules like
- * data-fetching.md, query-performance.md, and skills up to date even in
- * projects that were scaffolded months ago.
- *
- * Called from `deploy` (before uploading) and `pull` (after extracting).
- * User-owned files (company-context.md, this-dashboard.md, knowledge/)
- * are NOT touched — those sync via separate mechanisms.
- */
 const DASHBOARD_IMPORTS = [
     "@.claude/rules/this-dashboard.md",
     "@.claude/knowledge/INDEX.md",
@@ -1284,16 +1274,19 @@ const ANALYSIS_IMPORTS = [
 ];
 /**
  * Ensures the project's CLAUDE.md has the standard @import lines at the top.
- * Detects dashboard vs analysis by file presence. Injects only what's missing —
- * never overwrites existing content. Returns true if CLAUDE.md was modified.
+ * Detects dashboard vs analysis by reading the CLAUDE.md heading. Injects only
+ * what's missing — never overwrites existing content. Returns true if modified.
  */
 function syncClaudeMdImports(targetDir) {
     const claudeMdPath = path.join(targetDir, ".claude", "CLAUDE.md");
     if (!fs.existsSync(claudeMdPath))
         return false;
-    const isAnalysis = fs.existsSync(path.join(targetDir, "analysis-log.md"));
+    const content = fs.readFileSync(claudeMdPath, "utf-8");
+    const firstHeading = content.match(/^#\s+Overscore\s+(Dashboard|Analysis)/im);
+    if (!firstHeading)
+        return false;
+    const isAnalysis = firstHeading[1].toLowerCase() === "analysis";
     const requiredImports = isAnalysis ? ANALYSIS_IMPORTS : DASHBOARD_IMPORTS;
-    let content = fs.readFileSync(claudeMdPath, "utf-8");
     const missing = requiredImports.filter((imp) => !content.includes(imp));
     if (missing.length === 0)
         return false;
@@ -1307,6 +1300,12 @@ function syncClaudeMdImports(targetDir) {
     fs.writeFileSync(claudeMdPath, updated);
     return true;
 }
+/**
+ * Fetch the latest platform-owned Claude Code rules from the Hub and write
+ * them into the project's .claude/ directory. Called from `deploy` (before
+ * uploading), `pull` (after extracting), and `syncHubManagedRules` (analysis
+ * paths). User-owned files (knowledge/, this-dashboard.md) are NOT touched.
+ */
 async function syncPlatformRules(targetDir, baseUrl, apiKey) {
     try {
         const res = await fetch(`${baseUrl}/api/projects/platform-rules`, {
@@ -1337,11 +1336,15 @@ async function syncPlatformRules(targetDir, baseUrl, apiKey) {
 /**
  * Explicit on-demand platform rules sync. Reports what changed so Claude
  * can tell the user what's new. Called by `npx @overscore/cli platform update`.
+ *
+ * With `--check`: read-only mode. Reports what *would* change without writing.
  */
-async function platformUpdate() {
+async function platformUpdate(checkOnly = false) {
     const { apiUrl, apiKey } = await loadEnv();
     const baseUrl = apiUrl.replace(/\/api$/, "");
-    console.log("\n  Syncing platform rules from Hub...\n");
+    console.log(checkOnly
+        ? "\n  Checking platform rules (no changes will be written)...\n"
+        : "\n  Syncing platform rules from Hub...\n");
     const capPath = path.resolve(process.cwd(), ".claude", "platform-capabilities.md");
     const prevContent = fs.existsSync(capPath) ? fs.readFileSync(capPath, "utf-8") : null;
     const prevVersion = prevContent?.match(/^version:\s*(.+)$/m)?.[1]?.trim() ?? null;
@@ -1361,19 +1364,41 @@ async function platformUpdate() {
     for (const [relPath, content] of Object.entries(data.files)) {
         const fullPath = path.join(process.cwd(), ".claude", relPath);
         const existing = fs.existsSync(fullPath) ? fs.readFileSync(fullPath, "utf-8") : null;
-        fs.mkdirSync(path.dirname(fullPath), { recursive: true });
-        fs.writeFileSync(fullPath, content);
-        if (existing !== content)
+        if (existing !== content) {
             updated.push(relPath);
+            if (!checkOnly) {
+                fs.mkdirSync(path.dirname(fullPath), { recursive: true });
+                fs.writeFileSync(fullPath, content);
+            }
+        }
+    }
+    // syncClaudeMdImports only makes sense if we're actually writing files.
+    // In check mode, compute whether imports would be added by reading current state.
+    let claudeWouldChange = false;
+    if (checkOnly) {
+        const claudeMdPath = path.join(process.cwd(), ".claude", "CLAUDE.md");
+        if (fs.existsSync(claudeMdPath)) {
+            const cmContent = fs.readFileSync(claudeMdPath, "utf-8");
+            const heading = cmContent.match(/^#\s+Overscore\s+(Dashboard|Analysis)/im);
+            if (heading) {
+                const required = heading[1].toLowerCase() === "analysis" ? ANALYSIS_IMPORTS : DASHBOARD_IMPORTS;
+                claudeWouldChange = required.some((imp) => !cmContent.includes(imp));
+            }
+        }
+        if (claudeWouldChange)
+            updated.push(".claude/CLAUDE.md (@imports would be added)");
+    }
+    else {
+        const claudeUpdated = syncClaudeMdImports(process.cwd());
+        if (claudeUpdated)
+            updated.push(".claude/CLAUDE.md (@imports updated)");
     }
-    const claudeUpdated = syncClaudeMdImports(process.cwd());
-    if (claudeUpdated)
-        updated.push(".claude/CLAUDE.md (@imports updated)");
     if (updated.length === 0) {
         console.log("  Already up to date — no changes.\n");
         return;
     }
-    console.log(`  Updated ${updated.length} file${updated.length === 1 ? "" : "s"}:\n`);
+    const verb = checkOnly ? "Would update" : "Updated";
+    console.log(`  ${verb} ${updated.length} file${updated.length === 1 ? "" : "s"}:\n`);
     for (const f of updated)
         console.log(`    ${f}`);
     if (updated.includes("platform-capabilities.md")) {
@@ -2395,17 +2420,20 @@ else if (command === "analysis") {
 }
 else if (command === "platform") {
     if (subcommand === "update") {
-        platformUpdate();
+        const checkOnly = process.argv.includes("--check");
+        platformUpdate(checkOnly);
     }
     else {
         console.log(`
   overscore platform — Manage platform rules and capabilities
 
   Commands:
-    update    Sync latest platform rules, skills, and capabilities from Hub
+    update          Sync latest platform rules, skills, and capabilities from Hub
+    update --check  Show what would change without writing anything
 
-  Example:
+  Examples:
     npx @overscore/cli platform update
+    npx @overscore/cli platform update --check
 `);
     }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@overscore/cli",
-  "version": "0.11.10",
+  "version": "0.11.12",
   "description": "CLI for deploying Overscore dashboards and publishing analyses",
   "bin": {
     "overscore": "dist/index.js"