#9242: migrate unsaved standalone mod components in the Page Editor

src/analysis/analysisVisitors/varAnalysis/varAnalysis.ts

@@ -57,6 +57,7 @@ import { getOutputReference, isOutputKey } from "@/runtime/runtimeTypes";
 import { assertNotNullish } from "@/utils/nullishUtils";
 import { BusinessError } from "@/errors/businessErrors";
 import { adapterForComponent } from "@/pageEditor/starterBricks/adapter";
+import { isInnerDefinitionRegistryId } from "@/types/helpers";
 
 export const INVALID_VARIABLE_GENERIC_MESSAGE = "Invalid variable name";
 
@@ -305,11 +306,13 @@ async function setOptionsVars(
   formState: ModComponentFormState,
   contextVars: VarMap,
 ): Promise<void> {
-  if (formState.modMetadata == null) {
+  const modId = formState.modMetadata.id;
+
+  // Mod is unsaved, so definition won't be in the registry
+  if (isInnerDefinitionRegistryId(modId)) {
     return;
   }
 
-  const modId = formState.modMetadata.id;
   const mod = await modRegistry.lookup(modId);
   const optionsSchema = mod?.options?.schema;
   if (isEmpty(optionsSchema)) {

src/background/deploymentUpdater.test.ts

@@ -382,11 +382,9 @@ describe("syncDeployments", () => {
     };
     registryFindMock.mockResolvedValue(brick);
 
-    // An extension without a recipe. Exclude _recipe entirely to handle the case where the property is missing
     const modComponent = modComponentFactory({
       extensionPointId: starterBrick.metadata!.id,
     }) as ActivatedModComponent;
-    delete modComponent._recipe;
     delete modComponent._deployment;
 
     await saveModComponentState({

src/components/fields/schemaFields/integrations/integrationDependencyFieldUtils.test.ts

@@ -31,7 +31,10 @@ import { validateOutputKey } from "@/runtime/runtimeTypes";
 import { toExpression } from "@/utils/expressionUtils";
 import { normalizeAvailability } from "@/bricks/available";
 import { StarterBrickTypes } from "@/types/starterBrickTypes";
-import { emptyModVariablesDefinitionFactory } from "@/utils/modUtils";
+import {
+  createNewUnsavedModMetadata,
+  emptyModVariablesDefinitionFactory,
+} from "@/utils/modUtils";
 
 describe("selectVariables", () => {
   test("selects nothing when no services used", () => {
@@ -224,7 +227,7 @@ describe("selectVariables", () => {
       permissions: emptyPermissionsFactory(),
       optionsArgs: {},
       variablesDefinition: emptyModVariablesDefinitionFactory(),
-      modMetadata: undefined,
+      modMetadata: createNewUnsavedModMetadata({ modName: "Document Mod" }),
       modComponent: {
         brickPipeline: [
           {

src/pageEditor/hooks/useAddNewModComponent.ts

@@ -89,19 +89,20 @@ function useAddNewModComponent(modMetadata?: ModMetadata): AddNewModComponent {
         setInsertingStarterBrickType(null);
       }
 
-      const url = await getCurrentInspectedURL();
-      const metadata = internalStarterBrickMetaFactory();
-      const initialFormState = fromNativeElement(url, metadata, element);
+      const initialFormState = fromNativeElement({
+        url: await getCurrentInspectedURL(),
+        starterBrickMetadata: internalStarterBrickMetaFactory(),
+        modMetadata:
+          modMetadata ??
+          createNewUnsavedModMetadata({
+            modName: generateFreshModName(),
+          }),
+        element,
+      });
 
       initialFormState.modComponent.brickPipeline =
         getExampleBrickPipeline(starterBrickType);
 
-      initialFormState.modMetadata =
-        modMetadata ??
-        createNewUnsavedModMetadata({
-          modName: generateFreshModName(),
-        });
-
       return initialFormState as ModComponentFormState;
     },
     [

src/pageEditor/hooks/useClearModChanges.ts

@@ -47,7 +47,7 @@ function useClearModChanges(): (modId: RegistryId) => Promise<void> {
 
       await Promise.all(
         modComponentFormStates
-          .filter((x) => x.modMetadata?.id === modId)
+          .filter((x) => x.modMetadata.id === modId)
           .map(async (modComponentFormState) =>
             clearModComponentChanges({
               modComponentId: modComponentFormState.uuid,