@mycadet/assistant 0.12.7 → 0.12.8

package/src/__tests__/{managed-store.test.ts → cloud-store.test.ts}

@@ -17,11 +17,11 @@ const TEST_DIR = process.env.CADET_WORKSPACE_DIR!;
 import { loadSkillCatalog } from "../config/skills.js";
 import {
   buildSkillMarkdown,
-  createManagedSkill,
-  deleteManagedSkill,
+  createCloudSkill,
+  deleteCloudSkill,
+  validateCloudSkillId,
   validateCompanionPath,
-  validateManagedSkillId,
-} from "../skills/managed-store.js";
+} from "../skills/cloud-store.js";
 
 beforeEach(() => {
   mkdirSync(join(TEST_DIR, "skills"), { recursive: true });
@@ -48,31 +48,31 @@ function readInstallMetaFile(skillId: string): TestInstallMeta {
   );
 }
 
-describe("validateManagedSkillId", () => {
+describe("validateCloudSkillId", () => {
   test("accepts valid slug IDs", () => {
-    expect(validateManagedSkillId("my-skill")).toBeNull();
-    expect(validateManagedSkillId("skill123")).toBeNull();
-    expect(validateManagedSkillId("my.skill")).toBeNull();
-    expect(validateManagedSkillId("my_skill")).toBeNull();
+    expect(validateCloudSkillId("my-skill")).toBeNull();
+    expect(validateCloudSkillId("skill123")).toBeNull();
+    expect(validateCloudSkillId("my.skill")).toBeNull();
+    expect(validateCloudSkillId("my_skill")).toBeNull();
   });
 
   test("rejects empty string", () => {
-    expect(validateManagedSkillId("")).not.toBeNull();
+    expect(validateCloudSkillId("")).not.toBeNull();
   });
 
   test("rejects traversal patterns", () => {
-    expect(validateManagedSkillId("../escape")).not.toBeNull();
-    expect(validateManagedSkillId("foo/bar")).not.toBeNull();
-    expect(validateManagedSkillId("foo\\bar")).not.toBeNull();
+    expect(validateCloudSkillId("../escape")).not.toBeNull();
+    expect(validateCloudSkillId("foo/bar")).not.toBeNull();
+    expect(validateCloudSkillId("foo\\bar")).not.toBeNull();
   });
 
   test("rejects uppercase", () => {
-    expect(validateManagedSkillId("MySkill")).not.toBeNull();
+    expect(validateCloudSkillId("MySkill")).not.toBeNull();
   });
 
   test("rejects IDs starting with special chars", () => {
-    expect(validateManagedSkillId(".hidden")).not.toBeNull();
-    expect(validateManagedSkillId("-dash")).not.toBeNull();
+    expect(validateCloudSkillId(".hidden")).not.toBeNull();
+    expect(validateCloudSkillId("-dash")).not.toBeNull();
   });
 });
 
@@ -134,7 +134,7 @@ describe("buildSkillMarkdown", () => {
 
   test("round-trips special characters through write and load", () => {
     // Write a skill with special chars in name/description
-    createManagedSkill({
+    createCloudSkill({
       id: "roundtrip-test",
       name: 'Say "hello" & back\\slash',
       description: "Line1\nLine2\r\nLine3",
@@ -152,7 +152,7 @@ describe("buildSkillMarkdown", () => {
   test("round-trips backslash-n literal (not a newline) correctly", () => {
     // A name containing a literal backslash followed by 'n' (not a newline)
     const nameWithBackslashN = "path\\name";
-    createManagedSkill({
+    createCloudSkill({
       id: "backslash-n-test",
       name: nameWithBackslashN,
       description: "test",
@@ -249,7 +249,7 @@ describe("buildSkillMarkdown", () => {
   });
 
   test("includes round-trips through write and catalog load", () => {
-    createManagedSkill({
+    createCloudSkill({
       id: "roundtrip-includes",
       name: "Roundtrip",
       description: "Test roundtrip",
@@ -283,9 +283,9 @@ describe("buildSkillMarkdown", () => {
   });
 });
 
-describe("createManagedSkill", () => {
+describe("createCloudSkill", () => {
   test("creates skill and writes to expected path", () => {
-    const result = createManagedSkill({
+    const result = createCloudSkill({
       id: "test-skill",
       name: "Test Skill",
       description: "A test skill",
@@ -302,14 +302,14 @@ describe("createManagedSkill", () => {
   });
 
   test("rejects duplicate unless overwrite=true", () => {
-    createManagedSkill({
+    createCloudSkill({
       id: "dupe",
       name: "Original",
       description: "First version",
       bodyMarkdown: "V1.",
     });
 
-    const result2 = createManagedSkill({
+    const result2 = createCloudSkill({
       id: "dupe",
       name: "Duplicate",
       description: "Second version",
@@ -318,7 +318,7 @@ describe("createManagedSkill", () => {
     expect(result2.created).toBe(false);
     expect(result2.error).toContain("already exists");
 
-    const result3 = createManagedSkill({
+    const result3 = createCloudSkill({
       id: "dupe",
       name: "Overwritten",
       description: "Third version",
@@ -331,7 +331,7 @@ describe("createManagedSkill", () => {
   });
 
   test("an overwrite keeps the frontmatter the call omits and replaces the body", () => {
-    createManagedSkill({
+    createCloudSkill({
       id: "kept",
       name: "Kept",
       description: "V1",
@@ -343,7 +343,7 @@ describe("createManagedSkill", () => {
       category: "productivity",
     });
 
-    const result = createManagedSkill({
+    const result = createCloudSkill({
       id: "kept",
       name: "Kept",
       description: "V2",
@@ -388,7 +388,7 @@ describe("createManagedSkill", () => {
       ].join("\n"),
     );
 
-    const result = createManagedSkill({
+    const result = createCloudSkill({
       id: "hand-edited",
       name: "Hand Edited",
       description: "V2",
@@ -416,7 +416,7 @@ describe("createManagedSkill", () => {
   });
 
   test("an overwrite replaces a field it passes and clears one passed empty", () => {
-    createManagedSkill({
+    createCloudSkill({
       id: "patched",
       name: "Patched",
       description: "V1",
@@ -428,7 +428,7 @@ describe("createManagedSkill", () => {
       category: "productivity",
     });
 
-    createManagedSkill({
+    createCloudSkill({
       id: "patched",
       name: "Patched",
       description: "V2",
@@ -452,7 +452,7 @@ describe("createManagedSkill", () => {
   });
 
   test("writes category to frontmatter and round-trips through catalog load", () => {
-    createManagedSkill({
+    createCloudSkill({
       id: "categorized-skill",
       name: "Categorized",
       description: "Has a category",
@@ -473,7 +473,7 @@ describe("createManagedSkill", () => {
   });
 
   test("leaves category unset when omitted", () => {
-    createManagedSkill({
+    createCloudSkill({
       id: "no-category",
       name: "No Category",
       description: "Uncategorized",
@@ -487,7 +487,7 @@ describe("createManagedSkill", () => {
   });
 
   test("rejects invalid IDs", () => {
-    const result = createManagedSkill({
+    const result = createCloudSkill({
       id: "../escape",
       name: "Bad",
       description: "Bad",
@@ -498,7 +498,7 @@ describe("createManagedSkill", () => {
   });
 
   test("rejects empty name", () => {
-    const result = createManagedSkill({
+    const result = createCloudSkill({
       id: "no-name",
       name: "",
       description: "Has desc",
@@ -509,7 +509,7 @@ describe("createManagedSkill", () => {
   });
 
   test("rejects whitespace-only name", () => {
-    const result = createManagedSkill({
+    const result = createCloudSkill({
       id: "blank-name",
       name: "   ",
       description: "Has desc",
@@ -520,7 +520,7 @@ describe("createManagedSkill", () => {
   });
 
   test("rejects empty description", () => {
-    const result = createManagedSkill({
+    const result = createCloudSkill({
       id: "no-desc",
       name: "Has Name",
       description: "",
@@ -531,7 +531,7 @@ describe("createManagedSkill", () => {
   });
 
   test("does not create SKILLS.md and is discovered through SKILL.md directory", () => {
-    createManagedSkill({
+    createCloudSkill({
       id: "discovered-skill",
       name: "Discovered",
       description: "Found by directory discovery",
@@ -637,9 +637,9 @@ describe("validateCompanionPath", () => {
   });
 });
 
-describe("createManagedSkill companion files", () => {
+describe("createCloudSkill companion files", () => {
   test("writes companion files under the skill dir and round-trips on disk", () => {
-    const result = createManagedSkill({
+    const result = createCloudSkill({
       id: "with-files",
       name: "With Files",
       description: "Has companion files",
@@ -667,7 +667,7 @@ describe("createManagedSkill companion files", () => {
   });
 
   test("rejects a companion path colliding with an existing directory on overwrite, leaving SKILL.md intact", () => {
-    const first = createManagedSkill({
+    const first = createCloudSkill({
       id: "dir-collide",
       name: "Dir Collide",
       description: "v1",
@@ -679,7 +679,7 @@ describe("createManagedSkill companion files", () => {
     const before = readFileSync(skillMd, "utf-8");
 
     // Overwrite with a companion path that names the existing references/ dir.
-    const second = createManagedSkill({
+    const second = createCloudSkill({
       id: "dir-collide",
       name: "Dir Collide",
       description: "v2",
@@ -695,7 +695,7 @@ describe("createManagedSkill companion files", () => {
   });
 
   test("rejects path traversal and writes nothing", () => {
-    const result = createManagedSkill({
+    const result = createCloudSkill({
       id: "traversal",
       name: "Traversal",
       description: "Bad companion path",
@@ -713,7 +713,7 @@ describe("createManagedSkill companion files", () => {
   });
 
   test("rejects absolute companion paths and writes nothing", () => {
-    const result = createManagedSkill({
+    const result = createCloudSkill({
       id: "abs-path",
       name: "Absolute",
       description: "Absolute companion path",
@@ -729,7 +729,7 @@ describe("createManagedSkill companion files", () => {
   });
 
   test("rejects a path resolving outside the skill dir and writes nothing", () => {
-    const result = createManagedSkill({
+    const result = createCloudSkill({
       id: "outside",
       name: "Outside",
       description: "Resolves outside",
@@ -752,14 +752,14 @@ describe("createManagedSkill companion files", () => {
   });
 
   test("does not write companion files when create errors on an existing skill", () => {
-    createManagedSkill({
+    createCloudSkill({
       id: "exists-files",
       name: "Exists",
       description: "Already here",
       bodyMarkdown: "Body.",
     });
 
-    const result = createManagedSkill({
+    const result = createCloudSkill({
       id: "exists-files",
       name: "Exists Again",
       description: "Should not write",
@@ -780,7 +780,7 @@ describe("createManagedSkill companion files", () => {
     // Planting a TOOLS.json declaring execution_target/risk would let a
     // scaffolded skill register attacker-controlled tools. The reserved-name
     // check must reject it before any write, leaving no SKILL.md or manifest.
-    const result = createManagedSkill({
+    const result = createCloudSkill({
       id: "tools-manifest",
       name: "Tools Manifest",
       description: "Tries to plant a tool manifest",
@@ -814,7 +814,7 @@ describe("createManagedSkill companion files", () => {
   });
 
   test("overwrite re-writes companion files", () => {
-    createManagedSkill({
+    createCloudSkill({
       id: "overwrite-files",
       name: "V1",
       description: "First",
@@ -831,7 +831,7 @@ describe("createManagedSkill companion files", () => {
     );
     expect(readFileSync(companionPath, "utf-8")).toBe("v1 notes");
 
-    const result = createManagedSkill({
+    const result = createCloudSkill({
       id: "overwrite-files",
       name: "V2",
       description: "Second",
@@ -845,12 +845,12 @@ describe("createManagedSkill companion files", () => {
   });
 });
 
-describe("createManagedSkill copy_from companion sources", () => {
+describe("createCloudSkill copy_from companion sources", () => {
   test("copies a workspace source file into the skill dir", () => {
     const sourcePath = join(TEST_DIR, "ran-script.py");
     writeFileSync(sourcePath, "print('proven')\n", "utf-8");
 
-    const result = createManagedSkill({
+    const result = createCloudSkill({
       id: "copy-source",
       name: "Copy Source",
       description: "Copies a proven script",
@@ -871,7 +871,7 @@ describe("createManagedSkill copy_from companion sources", () => {
     const sourcePath = join(TEST_DIR, "both.py");
     writeFileSync(sourcePath, "x", "utf-8");
 
-    const result = createManagedSkill({
+    const result = createCloudSkill({
       id: "copy-both",
       name: "Both",
       description: "Both content and copyFrom",
@@ -885,7 +885,7 @@ describe("createManagedSkill copy_from companion sources", () => {
   });
 
   test("rejects an entry setting neither content nor copyFrom", () => {
-    const result = createManagedSkill({
+    const result = createCloudSkill({
       id: "copy-neither",
       name: "Neither",
       description: "Neither content nor copyFrom",
@@ -898,7 +898,7 @@ describe("createManagedSkill copy_from companion sources", () => {
   });
 
   test("rejects a relative copyFrom source", () => {
-    const result = createManagedSkill({
+    const result = createCloudSkill({
       id: "copy-relative",
       name: "Relative",
       description: "Relative source",
@@ -911,7 +911,7 @@ describe("createManagedSkill copy_from companion sources", () => {
   });
 
   test("rejects a missing copyFrom source, writing nothing", () => {
-    const result = createManagedSkill({
+    const result = createCloudSkill({
       id: "copy-missing",
       name: "Missing",
       description: "Missing source",
@@ -925,7 +925,7 @@ describe("createManagedSkill copy_from companion sources", () => {
   });
 
   test("rejects a source outside the workspace and temp dir", () => {
-    const result = createManagedSkill({
+    const result = createCloudSkill({
       id: "copy-outside",
       name: "Outside",
       description: "Out-of-bounds source",
@@ -943,7 +943,7 @@ describe("createManagedSkill copy_from companion sources", () => {
     const linkPath = join(TEST_DIR, "sneaky-link");
     fs.symlinkSync("/etc/hosts", linkPath);
 
-    const result = createManagedSkill({
+    const result = createCloudSkill({
       id: "copy-symlink",
       name: "Symlink",
       description: "Symlink escape",
@@ -963,7 +963,7 @@ describe("createManagedSkill copy_from companion sources", () => {
     writeFileSync(sourcePath, "console.log('ok');\n", "utf-8");
 
     try {
-      const result = createManagedSkill({
+      const result = createCloudSkill({
         id: "copy-tmp",
         name: "Tmp Source",
         description: "Copies from literal /tmp",
@@ -987,7 +987,7 @@ describe("createManagedSkill copy_from companion sources", () => {
     const keyPath = join(TEST_DIR, "backup.key");
     writeFileSync(keyPath, "secret", "utf-8");
 
-    const result = createManagedSkill({
+    const result = createCloudSkill({
       id: "copy-denied",
       name: "Denied",
       description: "Denied basename",
@@ -1005,7 +1005,7 @@ describe("createManagedSkill copy_from companion sources", () => {
     const linkPath = join(TEST_DIR, "looks-fine.txt");
     fs.symlinkSync(keyPath, linkPath);
 
-    const result = createManagedSkill({
+    const result = createCloudSkill({
       id: "copy-denied-link",
       name: "Denied Link",
       description: "Symlink to denied basename",
@@ -1021,7 +1021,7 @@ describe("createManagedSkill copy_from companion sources", () => {
     const dirPath = join(TEST_DIR, "a-directory");
     mkdirSync(dirPath, { recursive: true });
 
-    const result = createManagedSkill({
+    const result = createCloudSkill({
       id: "copy-dir",
       name: "Dir",
       description: "Directory source",
@@ -1037,7 +1037,7 @@ describe("createManagedSkill copy_from companion sources", () => {
     const sourcePath = join(TEST_DIR, "manifest.json");
     writeFileSync(sourcePath, "{}", "utf-8");
 
-    const result = createManagedSkill({
+    const result = createCloudSkill({
       id: "copy-reserved",
       name: "Reserved",
       description: "Reserved destination",
@@ -1054,7 +1054,7 @@ describe("atomic write safety", () => {
   test("SKILL.md is not partially written on concurrent create", () => {
     // Verify that atomicWriteFile prevents corruption: after creation,
     // the file is always complete (starts with --- and ends with newline)
-    const result = createManagedSkill({
+    const result = createCloudSkill({
       id: "atomic-test",
       name: "Atomic",
       description: "Tests atomic write",
@@ -1069,14 +1069,14 @@ describe("atomic write safety", () => {
   });
 
   test("overwrite replaces file atomically (no leftover temp files)", () => {
-    createManagedSkill({
+    createCloudSkill({
       id: "atomic-overwrite",
       name: "V1",
       description: "First",
       bodyMarkdown: "Original.",
     });
 
-    createManagedSkill({
+    createCloudSkill({
       id: "atomic-overwrite",
       name: "V2",
       description: "Second",
@@ -1122,7 +1122,7 @@ describe("atomic write failure", () => {
 
     try {
       expect(() => {
-        createManagedSkill({
+        createCloudSkill({
           id: "fail-write",
           name: "Should Fail",
           description: "This should not be written",
@@ -1141,16 +1141,16 @@ describe("atomic write failure", () => {
   });
 });
 
-describe("deleteManagedSkill", () => {
+describe("deleteCloudSkill", () => {
   test("deletes existing skill", () => {
-    createManagedSkill({
+    createCloudSkill({
       id: "to-delete",
       name: "Delete Me",
       description: "Will be deleted",
       bodyMarkdown: "Gone soon.",
     });
 
-    const result = deleteManagedSkill("to-delete");
+    const result = deleteCloudSkill("to-delete");
     expect(result.deleted).toBe(true);
     expect(existsSync(join(TEST_DIR, "skills", "to-delete"))).toBe(false);
 
@@ -1159,19 +1159,19 @@ describe("deleteManagedSkill", () => {
   });
 
   test("returns error for non-existent skill", () => {
-    const result = deleteManagedSkill("ghost");
+    const result = deleteCloudSkill("ghost");
     expect(result.deleted).toBe(false);
     expect(result.error).toContain("not found");
   });
 
   test("rejects invalid IDs", () => {
-    const result = deleteManagedSkill("../bad");
+    const result = deleteCloudSkill("../bad");
     expect(result.deleted).toBe(false);
     expect(result.error).toContain("traversal");
   });
 
   test("delete leaves a stale SKILLS.md index unchanged", () => {
-    createManagedSkill({
+    createCloudSkill({
       id: "keep-index",
       name: "Keep Index",
       description: "Index stays",
@@ -1182,7 +1182,7 @@ describe("deleteManagedSkill", () => {
       "- keep-index\n- survivor\n",
     );
 
-    const result = deleteManagedSkill("keep-index");
+    const result = deleteCloudSkill("keep-index");
     expect(result.deleted).toBe(true);
 
     const indexContent = readFileSync(
@@ -1197,7 +1197,7 @@ describe("deleteManagedSkill", () => {
   });
 
   test("delete does not create or edit SKILLS.md", () => {
-    createManagedSkill({
+    createCloudSkill({
       id: "delete-no-index",
       name: "Delete No Index",
       description: "No index write",
@@ -1205,7 +1205,7 @@ describe("deleteManagedSkill", () => {
     });
 
     const skillsDir = join(TEST_DIR, "skills");
-    const result = deleteManagedSkill("delete-no-index");
+    const result = deleteCloudSkill("delete-no-index");
     expect(result.deleted).toBe(true);
     expect(existsSync(join(skillsDir, "delete-no-index"))).toBe(false);
     expect(existsSync(join(skillsDir, "SKILLS.md"))).toBe(false);
@@ -1213,8 +1213,8 @@ describe("deleteManagedSkill", () => {
 });
 
 describe("version metadata", () => {
-  test("createManagedSkill writes install-meta.json without version when version is omitted", () => {
-    createManagedSkill({
+  test("createCloudSkill writes install-meta.json without version when version is omitted", () => {
+    createCloudSkill({
       id: "no-version",
       name: "No Version",
       description: "Created without version",
@@ -1226,8 +1226,8 @@ describe("version metadata", () => {
     expect(meta.version).toBeUndefined();
   });
 
-  test("createManagedSkill writes install-meta.json when version is provided", () => {
-    createManagedSkill({
+  test("createCloudSkill writes install-meta.json when version is provided", () => {
+    createCloudSkill({
       id: "versioned",
       name: "Versioned",
       description: "Has a version",
@@ -1240,7 +1240,7 @@ describe("version metadata", () => {
   });
 
   test("install-meta.json contains valid JSON with origin, version, and installedAt", () => {
-    createManagedSkill({
+    createCloudSkill({
       id: "version-meta",
       name: "Meta",
       description: "Check metadata shape",
@@ -1267,7 +1267,7 @@ describe("version metadata", () => {
   });
 
   test("install-meta.json includes installedBy when contactId is provided", () => {
-    createManagedSkill({
+    createCloudSkill({
       id: "with-contact",
       name: "With Contact",
       description: "Has contactId",
@@ -1288,7 +1288,7 @@ describe("version metadata", () => {
   });
 
   test("install-meta.json forwards author when provided", () => {
-    createManagedSkill({
+    createCloudSkill({
       id: "with-author",
       name: "With Author",
       description: "Has author",
@@ -1300,7 +1300,7 @@ describe("version metadata", () => {
   });
 
   test("install-meta.json omits author when not provided", () => {
-    createManagedSkill({
+    createCloudSkill({
       id: "no-author",
       name: "No Author",
       description: "No author",
@@ -1311,7 +1311,7 @@ describe("version metadata", () => {
   });
 
   test("overwrite updates install-meta.json", () => {
-    createManagedSkill({
+    createCloudSkill({
       id: "update-version",
       name: "V1",
       description: "First",
@@ -1321,7 +1321,7 @@ describe("version metadata", () => {
 
     expect(readInstallMetaFile("update-version").version).toBe("v1:first");
 
-    createManagedSkill({
+    createCloudSkill({
       id: "update-version",
       name: "V2",
       description: "Second",
@@ -1333,7 +1333,7 @@ describe("version metadata", () => {
   });
 
   test("overwrite removes legacy version.json", () => {
-    createManagedSkill({
+    createCloudSkill({
       id: "legacy-version",
       name: "Legacy",
       description: "Has legacy metadata",
@@ -1350,7 +1350,7 @@ describe("version metadata", () => {
     writeFileSync(legacyMetaPath, '{"version":"legacy"}', "utf-8");
     expect(existsSync(legacyMetaPath)).toBe(true);
 
-    createManagedSkill({
+    createCloudSkill({
       id: "legacy-version",
       name: "Legacy Updated",
       description: "Has current metadata",
@@ -1363,38 +1363,38 @@ describe("version metadata", () => {
   });
 });
 
-describe("validateManagedSkillId edge cases", () => {
+describe("validateCloudSkillId edge cases", () => {
   test("rejects non-string input", () => {
     // @ts-expect-error testing runtime validation
-    expect(validateManagedSkillId(null)).not.toBeNull();
+    expect(validateCloudSkillId(null)).not.toBeNull();
     // @ts-expect-error testing runtime validation
-    expect(validateManagedSkillId(undefined)).not.toBeNull();
+    expect(validateCloudSkillId(undefined)).not.toBeNull();
     // @ts-expect-error testing runtime validation
-    expect(validateManagedSkillId(123)).not.toBeNull();
+    expect(validateCloudSkillId(123)).not.toBeNull();
   });
 
   test("rejects IDs with only dots", () => {
-    expect(validateManagedSkillId("...")).not.toBeNull();
+    expect(validateCloudSkillId("...")).not.toBeNull();
   });
 
   test("rejects single dot (hidden dir)", () => {
-    expect(validateManagedSkillId(".")).not.toBeNull();
+    expect(validateCloudSkillId(".")).not.toBeNull();
   });
 
   test("accepts single character ID", () => {
-    expect(validateManagedSkillId("a")).toBeNull();
-    expect(validateManagedSkillId("0")).toBeNull();
+    expect(validateCloudSkillId("a")).toBeNull();
+    expect(validateCloudSkillId("0")).toBeNull();
   });
 
   test("accepts ID with all allowed character types", () => {
-    expect(validateManagedSkillId("a1.b2-c3_d4")).toBeNull();
+    expect(validateCloudSkillId("a1.b2-c3_d4")).toBeNull();
   });
 });
 
 describe("YAML metadata round-trip", () => {
   test("all cadet fields round-trip through write and load", () => {
-    // Create a managed skill with every cadet metadata field populated
-    createManagedSkill({
+    // Create a cloud skill with every cadet metadata field populated
+    createCloudSkill({
       id: "yaml-roundtrip-all",
       name: "Full Metadata Skill",
       description: "Tests all cadet fields round-trip correctly",
@@ -1418,10 +1418,10 @@ describe("YAML metadata round-trip", () => {
   });
 
   test("activation hints and avoid-when round-trip into SkillSummary", () => {
-    // An assistant-authored (retrospective) skill written via createManagedSkill
+    // An assistant-authored (retrospective) skill written via createCloudSkill
     // carries activation hints so the memory seeder emits a "Use when:" clause
     // for it, just like bundled skills.
-    createManagedSkill({
+    createCloudSkill({
       id: "hints-roundtrip",
       name: "Hints Roundtrip",
       description: "Trigger phrases round-trip through write and load",