storyblok 4.6.4 → 4.6.7

package/dist/index.mjs

@@ -856,7 +856,7 @@ const loginStrategy = {
       short: "Email"
     },
     {
-      name: "With Token (SSO)",
+      name: "With Token (Personal Access Token \u2013 works also for SSO accounts)",
       value: "login-with-token",
       short: "Token"
     }
@@ -915,8 +915,13 @@ program$i.command(commands.LOGIN).description("Login to the Storyblok CLI").opti
     try {
       const strategy = await select(loginStrategy);
       if (strategy === "login-with-token") {
+        konsola.info([
+          "\u{1F511} You can use a Personal Access Token to log in.",
+          "This works for all accounts, including SSO accounts.",
+          `Generate one in your Storyblok account settings: ${chalk.underline.blue("https://app.storyblok.com/#/me/account?tab=token")}`
+        ].join("\n"));
         const userToken = await password({
-          message: "Please enter your token:",
+          message: "Please enter your Personal Access Token:",
           validate: (value) => {
             return value.length > 0;
           }
@@ -1572,6 +1577,7 @@ componentsCommand.command("pull [componentName]").option("-f, --filename <filena
   }
 });
 
+const fieldTypesWithDependencies = ["bloks", "richtext"];
 function buildDependencyGraph(context) {
   const { spaceState } = context;
   const graph = { nodes: /* @__PURE__ */ new Map() };
@@ -1676,7 +1682,7 @@ function collectWhitelistDependencies(schema) {
   const componentNames = /* @__PURE__ */ new Set();
   const datasourceNames = /* @__PURE__ */ new Set();
   function traverseField(field) {
-    if (field.type === "bloks") {
+    if (fieldTypesWithDependencies.includes(field.type)) {
       if (field.component_group_whitelist && Array.isArray(field.component_group_whitelist)) {
         field.component_group_whitelist.forEach((uuid) => groupUuids.add(uuid));
       }
@@ -2024,7 +2030,7 @@ class ComponentNode extends GraphNode {
         return field.map(resolveField);
       }
       const resolvedField = { ...field };
-      if (resolvedField.type === "bloks") {
+      if (fieldTypesWithDependencies.includes(resolvedField.type)) {
         if (resolvedField.component_group_whitelist && Array.isArray(resolvedField.component_group_whitelist)) {
           resolvedField.component_group_whitelist = resolvedField.component_group_whitelist.map((groupUuid) => {
             const groupNodeId = `group:${groupUuid}`;
@@ -3074,7 +3080,8 @@ function applyMigrationToAllBlocks(content, migrationFunction, targetComponent)
     migratedContent = migrationFunction({ ...content });
     processed = true;
   }
-  for (const key of Object.keys(content)) {
+  const uniqueKeys = /* @__PURE__ */ new Set([...Object.keys(content), ...Object.keys(migratedContent || {})]);
+  for (const key of uniqueKeys) {
     if (migratedContent) {
       if (!(key in migratedContent)) {
         delete content[key];
@@ -3102,7 +3109,7 @@ async function saveRollbackData({
   path,
   story,
   migrationTimestamp,
-  migrationFile
+  migrationNames
 }) {
   const rollbackData = {
     storyId: story.id,
@@ -3110,7 +3117,9 @@ async function saveRollbackData({
     content: story.content
   };
   const rollbacksPath = resolvePath(path, `migrations/${space}/rollbacks`);
-  const rollbackFileName = `${migrationFile.replace(".js", "")}.${migrationTimestamp}.jsonl`;
+  const componentNames = migrationNames.map((n) => getComponentNameFromFilename(n));
+  const rollbackName = [...new Set(componentNames)].join("~");
+  const rollbackFileName = `${rollbackName}.${migrationTimestamp}.jsonl`;
   const rollbackFilePath = join(rollbacksPath, rollbackFileName);
   await appendToFile(
     rollbackFilePath,
@@ -3168,67 +3177,71 @@ class MigrationStream extends Transform {
       callback(error);
     }
   }
-  async processStory(story) {
-    if (!story.content) {
-      for (const migrationFile of this.options.migrationFiles) {
-        this.results.failed.push({
-          storyId: story.id,
-          migrationName: migrationFile.name,
-          error: new Error("Story content is missing")
-        });
-      }
-      return [];
+  async getOrLoadMigrationFunction(migrationFile) {
+    if (this.migrationFunctions.has(migrationFile.name)) {
+      return this.migrationFunctions.get(migrationFile.name);
     }
+    const migrationFunction = await getMigrationFunction(
+      migrationFile.name,
+      this.options.space,
+      this.options.path
+    );
+    this.migrationFunctions.set(migrationFile.name, migrationFunction);
+    return migrationFunction;
+  }
+  async processStory(story) {
     const relevantMigrations = this.options.componentName ? this.options.migrationFiles.filter((file) => {
       const targetComponent = getComponentNameFromFilename(file.name);
       return targetComponent.split(".")[0] === this.options.componentName;
     }) : this.options.migrationFiles;
+    if (!story.content) {
+      this.results.failed.push({
+        storyId: story.id,
+        migrationNames: relevantMigrations.map((m) => m.name),
+        error: new Error("Story content is missing")
+      });
+      return [];
+    }
     const successfulResults = [];
-    for (const migrationFile of relevantMigrations) {
-      const result = await this.applyMigrationToStory(story, migrationFile);
-      if (result) {
-        successfulResults.push(result);
-      }
+    const result = await this.applyMigrationsToStory(story, relevantMigrations);
+    if (result) {
+      successfulResults.push(result);
     }
     return successfulResults;
   }
-  async applyMigrationToStory(story, migrationFile) {
+  async applyMigrationsToStory(story, migrationFiles) {
+    const migrationNames = migrationFiles.map((f) => f.name);
     try {
-      let migrationFunction = this.migrationFunctions.get(migrationFile.name);
-      if (!migrationFunction) {
-        migrationFunction = await getMigrationFunction(
-          migrationFile.name,
-          this.options.space,
-          this.options.path
-        );
-        this.migrationFunctions.set(migrationFile.name, migrationFunction);
-      }
-      if (!migrationFunction) {
-        this.results.failed.push({
-          storyId: story.id,
-          migrationName: migrationFile.name,
-          error: new Error(`Failed to load migration function from file "${migrationFile.name}"`)
-        });
-        return null;
-      }
-      await saveRollbackData({
-        space: this.options.space,
-        path: this.options.path,
-        story: { id: story.id, name: story.name || "", content: story.content },
-        migrationTimestamp: this.timestamp,
-        migrationFile: migrationFile.name
-      });
       const storyContent = structuredClone(story.content);
       const originalContentHash = hash(storyContent);
-      const targetComponent = this.options.componentName || getComponentNameFromFilename(migrationFile.name);
-      const processed = applyMigrationToAllBlocks(storyContent, migrationFunction, targetComponent);
+      let processed = false;
+      for (const migrationFile of migrationFiles) {
+        const migrationFunction = await this.getOrLoadMigrationFunction(migrationFile);
+        if (!migrationFunction) {
+          this.results.failed.push({
+            storyId: story.id,
+            migrationNames,
+            error: new Error(`Failed to load migration function from file "${migrationFile.name}"`)
+          });
+          return null;
+        }
+        const targetComponent = this.options.componentName || getComponentNameFromFilename(migrationFile.name);
+        processed = applyMigrationToAllBlocks(storyContent, migrationFunction, targetComponent);
+      }
       const newContentHash = hash(storyContent);
       const contentChanged = originalContentHash !== newContentHash;
       if (processed && contentChanged) {
+        await saveRollbackData({
+          space: this.options.space,
+          path: this.options.path,
+          story: { id: story.id, name: story.name || "", content: story.content },
+          migrationTimestamp: this.timestamp,
+          migrationNames
+        });
         this.results.successful.push({
           storyId: story.id,
           name: story.name,
-          migrationName: migrationFile.name,
+          migrationNames,
           content: storyContent
         });
         return {
@@ -3240,24 +3253,28 @@ class MigrationStream extends Transform {
         this.results.skipped.push({
           storyId: story.id,
           name: story.name,
-          migrationName: migrationFile.name,
+          migrationNames,
           reason: "No changes detected after migration"
         });
         return null;
       } else {
-        const baseComponent = targetComponent.split(".")[0];
+        const reason = migrationFiles.map((migrationFile) => {
+          const targetComponent = this.options.componentName || getComponentNameFromFilename(migrationFile.name);
+          const baseComponent = targetComponent.split(".")[0];
+          return baseComponent === this.options.componentName ? `No matching components found for ${migrationFile.name}` : `Different component target ${migrationFile.name}`;
+        }).join("\n");
         this.results.skipped.push({
           storyId: story.id,
           name: story.name,
-          migrationName: migrationFile.name,
-          reason: baseComponent === this.options.componentName ? "No matching components found" : "Different component target"
+          migrationNames,
+          reason
         });
         return null;
       }
     } catch (error) {
       this.results.failed.push({
         storyId: story.id,
-        migrationName: migrationFile.name,
+        migrationNames,
         error
       });
       return null;
@@ -3355,8 +3372,9 @@ class UpdateStream extends Writable {
       } else if (this.options.publish === "all") {
         payload.publish = 1;
       }
-      const updatedStory = await updateStory(this.options.space, storyId, payload);
-      if (updatedStory) {
+      const updatedStory = !this.options.dryRun && await updateStory(this.options.space, storyId, payload);
+      const isStoryUpdated = Boolean(updatedStory);
+      if (isStoryUpdated || this.options.dryRun) {
         this.results.successful.push({ storyId, name: storyName });
         this.results.totalProcessed++;
         this.options.onProgress?.(this.results.totalProcessed);
@@ -3415,6 +3433,10 @@ class UpdateStream extends Writable {
 const program$8 = getProgram();
 migrationsCommand.command("run [componentName]").description("Run migrations").option("--fi, --filter <filter>", "glob filter to apply to the components before pushing").option("-d, --dry-run", "Preview changes without applying them to Storyblok").option("-q, --query <query>", 'Filter stories by content attributes using Storyblok filter query syntax. Example: --query="[highlighted][in]=true"').option("--starts-with <path>", 'Filter stories by path. Example: --starts-with="/en/blog/"').option("--publish <publish>", "Options for publication mode: all | published | published-with-changes").action(async (componentName, options) => {
   konsola.title(`${commands.MIGRATIONS}`, colorPalette.MIGRATIONS, componentName ? `Running migrations for component ${componentName}...` : "Running migrations...");
+  if (options.dryRun) {
+    konsola.warn(`DRY RUN MODE ENABLED: No changes will be made.
+`);
+  }
   const verbose = program$8.opts().verbose;
   const { filter, dryRun = false, query, startsWith, publish } = options;
   const { space, path } = migrationsCommand.opts();
@@ -4001,6 +4023,7 @@ const storyblokSchemas = /* @__PURE__ */ new Map([
 ]);
 
 const STORY_TYPE = "ISbStoryData";
+const DEFAULT_COMPONENT_FILENAME = "storyblok-components";
 const DEFAULT_TYPEDEFS_HEADER = [
   "// This file was generated by the storyblok CLI.",
   "// DO NOT MODIFY THIS FILE BY HAND."
@@ -4263,8 +4286,8 @@ const generateTypes = async (spaceData, options = {
     handleError(error);
   }
 };
-const saveTypesToFile = async (space, typedefString, options) => {
-  const { filename = "storyblok-components", path } = options;
+const saveTypesToComponentsFile = async (space, typedefString, options) => {
+  const { filename = DEFAULT_COMPONENT_FILENAME, path } = options;
   const resolvedPath = path ? resolve(process.cwd(), path, "types", space) : resolvePath(path, `types/${space}`);
   try {
     await saveToFile(join(resolvedPath, `${filename}.d.ts`), typedefString);
@@ -4273,7 +4296,7 @@ const saveTypesToFile = async (space, typedefString, options) => {
   }
 };
 const generateStoryblokTypes = async (options = {}) => {
-  const { filename = "storyblok", path } = options;
+  const { path } = options;
   try {
     const storyblokTypesPath = resolve(__dirname, "./index.d.ts");
     const storyblokTypesContent = readFileSync(storyblokTypesPath, "utf-8");
@@ -4284,7 +4307,7 @@ const generateStoryblokTypes = async (options = {}) => {
       storyblokTypesContent
     ].join("\n");
     const resolvedPath = path ? resolve(process.cwd(), path, "types") : resolvePath(path, "types");
-    await saveToFile(join(resolvedPath, `${filename}.d.ts`), typeDefs);
+    await saveToFile(join(resolvedPath, `storyblok.d.ts`), typeDefs);
     return true;
   } catch (error) {
     handleFileSystemError("read", error);
@@ -4293,7 +4316,10 @@ const generateStoryblokTypes = async (options = {}) => {
 };
 
 const program$5 = getProgram();
-typesCommand.command("generate").description("Generate types d.ts for your component schemas").option("--sf, --separate-files", "").option("--strict", "strict mode, no loose typing").option("--type-prefix <prefix>", "prefix to be prepended to all generated component type names").option("--type-suffix <suffix>", "suffix to be appended to all generated component type names").option("--suffix <suffix>", "Components suffix").option("--custom-fields-parser <path>", "Path to the parser file for Custom Field Types").option("--compiler-options <options>", "path to the compiler options from json-schema-to-typescript").action(async (options) => {
+typesCommand.command("generate").description("Generate types d.ts for your component schemas").option("--sf, --separate-files", "Generate one .d.ts file per component instead of a single combined file").option(
+  "--filename <name>",
+  "Base file name for all component types when generating a single declarations file (e.g. components.d.ts). Ignored when using --separate-files."
+).option("--strict", "strict mode, no loose typing").option("--type-prefix <prefix>", "prefix to be prepended to all generated component type names").option("--type-suffix <suffix>", "suffix to be appended to all generated component type names").option("--suffix <suffix>", "Components suffix").option("--custom-fields-parser <path>", "Path to the parser file for Custom Field Types").option("--compiler-options <options>", "path to the compiler options from json-schema-to-typescript").action(async (options) => {
   konsola.title(`${commands.TYPES}`, colorPalette.TYPES, "Generating types...");
   const verbose = program$5.opts().verbose;
   const { space, path } = typesCommand.opts();
@@ -4308,7 +4334,6 @@ typesCommand.command("generate").description("Generate types d.ts for your compo
       path
     });
     await generateStoryblokTypes({
-      ...options,
       path
     });
     const spaceDataWithDatasources = {
@@ -4320,8 +4345,8 @@ typesCommand.command("generate").description("Generate types d.ts for your compo
       path
     });
     if (typedefString) {
-      await saveTypesToFile(space, typedefString, {
-        ...options,
+      await saveTypesToComponentsFile(space, typedefString, {
+        filename: options.filename,
         path
       });
     }
@@ -5157,7 +5182,7 @@ program$1.command(`${commands.CREATE} [project-path]`).alias("c").description(`S
   konsola.br();
 });
 
-const version = "4.6.4";
+const version = "4.6.7";
 const pkg = {
 	version: version};