dlw-machine-setup 0.8.0 → 0.8.1

package/bin/installer.js

@@ -3888,7 +3888,9 @@ var claudeCodeProfile = {
   handlers: {
     agent: {
       supported: true,
-      destination: (name) => `.claude/agents/${name}.md`
+      // File agents land at `.claude/agents/<name>.md`; folder agents
+      // (persona bundles like jay/, monty/) land at `.claude/agents/<name>/`.
+      destination: (name, isDir) => `.claude/agents/${name}${isDir ? "" : ".md"}`
     },
     skill: {
       supported: true,
@@ -3903,6 +3905,13 @@ var claudeCodeProfile = {
     },
     "instructions-snippet": {
       supported: true
+    },
+    workspace: {
+      supported: true,
+      // Bundle workspace payload (workflow scripts, shared definitions,
+      // README) lands under `.claude/factory/`. Matches the path pattern
+      // used historically by Factory's v1 ops and gitignored entries.
+      destination: (name) => `.claude/factory/${name}`
     }
   }
 };
@@ -3914,19 +3923,37 @@ var githubCopilotProfile = {
   handlers: {
     agent: {
       supported: true,
-      destination: (name) => `.github/agents/${name}.agent.md`
+      // Copilot only registers agents that match `.agent.md` literally.
+      // File agents → `.github/agents/<name>.agent.md` (registered).
+      // Folder agents (persona bundles like jay/, monty/, shared/) →
+      // `.github/agents/<name>/` (placed alongside, but Copilot ignores
+      // them because they don't match `.agent.md`). That's intentional:
+      // these are Claude-specific personas, but shipping the files
+      // means a user who switches agents later still has them on disk.
+      destination: (name, isDir) => `.github/agents/${name}${isDir ? "" : ".agent.md"}`
     },
     skill: {
       supported: true,
       destination: (name) => `.github/skills/${name}/SKILL.md`
     },
     hook: {
-      // See header docblock for the three HookHandler extensions needed
-      // to flip this on. Until then, hook assets are skipped on Copilot.
+      // Hooks are Claude-only by policy (see plan doc). The bundle's
+      // hook-related ops are gated with `when: 'hooks-supported'`, so
+      // Copilot installs ship no hook events, no hook scripts, and no
+      // statusLine setting. Phase 4 (Copilot hook implementation) was
+      // canceled — this is the permanent final state.
       supported: false
     },
     "instructions-snippet": {
       supported: true
+    },
+    workspace: {
+      supported: true,
+      // Bundle workspace payload (workflow scripts, shared definitions,
+      // README) lands under `.github/factory/`. Keeps Factory's
+      // supporting material under the namespaced .github/ tree where
+      // Copilot already reads everything else from.
+      destination: (name) => `.github/factory/${name}`
     }
   }
 };
@@ -3939,7 +3966,8 @@ var cursorProfile = {
     agent: { supported: false },
     skill: { supported: false },
     hook: { supported: false },
-    "instructions-snippet": { supported: true }
+    "instructions-snippet": { supported: true },
+    workspace: { supported: false }
   }
 };
 
@@ -4008,6 +4036,7 @@ function runBundleV2(manifest, ctx) {
     runMergeJson({ op: "merge-json", file, patch }, manifest.name, ctx, result);
   }
   for (const op of manifest.ops ?? []) {
+    if (op.when === "hooks-supported" && !profile.handlers.hook.supported) continue;
     executeOp(op, manifest.name, ctx, result);
     result.opsExecuted++;
   }
@@ -4035,20 +4064,25 @@ function runAsset(asset, profile, hookPatches, ctx, result) {
         result.instructionsSnippet = asset.content;
       }
       return;
+    case "workspace":
+      runAssetCopy(asset, profile.handlers.workspace, ctx, result);
+      return;
   }
 }
 function runAssetCopy(asset, handler, ctx, result) {
   if (!handler.supported || !handler.destination) return;
   const source = resolveBundlePath(asset.source, ctx);
   if (!(0, import_fs6.existsSync)(source)) return;
-  const target = resolveProjectPath(handler.destination(asset.name), ctx);
-  if ((0, import_fs6.statSync)(source).isDirectory()) {
+  const isDirectory = (0, import_fs6.statSync)(source).isDirectory();
+  const targetRel = handler.destination(asset.name, isDirectory);
+  const target = resolveProjectPath(targetRel, ctx);
+  if (isDirectory) {
     copyDirectory(source, target);
   } else {
     (0, import_fs6.mkdirSync)((0, import_path6.dirname)(target), { recursive: true });
     (0, import_fs6.copyFileSync)(source, target);
   }
-  result.filesTouched.push(handler.destination(asset.name));
+  result.filesTouched.push(targetRel);
 }
 function accumulateHook(asset, handler, hookPatches, ctx, result) {
   if (!handler.supported) return;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dlw-machine-setup",
-  "version": "0.8.0",
+  "version": "0.8.1",
   "description": "One-shot installer for The Machine toolchain",
   "bin": {
     "dlw-machine-setup": "bin/installer.js"