#9257: fix heartbeat telemetry regression

src/types/contract.ts

@@ -271,5 +271,5 @@ export type ActivatedDeployment = {
   deployment: UUID;
   // Use legacy names - these are passed to the server
   blueprint: RegistryId;
-  blueprintVersion: string;
+  blueprintVersion: SemVerString;
 };

src/types/helpers.test.ts

@@ -125,6 +125,12 @@ describe("types/helpers.ts", () => {
         coerce: true,
         expected: "v1.2.3",
       },
+      {
+        value: "0.0.0",
+        allowLeadingV: false,
+        coerce: false,
+        expected: "0.0.0",
+      },
     ])(
       "$value with allowLeadingV: $allowLeadingV and coerce: $coerce returns $expected",
       ({ value, allowLeadingV, coerce, expected }) => {

src/types/registryTypes.ts

@@ -74,19 +74,11 @@ export type Metadata = {
    */
   readonly name: string;
 
-  /**
-   * An optional human-readable description.
-   */
   /**
    * An optional human-readable description.
    */
   readonly description?: string;
 
-  /**
-   * The semantic version of the package.
-   *
-   * Currently optional because it defaults to the browser extension version for bricks defined in JS.
-   */
   /**
    * The semantic version of the package.
    *

src/utils/deploymentUtils.ts

@@ -24,7 +24,7 @@ import {
   type IntegrationDependency,
   type SanitizedIntegrationConfig,
 } from "@/integrations/integrationTypes";
-import { validateUUID } from "@/types/helpers";
+import { normalizeSemVerString, validateUUID } from "@/types/helpers";
 import { type Except } from "type-fest";
 import { PIXIEBRIX_INTEGRATION_ID } from "@/integrations/constants";
 import getUnconfiguredComponentIntegrations from "@/integrations/util/getUnconfiguredComponentIntegrations";
@@ -150,11 +150,15 @@ export function selectActivatedDeployments(
   return uniqBy(
     modInstances
       .filter((x) => x.deploymentMetadata != null)
-      .map((x) => ({
+      .map((modInstance) => ({
         // eslint-disable-next-line @typescript-eslint/no-non-null-assertion -- Typescript not picking up filter
-        deployment: x.deploymentMetadata!.id,
-        blueprint: x.definition.metadata.id,
-        blueprintVersion: x.definition.metadata.id,
+        deployment: modInstance.deploymentMetadata!.id,
+        blueprint: modInstance.definition.metadata.id,
+        // In practice, all activated mods must have a version. But be defensive given that Metadata type used by
+        // ModDefinition does not currently require version.
+        blueprintVersion:
+          modInstance.definition.metadata.version ??
+          normalizeSemVerString("0.0.0"),
       })),
     (x) => x.deployment,
   );