@jxsuite/studio 6.1.1 → 6.1.2

package/src/grid/sources/connector-source.ts

@@ -9,9 +9,17 @@
  * deletion. The server is authoritative — there are no file fingerprints; conflicts surface as
  * per-row API errors.
  */
+import { isJsonPrimitive, isNumber, isString } from "@jxsuite/schema/guards";
+import type { JsonObject, JsonValue } from "@jxsuite/schema/types";
 import { deleteRow, fetchRows, insertRow, updateRow } from "../../services/data-service";
 import { makeGridTabId } from "../grid-source";
-import type { DataColumnMeta, DataRowsQuery } from "../../types";
+import type {
+  DataColumnMeta,
+  DataRowDelete,
+  DataRowInsert,
+  DataRowUpdate,
+  DataRowsQuery,
+} from "../../types";
 import type {
   CommitResult,
   GridCellValue,
@@ -48,11 +56,11 @@ function editableColumn(column: DataColumnMeta): boolean {
 }
 
 /** DB value → grid cell value (JSON-ish values surface as text). */
-function toCellValue(value: unknown): GridCellValue {
-  if (value === null || value === undefined) {
+function toCellValue(value: JsonValue | undefined): GridCellValue {
+  if (value === undefined) {
     return null;
   }
-  if (typeof value === "string" || typeof value === "number" || typeof value === "boolean") {
+  if (isJsonPrimitive(value)) {
     return value;
   }
   return JSON.stringify(value);
@@ -72,17 +80,23 @@ export function createConnectorSource(connection: string | undefined, table: str
     }
   };
 
-  const toQuery = (query?: GridQuery): DataRowsQuery => ({
-    table,
-    ...(connection ? { connection } : {}),
-    limit: query?.limit ?? DATA_PAGE_SIZE,
-    offset: query?.offset ?? 0,
-    ...(query?.orderBy ? { dir: query.dir ?? "asc", orderBy: query.orderBy } : {}),
-  });
+  const toQuery = (query?: GridQuery): DataRowsQuery => {
+    const rowsQuery: DataRowsQuery = { table };
+    if (connection) {
+      rowsQuery.connection = connection;
+    }
+    rowsQuery.limit = query?.limit ?? DATA_PAGE_SIZE;
+    rowsQuery.offset = query?.offset ?? 0;
+    if (query?.orderBy) {
+      rowsQuery.dir = query.dir ?? "asc";
+      rowsQuery.orderBy = query.orderBy;
+    }
+    return rowsQuery;
+  };
 
-  const rowKeyOf = (row: Record<string, unknown>, index: number): string => {
+  const rowKeyOf = (row: JsonObject, index: number): string => {
     const pk = row[pkName];
-    if (typeof pk === "string" || typeof pk === "number") {
+    if (isString(pk) || isNumber(pk)) {
       pkValues.set(String(pk), pk);
       return String(pk);
     }
@@ -132,11 +146,15 @@ export function createConnectorSource(connection: string | undefined, table: str
           continue;
         }
         try {
-          const set: Record<string, unknown> = {};
+          const set: JsonObject = {};
           for (const patch of patches) {
             set[patch.field] = patch.value;
           }
-          await updateRow({ pk, set, table, ...(connection ? { connection } : {}) });
+          const update: DataRowUpdate = { pk, set, table };
+          if (connection) {
+            update.connection = connection;
+          }
+          await updateRow(update);
           outcome(true);
         } catch (error) {
           outcome(false, error instanceof Error ? error.message : String(error));
@@ -145,16 +163,21 @@ export function createConnectorSource(connection: string | undefined, table: str
 
       for (const insert of batch.inserts) {
         try {
-          const values: Record<string, unknown> = {};
+          const values: JsonObject = {};
           for (const [field, value] of Object.entries(insert.cells)) {
             if (value !== null) {
               values[field] = value;
             }
           }
-          const { row } = await insertRow({ table, values, ...(connection ? { connection } : {}) });
+          const request: DataRowInsert = { table, values };
+          if (connection) {
+            request.connection = connection;
+          }
+          const { row } = await insertRow(request);
           const pk = row[pkName];
-          const newKey = typeof pk === "string" || typeof pk === "number" ? String(pk) : undefined;
-          if (newKey !== undefined && (typeof pk === "string" || typeof pk === "number")) {
+          let newKey: string | undefined;
+          if (isString(pk) || isNumber(pk)) {
+            newKey = String(pk);
             pkValues.set(newKey, pk);
           }
           result.inserts.push({ newKey, ok: true, tempKey: insert.tempKey });
@@ -178,7 +201,11 @@ export function createConnectorSource(connection: string | undefined, table: str
           continue;
         }
         try {
-          await deleteRow({ pk, table, ...(connection ? { connection } : {}) });
+          const request: DataRowDelete = { pk, table };
+          if (connection) {
+            request.connection = connection;
+          }
+          await deleteRow(request);
           result.deletes.push({ ok: true, rowKey: del.rowKey });
         } catch (error) {
           result.deletes.push({

package/src/grid/sources/content-source.ts

@@ -51,13 +51,16 @@ import type {
   GridSource,
 } from "../grid-source";
 import type { ContentSectionEntry } from "../../types";
+import type { EntryFields } from "../../content/entry-fields";
+import type { JxDocumentValue, JxMutableNode } from "@jxsuite/schema/types";
+import { isJsonPrimitive, isNonNullObject, isString } from "@jxsuite/schema/guards";
 
 /** The synthetic identity column: the entry's project path. Editable only on pending inserts. */
 export const PATH_FIELD = "__path";
 
 interface EntryRecord {
-  frontmatter: Record<string, unknown>;
-  document: Record<string, unknown>;
+  frontmatter: EntryFields;
+  document: JxMutableNode;
   rawText: string;
   formatName: string;
 }
@@ -69,13 +72,17 @@ interface CollectionInfo {
   ext: string;
 }
 
+/** The open project's `content` section (collection name → entry), or an empty one. */
+function contentSection(): Record<string, ContentSectionEntry> {
+  /* SAFETY: `content` is the parser extension's section — collection name → entry object, its
+     project fragment schema the source of truth. `ContentSectionEntry` names only the fields Studio
+     reads, every one optional, and both readers below test `source` before using it. */
+  return (projectState?.projectConfig?.content ?? {}) as Record<string, ContentSectionEntry>;
+}
+
 /** Resolve a content type's definition, source dir, and entry extension from project config. */
 export function collectionInfo(typeName: string): CollectionInfo | null {
-  const content = (projectState?.projectConfig?.content ?? {}) as Record<
-    string,
-    ContentSectionEntry
-  >;
-  const def = content[typeName];
+  const def = contentSection()[typeName];
   if (!def?.source) {
     return null;
   }
@@ -91,11 +98,7 @@ export function collectionInfo(typeName: string): CollectionInfo | null {
 
 /** Directory-backed collections in the project (for "Edit collection in Grid" affordances). */
 export function collectionDirs(): { name: string; dir: string }[] {
-  const content = (projectState?.projectConfig?.content ?? {}) as Record<
-    string,
-    ContentSectionEntry
-  >;
-  return Object.entries(content)
+  return Object.entries(contentSection())
     .filter(([, def]) => {
       const source = def.source?.replace(/^\.\//, "") ?? "";
       return source !== "" && !/\.[a-z0-9]+$/i.test(source);
@@ -160,14 +163,14 @@ async function mapLimit<T, R>(
 }
 
 /** Frontmatter value → grid cell value. Nested objects surface as read-only JSON text. */
-function toCellValue(value: unknown): GridCellValue {
-  if (value === null || value === undefined) {
+function toCellValue(value: JxDocumentValue | undefined): GridCellValue {
+  if (value === undefined) {
     return null;
   }
-  if (typeof value === "string" || typeof value === "number" || typeof value === "boolean") {
+  if (isJsonPrimitive(value)) {
     return value;
   }
-  if (Array.isArray(value) && value.every((v) => typeof v !== "object" || v === null)) {
+  if (Array.isArray(value) && value.every((v) => !isNonNullObject(v))) {
     return value.map(String);
   }
   return JSON.stringify(value);
@@ -187,9 +190,7 @@ function openTabFor(path: string) {
 function extraRowsAreStrings(rows: Record<string, GridCellValue>[], field: string): boolean {
   return rows.every((row) => {
     const value = row[field];
-    return value === null || value === undefined || typeof value !== "string"
-      ? true
-      : !value.startsWith("{") && !value.startsWith("[");
+    return !isString(value) || (!value.startsWith("{") && !value.startsWith("["));
   });
 }
 
@@ -294,7 +295,7 @@ function buildEntryFileSource(opts: EntryFileSourceOptions): GridSource {
           return {
             document: parsed.document,
             formatName: parsed.format.name,
-            frontmatter: parsed.frontmatter ?? {},
+            frontmatter: parsed.frontmatter,
             rawText: text,
           };
         } catch {
@@ -393,7 +394,7 @@ function buildEntryFileSource(opts: EntryFileSourceOptions): GridSource {
   };
 
   const rowFor = (path: string, entry: EntryRecord): GridRow => {
-    const cells: Record<string, GridCellValue> = { [PATH_FIELD]: path };
+    const cells: GridRow["cells"] = { [PATH_FIELD]: path };
     for (const column of buildColumns()) {
       if (column.field === PATH_FIELD) {
         continue;
@@ -490,7 +491,7 @@ function buildEntryFileSource(opts: EntryFileSourceOptions): GridSource {
       // Inserts: new entry files from the Path cell.
       for (const insert of batch.inserts) {
         const rawPath = insert.cells[PATH_FIELD];
-        if (typeof rawPath !== "string" || rawPath.trim() === "") {
+        if (!isString(rawPath) || rawPath.trim() === "") {
           result.inserts.push({ error: "Path is required", ok: false, tempKey: insert.tempKey });
           continue;
         }
@@ -522,7 +523,7 @@ function buildEntryFileSource(opts: EntryFileSourceOptions): GridSource {
           continue;
         }
         try {
-          const frontmatter: Record<string, unknown> = {};
+          const frontmatter: Record<string, GridCellValue> = {};
           for (const [field, value] of Object.entries(insert.cells)) {
             if (field !== PATH_FIELD && value !== null) {
               frontmatter[field] = value;
@@ -539,7 +540,7 @@ function buildEntryFileSource(opts: EntryFileSourceOptions): GridSource {
           entries.set(path, {
             document: parsed.document,
             formatName: parsed.format.name,
-            frontmatter: parsed.frontmatter ?? {},
+            frontmatter: parsed.frontmatter,
             rawText: text,
           });
           result.inserts.push({ newKey: path, ok: true, tempKey: insert.tempKey });

package/src/grid/sources/csv-file-source.ts

@@ -84,7 +84,7 @@ function buildModel(text: string, path: string): CsvModel {
 
   // Typed columns when this CSV is a collection source; sniffed kinds otherwise.
   const contentType = findContentTypeSchema(path, projectState?.projectConfig);
-  const schemaProps = contentType?.schema?.properties ?? {};
+  const schemaProps = contentType?.schema.properties ?? {};
   const sampleRows = doc.rows.slice(0, 50).map((row) => {
     const cells: Record<string, GridCellValue> = {};
     for (const [i, field] of fields.entries()) {
@@ -116,7 +116,7 @@ function buildModel(text: string, path: string): CsvModel {
 }
 
 /** Typed cell values for one matrix row. */
-function rowCells(model: CsvModel, rowIndex: number): Record<string, GridCellValue> {
+function rowCells(model: CsvModel, rowIndex: number): GridRow["cells"] {
   const cells: Record<string, GridCellValue> = {};
   const row = model.doc.rows[rowIndex] ?? [];
   for (const [i, field] of model.fields.entries()) {

package/src/grid/tabulator-tables.d.ts

@@ -1,26 +1,35 @@
 /**
  * Minimal ambient types for tabulator-tables 6.x — the package ships no declarations. Only the
  * surface grid-view.ts touches is typed; extend here as usage grows.
+ *
+ * Every component is generic over `Value`, the type of one cell value. Tabulator stores the row
+ * objects it is handed (`data`, `replaceData`) and gives the same values back (`getValue`,
+ * `getData`, sorter and filter arguments), so the caller that builds the table names the value type
+ * once and reads it back from every component. Values written in (`setValue`, an editor's
+ * `success`, `selectableRangeClearCellsValue`) must be of that same type for the promise to hold.
  */
 declare module "tabulator-tables" {
-  export interface CellComponent {
-    getValue: () => unknown;
+  /** One row object as the table stores it: cell values keyed by field. */
+  export type RowData<Value> = Record<string, Value>;
+
+  export interface CellComponent<Value> {
+    getValue: () => Value;
     getField: () => string;
     getElement: () => HTMLElement;
-    getRow: () => RowComponent;
-    setValue: (value: unknown, mutate?: boolean) => void;
+    getRow: () => RowComponent<Value>;
+    setValue: (value: Value, mutate?: boolean) => void;
   }
 
-  export interface RowComponent {
-    getData: () => Record<string, unknown>;
+  export interface RowComponent<Value> {
+    getData: () => RowData<Value>;
     getElement: () => HTMLElement;
-    getCells: () => CellComponent[];
+    getCells: () => CellComponent<Value>[];
     reformat: () => void;
   }
 
-  export interface RangeComponent {
-    getRows: () => RowComponent[];
-    getCells: () => CellComponent[][];
+  export interface RangeComponent<Value> {
+    getRows: () => RowComponent<Value>[];
+    getCells: () => CellComponent<Value>[][];
   }
 
   export interface ColumnComponent {
@@ -28,28 +37,29 @@ declare module "tabulator-tables" {
     getWidth: () => number;
   }
 
-  export type CellEditor = (
-    cell: CellComponent,
+  export type CellEditor<Value> = (
+    cell: CellComponent<Value>,
     onRendered: (fn: () => void) => void,
-    success: (value: unknown) => void,
+    success: (value: Value) => void,
     cancel: () => void,
   ) => HTMLElement | false;
 
-  export type CellFormatter = (
-    cell: CellComponent,
-    formatterParams: Record<string, unknown>,
+  export type CellFormatter<Value> = (
+    cell: CellComponent<Value>,
+    /** Always `{}`: no column definition here declares `formatterParams`. */
+    formatterParams: Record<string, never>,
     onRendered: (fn: () => void) => void,
   ) => HTMLElement | string;
 
-  export interface ColumnDefinition {
+  export interface ColumnDefinition<Value> {
     title: string;
     field?: string | undefined;
     width?: number | undefined;
     frozen?: boolean | undefined;
-    editor?: CellEditor | undefined;
-    editable?: boolean | ((cell: CellComponent) => boolean) | undefined;
-    formatter?: CellFormatter | undefined;
-    sorter?: string | ((a: unknown, b: unknown) => number) | undefined;
+    editor?: CellEditor<Value> | undefined;
+    editable?: boolean | ((cell: CellComponent<Value>) => boolean) | undefined;
+    formatter?: CellFormatter<Value> | undefined;
+    sorter?: string | ((a: Value, b: Value) => number) | undefined;
     headerSort?: boolean | undefined;
     headerFilter?: string | undefined;
     cssClass?: string | undefined;
@@ -57,9 +67,20 @@ declare module "tabulator-tables" {
     headerSortTristate?: boolean | undefined;
   }
 
-  export interface Options {
-    data?: Record<string, unknown>[];
-    columns?: ColumnDefinition[];
+  /** Which table regions a clipboard copy includes. */
+  export interface ClipboardCopyConfig {
+    columnHeaders?: boolean;
+    columnGroups?: boolean;
+    rowHeaders?: boolean;
+    rowGroups?: boolean;
+    columnCalcs?: boolean;
+    dataTree?: boolean;
+    formatCells?: boolean;
+  }
+
+  export interface Options<Value> {
+    data?: RowData<Value>[];
+    columns?: ColumnDefinition<Value>[];
     index?: string;
     layout?: string;
     height?: string | number;
@@ -73,27 +94,27 @@ declare module "tabulator-tables" {
     selectableRangeColumns?: boolean;
     selectableRangeRows?: boolean;
     selectableRangeClearCells?: boolean;
-    selectableRangeClearCellsValue?: unknown;
+    selectableRangeClearCellsValue?: Value;
     clipboard?: boolean;
     clipboardCopyStyled?: boolean;
-    clipboardCopyConfig?: Record<string, unknown>;
+    clipboardCopyConfig?: ClipboardCopyConfig;
     clipboardCopyRowRange?: string;
     clipboardPasteParser?: string;
     clipboardPasteAction?: string;
     movableColumns?: boolean;
-    columnDefaults?: Partial<ColumnDefinition>;
-    rowFormatter?: (row: RowComponent) => void;
+    columnDefaults?: Partial<ColumnDefinition<Value>>;
+    rowFormatter?: (row: RowComponent<Value>) => void;
     placeholder?: string;
   }
 
-  export class Tabulator {
-    constructor(host: HTMLElement, options: Options);
+  export class Tabulator<Value> {
+    constructor(host: HTMLElement, options: Options<Value>);
     static registerModule(modules: unknown[]): void;
     on(event: string, callback: (...args: never[]) => void): void;
-    replaceData(data: Record<string, unknown>[]): Promise<void>;
-    getRows(activeOnly?: string): RowComponent[];
-    getRanges(): RangeComponent[];
-    setFilter(filter: (data: Record<string, unknown>) => boolean): void;
+    replaceData(data: RowData<Value>[]): Promise<void>;
+    getRows(activeOnly?: string): RowComponent<Value>[];
+    getRanges(): RangeComponent<Value>[];
+    setFilter(filter: (data: RowData<Value>) => boolean): void;
     clearFilter(includeHeaderFilters?: boolean): void;
     redraw(force?: boolean): void;
     destroy(): void;

package/src/hosting/stage.ts

@@ -15,6 +15,7 @@ import { cp, mkdir, readFile, rm, stat, writeFile } from "node:fs/promises";
 import { existsSync } from "node:fs";
 import { dirname, join, resolve } from "node:path";
 import { Glob } from "bun";
+import { isNonNullObject } from "@jxsuite/schema/guards";
 import { STUDIO_ASSETS, STUDIO_CANVAS, STUDIO_SHELL } from "./layout";
 import type { AssetBase, StudioAsset, StudioAssetKind, StudioLayoutMode } from "./layout";
 import { canvasShellHtml } from "./document";
@@ -193,11 +194,10 @@ export async function writeAssetManifest(
 ): Promise<void> {
   const out = manifestPath(root);
   await mkdir(dirname(out), { recursive: true });
-  const manifest = {
-    assets: STUDIO_ASSETS,
-    ...(extra.preload === undefined ? {} : { preload: extra.preload }),
-    shell: STUDIO_SHELL,
-  };
+  const manifest =
+    extra.preload === undefined
+      ? { assets: STUDIO_ASSETS, shell: STUDIO_SHELL }
+      : { assets: STUDIO_ASSETS, preload: extra.preload, shell: STUDIO_SHELL };
   await writeFile(out, `${JSON.stringify(manifest, null, 2)}\n`);
 }
 
@@ -231,8 +231,8 @@ export async function studioPreload(root: string = STUDIO_PACKAGE_DIR): Promise<
   if (!existsSync(path)) {
     return [];
   }
-  const manifest = JSON.parse(await readFile(path, "utf8")) as { preload?: unknown };
-  const { preload } = manifest;
+  const manifest: unknown = JSON.parse(await readFile(path, "utf8"));
+  const preload = isNonNullObject(manifest) && "preload" in manifest ? manifest.preload : undefined;
   if (!Array.isArray(preload)) {
     return [];
   }

package/src/i18n/i18n-commands.ts

@@ -47,7 +47,6 @@ import {
 } from "../commands/command-args";
 import type { CommandArgValues } from "../commands/command-args";
 import type { AnyCommand, CommandRegistry } from "../commands/registry";
-import type { Tab } from "../tabs/tab";
 
 /** The tags a translation verb may be asked for — the project's declared locales, read live. */
 function declaredLocales(): string[] {
@@ -98,7 +97,7 @@ function addressedDocument(commandId: string, args: CommandArgValues): string {
     }
     return path;
   }
-  const tab = paneId === undefined ? (activeTab.value as Tab | null) : tabOfPane(paneId);
+  const tab = paneId === undefined ? activeTab.value : tabOfPane(paneId);
   if (!tab) {
     throw new RangeError(
       paneId === undefined

package/src/media/media-pane.ts

@@ -228,7 +228,10 @@ export function renderMediaMode(surface: CanvasSurface, tab: Tab): void {
   const scope = effectScope();
   const mounted = mountMediaSurface(canvasWrap, view(), {
     copyRef: () => {
-      void navigator.clipboard?.writeText(ref);
+      // `navigator.clipboard` is [SecureContext]: a plain-http host has none, whatever lib.dom says.
+      if ("clipboard" in navigator) {
+        void navigator.clipboard.writeText(ref);
+      }
     },
     imageLoaded: (width, height) => {
       /* The only honest source of pixel dimensions in a browser is an image that has loaded, and

package/src/new-project/add-repo-modal.ts

@@ -35,6 +35,7 @@
 
 import { actIfRequired } from "../account/action-flow";
 import { errorMessage } from "@jxsuite/schema/parse";
+import { isFunction } from "@jxsuite/schema/guards";
 import {
   getAccountStatus,
   getRepoAccessLinks,
@@ -90,7 +91,7 @@ let _resolve: ((result: { root: string } | null) => void) | null = null;
 /** True when the active platform can browse + adopt existing repositories. */
 export function platformSupportsAddRepo(): boolean {
   const platform = getPlatform();
-  return typeof platform.listRepos === "function" && typeof platform.importProject === "function";
+  return isFunction(platform.listRepos) && isFunction(platform.importProject);
 }
 
 /** True when the active platform routes Open Project through this repo picker. */
@@ -207,7 +208,7 @@ function loadRepos(): void {
         requestScan(_selected, "now");
       }
     })
-    .catch((error: unknown) => {
+    .catch((error) => {
       _repos = [];
       _error = errorMessage(error);
     })
@@ -492,11 +493,8 @@ async function adopt(fullName: string, dir: string, retried = false): Promise<vo
   _error = "";
   redraw();
   try {
-    const imported = await getPlatform().importProject?.({
-      name: repo.name,
-      owner: repo.owner,
-      ...(dir ? { dir } : {}),
-    });
+    const request = { name: repo.name, owner: repo.owner };
+    const imported = await getPlatform().importProject?.(dir ? { ...request, dir } : request);
     if (_handle !== handle) {
       return;
     }
@@ -604,13 +602,16 @@ function accessOf(): AddRepoView["access"] {
   ];
 }
 
-/** The chosen repository's pane: its state, its rows, and the sentence under the folder field. */
-function detailOf(repo: RepoInfo | undefined): Pick<
+/** The chosen repository's pane, plus the branch it is read at. */
+interface RepoDetail extends Pick<
   AddRepoView,
   "detailState" | "detailName" | "detailMeta" | "locations" | "folderHelp"
-> & {
+> {
   branch: string;
-} {
+}
+
+/** The chosen repository's pane: its state, its rows, and the sentence under the folder field. */
+function detailOf(repo: RepoInfo | undefined): RepoDetail {
   if (!repo) {
     return {
       branch: "",
@@ -623,12 +624,13 @@ function detailOf(repo: RepoInfo | undefined): Pick<
   }
   const scanned = _scans.get(repo.fullName);
   const branch = scanned?.status === "done" ? scanned.projects.branch : repo.defaultBranch;
+  const noLocations: AddRepoLocation[] = [];
   const base = {
     branch,
     detailMeta: [branch, repo.permission, ...(repo.private ? ["private"] : [])].join(" · "),
     detailName: repo.fullName,
     folderHelp: "The folder that holds project.json. Leave it empty for the repository root.",
-    locations: [] as AddRepoLocation[],
+    locations: noLocations,
   };
   if (!getPlatform().listRepoProjects) {
     return { ...base, detailState: "unlisted" };
@@ -640,17 +642,16 @@ function detailOf(repo: RepoInfo | undefined): Pick<
     return { ...base, detailState: "failed" };
   }
   const { locations, truncated } = scanned.projects;
-  return {
+  const detail: RepoDetail = {
     ...base,
     detailState: locations.length > 0 ? "listed" : "empty",
-    ...(truncated
-      ? {
-          folderHelp:
-            "This repository is too large to scan in full. Name the folder if its project isn't listed.",
-        }
-      : {}),
     locations: locations.map((location) => locationOf(location, repo)),
   };
+  if (truncated) {
+    detail.folderHelp =
+      "This repository is too large to scan in full. Name the folder if its project isn't listed.";
+  }
+  return detail;
 }
 
 /** Everything the dialog draws, as one record. The only place this module's state becomes a view. */

package/src/new-project/import-tab.ts

@@ -25,7 +25,9 @@ import { createModelPicker } from "../ui/ai-model-picker";
 import { destinationPath } from "./location-fields";
 import type { CreateProjectDestination, ImportBreakpointPolicy } from "../types";
 
-import { setPendingImportBrief } from "../services/import-seed";
+import { IMPORT_MAX_BREAKPOINTS } from "@jxsuite/protocol";
+
+import { MAX_IMPORT_DEPTH, setPendingImportBrief } from "../services/import-seed";
 import type { ImportBrief } from "../services/import-seed";
 
 /** The three answers the Breakpoints control offers. */
@@ -34,17 +36,6 @@ type BreakpointMode = "all" | "limit" | "explicit";
 /** How a kept width matches one the site declared. */
 type BreakpointRounding = "nearest" | "down" | "up";
 
-/** The most breakpoints the control will offer to keep — `@jxsuite/import`'s own ceiling. */
-const MAX_BREAKPOINTS = 12;
-
-/**
- * The deepest crawl the pipeline accepts.
- *
- * The field said 2 while `import_site` and the endpoint both clamped to 5, so the wizard could not
- * ask for a depth the backend was willing to run.
- */
-const MAX_DEPTH = 5;
-
 export interface ImportTabCtx {
   /** The wizard's shared name/directory state (the import destination slug). */
   form: { name: string; directory: string };
@@ -383,8 +374,8 @@ export function importView(ctx: ImportTabCtx): ImportView {
     canVerify: canVerify(),
     depth: String(_depth),
     error: _errorMsg,
-    maxBreakpoints: String(MAX_BREAKPOINTS),
-    maxDepth: String(MAX_DEPTH),
+    maxBreakpoints: String(IMPORT_MAX_BREAKPOINTS),
+    maxDepth: String(MAX_IMPORT_DEPTH),
     maxPages: String(_maxPages),
     minFidelity: String(_minFidelity),
     prompt: _prompt,
@@ -438,6 +429,8 @@ export function setImportMaxPages(value: string): void {
 
 /** Which of the three breakpoint answers the reader chose. */
 export function setBreakpointMode(value: string): void {
+  /* SAFETY: Studio's one caller is the `bp-mode` select's change handler (surfaces/new-project.json),
+     and that select's options are `importView().breakpointModes`: "limit", "explicit" and "all". */
   _breakpointMode = value as BreakpointMode;
 }
 
@@ -453,6 +446,9 @@ export function setBreakpointWidths(value: string): void {
 
 /** Which declared width backs a kept one. */
 export function setBreakpointRounding(value: string): void {
+  /* SAFETY: Studio's one caller is the `bp-rounding` select's change handler (surfaces/
+     new-project.json), and that select's options are `importView().roundings`: "nearest", "down"
+     and "up". */
   _breakpointRounding = value as BreakpointRounding;
 }
 

package/src/new-project/location-fields.ts

@@ -22,6 +22,7 @@
  */
 
 import { errorMessage } from "@jxsuite/schema/parse";
+import { isFunction } from "@jxsuite/schema/guards";
 import { getPlatform } from "../platform";
 import type { CreateProjectDestination, RepoInfo } from "../types";
 
@@ -211,7 +212,7 @@ export function locationView(slug: string): LocationView {
   return {
     browseLabel: _browsing ? "Choosing…" : "Browse…",
     browsing: _browsing,
-    canBrowse: typeof platform.pickDirectory === "function",
+    canBrowse: isFunction(platform.pickDirectory),
     destination: isRepo ? "repo" : "path",
     error: _error,
     owner: _owner,