@jxsuite/studio 6.1.1 → 6.1.2

package/src/settings/contributed-section.ts

@@ -39,6 +39,7 @@
  *   text. It is one value now, and the field shows it.
  */
 
+import { isFunction, isJsonObject, isNonNullObject, isString } from "@jxsuite/schema/guards";
 import { errorMessage } from "@jxsuite/schema/parse";
 import { getPlatform } from "../platform";
 import { commitProjectConfig } from "../tabs/project-config";
@@ -53,7 +54,8 @@ import { paneOfContainer } from "../canvas/canvas-surface";
 import { mountContributedSurface } from "../surfaces/settings-contributed";
 
 import type { JsonSchema, SchemaFormContext } from "../ui/schema-form";
-import type { ProjectConfig } from "@jxsuite/schema/types";
+import type { JsonObject, JsonValue } from "@jxsuite/schema/types";
+import type { ResolveContextOptions } from "../services/context-resolver";
 import type {
   ContributedActions,
   ContributedSurfaceHandle,
@@ -74,7 +76,7 @@ export interface SettingsContribution {
     entry?:
       | {
           ui?: Record<string, { control?: string; enum?: unknown }> | undefined;
-          newEntry?: Record<string, unknown> | undefined;
+          newEntry?: JsonObject | undefined;
         }
       | undefined;
   };
@@ -139,13 +141,8 @@ export function resetContributedSectionState(): void {
  *   such entry, so the caller's refusal can name both sides.
  */
 export function selectContributedEntry(sectionKey: string, entryKey: string): string[] | null {
-  const section = (projectState?.projectConfig as Record<string, unknown> | null | undefined)?.[
-    sectionKey
-  ];
-  const entries =
-    section && typeof section === "object" && !Array.isArray(section)
-      ? (section as Record<string, unknown>)
-      : {};
+  const section = projectState?.projectConfig?.[sectionKey];
+  const entries = isJsonObject(section) ? section : {};
   if (!Object.hasOwn(entries, entryKey)) {
     return Object.keys(entries);
   }
@@ -170,6 +167,14 @@ export function resetContributedDiagnostics(): void {
   diagnostics.clear();
 }
 
+/** {@link routeDiagnostics}' answer: per-field messages, and the ones about the record itself. */
+export interface RoutedDiagnostics {
+  /** The first message for each property one level under the base, keyed by property name. */
+  fields: Record<string, string>;
+  /** Messages whose pointer is the base itself, for the section line. */
+  section: string[];
+}
+
 /**
  * Route `jx-validate` messages ("/search/index: must be string") to the field they are about.
  *
@@ -182,10 +187,7 @@ export function resetContributedDiagnostics(): void {
  * @param {string[]} messages
  * @returns
  */
-export function routeDiagnostics(
-  base: string,
-  messages: string[],
-): { fields: Record<string, string>; section: string[] } {
+export function routeDiagnostics(base: string, messages: string[]): RoutedDiagnostics {
   const fields: Record<string, string> = {};
   const section: string[] = [];
   for (const message of messages) {
@@ -230,7 +232,12 @@ export function routeDiagnostics(
  * @param {() => void} rerender
  */
 async function saveProjectConfig(base: string, rerender: () => void) {
-  const config = (projectState as { projectConfig: ProjectConfig }).projectConfig;
+  /* Every caller reached this through `sectionValue`, which returns null without a config, so a
+     project is open here; the guard only states that to the compiler. */
+  const config = projectState?.projectConfig;
+  if (!config) {
+    return;
+  }
 
   const commit = await commitProjectConfig();
   if (!commit.ok) {
@@ -258,7 +265,7 @@ async function saveProjectConfig(base: string, rerender: () => void) {
 // ─── Helpers ──────────────────────────────────────────────────────────────────
 
 /** Apply a schema-form patch onto a record in place; `undefined` values unset the key. */
-function applyPatch(target: Record<string, unknown>, patch: Record<string, unknown>) {
+function applyPatch<V>(target: Record<string, V>, patch: Record<string, V | undefined>) {
   for (const [key, value] of Object.entries(patch)) {
     if (value === undefined) {
       delete target[key];
@@ -277,30 +284,30 @@ function slugify(name: string): string {
 }
 
 /** Instantiate a newEntry template, substituting `${key}` in every string value. */
-function instantiateNewEntry(
-  template: Record<string, unknown> | undefined,
-  key: string,
-): Record<string, unknown> {
+function instantiateNewEntry(template: JsonObject | undefined, key: string): JsonObject {
   if (!template) {
     return {};
   }
-  const substitute = (value: unknown): unknown => {
-    if (typeof value === "string") {
+  const substituteObject = (value: JsonObject): JsonObject => {
+    const out: JsonObject = {};
+    for (const [k, v] of Object.entries(value)) {
+      out[k] = substitute(v);
+    }
+    return out;
+  };
+  const substitute = (value: JsonValue): JsonValue => {
+    if (isString(value)) {
       return value.replaceAll("${key}", key);
     }
     if (Array.isArray(value)) {
       return value.map((item) => substitute(item));
     }
-    if (value && typeof value === "object") {
-      const out: Record<string, unknown> = {};
-      for (const [k, v] of Object.entries(value)) {
-        out[k] = substitute(v);
-      }
-      return out;
+    if (isJsonObject(value)) {
+      return substituteObject(value);
     }
     return value;
   };
-  return substitute(template) as Record<string, unknown>;
+  return substituteObject(template);
 }
 
 /**
@@ -314,37 +321,44 @@ function buildContext(
   entryKey: string | null = null,
 ): SchemaFormContext {
   const platform = getPlatform();
-  return {
+  const context: SchemaFormContext = {
     fieldKeyPrefix: `$settings.${sectionKey}`,
-    resolvePointer: (pointer, scope) =>
-      resolveContextPointer(pointer, {
-        projectConfig: (projectState?.projectConfig ?? {}) as Record<string, unknown>,
-        ...(scope !== undefined && { scope }),
-        ...(opts.formats !== undefined && { formats: opts.formats }),
-      }),
-    ...(typeof platform.setSecrets === "function"
-      ? {
-          commitSecret: async (key: string, value: string) => {
-            const envName = deriveSecretEnvName(sectionKey, entryKey, key);
-            await platform.setSecrets!({ set: { [envName]: value } });
-            return envName;
-          },
-        }
-      : {}),
+    resolvePointer: (pointer, scope) => {
+      const resolveOpts: ResolveContextOptions = {
+        projectConfig: projectState?.projectConfig ?? {},
+      };
+      /* The scope is the form's current value record, which is section JSON; the guard is what
+         lets the compiler see that, and it also stands for the old `!== undefined` test. */
+      if (isJsonObject(scope)) {
+        resolveOpts.scope = scope;
+      }
+      if (opts.formats !== undefined) {
+        resolveOpts.formats = opts.formats;
+      }
+      return resolveContextPointer(pointer, resolveOpts);
+    },
   };
+  if (isFunction(platform.setSecrets)) {
+    context.commitSecret = async (key: string, value: string) => {
+      const envName = deriveSecretEnvName(sectionKey, entryKey, key);
+      await platform.setSecrets!({ set: { [envName]: value } });
+      return envName;
+    };
+  }
+  return context;
 }
 
 /** The section's value object in the project config, created on demand. */
-function sectionValue(key: string): Record<string, unknown> | null {
-  const config = projectState?.projectConfig as Record<string, unknown> | null | undefined;
+function sectionValue(key: string): JsonObject | null {
+  const config = projectState?.projectConfig;
   if (!config) {
     return null;
   }
   const existing = config[key];
-  if (existing && typeof existing === "object" && !Array.isArray(existing)) {
-    return existing as Record<string, unknown>;
+  if (isJsonObject(existing)) {
+    return existing;
   }
-  const fresh: Record<string, unknown> = {};
+  const fresh: JsonObject = {};
   config[key] = fresh;
   return fresh;
 }
@@ -360,7 +374,8 @@ function sectionValue(key: string): Record<string, unknown> | null {
 interface SectionMount {
   /** The element the registry handed the renderer, and this record's identity. */
   container: HTMLElement;
-  handle: ContributedSurfaceHandle;
+  /** The mounted document, or `null` for the instant between creating this record and mounting. */
+  handle: ContributedSurfaceHandle | null;
   contribution: SettingsContribution;
   opts: ContributedSectionOptions;
   /** Where the schema form goes, once the document has made one. */
@@ -423,8 +438,7 @@ function project(record: SectionMount): ContributedView {
   const entries = sectionValue(sectionKey) ?? {};
   const selected = selectedEntries.get(sectionKey) ?? null;
   const value = selected === null ? undefined : entries[selected];
-  const editing =
-    selected !== null && value !== undefined && value !== null && typeof value === "object";
+  const editing = selected !== null && isNonNullObject(value);
   return {
     actionsState,
     editorRegion,
@@ -499,11 +513,14 @@ function paintForm(record: SectionMount): void {
   const entries = sectionValue(sectionKey) ?? {};
   const selected = selectedEntries.get(sectionKey) ?? null;
   const value = selected === null ? undefined : entries[selected];
-  if (selected === null || !value || typeof value !== "object") {
+  if (selected === null || !isNonNullObject(value)) {
     host.replaceChildren();
     return;
   }
-  const entry = value as Record<string, unknown>;
+  /* SAFETY: a map section's entries are objects by the section schema (`additionalProperties`
+     of type object), and the check above admits only objects; an array entry is malformed config
+     the form still treats as the record it should be, reading and writing named members. */
+  const entry = value as JsonObject;
   const base = `/${sectionKey}/${selected}`;
   place(
     mountSchemaForm(`section:${sectionKey}:${selected}`, contribution.entrySchema, entry, {
@@ -619,7 +636,7 @@ function actionsFor(record: SectionMount): ContributedActions {
         return;
       }
       // Rebuild the map to preserve entry order under the new key
-      const next: Record<string, unknown> = {};
+      const next: JsonObject = {};
       for (const [k, v] of Object.entries(target)) {
         next[k === selected ? slug : k] = v;
         delete target[k];
@@ -655,19 +672,20 @@ export function renderContributedSection(
     record.contribution = contribution;
     record.opts = opts;
   }
-  if (!record?.handle.attached()) {
-    record?.handle.dispose();
+  let handle = record?.handle ?? null;
+  if (!record || !handle?.attached()) {
+    handle?.dispose();
     const fresh: SectionMount = {
       actionsHost: null,
       container,
       contribution,
       echo: null,
       formHost: null,
-      handle: undefined as unknown as ContributedSurfaceHandle,
+      handle: null,
       opts,
     };
     mounts.set(container, fresh);
-    fresh.handle = mountContributedSurface(container, actionsFor(fresh), {
+    handle = mountContributedSurface(container, actionsFor(fresh), {
       actionsSlot: (host) => {
         fresh.actionsHost = host;
         paintActions(fresh);
@@ -677,9 +695,10 @@ export function renderContributedSection(
         paintForm(fresh);
       },
     });
+    fresh.handle = handle;
     record = fresh;
   }
-  record.handle.update(project(record));
+  handle.update(project(record));
   paintActions(record);
   paintForm(record);
 }

package/src/settings/css-vars-editor.ts

@@ -26,6 +26,7 @@
  *   change" true of the specimen canvas as well as the page one.
  */
 
+import { isNumber, isString } from "@jxsuite/schema/guards";
 import { errorMessage } from "@jxsuite/schema/parse";
 import { notify } from "../services/notify";
 import { projectState } from "../store";
@@ -110,7 +111,7 @@ function pending(container: HTMLElement, group: TokenGroupId): { name: string; v
 
 /** The live project configuration, or an empty one before a project is open. */
 function config(): ProjectConfig {
-  return (projectState?.projectConfig || {}) as ProjectConfig;
+  return projectState?.projectConfig || {};
 }
 
 /**
@@ -121,7 +122,7 @@ function config(): ProjectConfig {
  * write, because there is no object here for it to have been written into.
  */
 function style(): JxStyle {
-  return (config().style || {}) as JxStyle;
+  return config().style || {};
 }
 
 /** Every rendering context this project declares, schemes first. */
@@ -159,10 +160,18 @@ async function persist(container: HTMLElement): Promise<void> {
 // ─── Projection ──────────────────────────────────────────────────────────────
 
 /** The colour a well is painted, and the colour the native input under it opens on. */
-function well(
-  resolved: string | number | undefined,
-  fallback: string,
-): { swatchBg: string; swatchInput: string } {
+interface SwatchWell {
+  /** The resolved value as a CSS background, or `transparent` when the token resolves to nothing. */
+  swatchBg: string;
+  /**
+   * What the native colour input opens on: the background when it is a hex colour, else the
+   * fallback.
+   */
+  swatchInput: string;
+}
+
+/** Paint one well from what the token resolves to. */
+function well(resolved: string | number | undefined, fallback: string): SwatchWell {
   const swatchBg = resolved === undefined ? "transparent" : String(resolved);
   return { swatchBg, swatchInput: swatchBg.startsWith("#") ? swatchBg : fallback };
 }
@@ -344,13 +353,25 @@ function addOverride(container: HTMLElement, token: string, ctx: string): void {
     return;
   }
   const base = rootStyle[token];
-  if (typeof base !== "string" && typeof base !== "number") {
+  if (!isString(base) && !isNumber(base)) {
     return;
   }
   writeTokenOverride(rootStyle, context, token, String(base));
   commit(container);
 }
 
+/** Switch Settings to the Contexts section, saying so when the section registry will not load. */
+async function openContextsSection(): Promise<void> {
+  try {
+    const { setSettingsSection } = await import("./section-registry");
+    setSettingsSection("contexts");
+  } catch (error) {
+    notify.error(`Could not open Settings › Contexts — ${errorMessage(error)}`, {
+      source: "Settings",
+    });
+  }
+}
+
 /**
  * What the reader may do, bound to one container.
  *
@@ -378,13 +399,7 @@ function actions(container: HTMLElement): CssVarsActions {
        * breaks the css-vars-editor ↔ section-registry cycle.
        */
       manageContexts: () => {
-        void import("./section-registry")
-          .then(({ setSettingsSection }) => setSettingsSection("contexts"))
-          .catch((error: unknown) => {
-            notify.error(`Could not open Settings › Contexts — ${errorMessage(error)}`, {
-              source: "Settings",
-            });
-          });
+        void openContextsSection();
       },
       removeToken: (name: string) => {
         delete style()[name];
@@ -399,10 +414,16 @@ function actions(container: HTMLElement): CssVarsActions {
        * predecessor's `ref()` into the DOM could never do.
        */
       setAddName: (groupId: string, value: string) => {
-        pending(container, groupId as TokenGroupId).name = value;
+        const group = groupOf(groupId);
+        if (group) {
+          pending(container, group.id).name = value;
+        }
       },
       setAddValue: (groupId: string, value: string) => {
-        pending(container, groupId as TokenGroupId).value = value;
+        const group = groupOf(groupId);
+        if (group) {
+          pending(container, group.id).value = value;
+        }
       },
       setOverride: (token: string, ctx: string, value: string) => {
         const context = contexts().find((c) => c.name === ctx);

package/src/settings/defs-editor.ts

@@ -40,10 +40,12 @@
  * @docs studio/projects/settings
  */
 
+import { isBoolean, isJsonObject, isString } from "@jxsuite/schema/guards";
 import { projectState } from "../store";
 import { commitProjectConfig } from "../tabs/project-config";
 import { camelToLabel } from "../utils/studio-utils";
 import {
+  BLANK_FIELD,
   FIELD_TYPES,
   FORMAT_OPTIONS,
   detectFieldFormat,
@@ -52,39 +54,30 @@ import {
 } from "./schema-field-ui";
 import { mountDefsSurface } from "../surfaces/settings-defs";
 
-import type { SchemaProperty } from "./schema-field-ui";
+import type { NestedFieldDraft, SchemaProperty } from "./schema-field-ui";
 import type {
   DefsActions,
-  DefsChoice,
   DefsFieldView,
   DefsSurfaceHandle,
   DefsView,
   NestedFieldView,
 } from "../surfaces/settings-defs";
-import type { ContentTypeSchema, ContentTypeSchemaField } from "@jxsuite/schema/types";
+import type { ContentTypeSchema } from "@jxsuite/schema/types";
+import type { SurfaceOption } from "../ui/surface";
 
 // ─── The pickers ──────────────────────────────────────────────────────────────
 
 /** The field types, as picker rows. One array, made once: an equal write re-runs no binding. */
-const TYPE_ROWS: DefsChoice[] = FIELD_TYPES.map((type) => ({ label: type, value: type }));
+const TYPE_ROWS: SurfaceOption[] = FIELD_TYPES.map((type) => ({ label: type, value: type }));
 
 /** The formats, with the empty one named: a blank row says nothing about what choosing it does. */
-const FORMAT_ROWS: DefsChoice[] = FORMAT_OPTIONS.map((format) => ({
+const FORMAT_ROWS: SurfaceOption[] = FORMAT_OPTIONS.map((format) => ({
   label: format || "(none)",
   value: format,
 }));
 
-/** What a new field starts as. */
-const BLANK_FIELD = { format: "", name: "", required: false, type: "string" };
-
 // ─── Per-container state ──────────────────────────────────────────────────────
 
-/** One object field's unsubmitted nested field. */
-interface Draft {
-  name: string;
-  type: string;
-}
-
 /**
  * What the reader is part-way through, per rendered container.
  *
@@ -100,7 +93,7 @@ interface DefsUi {
   addOpen: boolean;
   add: { format: string; name: string; required: boolean; type: string };
   /** Each object field's nested add row, by the field's name on disk. */
-  drafts: Map<string, Draft>;
+  drafts: Map<string, NestedFieldDraft>;
   /**
    * The ONE control whose value has parted from what is on disk, and what it holds.
    *
@@ -166,9 +159,19 @@ function hush(container: HTMLElement): void {
   uiFor(container).echo = null;
 }
 
-/** What a control shows: what it was last told it holds, or what is on disk. */
-function shown<T extends string | boolean>(ui: DefsUi, id: string, disk: T): T {
-  return ui.echo?.id === id ? (ui.echo.value as T) : disk;
+/**
+ * What a text control shows: what it was last told it holds, or what is on disk.
+ *
+ * The echo's value is a string whenever its id is a {@link nameId}, because only the rename setters
+ * echo under one; the check states that rather than assuming it.
+ */
+function shownText(ui: DefsUi, id: string, disk: string): string {
+  return ui.echo?.id === id && isString(ui.echo.value) ? ui.echo.value : disk;
+}
+
+/** What a checkbox shows: what it was last told it holds, or what is on disk. */
+function shownFlag(ui: DefsUi, id: string, disk: boolean): boolean {
+  return ui.echo?.id === id && isBoolean(ui.echo.value) ? ui.echo.value : disk;
 }
 
 // ─── Persistence ──────────────────────────────────────────────────────────────
@@ -200,6 +203,9 @@ function persist(): void {
  * outlive a projection, which makes reading through to the store the thing that keeps them honest.
  */
 function currentDefs(): Record<string, ContentTypeSchema> | undefined {
+  /* SAFETY: project `$defs` holds JSON Schema type definitions (project-schema.json, "Global type
+     definitions") and this editor is their writer; ContentTypeSchema is that schema object's view,
+     every member optional, and each read below falls back (`?? {}`, `?? []`) when one is absent. */
   return projectState?.projectConfig?.$defs as Record<string, ContentTypeSchema> | undefined;
 }
 
@@ -213,19 +219,16 @@ function selectedDef(ui: DefsUi): ContentTypeSchema | undefined {
  * resolves through `#/$context/content`, read here without a schema-form context to go through.
  */
 function contentTypeNames(): string[] {
-  const content = (projectState?.projectConfig as Record<string, unknown> | null | undefined)
-    ?.content;
-  return content && typeof content === "object" && !Array.isArray(content)
-    ? Object.keys(content)
-    : [];
+  const content = projectState?.projectConfig?.content;
+  return isJsonObject(content) ? Object.keys(content) : [];
 }
 
 /** The last target list, so an unchanged one is the same array and re-runs no binding. */
-let targetRows: DefsChoice[] = [];
+let targetRows: SurfaceOption[] = [];
 let targetNames = " ";
 
 /** The reference targets, as picker rows. */
-function targets(): DefsChoice[] {
+function targets(): SurfaceOption[] {
   const names = contentTypeNames();
   const id = names.join(" ");
   if (id !== targetNames) {
@@ -256,9 +259,9 @@ function nestedView(
     format: detectFieldFormat(schema),
     hasFormat: takesFormat(type),
     key,
-    name: shown(ui, nameId(id), key),
+    name: shownText(ui, nameId(id), key),
     parent,
-    required: shown(ui, requiredId(id), required),
+    required: shownFlag(ui, requiredId(id), required),
     type,
   };
 }
@@ -290,9 +293,9 @@ function fieldView(
     hasFormat: takesFormat(type),
     key,
     label: camelToLabel(key),
-    name: shown(ui, nameId(key), key),
+    name: shownText(ui, nameId(key), key),
     refTarget: schema.$ref ? schema.$ref.replace(/^#\/[^/]+\//, "") : "",
-    required: shown(ui, requiredId(key), required),
+    required: shownFlag(ui, requiredId(key), required),
     type,
   };
 }
@@ -312,15 +315,15 @@ function project(container: HTMLElement): DefsView {
     addRequired: ui.add.required,
     addState: ui.addOpen && def ? "open" : "closed",
     addType: ui.add.type,
+    defs: Object.keys(defs).map((name) => ({ name, selected: name === ui.selected })),
     editorState: def ? "editing" : "empty",
     fields: Object.entries(properties).map(([key, schema]) =>
-      fieldView(ui, key, schema as SchemaProperty, required.includes(key), rows.length > 0),
+      fieldView(ui, key, schema, required.includes(key), rows.length > 0),
     ),
     formatOptions: FORMAT_ROWS,
     newName: ui.newName,
     newState: ui.newOpen ? "open" : "closed",
     selected: def && ui.selected !== null ? ui.selected : "",
-    shapes: Object.keys(defs).map((name) => ({ name, selected: name === ui.selected })),
     targets: rows,
     typeOptions: TYPE_ROWS,
   };
@@ -338,10 +341,10 @@ function actionsFor(container: HTMLElement): DefsActions {
 
   /** The parent object a nested edit is about, or `undefined` when it has gone underneath. */
   const parentOf = (parent: string): SchemaProperty | undefined =>
-    selectedDef(ui)?.properties?.[parent] as SchemaProperty | undefined;
+    selectedDef(ui)?.properties?.[parent];
 
   /** One object field's add-row draft, made on first use. */
-  const draftOf = (parent: string): Draft => {
+  const draftOf = (parent: string): NestedFieldDraft => {
     let draft = ui.drafts.get(parent);
     if (!draft) {
       draft = { name: "", type: "string" };
@@ -513,7 +516,7 @@ function actionsFor(container: HTMLElement): DefsActions {
       }
       done();
     },
-    removeShape() {
+    removeSelected() {
       const defs = currentDefs();
       if (ui.selected === null || !defs?.[ui.selected]) {
         return;
@@ -613,7 +616,7 @@ function actionsFor(container: HTMLElement): DefsActions {
       }
       /* The pointer form is `#/content/<type>` — the same one `ui/form-controls.ts` writes, so a
          reference authored here and one authored in the content-types builder are the same value. */
-      def.properties[key] = { $ref: `#/content/${value}` } as ContentTypeSchemaField;
+      def.properties[key] = { $ref: `#/content/${value}` };
       done();
     },
     setType(key, value) {

package/src/settings/extension-rows.ts

@@ -42,7 +42,7 @@ export interface ExtensionRow {
 
 /** The live project configuration, or an empty one before a project is open. */
 export function projectConfig(): ProjectConfig {
-  return (projectState?.projectConfig ?? {}) as ProjectConfig;
+  return projectState?.projectConfig ?? {};
 }
 
 /** The specifiers `project.json` currently enables. */
@@ -71,7 +71,7 @@ export function buildRows(packages: PackageInfo[] | null = null): ExtensionRow[]
     const here = entry.installed ?? installed.has(entry.name);
     const bundled = entry.bundled ?? false;
     const info = resolved.get(entry.name);
-    rows.push({
+    const row: ExtensionRow = {
       broken: enabled.has(specifier) && !here && !bundled,
       bundled,
       enabled: enabled.has(specifier),
@@ -83,9 +83,14 @@ export function buildRows(packages: PackageInfo[] | null = null): ExtensionRow[]
         : entry.sections.map((s) => s.key),
       specifier,
       title: entry.title ?? entry.name,
-      ...(entry.description === undefined ? {} : { description: entry.description }),
-      ...(entry.problem === undefined ? {} : { unavailable: entry.problem }),
-    });
+    };
+    if (entry.description !== undefined) {
+      row.description = entry.description;
+    }
+    if (entry.problem !== undefined) {
+      row.unavailable = entry.problem;
+    }
+    rows.push(row);
   }
 
   // Anything project.json names that the backend did not describe. It can still be turned OFF,
@@ -95,7 +100,7 @@ export function buildRows(packages: PackageInfo[] | null = null): ExtensionRow[]
       continue;
     }
     const info = resolved.get(specifier);
-    rows.push({
+    const row: ExtensionRow = {
       broken: !installed.has(specifier) && info === undefined,
       bundled: false,
       enabled: true,
@@ -105,8 +110,11 @@ export function buildRows(packages: PackageInfo[] | null = null): ExtensionRow[]
       sections: info ? info.contributions.map((c) => c.project.key) : [],
       specifier,
       title: info?.title ?? specifier,
-      ...(info?.description === undefined ? {} : { description: info.description }),
-    });
+    };
+    if (info?.description !== undefined) {
+      row.description = info.description;
+    }
+    rows.push(row);
   }
 
   return rows;