dlw-machine-setup 0.7.1 → 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,26 +3923,37 @@ var githubCopilotProfile = {
   handlers: {
     agent: {
       supported: true,
-      // TODO(phase-3): verify destination path against Copilot CLI docs.
-      destination: (name) => `.github/agents/${name}.md`
-      // TODO(phase-3): supply frontmatter rewriter that maps Claude's
-      // {name, description, disable-model-invocation, ...} to whatever
-      // Copilot expects ({name, description, category, version, ...}).
+      // 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,
-      // TODO(phase-3): verify destination path against Copilot CLI docs.
-      destination: (name) => `.github/skills/${name}.skill.md`
-      // TODO(phase-3): supply frontmatter rewriter.
+      destination: (name) => `.github/skills/${name}/SKILL.md`
     },
     hook: {
-      // TODO(phase-3): verify whether Copilot CLI has a hook primitive.
-      // If yes, flip to supported: true and fill in configFile/configKey/scriptDir.
-      // If no, leave as unsupported — assets of type 'hook' are silently skipped.
+      // 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}`
     }
   }
 };
@@ -3946,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 }
   }
 };
 
@@ -4015,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++;
   }
@@ -4042,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;
@@ -4227,14 +4254,8 @@ function assertBundleRootExists(ctx) {
 }
 
 // src/steps/resources/fetch-factory.ts
-function getFactoryAsset(agent) {
-  switch (agent) {
-    case "github-copilot":
-      return { assetName: "factory-copilot.tar.gz", rootFolder: "factory-copilot" };
-    default:
-      return { assetName: "factory.tar.gz", rootFolder: "factory" };
-  }
-}
+var FACTORY_ASSET_NAME = "factory.tar.gz";
+var FACTORY_ROOT_FOLDER = "factory";
 var fetch_factory_default = defineStep({
   name: "fetch-factory",
   label: "Installing Factory framework",
@@ -4253,7 +4274,6 @@ var fetch_factory_default = defineStep({
 });
 async function fetchFactory(token, repo, targetDir, agent) {
   const result = { success: false, filesInstalled: [] };
-  const { assetName, rootFolder } = getFactoryAsset(agent);
   let release;
   try {
     release = await fetchLatestRelease(token, repo);
@@ -4261,18 +4281,18 @@ async function fetchFactory(token, repo, targetDir, agent) {
     result.failureReason = err instanceof Error ? err.message : String(err);
     return result;
   }
-  const asset = release.assets.find((a) => a.name === assetName);
+  const asset = release.assets.find((a) => a.name === FACTORY_ASSET_NAME);
   if (!asset) {
-    result.failureReason = `${assetName} not found in release`;
+    result.failureReason = `${FACTORY_ASSET_NAME} not found in release`;
     return result;
   }
   let archive = null;
   try {
     archive = await downloadAndExtractAsset(token, asset, targetDir, ".temp-factory-download");
     const extractedEntries = (0, import_fs7.readdirSync)(archive.extractedRoot);
-    const extractedFolder = extractedEntries.find((e) => e.toLowerCase() === rootFolder.toLowerCase());
+    const extractedFolder = extractedEntries.find((e) => e.toLowerCase() === FACTORY_ROOT_FOLDER.toLowerCase());
     if (!extractedFolder) {
-      result.failureReason = `No ${rootFolder}/ folder in archive`;
+      result.failureReason = `No ${FACTORY_ROOT_FOLDER}/ folder in archive`;
       return result;
     }
     const extractedPath = (0, import_path7.join)(archive.extractedRoot, extractedFolder);

package/package.json

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