@sitecoreai-labs/sitecoreai-cli 0.1.2 → 0.2.1

package/dist/mcp/tools/brief.js

@@ -186,21 +186,21 @@ const registerBriefTools = (registry) => {
                 .describe("Which Brief resource the verb targets. 'brief-type' supports create/update/delete; 'brief' supports set-status/delete; 'comment' supports create."),
             verb: zod_1.z
                 .enum(["create", "update", "delete", "set-status"])
-                .describe("Mutation verb. brief-type: create (POST), update (PUT-replace), delete (irreversible). brief: set-status, delete. comment: create."),
+                .describe("Mutation verb. brief-type: create (POST), update (PUT-replace), delete (irreversible). brief: create (POST), update (partial PUT), set-status, delete. comment: create."),
             briefTypeId: zod_1.z
                 .string()
                 .uuid()
                 .optional()
-                .describe("Brief type UUID. Required for brief-type verb='update' and verb='delete'."),
+                .describe("Brief type UUID. Required for brief-type verb='update' and verb='delete', and for brief verb='create' (the type to build the brief against)."),
             briefId: zod_1.z
                 .string()
                 .uuid()
                 .optional()
-                .describe("Brief UUID. Required for resource='brief' verb='set-status'."),
+                .describe("Brief UUID. Required for resource='brief' verbs 'update', 'set-status', and 'delete'."),
             status: zod_1.z
                 .enum(["Draft", "InReview", "Approved", "Canceled", "Archived"])
                 .optional()
-                .describe("Target brief status for verb='set-status'. Wire form — 'InReview' is the 'In Review' UI label. A brief must leave 'Draft' before it can be linked to a campaign."),
+                .describe("Target brief status. Required for verb='set-status'; optional on brief 'create'/'update' (defaults to server 'Draft' on create). Wire form — 'InReview' is the 'In Review' UI label. A brief must leave 'Draft' before it can be linked to a campaign."),
             commentText: zod_1.z
                 .string()
                 .optional()
@@ -221,7 +221,19 @@ const registerBriefTools = (registry) => {
                     .describe("Field definitions (RichText | DateTime | Timeline | Budget). Use brief_inspect verb='type' on an existing type for a worked example."),
             })
                 .optional()
-                .describe("Full BriefType body for verb='create' or verb='update'. Required on writes; ignored on delete."),
+                .describe("Full BriefType body for resource='brief-type' verb='create' or verb='update'. Required on brief-type writes; ignored on delete."),
+            brief: zod_1.z
+                .object({
+                name: zod_1.z.string().min(1).describe("Brief display name."),
+                locale: zod_1.z.string().optional().describe("BCP-47-ish locale, e.g. 'en-us'."),
+                fields: zod_1.z
+                    .record(zod_1.z.string(), zod_1.z.unknown())
+                    .optional()
+                    .describe("Field values keyed by BriefField.name; the per-field shape follows the brief type's field definitions."),
+                isTemplate: zod_1.z.boolean().optional().describe("Whether the brief is a template."),
+            })
+                .optional()
+                .describe("Brief body. Required for resource='brief' verb='create' (with `briefTypeId`); optional for verb='update' (any subset of name/locale/fields/isTemplate plus the top-level `status`)."),
             ...common_1.environmentBindingShape,
             ...common_1.allowWriteShape,
             ...common_1.whatIfShape,
