Create a transformer to enforce current terminology in reportEvent without breaking Mixpanel events

src/activation/useActivateMod.ts

@@ -53,8 +53,8 @@ type ActivationSource = "marketplace" | "extensionConsole";
 
 function selectActivateEventData(modDefinition: ModDefinition) {
   return {
-    blueprintId: modDefinition.metadata.id,
-    extensions: modDefinition.extensionPoints.map((x) => x.label),
+    modId: modDefinition.metadata.id,
+    modComponents: modDefinition.extensionPoints.map((x) => x.label),
   };
 }
 

src/background/executor.test.ts

@@ -43,11 +43,11 @@ const runBrickMock = jest.mocked(runBrick);
 const optionsFactory = define<RemoteBrickOptions>({
   ctxt: () => ({}),
   messageContext: (i: number) => ({
-    extensionId: uuidSequence(i),
+    modComponentId: uuidSequence(i),
   }),
   meta: derive<RemoteBrickOptions, RemoteBrickOptions["meta"]>(
     (options) => ({
-      extensionId: options.messageContext!.extensionId,
+      extensionId: options.messageContext!.modComponentId,
       runId: null,
       branches: [],
     }),

src/background/messenger/registration.ts

@@ -41,7 +41,7 @@ import * as registry from "@/registry/packageRegistry";
 import serviceRegistry from "@/integrations/registry";
 import { getUserData } from "@/auth/authStorage";
 import {
-  clearExtensionDebugLogs,
+  clearModComponentDebugLogs,
   clearLog,
   clearLogs,
   recordError,
@@ -108,7 +108,7 @@ declare global {
     RECORD_ERROR: typeof recordError;
     CLEAR_LOGS: typeof clearLogs;
     CLEAR_LOG: typeof clearLog;
-    CLEAR_EXTENSION_DEBUG_LOGS: typeof clearExtensionDebugLogs;
+    CLEAR_EXTENSION_DEBUG_LOGS: typeof clearModComponentDebugLogs;
 
     LOCATE_SERVICES_FOR_ID: typeof locator.locateAllForService;
     LOCATE_SERVICE: typeof locator.locate;
@@ -181,7 +181,7 @@ export default function registerMessenger(): void {
     RECORD_ERROR: recordError,
     CLEAR_LOGS: clearLogs,
     CLEAR_LOG: clearLog,
-    CLEAR_EXTENSION_DEBUG_LOGS: clearExtensionDebugLogs,
+    CLEAR_EXTENSION_DEBUG_LOGS: clearModComponentDebugLogs,
 
     LOCATE_SERVICES_FOR_ID: locator.locateAllForService.bind(locator),
     LOCATE_SERVICE: locator.locate.bind(locator),

src/background/removeModComponentForEveryTab.ts

@@ -39,5 +39,5 @@ export async function removeModComponentForEveryTab(
   });
   await uninstallContextMenu({ extensionId: modComponentId });
   await clearModComponentTraces(modComponentId);
-  await clearLog({ extensionId: modComponentId });
+  await clearLog({ modComponentId });
 }

src/background/requests.ts

@@ -347,8 +347,8 @@ async function getIntegrationMessageContext(
   }
 
   return {
-    serviceId: config.serviceId,
-    serviceVersion: resolvedIntegration?.version,
+    integrationId: config.serviceId,
+    integrationVersion: resolvedIntegration?.version,
     authId: config.id,
   };
 }

src/bricks/effects/AddDynamicTextSnippet.test.ts

@@ -49,8 +49,8 @@ describe("AddDynamicTextSnippet", () => {
     async (shortcut) => {
       const extensionId = uuidv4();
       const logger = new ConsoleLogger({
-        extensionId,
-        blueprintId: registryIdFactory(),
+        modComponentId: extensionId,
+        modId: registryIdFactory(),
       });
 
       const pipeline = {
@@ -81,8 +81,8 @@ describe("AddDynamicTextSnippet", () => {
           componentId: extensionId,
           context: {
             ...logger.context,
-            blockId: brick.id,
-            blockVersion: expect.toBeString(),
+            brickId: brick.id,
+            brickVersion: expect.toBeString(),
             label: brick.name,
           },
         },
@@ -98,7 +98,7 @@ describe("AddDynamicTextSnippet", () => {
     "passes preview directly: %s",
     async (preview) => {
       const extensionId = uuidv4();
-      const logger = new ConsoleLogger({ extensionId });
+      const logger = new ConsoleLogger({ modComponentId: extensionId });
 
       const pipeline = {
         id: brick.id,
@@ -127,8 +127,8 @@ describe("AddDynamicTextSnippet", () => {
           componentId: extensionId,
           context: {
             ...logger.context,
-            blockId: brick.id,
-            blockVersion: expect.toBeString(),
+            brickId: brick.id,
+            brickVersion: expect.toBeString(),
             label: brick.name,
           },
         },

src/bricks/effects/AddDynamicTextSnippet.ts

@@ -114,15 +114,15 @@ class AddDynamicTextSnippet extends EffectABC {
       return;
     }
 
-    if (logger.context.extensionId == null) {
+    if (logger.context.modComponentId == null) {
       throw new Error("Must be run in the context of a mod");
     }
 
     // Counter to keep track of the action run number for tracing
     let counter = 0;
 
     platform.snippetShortcutMenu.register({
-      componentId: logger.context.extensionId,
+      componentId: logger.context.modComponentId,
       context: logger.context,
       // Trim leading command key in shortcut to be resilient to user input
       shortcut: normalizeShortcut(shortcut),

src/bricks/effects/AddQuickBarAction.tsx

@@ -151,11 +151,11 @@ class AddQuickBarAction extends EffectABC {
     let counter = 0;
 
     // Expected parent id from QuickBarProviderExtensionPoint
-    const parentId = `provider-${logger.context.extensionId}`;
+    const parentId = `provider-${logger.context.modComponentId}`;
 
     const action: CustomAction = {
       // XXX: old actions will still appear in the quick bar unless the extension point clears out the old actions
-      id: `${logger.context.extensionId}-${title}`,
+      id: `${logger.context.modComponentId}-${title}`,
       // Additional metadata, for enabling clearing out old actions
       modComponentRef: mapMessageContextToModComponentRef(logger.context),
       // Can only provide a parent if the parent exists

src/bricks/effects/AddTextSnippets.test.ts

@@ -53,7 +53,7 @@ describe("AddTextSnippets", () => {
           title: "Test",
           preview: "test text",
           handler: expect.toBeFunction(),
-          componentId: options.logger.context.extensionId,
+          componentId: options.logger.context.modComponentId,
           context: options.logger.context,
         },
       ]);

src/bricks/effects/AddTextSnippets.ts

@@ -115,13 +115,13 @@ class AddTextSnippets extends EffectABC {
       return;
     }
 
-    if (logger.context.extensionId == null) {
+    if (logger.context.modComponentId == null) {
       throw new Error("Must be run in the context of a mod component");
     }
 
     for (const { shortcut, title, text } of snippets) {
       platform.snippetShortcutMenu.register({
-        componentId: logger.context.extensionId,
+        componentId: logger.context.modComponentId,
         context: logger.context,
         shortcut: normalizeShortcut(shortcut),
         title,

src/bricks/effects/assignModVariable.test.ts

@@ -35,8 +35,8 @@ const modId = validateRegistryId("test/123");
 const brick = new AssignModVariable();
 
 const logger = new ConsoleLogger({
-  extensionId: modComponentId,
-  blueprintId: modId,
+  modComponentId,
+  modId,
 });
 
 const brickOptions = brickOptionsFactory({ logger });

src/bricks/effects/assignModVariable.ts

@@ -96,7 +96,7 @@ class AssignModVariable extends EffectABC {
     }>,
     { logger }: BrickOptions,
   ): Promise<void> {
-    const { blueprintId: modId, extensionId: modComponentId } = logger.context;
+    const { modId, modComponentId } = logger.context;
 
     setState({
       namespace: StateNamespaces.MOD,

src/bricks/effects/attachAutocomplete.test.ts

@@ -24,7 +24,7 @@ import { brickOptionsFactory } from "@/testUtils/factories/runtimeFactories";
 const brick = new AttachAutocomplete();
 
 const logger = new ConsoleLogger({
-  extensionId: uuidSequence(0),
+  modComponentId: uuidSequence(0),
 });
 
 describe("AttachAutocomplete", () => {

src/bricks/effects/customEvent.test.ts

@@ -24,7 +24,7 @@ import { brickOptionsFactory } from "@/testUtils/factories/runtimeFactories";
 const brick = new CustomEventEffect();
 
 const logger = new ConsoleLogger({
-  extensionId: uuidSequence(0),
+  modComponentId: uuidSequence(0),
 });
 
 describe("CustomEventEffect", () => {

src/bricks/effects/pageState.test.ts

@@ -34,8 +34,8 @@ describe("@pixiebrix/state/get", () => {
   test("default to blueprint state", async () => {
     const brick = new GetPageState();
     const logger = new ConsoleLogger({
-      extensionId: uuidv4(),
-      blueprintId: validateRegistryId("test/123"),
+      modComponentId: uuidv4(),
+      modId: validateRegistryId("test/123"),
     });
     await brick.transform(
       unsafeAssumeValidArg({}),
@@ -53,8 +53,8 @@ describe("@pixiebrix/state/set", () => {
   test("shallow merge", async () => {
     const brick = new SetPageState();
     const logger = new ConsoleLogger({
-      extensionId: uuidv4(),
-      blueprintId: validateRegistryId("test/123"),
+      modComponentId: uuidv4(),
+      modId: validateRegistryId("test/123"),
     });
 
     let result = await brick.transform(
@@ -78,8 +78,8 @@ describe("@pixiebrix/state/set", () => {
 
     const brick = new SetPageState();
     const logger = new ConsoleLogger({
-      extensionId: uuidv4(),
-      blueprintId: validateRegistryId("test/123"),
+      modComponentId: uuidv4(),
+      modId: validateRegistryId("test/123"),
     });
 
     const original = {
@@ -164,8 +164,8 @@ describe("set and get", () => {
     const setState = new SetPageState();
     const getState = new GetPageState();
     const logger = new ConsoleLogger({
-      extensionId: uuidv4(),
-      blueprintId: validateRegistryId("test/123"),
+      modComponentId: uuidv4(),
+      modId: validateRegistryId("test/123"),
     });
 
     await setState.transform(
@@ -196,7 +196,7 @@ describe("set and get", () => {
     const setState = new SetPageState();
     const getState = new GetPageState();
     const logger = new ConsoleLogger({
-      extensionId: uuidv4(),
+      modComponentId: uuidv4(),
     });
 
     await setState.transform(

src/bricks/effects/pageState.ts

@@ -150,7 +150,7 @@ export class SetPageState extends TransformerABC {
     }>,
     { logger, platform }: BrickOptions,
   ): Promise<JsonObject> {
-    const { blueprintId, extensionId } = logger.context;
+    const { modId: blueprintId, modComponentId: extensionId } = logger.context;
 
     return platform.state.setState({
       namespace,
@@ -205,7 +205,7 @@ export class GetPageState extends TransformerABC {
     }: BrickArgs<{ namespace?: StateNamespace }>,
     { logger, platform }: BrickOptions,
   ): Promise<JsonObject> {
-    const { blueprintId: modId, extensionId: modComponentId } = logger.context;
+    const { modId, modComponentId } = logger.context;
 
     return platform.state.getState({
       namespace,

src/bricks/effects/sidebarEffects.ts

@@ -79,7 +79,7 @@ export class ShowSidebar extends EffectABC {
       sidebarController.updateSidebar({
         force: forcePanel,
         panelHeading,
-        blueprintId: logger.context.blueprintId,
+        blueprintId: logger.context.modId,
       }),
     );
   }

src/bricks/effects/tourEffect.ts

@@ -133,7 +133,7 @@ export class TourEffect extends EffectABC {
       );
     }
 
-    const { extensionId } = logger.context;
+    const { modComponentId } = logger.context;
     const abortController = new AbortController();
     const stylesheetLink = await injectStylesheet(stylesheetUrl);
 
@@ -166,7 +166,10 @@ export class TourEffect extends EffectABC {
       );
     }
 
-    assertNotNullish(extensionId, "extensionId is required to run a tour");
+    assertNotNullish(
+      modComponentId,
+      "modComponentId is required to run a tour",
+    );
 
     const tour = introJs()
       .setOptions({

src/bricks/renderers/customForm.test.tsx

@@ -352,8 +352,8 @@ describe("CustomFormRenderer", () => {
       expect(
         getState({
           namespace: StateNamespaces.MOD,
-          modComponentId: options.logger.context.extensionId,
-          modId: options.logger.context.blueprintId,
+          modComponentId: options.logger.context.modComponentId,
+          modId: options.logger.context.modId,
         }),
       ).toStrictEqual({
         name: value,

src/bricks/renderers/customForm.tsx

@@ -284,8 +284,8 @@ export class CustomFormRenderer extends RendererABC {
     }>,
     { logger, runPipeline, platform }: BrickOptions,
   ): Promise<ComponentRef> {
-    if (logger.context.extensionId == null) {
-      throw new Error("extensionId is required");
+    if (logger.context.modComponentId == null) {
+      throw new Error("modComponentId is required");
     }
 
     // Redundant with the JSON Schema input validation for `required`. But keeping here for clarity
@@ -311,7 +311,7 @@ export class CustomFormRenderer extends RendererABC {
       assumeNotNullish_UNSAFE(recordId);
     }
 
-    const { blueprintId, extensionId } = logger.context;
+    const { modId: blueprintId, modComponentId: extensionId } = logger.context;
 
     const initialData = await getInitialData(storage, recordId, {
       blueprintId,

src/bricks/transformers/RunBrickByIdTransformer.ts

@@ -95,7 +95,7 @@ class RunBrickByIdTransformer extends TransformerABC {
     const { logger, ...otherOptions } = options;
 
     return brick.run(rawArguments as BrickArgs, {
-      logger: logger.childLogger({ blockId: brick.id }),
+      logger: logger.childLogger({ brickId: brick.id }),
       ...otherOptions,
     });
   }