@vellumai/cli 0.11.10-dev.202609110043.1ff55a1 → 0.11.10-staging.2

package/node_modules/@vellumai/avatar-manifest/package.json

@@ -6,8 +6,7 @@
   "type": "module",
   "exports": {
     ".": "./src/index.ts",
-    "./accent": "./src/accent.ts",
-    "./notification-avatar": "./src/notification-avatar.ts"
+    "./accent": "./src/accent.ts"
   },
   "scripts": {
     "typecheck": "bunx tsc --noEmit",

package/node_modules/@vellumai/local-mode/node_modules/@vellumai/avatar-manifest/package.json

@@ -6,8 +6,7 @@
   "type": "module",
   "exports": {
     ".": "./src/index.ts",
-    "./accent": "./src/accent.ts",
-    "./notification-avatar": "./src/notification-avatar.ts"
+    "./accent": "./src/accent.ts"
   },
   "scripts": {
     "typecheck": "bunx tsc --noEmit",

package/node_modules/@vellumai/local-mode/node_modules/@vellumai/service-contracts/package.json

@@ -26,8 +26,7 @@
     "./error": "./src/error.ts",
     "./reactions": "./src/reactions.ts",
     "./guardian-requests": "./src/guardian-requests.ts",
-    "./platform-credential": "./src/platform-credential.ts",
-    "./plan-credit": "./src/plan-credit.ts"
+    "./platform-credential": "./src/platform-credential.ts"
   },
   "scripts": {
     "typecheck": "bunx tsc --noEmit",

package/node_modules/@vellumai/local-mode/node_modules/@vellumai/service-contracts/src/__tests__/channels.test.ts

@@ -1,10 +1,8 @@
 import { describe, expect, test } from "bun:test";
 
 import {
-  botProviderForChannel,
   CHANNEL_BOT_PROVIDER,
   CHANNEL_IDS,
-  channelForBotProvider,
   isChannelBotProvider,
   isChannelUserIntegration,
   isChannelId,
@@ -43,35 +41,6 @@ describe("isChannelId", () => {
   });
 });
 
-describe("botProviderForChannel and channelForBotProvider", () => {
-  test("are inverses over every entry of the map", () => {
-    for (const [channelId, botProviderKey] of Object.entries(
-      CHANNEL_BOT_PROVIDER,
-    )) {
-      expect(botProviderForChannel(channelId)).toBe(botProviderKey);
-      expect(channelForBotProvider(botProviderKey)).toBe(
-        channelId as keyof typeof CHANNEL_BOT_PROVIDER,
-      );
-    }
-  });
-
-  test("answer nothing for a channel without a bot, a person's grant, or an unknown key", () => {
-    expect(botProviderForChannel("phone")).toBeUndefined();
-    expect(botProviderForChannel("vellum")).toBeUndefined();
-    expect(botProviderForChannel("not-a-channel")).toBeUndefined();
-    // `slack` is the person's integration standing beside the bot's key.
-    expect(channelForBotProvider("slack")).toBeUndefined();
-    expect(channelForBotProvider("google")).toBeUndefined();
-  });
-
-  test("never resolve an inherited object property as a channel or a key", () => {
-    for (const name of ["constructor", "toString", "__proto__"]) {
-      expect(botProviderForChannel(name)).toBeUndefined();
-      expect(channelForBotProvider(name)).toBeUndefined();
-    }
-  });
-});
-
 describe("isChannelUserIntegration", () => {
   test("names the grant standing beside a bot of the same brand", () => {
     expect(isChannelUserIntegration("slack")).toBe(true);

package/node_modules/@vellumai/local-mode/node_modules/@vellumai/service-contracts/src/channels.ts

@@ -93,34 +93,9 @@ export const CHANNEL_BOT_PROVIDER = {
  * connected" question, for any provider key.
  */
 export function isChannelBotProvider(providerKey: string): boolean {
-  return channelForBotProvider(providerKey) !== undefined;
-}
-
-/**
- * The provider key holding a channel's bot credential, or `undefined` when
- * the channel has none, or is not a channel at all. Walks the map's own
- * entries, so an inherited object property name is never a channel.
- */
-export function botProviderForChannel(channelId: string): string | undefined {
-  return Object.entries(CHANNEL_BOT_PROVIDER).find(
-    ([id]) => id === channelId,
-  )?.[1];
-}
-
-/**
- * The channel whose bot credential a provider key holds, or `undefined` when
- * the key is not a bot's. The inverse of `botProviderForChannel`, for a
- * surface that has only the provider key in hand and needs to speak in the
- * channel's vocabulary.
- */
-export function channelForBotProvider(
-  providerKey: string,
-): keyof typeof CHANNEL_BOT_PROVIDER | undefined {
-  return (
-    Object.entries(CHANNEL_BOT_PROVIDER) as Array<
-      [keyof typeof CHANNEL_BOT_PROVIDER, string]
-    >
-  ).find(([, key]) => key === providerKey)?.[0];
+  return (Object.values(CHANNEL_BOT_PROVIDER) as readonly string[]).includes(
+    providerKey,
+  );
 }
 
 /**

package/node_modules/@vellumai/local-mode/node_modules/@vellumai/service-contracts/src/index.ts

@@ -28,7 +28,6 @@ export * from "./rpc.js";
 export * from "./trust-rules.js";
 export * from "./ingress.js";
 export * from "./no-response.js";
-export * from "./plan-credit.js";
 export * from "./platform-credential.js";
 export * from "./remote-web-pairing.js";
 export * from "./twilio-ingress.js";

package/node_modules/@vellumai/local-mode/node_modules/@vellumai/service-contracts/src/reactions.ts

@@ -7,13 +7,11 @@
  * is spelled. Modelled on Zulip's `reaction_type`, the one surveyed system
  * that separates the namespace from the name.
  *
- * `shortcode` is a name in a channel's own namespace that the channel's
- * adapter could not place among the standard emoji. Slack sends `+1` and
- * `blob_wave` alike, as names; its normalizer resolves the standard ones to
- * `unicode` from Slack's own list, and what remains is a workspace upload
- * only the workspace can render. It is a distinct kind from `unicode`, not
- * a stand-in for an unknown one, and nothing past an adapter needs a
- * channel's naming to read it.
+ * `shortcode` is a name in a channel's own namespace whose kind the channel
+ * does not disclose: Slack sends `+1` for the standard emoji and `blob_wave`
+ * for a workspace upload with nothing to tell them apart, and only the
+ * workspace token can resolve the second. It is a distinct kind from
+ * `unicode`, not a stand-in for an unknown one.
  */
 import { z } from "zod";
 
@@ -41,12 +39,6 @@ export const ReactionEmojiFieldsSchema = z.object({
 });
 export type ReactionEmojiFields = z.infer<typeof ReactionEmojiFieldsSchema>;
 
-/** The typed fields with the kind and name known. */
-export type ReactionEmojiIdentity = ReactionEmojiFields & {
-  emojiKind: ReactionEmojiKind;
-  emojiName: string;
-};
-
 /**
  * The typed emoji fields a source actually carries, with undefined ones
  * omitted: an absent key and a present-but-undefined one serialize alike, but the
@@ -66,75 +58,3 @@ export function pickReactionEmojiFields(
       : {}),
   };
 }
-
-/**
- * Parse Discord's custom-emoji mention form. The form travels the wire as a
- * reaction's spelling, so more than one package reads it: the daemon rebuilds
- * a REST path from it when the assistant reacts, and a row carrying only the
- * spelling recovers its kind from it. One parser, so the two cannot disagree
- * about what counts as one.
- *
- * `animated` reports whether the spelling carries the `a` marker. Discord's
- * normalizer never writes that marker, so it is read only from spellings that
- * arrive from elsewhere: a row written by another producer, or a value the
- * model hands back as it received it.
- */
-export function parseDiscordEmojiMention(
-  emoji: string,
-): { name: string; id: string; animated: boolean } | null {
-  const match = /^<(a?):([^:>]+):(\d+)>$/.exec(emoji);
-  return match
-    ? { name: match[2]!, id: match[3]!, animated: match[1] === "a" }
-    : null;
-}
-
-/**
- * Recover an emoji's kind from its spelling alone. This is the one inference
- * the design permits, reserved for a value that carries the string and no
- * typed fields: a persisted row or a replayed retry payload. A value that
- * declares its kind never reaches this; `reactionEmojiIdentity` makes that
- * choice for every reader.
- *
- * A mention form is unambiguous. Past that the two remaining kinds are told
- * apart by whether the string is a name at all: a channel's shortcode is
- * ASCII word characters, optionally carrying Slack's `::skin-tone-N`
- * suffix, and anything else is the character itself.
- */
-export function classifyReactionEmojiSpelling(
-  emoji: string,
-): ReactionEmojiIdentity {
-  const custom = parseDiscordEmojiMention(emoji);
-  if (custom) {
-    // The plain `<:name:id>` form says nothing about animation: the
-    // normalizer spells every custom emoji that way and reports animation
-    // in the typed field instead, so a spelling without the `a` marker is
-    // "unrecorded", not "not animated". Only the `<a:` form asserts it.
-    return {
-      emojiKind: "custom",
-      emojiName: custom.name,
-      emojiId: custom.id,
-      ...(custom.animated ? { emojiAnimated: true } : {}),
-    };
-  }
-  return /^[\w+-]+(::skin-tone-[2-6])?$/.test(emoji)
-    ? { emojiKind: "shortcode", emojiName: emoji }
-    : { emojiKind: "unicode", emojiName: emoji };
-}
-
-/**
- * The typed identity of a reaction's emoji: the fields it declares when it
- * declares a kind, otherwise the kind recovered from its spelling. Every
- * reader that must know what an emoji is goes through this, so the wire
- * contract and the web agree on when the spelling is consulted.
- */
-export function reactionEmojiIdentity(
-  reaction: { emoji: string } & ReactionEmojiFields,
-): ReactionEmojiIdentity {
-  return reaction.emojiKind !== undefined && reaction.emojiName !== undefined
-    ? {
-        ...pickReactionEmojiFields(reaction),
-        emojiKind: reaction.emojiKind,
-        emojiName: reaction.emojiName,
-      }
-    : classifyReactionEmojiSpelling(reaction.emoji);
-}

package/node_modules/@vellumai/service-contracts/package.json

@@ -26,8 +26,7 @@
     "./error": "./src/error.ts",
     "./reactions": "./src/reactions.ts",
     "./guardian-requests": "./src/guardian-requests.ts",
-    "./platform-credential": "./src/platform-credential.ts",
-    "./plan-credit": "./src/plan-credit.ts"
+    "./platform-credential": "./src/platform-credential.ts"
   },
   "scripts": {
     "typecheck": "bunx tsc --noEmit",

package/node_modules/@vellumai/service-contracts/src/__tests__/channels.test.ts

@@ -1,10 +1,8 @@
 import { describe, expect, test } from "bun:test";
 
 import {
-  botProviderForChannel,
   CHANNEL_BOT_PROVIDER,
   CHANNEL_IDS,
-  channelForBotProvider,
   isChannelBotProvider,
   isChannelUserIntegration,
   isChannelId,
@@ -43,35 +41,6 @@ describe("isChannelId", () => {
   });
 });
 
-describe("botProviderForChannel and channelForBotProvider", () => {
-  test("are inverses over every entry of the map", () => {
-    for (const [channelId, botProviderKey] of Object.entries(
-      CHANNEL_BOT_PROVIDER,
-    )) {
-      expect(botProviderForChannel(channelId)).toBe(botProviderKey);
-      expect(channelForBotProvider(botProviderKey)).toBe(
-        channelId as keyof typeof CHANNEL_BOT_PROVIDER,
-      );
-    }
-  });
-
-  test("answer nothing for a channel without a bot, a person's grant, or an unknown key", () => {
-    expect(botProviderForChannel("phone")).toBeUndefined();
-    expect(botProviderForChannel("vellum")).toBeUndefined();
-    expect(botProviderForChannel("not-a-channel")).toBeUndefined();
-    // `slack` is the person's integration standing beside the bot's key.
-    expect(channelForBotProvider("slack")).toBeUndefined();
-    expect(channelForBotProvider("google")).toBeUndefined();
-  });
-
-  test("never resolve an inherited object property as a channel or a key", () => {
-    for (const name of ["constructor", "toString", "__proto__"]) {
-      expect(botProviderForChannel(name)).toBeUndefined();
-      expect(channelForBotProvider(name)).toBeUndefined();
-    }
-  });
-});
-
 describe("isChannelUserIntegration", () => {
   test("names the grant standing beside a bot of the same brand", () => {
     expect(isChannelUserIntegration("slack")).toBe(true);

package/node_modules/@vellumai/service-contracts/src/channels.ts

@@ -93,34 +93,9 @@ export const CHANNEL_BOT_PROVIDER = {
  * connected" question, for any provider key.
  */
 export function isChannelBotProvider(providerKey: string): boolean {
-  return channelForBotProvider(providerKey) !== undefined;
-}
-
-/**
- * The provider key holding a channel's bot credential, or `undefined` when
- * the channel has none, or is not a channel at all. Walks the map's own
- * entries, so an inherited object property name is never a channel.
- */
-export function botProviderForChannel(channelId: string): string | undefined {
-  return Object.entries(CHANNEL_BOT_PROVIDER).find(
-    ([id]) => id === channelId,
-  )?.[1];
-}
-
-/**
- * The channel whose bot credential a provider key holds, or `undefined` when
- * the key is not a bot's. The inverse of `botProviderForChannel`, for a
- * surface that has only the provider key in hand and needs to speak in the
- * channel's vocabulary.
- */
-export function channelForBotProvider(
-  providerKey: string,
-): keyof typeof CHANNEL_BOT_PROVIDER | undefined {
-  return (
-    Object.entries(CHANNEL_BOT_PROVIDER) as Array<
-      [keyof typeof CHANNEL_BOT_PROVIDER, string]
-    >
-  ).find(([, key]) => key === providerKey)?.[0];
+  return (Object.values(CHANNEL_BOT_PROVIDER) as readonly string[]).includes(
+    providerKey,
+  );
 }
 
 /**

package/node_modules/@vellumai/service-contracts/src/index.ts

@@ -28,7 +28,6 @@ export * from "./rpc.js";
 export * from "./trust-rules.js";
 export * from "./ingress.js";
 export * from "./no-response.js";
-export * from "./plan-credit.js";
 export * from "./platform-credential.js";
 export * from "./remote-web-pairing.js";
 export * from "./twilio-ingress.js";

package/node_modules/@vellumai/service-contracts/src/reactions.ts

@@ -7,13 +7,11 @@
  * is spelled. Modelled on Zulip's `reaction_type`, the one surveyed system
  * that separates the namespace from the name.
  *
- * `shortcode` is a name in a channel's own namespace that the channel's
- * adapter could not place among the standard emoji. Slack sends `+1` and
- * `blob_wave` alike, as names; its normalizer resolves the standard ones to
- * `unicode` from Slack's own list, and what remains is a workspace upload
- * only the workspace can render. It is a distinct kind from `unicode`, not
- * a stand-in for an unknown one, and nothing past an adapter needs a
- * channel's naming to read it.
+ * `shortcode` is a name in a channel's own namespace whose kind the channel
+ * does not disclose: Slack sends `+1` for the standard emoji and `blob_wave`
+ * for a workspace upload with nothing to tell them apart, and only the
+ * workspace token can resolve the second. It is a distinct kind from
+ * `unicode`, not a stand-in for an unknown one.
  */
 import { z } from "zod";
 
@@ -41,12 +39,6 @@ export const ReactionEmojiFieldsSchema = z.object({
 });
 export type ReactionEmojiFields = z.infer<typeof ReactionEmojiFieldsSchema>;
 
-/** The typed fields with the kind and name known. */
-export type ReactionEmojiIdentity = ReactionEmojiFields & {
-  emojiKind: ReactionEmojiKind;
-  emojiName: string;
-};
-
 /**
  * The typed emoji fields a source actually carries, with undefined ones
  * omitted: an absent key and a present-but-undefined one serialize alike, but the
@@ -66,75 +58,3 @@ export function pickReactionEmojiFields(
       : {}),
   };
 }
-
-/**
- * Parse Discord's custom-emoji mention form. The form travels the wire as a
- * reaction's spelling, so more than one package reads it: the daemon rebuilds
- * a REST path from it when the assistant reacts, and a row carrying only the
- * spelling recovers its kind from it. One parser, so the two cannot disagree
- * about what counts as one.
- *
- * `animated` reports whether the spelling carries the `a` marker. Discord's
- * normalizer never writes that marker, so it is read only from spellings that
- * arrive from elsewhere: a row written by another producer, or a value the
- * model hands back as it received it.
- */
-export function parseDiscordEmojiMention(
-  emoji: string,
-): { name: string; id: string; animated: boolean } | null {
-  const match = /^<(a?):([^:>]+):(\d+)>$/.exec(emoji);
-  return match
-    ? { name: match[2]!, id: match[3]!, animated: match[1] === "a" }
-    : null;
-}
-
-/**
- * Recover an emoji's kind from its spelling alone. This is the one inference
- * the design permits, reserved for a value that carries the string and no
- * typed fields: a persisted row or a replayed retry payload. A value that
- * declares its kind never reaches this; `reactionEmojiIdentity` makes that
- * choice for every reader.
- *
- * A mention form is unambiguous. Past that the two remaining kinds are told
- * apart by whether the string is a name at all: a channel's shortcode is
- * ASCII word characters, optionally carrying Slack's `::skin-tone-N`
- * suffix, and anything else is the character itself.
- */
-export function classifyReactionEmojiSpelling(
-  emoji: string,
-): ReactionEmojiIdentity {
-  const custom = parseDiscordEmojiMention(emoji);
-  if (custom) {
-    // The plain `<:name:id>` form says nothing about animation: the
-    // normalizer spells every custom emoji that way and reports animation
-    // in the typed field instead, so a spelling without the `a` marker is
-    // "unrecorded", not "not animated". Only the `<a:` form asserts it.
-    return {
-      emojiKind: "custom",
-      emojiName: custom.name,
-      emojiId: custom.id,
-      ...(custom.animated ? { emojiAnimated: true } : {}),
-    };
-  }
-  return /^[\w+-]+(::skin-tone-[2-6])?$/.test(emoji)
-    ? { emojiKind: "shortcode", emojiName: emoji }
-    : { emojiKind: "unicode", emojiName: emoji };
-}
-
-/**
- * The typed identity of a reaction's emoji: the fields it declares when it
- * declares a kind, otherwise the kind recovered from its spelling. Every
- * reader that must know what an emoji is goes through this, so the wire
- * contract and the web agree on when the spelling is consulted.
- */
-export function reactionEmojiIdentity(
-  reaction: { emoji: string } & ReactionEmojiFields,
-): ReactionEmojiIdentity {
-  return reaction.emojiKind !== undefined && reaction.emojiName !== undefined
-    ? {
-        ...pickReactionEmojiFields(reaction),
-        emojiKind: reaction.emojiKind,
-        emojiName: reaction.emojiName,
-      }
-    : classifyReactionEmojiSpelling(reaction.emoji);
-}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vellumai/cli",
-  "version": "0.11.10-dev.202609110043.1ff55a1",
+  "version": "0.11.10-staging.2",
   "description": "CLI tools for vellum-assistant",
   "type": "module",
   "exports": {

package/node_modules/@vellumai/avatar-manifest/src/__tests__/notification-avatar.test.ts

@@ -1,220 +0,0 @@
-import { describe, expect, test } from "bun:test";
-
-import {
-  NOTIFICATION_AVATAR_FALLBACK_DISC_HEX,
-  NOTIFICATION_AVATAR_INSET,
-  NOTIFICATION_AVATAR_MAX_BYTES,
-  NOTIFICATION_AVATAR_MAX_LOCAL_BYTES,
-  NOTIFICATION_AVATAR_SIZE,
-  NOTIFICATION_AVATAR_SPEC_VERSION,
-  notificationAvatarDiscHex,
-  notificationAvatarGeometry,
-  notificationAvatarSvg,
-} from "../notification-avatar.js";
-
-function allAttributes(tag: string, svg: string): Record<string, string>[] {
-  return [...svg.matchAll(new RegExp(`<${tag}\\s([^>]*)/>`, "g"))].map(
-    (match) => {
-      const out: Record<string, string> = {};
-      for (const [, name, value] of match[1]!.matchAll(
-        /([\w:-]+)="([^"]*)"/g,
-      )) {
-        out[name!] = value!;
-      }
-      return out;
-    },
-  );
-}
-
-function attributes(tag: string, svg: string): Record<string, string> {
-  const all = allAttributes(tag, svg);
-  expect(all.length).toBeGreaterThan(0);
-  return all[0]!;
-}
-
-function countOf(tag: string, svg: string): number {
-  return svg.split(`<${tag}`).length - 1;
-}
-
-const PNG_BASE64 = "iVBORw0KGgo=";
-
-describe("notificationAvatarDiscHex", () => {
-  test("mixes the accent into white", () => {
-    expect(notificationAvatarDiscHex("#E9642F")).toBe("#FCE9E2");
-    expect(notificationAvatarDiscHex("#4C9B50")).toBe("#E6F1E7");
-  });
-
-  test("is case insensitive and always returns uppercase #RRGGBB", () => {
-    expect(notificationAvatarDiscHex("#e9642f")).toBe("#FCE9E2");
-  });
-
-  test("keeps white white and lifts black to near white", () => {
-    expect(notificationAvatarDiscHex("#FFFFFF")).toBe("#FFFFFF");
-    expect(notificationAvatarDiscHex("#000000")).toBe("#DBDBDB");
-  });
-
-  test.each([
-    ["null", null],
-    ["short form", "#abc"],
-    ["no hash", "e9642f"],
-    ["garbage", "not a colour"],
-    ["empty", ""],
-  ])("falls back to the neutral disc for %s", (_label, value) => {
-    expect(notificationAvatarDiscHex(value as string | null)).toBe(
-      NOTIFICATION_AVATAR_FALLBACK_DISC_HEX,
-    );
-  });
-});
-
-describe("notificationAvatarSvg", () => {
-  test("draws one disc and one inset avatar at the default size", () => {
-    const svg = notificationAvatarSvg({
-      innerPngBase64: PNG_BASE64,
-      accentHex: "#E9642F",
-    });
-
-    expect(countOf("svg", svg)).toBe(1);
-    expect(countOf("image", svg)).toBe(1);
-
-    expect(svg.startsWith("<svg ")).toBe(true);
-    expect(svg).toContain(`width="256" height="256"`);
-    expect(svg).toContain(`viewBox="0 0 256 256"`);
-
-    const circle = attributes("circle", svg);
-    expect(circle).toMatchObject({ cx: "128", cy: "128", r: "128" });
-    expect(circle.fill).toBe("#FCE9E2");
-
-    const inset = NOTIFICATION_AVATAR_SIZE * NOTIFICATION_AVATAR_INSET;
-    const inner = NOTIFICATION_AVATAR_SIZE - 2 * inset;
-    const image = attributes("image", svg);
-    expect(Number(image.x)).toBeCloseTo(inset, 3);
-    expect(Number(image.y)).toBeCloseTo(inset, 3);
-    expect(Number(image.width)).toBeCloseTo(inner, 3);
-    expect(Number(image.height)).toBeCloseTo(inner, 3);
-    expect(image.href).toBe(`data:image/png;base64,${PNG_BASE64}`);
-    expect(image["xlink:href"]).toBe(image.href);
-  });
-
-  test("cover-crops a non-square raster and clips it to the disc", () => {
-    // The builder never sees the source dimensions, so what makes a 1024x512
-    // upload fill the square instead of letterboxing is the pair of attributes
-    // asserted here.
-    const svg = notificationAvatarSvg({
-      innerPngBase64: PNG_BASE64,
-      accentHex: "#E9642F",
-    });
-
-    const image = attributes("image", svg);
-    expect(image.preserveAspectRatio).toBe("xMidYMid slice");
-
-    const clipRef = image["clip-path"]!;
-    const clipId = clipRef.match(/^url\(#([\w-]+)\)$/)?.[1];
-    expect(clipId).toBeTruthy();
-    expect(svg).toContain(`<clipPath id="${clipId}">`);
-
-    // The clip circle carries the disc geometry and nothing else, so the
-    // painted disc and the boundary can never drift apart.
-    const [disc, clip] = allAttributes("circle", svg);
-    expect(clip).toEqual({ cx: "128", cy: "128", r: "128" });
-    expect(disc).toMatchObject({ cx: "128", cy: "128", r: "128" });
-  });
-
-  test("scales the geometry to a custom size", () => {
-    const svg = notificationAvatarSvg({
-      innerPngBase64: PNG_BASE64,
-      accentHex: "#E9642F",
-      size: 100,
-    });
-
-    expect(svg).toContain(`width="100" height="100"`);
-    expect(attributes("circle", svg)).toMatchObject({
-      cx: "50",
-      cy: "50",
-      r: "50",
-    });
-    expect(attributes("image", svg)).toMatchObject({
-      x: "11",
-      y: "11",
-      width: "78",
-      height: "78",
-    });
-  });
-
-  test("carries a non-PNG inner raster with its own media type", () => {
-    const svg = notificationAvatarSvg({
-      innerPngBase64: PNG_BASE64,
-      innerMediaType: "image/jpeg",
-      accentHex: "#E9642F",
-    });
-
-    const image = attributes("image", svg);
-    expect(image.href).toBe(`data:image/jpeg;base64,${PNG_BASE64}`);
-    expect(image["xlink:href"]).toBe(image.href);
-  });
-
-  test("paints the fallback disc when there is no accent", () => {
-    const svg = notificationAvatarSvg({
-      innerPngBase64: PNG_BASE64,
-      accentHex: null,
-    });
-    expect(attributes("circle", svg).fill).toBe(
-      NOTIFICATION_AVATAR_FALLBACK_DISC_HEX,
-    );
-  });
-});
-
-describe("notificationAvatarGeometry", () => {
-  test("derives the disc, the border and the avatar edge from the size", () => {
-    expect(notificationAvatarGeometry(100)).toEqual({
-      radius: 50,
-      offset: 11,
-      inner: 78,
-    });
-  });
-
-  test("measures the default size when given none", () => {
-    expect(notificationAvatarGeometry()).toEqual(
-      notificationAvatarGeometry(NOTIFICATION_AVATAR_SIZE),
-    );
-  });
-
-  test("is the geometry the SVG is drawn with", () => {
-    const size = 100;
-    const { radius, offset, inner } = notificationAvatarGeometry(size);
-    const svg = notificationAvatarSvg({
-      innerPngBase64: PNG_BASE64,
-      accentHex: null,
-      size,
-    });
-
-    expect(attributes("circle", svg)).toMatchObject({
-      cx: String(radius),
-      cy: String(radius),
-      r: String(radius),
-    });
-    expect(attributes("image", svg)).toMatchObject({
-      x: String(offset),
-      y: String(offset),
-      width: String(inner),
-      height: String(inner),
-    });
-  });
-});
-
-describe("the transport contract", () => {
-  test("caps a notification PNG at 128 KB for the platform sync", () => {
-    expect(NOTIFICATION_AVATAR_MAX_BYTES).toBe(128 * 1024);
-  });
-
-  test("gives the local desktop path a looser cap of its own", () => {
-    expect(NOTIFICATION_AVATAR_MAX_LOCAL_BYTES).toBe(512 * 1024);
-    expect(NOTIFICATION_AVATAR_MAX_LOCAL_BYTES).toBeGreaterThan(
-      NOTIFICATION_AVATAR_MAX_BYTES,
-    );
-  });
-
-  test("stamps the drawing with a spec version", () => {
-    expect(Number.isInteger(NOTIFICATION_AVATAR_SPEC_VERSION)).toBe(true);
-    expect(NOTIFICATION_AVATAR_SPEC_VERSION).toBeGreaterThan(0);
-  });
-});