@@ -230,6 +242,71 @@ const registerBriefTools = (registry) => {
             const taskOpts = baseTaskOptions(context.configPath, input.environmentName ?? context.envName);
             const whatIf = input.whatIf;
             if (input.resource === "brief") {
+                if (input.verb === "create") {
+                    if (!input.brief) {
+                        throw (0, errors_1.createScaiError)("verb='create' on resource='brief' requires `brief`.", "INPUT_INVALID");
+                    }
+                    if (!input.briefTypeId) {
+                        throw (0, errors_1.createScaiError)("verb='create' on resource='brief' requires `briefTypeId`.", "INPUT_INVALID");
+                    }
+                    const createInput = {
+                        name: input.brief.name,
+                        briefTypeId: input.briefTypeId,
+                        ...(input.brief.locale !== undefined && { locale: input.brief.locale }),
+                        ...(input.brief.fields !== undefined && { fields: input.brief.fields }),
+                        ...(input.brief.isTemplate !== undefined && { isTemplate: input.brief.isTemplate }),
+                    };
+                    const result = await (0, tasks_1.runBriefCreate)({
+                        ...taskOpts,
+                        input: createInput,
+                        whatIf,
+                    });
+                    const isPlan = "plan" in result;
+                    return {
+                        content: [
+                            {
+                                type: "text",
+                                text: isPlan
+                                    ? `Plan: create brief '${input.brief.name}'.`
+                                    : `Created brief '${input.brief.name}'.`,
+                            },
+                        ],
+                        structuredContent: { resource: input.resource, verb: input.verb, result },
+                    };
+                }
+                if (input.verb === "update") {
+                    if (!input.briefId) {
+                        throw (0, errors_1.createScaiError)("verb='update' on resource='brief' requires `briefId`.", "INPUT_INVALID");
+                    }
+                    const patch = {
+                        ...(input.brief?.name !== undefined && { name: input.brief.name }),
+                        ...(input.brief?.locale !== undefined && { locale: input.brief.locale }),
+                        ...(input.brief?.fields !== undefined && { fields: input.brief.fields }),
+                        ...(input.brief?.isTemplate !== undefined && { isTemplate: input.brief.isTemplate }),
+                        ...(input.status !== undefined && { status: input.status }),
+                    };
+                    if (Object.keys(patch).length === 0) {
+                        throw (0, errors_1.createScaiError)("verb='update' on resource='brief' requires at least one field in `brief` or a top-level `status`.", "INPUT_INVALID");
+                    }
+                    const result = await (0, tasks_1.runBriefUpdate)({
+                        ...taskOpts,
+                        briefId: input.briefId,
+                        patch,
+                        whatIf,
+                    });
+                    const isPlan = "plan" in result;
+                    return {
+                        content: [
+                            {
+                                type: "text",
+                                text: isPlan
+                                    ? `Plan: update brief ${input.briefId} (${Object.keys(patch).join(", ")}).`
+                                    : `Updated brief ${input.briefId}.`,
+                            },
+                        ],
+                        structuredContent: { resource: input.resource, verb: input.verb, result },
+                    };
+                }
                 if (input.verb === "set-status") {
                     if (!input.briefId) {
                         throw (0, errors_1.createScaiError)("verb='set-status' requires `briefId`.", "INPUT_INVALID");
@@ -277,7 +354,7 @@ const registerBriefTools = (registry) => {
                         structuredContent: { resource: input.resource, verb: input.verb, result },
                     };
                 }
-                throw (0, errors_1.createScaiError)("resource='brief' supports verb='set-status' and verb='delete'.", "INPUT_INVALID");
+                throw (0, errors_1.createScaiError)("resource='brief' supports verbs 'create', 'update', 'set-status', and 'delete'.", "INPUT_INVALID");
             }
             if (input.resource === "comment") {
                 if (input.verb !== "create") {

package/dist/recipe/compile/design-parameters-template.js

@@ -6,6 +6,7 @@ const operations_1 = require("../ir/operations");
 const policy_1 = require("../runtime/policy");
 const sitecore_templates_1 = require("../ir/sitecore-templates");
 const recipe_1 = require("../schema/recipe");
+const component_section_1 = require("./component-section");
 const shared_1 = require("./shared");
 /**
  * Compile a standalone `DesignParametersTemplateRecipe` to an Operation IR.
@@ -22,9 +23,10 @@ function compileDesignParametersTemplateRecipe(input, context, emittedFolders =
     const policy = (0, policy_1.defaultPolicyForRecipe)(recipe.kind);
     const icon = recipe.icon ?? sitecore_templates_1.DEFAULT_ICON;
     const site = (0, shared_1.siteOf)(context);
-    (0, shared_1.ensureSectionFolder)(operations, context, recipe.section, emittedFolders);
-    const bucketRefKey = (0, shared_1.ensurePresentationDesignParametersBucket)(operations, context, recipe.section, emittedFolders);
-    const parentPath = (0, shared_1.resolvePresentationDesignParametersBucketPath)(context, recipe.section);
+    const sectionName = (0, component_section_1.resolveSectionRecipe)(recipe.handle, recipe.section.handle, context.sectionsByHandle).name;
+    (0, shared_1.ensureSectionFolder)(operations, context, sectionName, emittedFolders);
+    const bucketRefKey = (0, shared_1.ensurePresentationDesignParametersBucket)(operations, context, sectionName, emittedFolders);
+    const parentPath = (0, shared_1.resolvePresentationDesignParametersBucketPath)(context, sectionName);
     // The standalone parameters template lands at the same identity
     // (designParametersTemplateId) as inline-hoisted ones — keeps re-pushes
     // idempotent if a recipe migrates from inline to standalone.

package/dist/recipe/compile/enumeration.js

@@ -118,17 +118,12 @@ function compileEnumerationRecipe(input, context, emittedFolders = new Set()) {
         kind: "ref-path",
         value: context.enumerationsRoot,
     };
-    const folder = recipe.location?.folder;
-    if (folder) {
-        const folderSegments = folder
-            .split("/")
-            .map((s) => s.trim())
-            .filter(Boolean);
-        if (folderSegments.length === 0) {
-            throw (0, errors_1.createScaiError)(`Recipe '${recipe.handle}' declares location.folder that is empty after trimming.`, "INPUT_INVALID", {
-                hint: "Use a non-empty folder string like 'Theme' or 'Components/Card'.",
-            });
-        }
+    // `recipe.location.folder` is normalized to `string[]` by the schema
+    // (`FolderPath` in schema/recipe.ts) — both `"Theme/Color"` and
+    // `["Theme", "Color"]` input shapes land here as `["Theme", "Color"]`,
+    // already trimmed and non-empty.
+    const folderSegments = recipe.location?.folder;
+    if (folderSegments && folderSegments.length > 0) {
         const cumulativeSegments = [];
         for (const segment of folderSegments) {
             cumulativeSegments.push(segment);

package/dist/recipe/compile/shared.js

@@ -50,9 +50,12 @@ const resolveEnumFolderPath = (context, enumHandle, consumerHandle) => {
             hint: "Add an `EnumerationRecipe` (kind: 'enumeration') with the matching handle to the recipe set, or change the field's `sitecore.enumHandle` to point at an existing one.",
         });
     }
-    const folder = enumRecipe.location?.folder;
-    return folder
-        ? (0, exports.joinPath)((0, exports.joinPath)(context.enumerationsRoot, folder), enumRecipe.name)
+    // `location.folder` is a `string[]` of grouping segments after the
+    // schema's `FolderPath` normalisation — join with `/` here to build
+    // the cumulative path under enumerationsRoot.
+    const folderSegments = enumRecipe.location?.folder;
+    return folderSegments && folderSegments.length > 0
+        ? (0, exports.joinPath)((0, exports.joinPath)(context.enumerationsRoot, folderSegments.join("/")), enumRecipe.name)
         : (0, exports.joinPath)(context.enumerationsRoot, enumRecipe.name);
 };
 exports.resolveEnumFolderPath = resolveEnumFolderPath;
@@ -799,19 +802,16 @@ function encodeStandardValueDefault(raw, type) {
 function resolveFieldSource(field, type, site, recipeHandle, context) {
     const sc = field.sitecore;
     if (sc) {
-        const fields = {
-            sourceTypes: sc.sourceTypes,
-            sourceQuery: sc.sourceQuery,
-            sourceScope: sc.sourceScope,
-            sourceRaw: sc.sourceRaw,
-        };
+        const fields = (0, source_fields_1.augmentSourceToFields)(sc.source);
         if ((0, source_fields_1.sourceFieldsNeedHandleResolution)(fields)) {
+            // `types` is non-empty here because `sourceFieldsNeedHandleResolution`
+            // returned true; the cast is to satisfy the IR's `.min(1)` constraint.
             return {
                 kind: "ref-source-fields",
                 site,
-                sourceTypes: sc.sourceTypes,
-                sourceQuery: sc.sourceQuery,
-                sourceScope: sc.sourceScope,
+                sourceTypes: fields.sourceTypes,
+                sourceQuery: fields.sourceQuery,
+                sourceScope: fields.sourceScope,
             };
         }
         const rendered = (0, source_fields_1.renderSourceFields)(fields, () => {

package/dist/recipe/compile.js

@@ -720,10 +720,10 @@ const buildPlaceholderSettingsAggregate = (recipes, context, site) => {
     // parent ref + path the leaf placeholder item lands under.
     const emittedFolders = new Set();
     const resolveFolderParent = (folder) => {
-        const segments = (folder ?? "")
-            .split("/")
-            .map((segment) => segment.trim())
-            .filter(Boolean);
+        // `folder` is normalised to `string[]` at the schema boundary
+        // (`FolderPath` in schema/recipe.ts) — both array and legacy
+        // slash-string inputs land here as a clean segment list.
+        const segments = folder ?? [];
         let parentPath = root;
         let parentRef = { kind: "ref-path", value: root };
         let cumulative = "";

package/dist/recipe/io.d.ts

@@ -4,11 +4,16 @@ import type { Plan } from "./runtime/plan";
 /**
  * Load a recipe file. Supports both shapes:
  *
- *   - `.recipe.ts` — TypeScript source compiled at runtime via `tsx/cjs/api`.
- *     Recipes export the recipe object as either the default export or a
- *     named export (the first one found wins).
+ *   - `.recipe.ts` (or `.tsx`/`.mts`/`.cts`) — TypeScript source compiled
+ *     and executed in the recipe sandbox. Recipes export the recipe
+ *     object as either the default export or a named export (the first
+ *     one found wins).
  *   - `.recipe.json` — pre-serialized recipe JSON.
  *
+ * The TypeScript path is shared with the schema-aware loader at
+ * `@/sync/typescript-recipe` so brand-kit, agent, campaign, and brief
+ * recipes can also be authored as `.recipe.ts`.
+ *
  * Validates against `RecipeSchema` (the `Recipe = ComponentTemplateRecipe |
  * ContentTemplateRecipe` discriminated union) so both kinds parse uniformly.
  */

package/dist/recipe/io.js

@@ -7,9 +7,9 @@ exports.defaultPlanPath = exports.defaultIrPath = exports.writePlan = exports.lo
 const node_fs_1 = require("node:fs");
 const node_path_1 = __importDefault(require("node:path"));
 const errors_1 = require("../shared/errors");
+const typescript_recipe_1 = require("../sync/typescript-recipe");
 const recipe_1 = require("./schema/recipe");
 const operations_1 = require("./ir/operations");
-const load_1 = require("./sandbox/load");
 /**
  * Recipe + IR + Plan I/O.
  *
@@ -46,18 +46,22 @@ const writeJson = async (filePath, value) => {
 /**
  * Load a recipe file. Supports both shapes:
  *
- *   - `.recipe.ts` — TypeScript source compiled at runtime via `tsx/cjs/api`.
- *     Recipes export the recipe object as either the default export or a
- *     named export (the first one found wins).
+ *   - `.recipe.ts` (or `.tsx`/`.mts`/`.cts`) — TypeScript source compiled
+ *     and executed in the recipe sandbox. Recipes export the recipe
+ *     object as either the default export or a named export (the first
+ *     one found wins).
  *   - `.recipe.json` — pre-serialized recipe JSON.
  *
+ * The TypeScript path is shared with the schema-aware loader at
+ * `@/sync/typescript-recipe` so brand-kit, agent, campaign, and brief
+ * recipes can also be authored as `.recipe.ts`.
+ *
  * Validates against `RecipeSchema` (the `Recipe = ComponentTemplateRecipe |
  * ContentTemplateRecipe` discriminated union) so both kinds parse uniformly.
  */
 const loadRecipe = async (filePath) => {
-    const ext = node_path_1.default.extname(filePath).toLowerCase();
-    const raw = ext === ".ts" || ext === ".tsx" || ext === ".mts" || ext === ".cts"
-        ? await loadRecipeFromTypeScript(filePath)
+    const raw = (0, typescript_recipe_1.isTypeScriptRecipePath)(filePath)
+        ? await (0, typescript_recipe_1.loadTypeScriptRecipe)(filePath)
         : await readJson(filePath);
     const result = recipe_1.RecipeSchema.safeParse(raw);
     if (!result.success) {
@@ -69,80 +73,6 @@ const loadRecipe = async (filePath) => {
     return result.data;
 };
 exports.loadRecipe = loadRecipe;
-/**
- * Lazily install tsx's CommonJS require hook *once* per process. Earlier
- * versions of this file called `register()` and the matching `unregister`
- * cleanup on every `loadRecipe` call — fine for one-shot CLI runs but
- * wasteful in long-running surfaces like `scai cli shell` where N recipe
- * loads = N register/unregister cycles. Once installed, the hook stays
- * for the rest of the process lifetime; recipe changes mid-shell-session
- * therefore require an exit (the per-recipe `require.cache` clear below
- * handles re-imports of the *same* file path within the same process).
- */
-let tsxRegistered = false;
-const ensureTsxRegistered = () => {
-    if (tsxRegistered) {
-        return;
-    }
-    /* eslint-disable @typescript-eslint/no-require-imports */
-    const tsxApi = require("tsx/cjs/api");
-    /* eslint-enable @typescript-eslint/no-require-imports */
-    tsxApi.register();
-    tsxRegistered = true;
-};
-let sandboxOptOutWarned = false;
-const warnSandboxDisabled = () => {
-    if (sandboxOptOutWarned) {
-        return;
-    }
-    sandboxOptOutWarned = true;
-    process.stderr.write("scai: SITECOREAI_RECIPE_SANDBOX=0 — loading .recipe.ts in-process; " +
-        "a hostile recipe runs with scai's full privileges.\n");
-};
-/**
- * Load a `.recipe.ts` and return its exported recipe object. By default the
- * file is loaded in a confined child process (see docs/recipe-sandbox.md) so
- * a hostile recipe a weaponized config could point at cannot run with scai's
- * privileges. `SITECOREAI_RECIPE_SANDBOX=0` forces the legacy in-process load.
- */
-const loadRecipeFromTypeScript = async (filePath) => {
-    if ((0, load_1.isRecipeSandboxEnabled)()) {
-        return (0, load_1.loadRecipeInSandbox)(filePath);
-    }
-    warnSandboxDisabled();
-    return loadRecipeInProcess(filePath);
-};
-const loadRecipeInProcess = async (filePath) => {
-    const absolute = node_path_1.default.resolve(filePath);
-    try {
-        ensureTsxRegistered();
-        if (require.cache[absolute]) {
-            delete require.cache[absolute];
-        }
-        /* eslint-disable @typescript-eslint/no-require-imports */
-        const mod = require(absolute);
-        /* eslint-enable @typescript-eslint/no-require-imports */
-        if (mod.default !== undefined) {
-            return mod.default;
-        }
-        // Fall back to the first non-default exported value.
-        for (const key of Object.keys(mod)) {
-            if (key !== "default" && mod[key] !== undefined) {
-                return mod[key];
-            }
-        }
-        throw (0, errors_1.createScaiError)(`Recipe at ${filePath} has no exports.`, "INPUT_INVALID", {
-            hint: "Add `export default <recipe>` (or any named export) to the file.",
-        });
-    }
-    catch (error) {
-        // Don't double-wrap our own CliErrors (e.g. "Recipe at <path> has no exports.").
-        if (error instanceof errors_1.ScaiError) {
-            throw error;
-        }
-        throw (0, errors_1.createScaiError)(`Failed to load recipe TypeScript file ${filePath}: ${error instanceof Error ? error.message : String(error)}`, "INPUT_INVALID", { hint: "Confirm the file compiles standalone (try `pnpm exec tsx <file>`)." });
-    }
-};
 const writeIr = async (filePath, ir) => {
     await writeJson(filePath, ir);
 };

package/dist/recipe/items/read-current.js

@@ -237,11 +237,11 @@ const shapeFromSitecoreType = (type) => {
  * `sitecore.type`), the section it lives under (`sitecore.section`),
  * `sitecore.sortOrder`, and the storage axis (`sitecore.storage`, recovered
  * from the field's `Shared` / `Unversioned` flags). The `Source` value is
- * preserved verbatim via `sitecore.sourceRaw` — the structured
- * `sourceTypes`/`sourceQuery`/`sourceScope` decomposition is intentionally
- * NOT reverse-engineered (it would require parsing the URL-encoded Source
- * and resolving GUIDs back to handles); `sourceRaw` round-trips to the
- * identical wire string.
+ * preserved verbatim via `sitecore.source = { kind: "raw", value }` —
+ * the structured `filter` decomposition (`types`/`query`/`scope`) is
+ * intentionally NOT reverse-engineered (it would require parsing the
+ * URL-encoded Source and resolving GUIDs back to recipe handles);
+ * `kind: "raw"` round-trips to the identical wire string.
  *
  * LOSSY / omitted: `required`, `hint`, `default`, `enumHandle`, and the
  * abstract `multiple` flag are not recoverable from a field item alone and
@@ -259,8 +259,9 @@ const fieldFromItem = (fieldItem, sectionName) => {
         augment.type = sitecoreType;
     const source = fieldValue(fieldItem, sitecore_templates_1.TEMPLATE_FIELD_FIELDS.SOURCE, "Source");
     if (source !== undefined && source !== "") {
-        // Verbatim round-trip: sourceRaw re-emits the identical Source string.
-        augment.sourceRaw = source;
+        // Verbatim round-trip: `source: { kind: "raw", value }` re-emits
+        // the identical Source string at compile time.
+        augment.source = { kind: "raw", value: source };
     }
     // Field storage axis — `Shared` / `Unversioned` are shared flags on the
     // field item. `versioned` is the Sitecore default; omit it rather than
@@ -473,7 +474,7 @@ const componentSectionFromItem = (folderItem) => {
  * schema requires `values.min(1)`) — such a container is skipped by the
  * orchestrator with no error.
  */
-const enumerationFromItem = async (containerItem, folder, client) => {
+const enumerationFromItem = async (containerItem, folderSegments, client) => {
     const valueItems = (await client.getChildren({ itemId: containerItem.itemId }))
         .filter((child) => child.name !== "__Standard Values")
         .sort(byTreeOrder);
@@ -507,8 +508,9 @@ const enumerationFromItem = async (containerItem, folder, client) => {
     }
     if (description !== undefined && description !== "")
         recipe.description = description;
-    if (folder)
-        recipe.location = { scope: "site", folder };
+    if (folderSegments.length > 0) {
+        recipe.location = { scope: "site", folder: folderSegments };
+    }
     // Only carry `default` when it names a real value — the compiler rejects
     // an out-of-range default, and a stale container `Value` is not intent.
     if (defaultValue !== undefined &&
@@ -666,7 +668,11 @@ const walkEnumerationsTree = async (rootPath, client) => {
      * a container. `Enumerations Folder` items are recursed; everything else
      * with ≥1 child is treated as a container.
      */
-    const walk = async (parent, folderPath) => {
+    // `folderSegments` carries the grouping-folder path as `string[]` —
+    // matches the canonical array shape on `EnumerationRecipe.location.folder`
+    // so the reverse-projected recipe emits the same wire shape authors hand
+    // to scai (no slash-joined fallback).
+    const walk = async (parent, folderSegments) => {
         const children = (await client.getChildren({ itemId: parent.itemId }))
             .filter((c) => c.name !== "__Standard Values")
             .sort(byTreeOrder);
@@ -690,17 +696,16 @@ const walkEnumerationsTree = async (rootPath, client) => {
                 }
             }
             if (groupsContainers) {
-                const nextFolder = folderPath ? `${folderPath}/${child.name}` : child.name;
-                await walk(child, nextFolder);
+                await walk(child, [...folderSegments, child.name]);
             }
             else {
-                const recipe = await enumerationFromItem(child, folderPath, client);
+                const recipe = await enumerationFromItem(child, folderSegments, client);
                 if (recipe)
                     recipes.push(recipe);
             }
         }
     };
-    await walk(root, undefined);
+    await walk(root, []);
     return recipes;
 };
 /**
@@ -1022,7 +1027,7 @@ const pageFromItem = (item, templateHandle, guidIndex) => {
  * `PlaceholderRecipe` REQUIRES a non-empty `key`, and a key-less
  * Placeholder Settings item is not reverse-projectable.
  */
-const placeholderFromItem = (item, folder, guidIndex) => {
+const placeholderFromItem = (item, folderSegments, guidIndex) => {
     const key = fieldValue(item, sitecore_templates_1.PLACEHOLDER_FIELDS.PLACEHOLDER_KEY, "Placeholder Key");
     if (key === undefined || key.trim() === "") {
         // No Placeholder Key — schema requires `key.min(1)`. Skip.
@@ -1056,8 +1061,8 @@ const placeholderFromItem = (item, folder, guidIndex) => {
         recipe.description = description;
     if (icon !== undefined && icon !== "")
         recipe.icon = icon;
-    if (folder)
-        recipe.folder = folder;
+    if (folderSegments.length > 0)
+        recipe.folder = folderSegments;
     return recipe;
 };
 /**
@@ -1171,24 +1176,26 @@ const walkPlaceholderSettingsTree = async (rootPath, client, guidIndex) => {
     const root = rootPath ? await client.getItem({ path: rootPath }) : null;
     if (!root)
         return recipes;
-    const visit = async (parent, folderPath) => {
+    // `folderSegments` carries the grouping-folder path as `string[]` so
+    // reverse-projected placeholder recipes emit the canonical array
+    // shape that schemas/recipe.ts's `FolderPath` accepts.
+    const visit = async (parent, folderSegments) => {
         const children = (await client.getChildren({ itemId: parent.itemId }))
             .filter((c) => c.name !== "__Standard Values")
             .sort(byTreeOrder);
         for (const child of children) {
             if (conformsTo(child, sitecore_templates_1.PLACEHOLDER_TEMPLATE_ID)) {
-                const recipe = placeholderFromItem(child, folderPath, guidIndex);
+                const recipe = placeholderFromItem(child, folderSegments, guidIndex);
                 if (recipe)
                     recipes.push(recipe);
                 continue;
             }
             // Anything that isn't a Placeholder leaf is a grouping folder —
-            // descend, extending the cumulative folder path.
-            const nextFolder = folderPath ? `${folderPath}/${child.name}` : child.name;
-            await visit(child, nextFolder);
+            // descend, extending the cumulative segment list.
+            await visit(child, [...folderSegments, child.name]);
         }
     };
-    await visit(root, undefined);
+    await visit(root, []);
     return recipes;
 };
 /**