feat(cli): complete phase 2 canonical distribution and hardening

Author: gpolanco

packages/cli/src/commands/compile.ts

@@ -15,6 +15,7 @@ import { copilotBridge } from '../bridges/copilot.js';
 import { mergeMarkedContent, removeMarkedBlock } from '../core/markers.js';
 import { cleanStaleFiles } from '../core/scope-filename.js';
 import { detectLegacyFiles, migrateLegacyFiles } from '../core/cleanup.js';
+import { buildCanonicalOutputs, writeCanonical } from '../core/canonical.js';
 import { fileExists } from '../utils/fs.js';
 import * as ui from '../utils/ui.js';
 import { ICONS } from '../utils/ui.js';
@@ -45,6 +46,8 @@ export interface MigrationResult {
 export interface CompileResult {
   results: BridgeResult[];
   activeRuleCount: number;
+  canonicalFileCount: number;
+  canonicalError?: string;
   assetPaths: string[];
   elapsedMs: number;
   staleResults: StaleFileResult[];
@@ -235,6 +238,35 @@ export async function executePipeline(options: PipelineOptions): Promise<Compile
     }
   }
 
+  // Canonical output intentionally always runs, even when --tool filters bridges.
+  // This keeps `.agents/rules/devw` as the source-of-truth for doctor checks and distribution.
+  const canonicalOutputs = buildCanonicalOutputs(rules);
+  let canonicalPaths: string[] = [];
+  let canonicalError: string | undefined;
+  if (write) {
+    try {
+      canonicalPaths = await writeCanonical(cwd, canonicalOutputs);
+      for (const relativePath of canonicalPaths) {
+        results.push({ bridgeId: 'canonical', outputPath: relativePath, success: true });
+      }
+    } catch (err) {
+      canonicalError = err instanceof Error ? err.message : String(err);
+      const errorPaths = [...canonicalOutputs.keys()];
+      if (errorPaths.length > 0) {
+        for (const relativePath of errorPaths) {
+          results.push({ bridgeId: 'canonical', outputPath: relativePath, success: false, error: canonicalError });
+        }
+      } else {
+        results.push({ bridgeId: 'canonical', outputPath: '.agents/rules/devw', success: false, error: canonicalError });
+      }
+    }
+  } else {
+    for (const [relativePath, content] of canonicalOutputs) {
+      canonicalPaths.push(relativePath);
+      results.push({ bridgeId: 'canonical', outputPath: relativePath, success: true, content });
+    }
+  }
+
   let assetPaths: string[] = [];
   if (write) {
     const hash = computeRulesHash(activeRules);
@@ -245,7 +277,16 @@ export async function executePipeline(options: PipelineOptions): Promise<Compile
   }
 
   const elapsedMs = performance.now() - startTime;
-  return { results, activeRuleCount: activeRules.length, assetPaths, elapsedMs, staleResults, migration };
+  return {
+    results,
+    activeRuleCount: activeRules.length,
+    canonicalFileCount: canonicalPaths.length,
+    canonicalError,
+    assetPaths,
+    elapsedMs,
+    staleResults,
+    migration,
+  };
 }
 
 export async function runCompile(options: CompileOptions): Promise<void> {
@@ -286,12 +327,23 @@ export async function runCompile(options: CompileOptions): Promise<void> {
       // Summary of what would be generated
       const fileCount = result.results.filter((r) => r.success).length;
       ui.newline();
-      ui.info(`Would generate ${String(fileCount)} file${fileCount !== 1 ? 's' : ''} from ${String(result.activeRuleCount)} rules`);
+      ui.info(
+        `Would generate ${String(fileCount)} file${fileCount !== 1 ? 's' : ''} (${String(result.canonicalFileCount)} canonical) from ${String(result.activeRuleCount)} rules`,
+      );
       return;
     }
 
     const result = await executePipeline({ cwd, tool: options.tool });
 
+    if (options.tool) {
+      ui.info('Note: canonical output is always refreshed in .agents/rules/devw');
+    }
+
+    if (result.canonicalError) {
+      ui.warn(`Canonical write failed: ${result.canonicalError}`);
+      ui.warn('Tool-specific outputs were still written');
+    }
+
     // Show migration messages if any
     if (result.migration.actions.length > 0) {
       ui.newline();
@@ -306,6 +358,7 @@ export async function runCompile(options: CompileOptions): Promise<void> {
 
     ui.newline();
     ui.success(`Compiled ${String(result.activeRuleCount)} rules ${ICONS.arrow} ${String(allPaths.length)} file${allPaths.length !== 1 ? 's' : ''} ${ui.timing(result.elapsedMs)}`);
+    ui.info(`Canonical files: ${String(result.canonicalFileCount)}`);
     ui.newline();
 
     if (options.verbose) {