@bakery-framework/plugin-db-explorer 2.0.0-alpha.9 → 2.0.0-rc.2

package/src/client/csv-model.ts

@@ -1,14 +1,14 @@
 /**
  * The import wizard's state, with no DOM anywhere in it.
  *
- * The wizard has five steps and the interesting behaviour is all in the
- * transitions between them — a re-pick *moves* a database column rather than
+ * The wizard has five steps and the interesting behavior is all in the
+ * transitions between them: a re-pick *moves* a database column rather than
  * duplicating it, an unmapped NOT NULL column blocks, `empty → NULL` defaults
  * differently per kind, and the preview recomputes from the mapping every
  * time. None of that is testable through a rendered `<select>`, so none of it
  * lives there. `csv.ts` is the view; this is the machine.
  *
- * Parsing, coercion and mapping are **not reimplemented** — `shared/csv.ts`,
+ * Parsing, coercion and mapping are **not reimplemented**: `shared/csv.ts`,
  * `shared/coerce.ts` and `shared/plan.ts` are the same modules the server runs,
  * which is the only way the preview and the outcome agree.
  */
@@ -58,7 +58,7 @@ export function targetOf(assignment: Assignment): string | null {
 /**
  * Does row 0 name the columns?
  *
- * Non-empty, unique, and not a number — the three properties a header row has
+ * Non-empty, unique, and not a number: the three properties a header row has
  * and a data row almost never does. Deliberately conservative in the direction
  * of *yes*: guessing "header" for a data file loses one row in a preview the
  * user is looking at, while guessing "data" for a header file maps nothing and
@@ -91,8 +91,8 @@ function square(fields: string[], width: number): string[] {
 /**
  * Empty means NULL by default for everything except text.
  *
- * `''` in a numeric or date column is never a value — `coerceValue` refuses it
- * outright with `empty_string` — so defaulting the toggle off there guarantees
+ * `''` in a numeric or date column is never a value (`coerceValue` refuses it
+ * outright with `empty_string`), so defaulting the toggle off there guarantees
  * a failure the user then has to diagnose. In a text column `''` *is* a value,
  * and silently turning every blank cell into NULL would be the dashboard's bug
  * with a checkbox on it.
@@ -161,7 +161,7 @@ export function buildModel(
  *
  * Any other header holding the same database column falls back to skip. That
  * makes a duplicate mapping impossible to express, which is a stronger promise
- * than `blockingIssues`' `duplicate_target` — that check stays as the
+ * than `blockingIssues`' `duplicate_target`: that check stays as the
  * server-side guarantee, because the server accepts a mapping this UI did not
  * build.
  */
@@ -345,10 +345,54 @@ export interface BuildResult {
  * walks over fifty thousand rows is the difference between a responsive dialog
  * and a frozen tab.
  */
+/**
+ * The last build, so a change that cannot alter it does not redo it.
+ *
+ * One slot, so it is bounded by construction (convention 6). Compared by
+ * **reference**, not by value: every update goes through a spread
+ * (`{ ...model, onBadRow }`), so an untouched field keeps its identity and a
+ * touched one does not. Comparing by value would mean walking the whole file
+ * to decide whether to walk the whole file.
+ *
+ * The key is exactly what this function reads - `headers`, `rows`, `assign`,
+ * `emptyToNull` and the columns - and deliberately not `onBadRow`, `ragged`,
+ * `delimiter` or `hasHeader`. Those change what the *import* does, never what
+ * the records are, and the bad-row policy is a dropdown in the same footer
+ * that displays this result. Picking one made the footer coerce every row of
+ * the file to arrive at the count it already had on screen.
+ *
+ * Measured on 50,000 rows of four columns: 153-250 ms per change, on the main
+ * thread, for a number that did not move.
+ */
+let lastBuild: {
+  headers: unknown
+  rows: unknown
+  assign: unknown
+  emptyToNull: unknown
+  columns: unknown
+  result: BuildResult
+} | null = null
+
+/** Test seam (convention 9): one test's file must not answer for another's. */
+export function __resetBuildRecords(): void {
+  lastBuild = null
+}
+
 export function buildRecords(
   model: ImportModel,
   columns: readonly SchemaColumn[],
 ): BuildResult {
+  if (
+    lastBuild !== null &&
+    lastBuild.headers === model.headers &&
+    lastBuild.rows === model.rows &&
+    lastBuild.assign === model.assign &&
+    lastBuild.emptyToNull === model.emptyToNull &&
+    lastBuild.columns === columns
+  ) {
+    return lastBuild.result
+  }
+
   const feeds = feedsOf(model, columns)
   const records: Record<string, unknown>[] = []
   const failures: RowFailure[] = []
@@ -369,7 +413,16 @@ export function buildRecords(
     records.push(record)
   })
 
-  return { records, failures }
+  const result = { records, failures }
+  lastBuild = {
+    headers: model.headers,
+    rows: model.rows,
+    assign: model.assign,
+    emptyToNull: model.emptyToNull,
+    columns,
+    result,
+  }
+  return result
 }
 
 /** RFC 4180 quoting: only when the field needs it, and `"` doubles. */

package/src/client/csv-pick.ts

@@ -1,7 +1,7 @@
 /**
  * Step one of the import wizard: get the text.
  *
- * A file input and a drop zone, and both end in the same `onText` callback —
+ * A file input and a drop zone, and both end in the same `onText` callback,
  * which is the whole reason this is a module rather than two handlers. Nothing
  * downstream knows or cares which way the file arrived.
  */

package/src/client/csv-preview.ts

@@ -7,7 +7,7 @@
  *
  * A coerced value that differs from the text carries the text as its `title`,
  * so `"007"` landing in an integer column is visibly `7` and traceably `007`.
- * A failure is ringed and shows the reason `coerceValue` gave — the same
+ * A failure is ringed and shows the reason `coerceValue` gave: the same
  * sentence the server would have returned.
  */
 
@@ -46,7 +46,7 @@ export function paintPreview(
  * column it governs is.
  *
  * On by default for every kind except text, because `''` is a value in a text
- * column and an error in every other one — see `defaultEmptyToNull`.
+ * column and an error in every other one. See `defaultEmptyToNull`.
  */
 function nullToggleRow(
   model: ImportModel,

package/src/client/csv.ts

@@ -1,8 +1,8 @@
 /**
  * The CSV import wizard: pick → sniff → map → preview → commit.
  *
- * This file is the composition and nothing else. Each step is its own module —
- * `csv-pick.ts`, `csv-map.ts`, `csv-preview.ts`, `csv-commit.ts` — and
+ * This file is the composition and nothing else. Each step is its own module:
+ * `csv-pick.ts`, `csv-map.ts`, `csv-preview.ts`, `csv-commit.ts`, and
  * everything that *decides* anything is in `csv-model.ts` and is pure. What is
  * left here is one mutable `model` reference and the repaint that follows it.
  *

package/src/client/dom.ts

@@ -2,8 +2,7 @@
  * The only place this client creates a node.
  *
  * **Nothing here writes `innerHTML`, and `client/safety.test.ts` asserts that
- * no client module does.** Every value the grid renders is a database row —
- * whatever the last writer put there — and the operator's origin owns
+ * no client module does.** Every value the grid renders is a database row ( * whatever the last writer put there), and the operator's origin owns
  * `/api/_db/rows`, so a string that reached the DOM as markup would be stored
  * XSS to arbitrary row writes. `textContent` is not a style preference; it is
  * the reason this plugin has no XSS surface. The dashboard's grid, which
@@ -16,7 +15,7 @@ export interface ElOptions {
   text?: string
   title?: string
   id?: string
-  /** `aria-*`, `role`, `data-*` — anything set through `setAttribute`. */
+  /** `aria-*`, `role`, `data-*`: anything set through `setAttribute`. */
   attrs?: Record<string, string | number | boolean | null | undefined>
 }
 
@@ -88,8 +87,7 @@ export function each<T>(
 /**
  * A `<table class="grid">` with its heading row already in it.
  *
- * The two lines this replaces — build the table, loop `<th>` into a `<tr>` —
- * had been written out three times, twice in `structure.ts` and once in
+ * The two lines this replaces (build the table, loop `<th>` into a `<tr>`)  * had been written out three times, twice in `structure.ts` and once in
  * `csv-preview.ts`. Rows are appended by the caller, because the three differ
  * in what a row is: `structure.ts` has plain text cells, and `csv-preview.ts`
  * puts a checkbox row directly under the headings.

package/src/client/edit-session.ts

@@ -1,7 +1,7 @@
 /**
  * The per-row dirty buffer, and the statement it becomes.
  *
- * **Pure — no DOM, no fetch.** This is where the editing model actually lives,
+ * **Pure: no DOM, no fetch.** This is where the editing model actually lives,
  * so it is the part with tests. The grid is a view over it.
  *
  * Three properties it exists to hold:
@@ -11,7 +11,7 @@
  *     latter, which means a reader can observe the row a third of the way
  *     through an edit the user considers atomic.
  *  2. **The pre-image is kept.** `expect` is built from the row as it was
- *     *read*, never from a re-read at save time — a re-read would defeat the
+ *     *read*, never from a re-read at save time: a re-read would defeat the
  *     concurrency check it exists to perform.
  *  3. **Unchanged columns are dropped**, by `updatePlan`, so two people editing
  *     different columns of one row do not collide.
@@ -23,7 +23,7 @@ import { updatePlan } from '../shared/plan'
 /**
  * A row's identity, flattened to a string.
  *
- * `JSON.stringify` over the identity columns in their declared order — the
+ * `JSON.stringify` over the identity columns in their declared order: the
  * order is the server's, from `identity.cols`, so it is stable across pages and
  * across a composite key's spelling. `null` when the row does not carry every
  * identity column, which is the same condition `updatePlan` reports as
@@ -50,7 +50,7 @@ export interface RowPlan {
   /** The pre-image of every changed column that SQL can compare. */
   expect: Record<string, unknown>
   /**
-   * Changed columns SQL cannot compare — `json` and `buffer`. The server
+   * Changed columns SQL cannot compare: `json` and `buffer`. The server
    * refuses these in `expect` and demands `force: true` to write them without
    * a concurrency check, so the UI has to say so rather than retry blindly.
    */
@@ -68,7 +68,7 @@ export type KindLookup = (column: string) => ColumnKind | undefined
  *
  * Bounded by construction: an entry exists only for a row the user has typed
  * into, they are dropped on save and on revert, and changing table or page
- * clears the whole session — there is no key here a page of data does not
+ * clears the whole session. There is no key here a page of data does not
  * already hold (convention 6).
  */
 export class EditSession {
@@ -144,7 +144,7 @@ export class EditSession {
    * One row's edits as an UPDATE.
    *
    * `where` comes from the pre-image and `set` from the edits, which is what
-   * lets a primary key itself be edited — see `updatePlan`'s own note.
+   * lets a primary key itself be edited. See `updatePlan`'s own note.
    */
   plan(
     id: string,
@@ -189,7 +189,7 @@ export interface UndoEntry {
  * Twenty rather than unlimited because each entry closes over a row's
  * pre-image, and an unbounded stack of those is exactly the module-level
  * unbounded cache convention 6 forbids. The pre-image is already being kept for
- * the concurrency check, so undo costs nothing extra to record — only to keep.
+ * the concurrency check, so undo costs nothing extra to record: only to keep.
  */
 export class UndoStack {
   private readonly entries: UndoEntry[] = []

package/src/client/editors.ts

@@ -28,7 +28,7 @@ export interface EditorHooks {
 export interface EditorHandle {
   node: HTMLElement
   focus: () => void
-  /** The current value, already NULL-aware. Not coerced — that is the caller's. */
+  /** The current value, already NULL-aware. Not coerced: that is the caller's. */
   read: () => unknown
 }
 
@@ -100,7 +100,7 @@ function enumControl(column: SchemaColumn, value: unknown): Control {
  * A checkbox, and it reads back a real boolean.
  *
  * A driver hands a boolean column back as `1`/`0` on SQLite and MySQL, so the
- * *initial* state has to accept those spellings — but what leaves here is
+ * *initial* state has to accept those spellings, but what leaves here is
  * `true`/`false`, which is what `coerceValue` wants and what makes `sameValue`
  * agree that nothing changed when nothing did.
  */
@@ -123,7 +123,7 @@ function booleanControl(_column: SchemaColumn, value: unknown): Control {
  *
  * It refuses an ISO string with a `Z` and refuses one carrying milliseconds, so
  * the value is trimmed to `YYYY-MM-DDTHH:mm` going in and handed back as a full
- * ISO string going out — `coerceValue`'s `date` branch parses either, but the
+ * ISO string going out: `coerceValue`'s `date` branch parses either, but the
  * round trip has to be lossless in the direction the user sees.
  */
 function dateControl(_column: SchemaColumn, value: unknown): Control {
@@ -199,8 +199,8 @@ export interface EditorOptions {
  * An editor for one cell.
  *
  * The wrapper owns three things the control does not: the null toggle, the
- * validity message, and key forwarding. Validation runs `coerceValue` — the
- * same function the server runs — so the message the user sees before sending
+ * validity message, and key forwarding. Validation runs `coerceValue` (the
+ * same function the server runs), so the message the user sees before sending
  * is the message they would have got back.
  */
 export function createEditor(

package/src/client/filter-builder.ts

@@ -36,7 +36,7 @@ const OPTIONS = FILTER_OPS.map(op => ({ value: op, label: OP_LABELS[op] }))
  * The bar.
  *
  * Every control writes a **whole new list** through `onChange` rather than
- * mutating the one it was given — same discipline as the CSV wizard's model,
+ * mutating the one it was given: same discipline as the CSV wizard's model,
  * and the reason a chip's state can never disagree with the URL.
  */
 export function filterBar(ctx: FilterBuilderContext): HTMLElement {
@@ -54,7 +54,7 @@ export function filterBar(ctx: FilterBuilderContext): HTMLElement {
  *
  * The wire shape is a record keyed by column, so only the last filter on a
  * column is sent. Rendering both chips and silently dropping one would be a
- * screen that disagrees with the query — the one failure mode a filter builder
+ * screen that disagrees with the query: the one failure mode a filter builder
  * must not have.
  */
 function clashNote(columns: readonly string[]): HTMLElement {
@@ -130,9 +130,9 @@ function nextOnOp(entry: Filter, op: FilterOp): Filter {
 }
 
 /**
- * The operand — **absent** for `null` and `notnull`.
+ * The operand: **absent** for `null` and `notnull`.
  *
- * Not disabled, not hidden by CSS: not built at all. A greyed-out box still
+ * Not disabled, not hidden by CSS: not built at all. A grayed-out box still
  * says "there is a value here", and there is not one.
  */
 function valueInput(
@@ -169,7 +169,7 @@ function removeButton(
  *
  * Exported because they are the two operations the chip callbacks are made of,
  * and an off-by-one in either would present as "removing one filter removed a
- * different one" — a bug worth a test rather than a careful read.
+ * different one": a bug worth a test rather than a careful read.
  */
 export function withAt(
   filters: readonly Filter[],
@@ -186,7 +186,7 @@ export function withoutAt(filters: readonly Filter[], index: number): Filter[] {
 /**
  * The filter a foreign-key jump becomes.
  *
- * One `eq` per referenced column, which is exactly a row identity — this is the
+ * One `eq` per referenced column, which is exactly a row identity: this is the
  * function that replaced `ViewState.focus`. Values stringify because the wire
  * carries text and the ORM binds it as a parameter; the comparison happens in
  * the database, against the column's own type.

package/src/client/fk.ts

@@ -15,7 +15,7 @@
  *    per-row and therefore unbounded in principle; the bound is what makes that
  *    safe.
  *
- * The pure half — which column points where, and what key a row implies — is
+ * The pure half (which column points where, and what key a row implies) is
  * separated out so the graph reasoning can be read without the timers.
  */
 
@@ -25,8 +25,8 @@ import { type ForeignKeyInfo, type SchemaGraph, sameTable } from './meta'
 /**
  * A least-recently-used map, in nine lines.
  *
- * Core has `LRUCache` and this is not a preference for a private copy —
- * `bundleModule` externalises every installed package, so importing it leaves
+ * Core has `LRUCache` and this is not a preference for a private copy:
+ * `bundleModule` externalizes every installed package, so importing it leaves
  * `@bakery-framework/core/cache/lru` as a bare specifier in the emitted bundle
  * and the browser cannot resolve it. Client code imports nothing from
  * `@bakery-framework/*`; see the header of `shared/coerce.ts`.
@@ -101,7 +101,7 @@ export function fkForColumn(
  * Every foreign-key column of one table, resolved once.
  *
  * `fkForColumn` is a linear scan of the whole graph, and the grid asked it per
- * cell, per paint — a fifty-row page of twenty columns against forty foreign
+ * cell, per paint: a fifty-row page of twenty columns against forty foreign
  * keys is forty thousand comparisons, each of which had `sameTable` allocate
  * two lowercased strings, and `repaintRow`/`repaintCell` run it again. The scan
  * does not depend on the row, so it happens once when the grid is built.
@@ -139,7 +139,7 @@ export function reverseFks(
 /**
  * The referenced row's identity, as this row names it.
  *
- * `null` when any participating column is NULL — an optional foreign key with
+ * `null` when any participating column is NULL: an optional foreign key with
  * no value points at nothing, and a predicate containing `NULL` matches no row
  * anyway, so issuing the lookup would spend a request to learn that.
  */
@@ -156,7 +156,7 @@ export function fkKeyOf(
   return key
 }
 
-/** `table|col|value` — stable across pages, which is what makes caching pay. */
+/** `table|col|value`: stable across pages, which is what makes caching pay. */
 export function cacheKey(
   refTable: string,
   key: Record<string, unknown>,
@@ -171,8 +171,8 @@ export function cacheKey(
  * What to show for a resolved row: the label column if the table has one, the
  * identity otherwise.
  *
- * `graph.labels` is the server's pick — the first text column that is not part
- * of the identity — so a `users` row reads as `ada` rather than `41`.
+ * `graph.labels` is the server's pick (the first text column that is not part
+ * of the identity), so a `users` row reads as `ada` rather than `41`.
  */
 export function fkLabel(
   graph: SchemaGraph | null,
@@ -197,7 +197,7 @@ export type Resolved = Record<string, unknown> | null
  */
 export class FkResolver {
   /**
-   * 500 entries — ten pages of a fifty-row grid with a foreign key on every
+   * 500 entries: ten pages of a fifty-row grid with a foreign key on every
    * row. Past that the earliest are the ones the user has scrolled away from.
    */
   private readonly cache = new BoundedCache<Resolved>(500)

package/src/client/grid-body.ts

@@ -2,8 +2,8 @@
  * What goes *inside* a cell: the text, the classes, and the foreign-key button.
  *
  * Split out of `grid.ts` alongside the header and the row bar. This half is the
- * one with no knowledge of the cursor or the editor — it is handed a value and
- * returns a node — which is why it could leave.
+ * one with no knowledge of the cursor or the editor (it is handed a value and
+ * returns a node), which is why it could leave.
  *
  * The `<button>` for a foreign key is a real button rather than a styled span,
  * because it is activated by keyboard and read as an action by a screen reader.
@@ -33,7 +33,7 @@ export interface CellPaintContext {
  * Write one cell's value into its `<td>`.
  *
  * The staged flag is passed in rather than read from a session here, so this
- * module needs no `EditSession` — a staged value and a stored value take
+ * module needs no `EditSession`: a staged value and a stored value take
  * exactly the same path and cannot diverge in how they look.
  */
 export function paintCell(
@@ -69,7 +69,7 @@ function cellBody(
 /**
  * A foreign key as a link, resolving its label on hover.
  *
- * The resolver owns the delay, the cache and the cancellation — see `fk.ts`.
+ * The resolver owns the delay, the cache and the cancellation. See `fk.ts`.
  * All that happens here is arming it on `pointerenter` and disarming it on
  * `pointerleave`, which is what keeps a sweep across the grid from costing
  * fifty round trips.

package/src/client/grid-header.ts

@@ -2,7 +2,7 @@
  * The grid's header row: the select-all box and the sortable column headings.
  *
  * Split out of `grid.ts` when that file passed six hundred lines. It is a clean
- * seam because the header depends on nothing the grid mutates — it is built
+ * seam because the header depends on nothing the grid mutates: it is built
  * once from the columns and the current sort, and never repainted. Cursor,
  * editor and selection state all live on the other side of it.
  */
@@ -30,7 +30,7 @@ export function buildHead(ctx: HeaderContext): HTMLTableSectionElement {
 
 /**
  * The leading column exists on every table, editable or not: it carries the row
- * panel's opener, which a read-only table needs just as much — a forty-column
+ * panel's opener, which a read-only table needs just as much. A forty-column
  * row is unreadable in a grid whether or not it can be changed. The select-all
  * checkbox joins it only when there is something to select *for*.
  */

package/src/client/grid-rowbar.ts

@@ -4,7 +4,7 @@
  * Under the row rather than in a toolbar, because "which row is this Save for"
  * has exactly one unambiguous answer and it is proximity. It carries the
  * changed-column list, the two buttons, and the place a failed save reports
- * itself — a 409 belongs next to the row it is about, not in a toast.
+ * itself: a 409 belongs next to the row it is about, not in a toast.
  */
 
 import { append, box, button, el } from './dom'

package/src/client/grid.ts

@@ -3,20 +3,20 @@
  *
  * Two rules keep this file's functions under biome's complexity ceiling, and
  * they are worth stating because the old `renderTable` broke both and scored
- * 34 with a fraction of the behaviour:
+ * 34 with a fraction of the behavior:
  *
  *  1. **No function both fetches and renders.** Everything here takes rows it
  *     was handed. `api.ts` does the asking.
  *  2. **Every loop body is a named function.** `each()` from `dom.ts` takes the
  *     factory; an inline body would fold its branches into the caller's score.
  *
- * Three neighbours carry the parts that do not need the cursor:
+ * Three neighbors carry the parts that do not need the cursor:
  * `grid-header.ts` builds the heading row, `grid-body.ts` paints a cell's
  * contents, and `grid-rowbar.ts` owns the per-row save strip. The editing model
  * itself is in `edit-session.ts` and the keyboard model in `cell.ts`, both pure
  * and both tested. What is left here genuinely needs a DOM.
  *
- * `focus` used to live here too — a row identity carried alongside the filters,
+ * `focus` used to live here too: a row identity carried alongside the filters,
  * because a substring `LIKE` could not name a row. With `eq` in the filter
  * vocabulary a foreign-key jump is an ordinary filter, and the highlight, the
  * `focusedMissing()` note and the whole concept are gone.
@@ -79,7 +79,7 @@ export class Grid {
     this.paintCtx = {
       table: ctx.table.name,
       graph: ctx.graph,
-      // Once per grid, not once per cell per paint — see `fkMapFor`.
+      // Once per grid, not once per cell per paint. See `fkMapFor`.
       fks: fkMapFor(ctx.graph, ctx.table.name),
       resolver: ctx.resolver,
       onFollowFk: ctx.onFollowFk,
@@ -174,7 +174,7 @@ export class Grid {
   /**
    * Render one cell from the session's current view of it.
    *
-   * Called on build, after every stage, and after an editor closes — so a
+   * Called on build, after every stage, and after an editor closes, so a
    * staged value and a saved value take exactly the same path.
    */
   private repaintCell(
@@ -248,7 +248,7 @@ export class Grid {
   // ---------------------------------------------------------------- editing
 
   /**
-   * Open an editor — or explain why not.
+   * Open an editor, or explain why not.
    *
    * A double-click on a read-only table says what is wrong *here*, at the
    * moment the user asked. The alternative is a grid that looks editable and
@@ -293,8 +293,7 @@ export class Grid {
     // write, and a multi-column edit must stay one statement.
     //
     // `relatedTarget` is checked because `focusout` also fires for movement
-    // *inside* the editor — tabbing from the input to the null toggle is one —
-    // and committing there would close the editor the user is still in.
+    // *inside* the editor (tabbing from the input to the null toggle is one),     // and committing there would close the editor the user is still in.
     handle.node.addEventListener('focusout', event => {
       const next = (event as FocusEvent).relatedTarget
       if (next instanceof Node && handle.node.contains(next)) return
@@ -425,7 +424,7 @@ export class Grid {
     return this.slots[index]?.row
   }
 
-  /** The row under the cursor — what a keyboard-only user means by "this row". */
+  /** The row under the cursor: what a keyboard-only user means by "this row". */
   focusedRow(): Record<string, unknown> | undefined {
     return this.slots[this.cursor.row]?.row
   }

package/src/client/meta.ts

@@ -2,7 +2,7 @@
  * What the server said about a column, and the four decisions the grid makes
  * from it.
  *
- * **Pure — no DOM.** Every branch the editor would otherwise take on a column's
+ * **Pure, no DOM.** Every branch the editor would otherwise take on a column's
  * type is resolved here, once, into a `WidgetKind`, and every consumer
  * dispatches through a record of small functions rather than an if-chain. A
  * record of five functions has no cognitive complexity; five `else if`s have
@@ -11,7 +11,7 @@
  *
  * The shapes below mirror `endpoints/read.ts` exactly. They are re-declared
  * rather than imported because that module reaches for the ORM and for
- * `node:async_hooks` through its neighbours, and this one is compiled into the
+ * `node:async_hooks` through its neighbors, and this one is compiled into the
  * browser bundle.
  */
 
@@ -48,7 +48,7 @@ export interface SchemaTable {
   rowCount: number
   columns: SchemaColumn[]
   identity: Identity
-  /** Optional so a fixture — and an older server — need not carry them. */
+  /** Optional so a fixture (and an older server) need not carry them. */
   indexes?: SchemaIndex[]
   isView?: boolean
   writable: boolean
@@ -87,7 +87,7 @@ export type WidgetKind = 'enum' | 'boolean' | 'date' | 'json' | 'text'
 
 /**
  * `enum` first, because it is a constraint on a `string` column rather than a
- * kind of its own — the server reports `kind: 'string'` with an `enum` list,
+ * kind of its own: the server reports `kind: 'string'` with an `enum` list,
  * and a free-text box over a checked list is a guaranteed round trip to a
  * `not_in_enum` error.
  */
@@ -124,7 +124,7 @@ export function columnMeta(column: SchemaColumn): ColumnMeta {
  * `NULL` is rendered as the word and styled as absent, which is the
  * distinction the whole editor turns on: an empty string is a value and a
  * blank cell would make the two look identical. Binary is summarised rather
- * than dumped — a blob rendered as its bytes is thousands of characters of
+ * than dumped: a blob rendered as its bytes is thousands of characters of
  * noise that also makes the row unselectable.
  */
 export function cellText(value: unknown): string {
@@ -139,7 +139,7 @@ function safeJson(value: unknown): string {
     return JSON.stringify(value) ?? String(value)
   } catch {
     // A cycle or a BigInt. The cell still has to render something, and the
-    // failure is cosmetic — the value itself is untouched.
+    // failure is cosmetic: the value itself is untouched.
     return String(value)
   }
 }

package/src/client/page.ts

@@ -6,7 +6,7 @@
  * what stopped the entry module growing back into the 197-line
  * fetch-and-render `renderTable` it replaced.
  *
- * The shape is now three regions rather than two — sidebar, then a column
+ * The shape is now three regions rather than two: sidebar, then a column
  * holding the tab strip, the active view and the status bar. The sidebar and
  * the status bar outlive a tab switch; only `#main` is replaced. The painters
  * take callbacks rather than reaching for the entry's state, so the page has no
@@ -78,7 +78,7 @@ export class Page {
    * The Data view: filters, the bulk toolbar, the grid, the pager.
    *
    * `data` is the page the server just returned and `ms` is what it said it
-   * cost — the status bar's timing is the envelope's own number rather than a
+   * cost: the status bar's timing is the envelope's own number rather than a
    * round trip timed here, so a slow filter is attributable.
    */
   paint(
@@ -232,14 +232,14 @@ export class Page {
   private readOnlyBanner(table: SchemaTable): HTMLElement {
     return el('p', {
       class: 'banner warn',
-      text: `read-only — ${readOnlyReason(this.state, table)}`,
+      text: `read-only: ${readOnlyReason(this.state, table)}`,
     })
   }
 
   /**
    * The filter builder.
    *
-   * A chip per condition — column, operator, value, remove — where there used
+   * A chip per condition (column, operator, value, remove) where there used
    * to be one text box per column that could only ever mean "contains". Any
    * change resets to page 1, because a filtered result has different pages and
    * staying on page 7 of a set that now has two is a blank screen.