@1agh/maude 0.28.1 → 0.30.0

package/plugins/design/dev-server/dom-selection.ts

@@ -1,156 +0,0 @@
-/**
- * @file       dom-selection.ts — selection-from-DOM helpers (leaf module)
- * @scope      plugins/design/dev-server/dom-selection.ts
- * @purpose    Pure DOM → Selection builders shared by the canvas chrome
- *             (canvas-shell.tsx) and the shell-owned comment mount layer
- *             (canvas-comment-mount.tsx). Lives in its own leaf module — no
- *             React, no canvas-lib import — so both consumers can lift the
- *             same `hoverTargetToSelection` / `deriveFile` logic without a
- *             cycle and without bundling the heavy DesignCanvas tree into the
- *             lite comment mount.
- */
-
-import type { HoverTarget } from './input-router.tsx';
-import type { Selection } from './use-selection-set.tsx';
-
-/**
- * Canvas file path for the current page. Under the mount harness the page is
- * `/_canvas-shell.html?canvas=<rel>&designRel=<root>`; for legacy `.html`
- * mocks it's the served file path itself.
- */
-export function deriveFile(): string | undefined {
-  if (typeof window === 'undefined') return undefined;
-  try {
-    const p = window.location.pathname;
-    if (p === '/_canvas-shell.html' || p === '/_canvas-shell') {
-      const qs = new URLSearchParams(window.location.search);
-      const canvas = qs.get('canvas') ?? '';
-      const designRel = (qs.get('designRel') ?? '.design').replace(/^\/+|\/+$/g, '');
-      return canvas ? `${designRel}/${canvas}` : undefined;
-    }
-    return decodeURIComponent(p).replace(/^\//, '');
-  } catch {
-    return undefined;
-  }
-}
-
-export function realClasses(el: Element | null): string {
-  if (!el) return '';
-  return (el.getAttribute('class') ?? '')
-    .trim()
-    .split(/\s+/)
-    .filter((c) => c && !c.startsWith('dgn-') && !c.startsWith('dc-cv-'))
-    .join(' ');
-}
-
-export function shortText(el: Element | null, max: number): string {
-  if (!el) return '';
-  const t = ((el as HTMLElement).innerText || el.textContent || '').replace(/\s+/g, ' ').trim();
-  return t.length > max ? `${t.slice(0, max - 1)}…` : t;
-}
-
-export function cssPath(el: Element | null): string {
-  if (!el) return '';
-  const path: string[] = [];
-  let cur: Element | null = el;
-  while (cur && cur.nodeType === 1 && path.length < 8) {
-    const dscEl = cur.getAttribute?.('data-dc-element');
-    if (dscEl) {
-      path.unshift(`[data-dc-element="${dscEl}"]`);
-      break;
-    }
-    const dscSc = cur.getAttribute?.('data-dc-screen');
-    if (dscSc) {
-      path.unshift(`[data-dc-screen="${dscSc}"]`);
-      break;
-    }
-    let sel = cur.nodeName.toLowerCase();
-    if (cur.id) {
-      sel = `#${cur.id}`;
-      path.unshift(sel);
-      break;
-    }
-    const cls = realClasses(cur).split(/\s+/).filter(Boolean).slice(0, 2);
-    if (cls.length) sel += `.${cls.join('.')}`;
-    let sib = 1;
-    let n: Element | null = cur.previousElementSibling;
-    while (n) {
-      sib++;
-      n = n.previousElementSibling;
-    }
-    sel += `:nth-child(${sib})`;
-    path.unshift(sel);
-    cur = cur.parentElement;
-  }
-  return path.join(' > ');
-}
-
-export function domPath(el: Element | null): string[] {
-  const hops: string[] = [];
-  let cur = el;
-  while (cur && cur.nodeType === 1 && hops.length < 8) {
-    let label = cur.nodeName.toLowerCase();
-    const dEl = cur.getAttribute?.('data-dc-element');
-    const dSc = cur.getAttribute?.('data-dc-screen');
-    if (dEl) label += `[data-dc-element="${dEl}"]`;
-    else if (dSc) label += `[data-dc-screen="${dSc}"]`;
-    else if (cur.id) label += `#${cur.id}`;
-    const cls = realClasses(cur).split(/\s+/).filter(Boolean).slice(0, 2);
-    if (cls.length && !dEl && !dSc) label += `.${cls.join('.')}`;
-    hops.unshift(label);
-    cur = cur.parentElement;
-  }
-  return hops;
-}
-
-export function cssEscape(s: string): string {
-  // Minimal CSS.escape polyfill — only handles chars actually present in
-  // pipeline-stamped IDs (alphanumerics + `-` + `_`).
-  return s.replace(/[^a-zA-Z0-9_-]/g, (c) => `\\${c}`);
-}
-
-/**
- * Build the wire-shape `Selection` for a resolved hover target. `file`
- * defaults to `deriveFile()`; the comment mount layer passes it explicitly
- * so all three consumers (router, overlay, mount) agree on the same key.
- */
-export function hoverTargetToSelection(target: HoverTarget, file?: string): Selection {
-  const el = target.el;
-  const rect =
-    el && (el as HTMLElement).getBoundingClientRect
-      ? (el as HTMLElement).getBoundingClientRect()
-      : null;
-  // `cdId` is the hit element's OWN data-cd-id (deep mode); resolver never
-  // climbs to an ancestor. Falls back to cssPath of the hit when no stable
-  // anchor exists.
-  const cdId = target.cdId;
-  // Selector resolution order:
-  //   1. data-cd-id anchor — stable pipeline-stamped id (preferred).
-  //   2. data-dc-screen — chrome click promoted to whole-artboard select
-  //      (T24.5 G8 multi-artboard gesture).
-  //   3. cssPath of the hit — last-resort path string.
-  const selector = cdId
-    ? `[data-cd-id="${cdId}"]`
-    : !cdId && target.artboardId
-      ? `[data-dc-screen="${target.artboardId}"]`
-      : cssPath(el);
-  return {
-    file: file ?? deriveFile(),
-    id: cdId ?? undefined,
-    selector,
-    artboardId: target.artboardId,
-    tag: el?.tagName.toLowerCase() ?? '',
-    classes: realClasses(el),
-    text: shortText(el, 240),
-    dom_path: domPath(el),
-    bounds: rect
-      ? {
-          x: Math.round(rect.left),
-          y: Math.round(rect.top),
-          w: Math.round(rect.width),
-          h: Math.round(rect.height),
-        }
-      : null,
-    html: el ? (el.outerHTML ?? '').slice(0, 4000) : '',
-  };
-}

package/plugins/design/dev-server/sync/migrate-seed.ts

@@ -1,163 +0,0 @@
-// One-time authoritative seed for the shared-doc cutover — Phase 9.2 (DDR-064
-// Task 9). The fix for Risk 1 (the duplication-on-merge trap).
-//
-// The trap (dmonad-confirmed, discuss.yjs.dev/t/.../2538): if a shared doc is
-// populated from TWO independent sources — the local file-seed (room.seed
-// pushing `_comments/<slug>.json` as fresh Y.Array items) AND the hub provider
-// (syncing the hub's canonical items) — the two item-sets have different client
-// IDs, so the CRDT merge CONCATENATES them: comment "c1" appears twice. You
-// cannot fix this with `applyUpdate` of two docs; you must pick ONE authoritative
-// source and build the doc from it.
-//
-// This module is that decision, run ONCE per canvas at cutover, AFTER the
-// provider's first sync (so the doc already holds hub state if the hub had any):
-//
-//   - hub HAD state (doc non-empty)  → HUB WINS. Leave the doc; the projection
-//     materializes hub state to disk. Local divergent files are snapshotted to
-//     `_history/` first (rollback) then overwritten by the room persist.
-//   - hub was EMPTY (doc empty)      → ADOPT. Clear+rebuild the doc from the
-//     local files inside `transact(fn, MIGRATION)` (the apply* codecs already
-//     delete-then-insert, so this is a true rebuild, not an append). The
-//     provider then pushes it up as the canonical first version.
-//
-// Idempotent: re-running with hub state present is a no-op (doc already holds
-// it); re-running an adopt rebuilds from the same files → byte-identical, no
-// duplicate items. The companion guard is the room's seed being disabled for
-// pinned (provider-attached) slugs under sharedDoc (see collab/index.ts
-// `shouldSeed`), so the file-seed can never re-introduce duplicate items after
-// this runs.
-
-import { copyFileSync, existsSync, mkdirSync, readFileSync } from 'node:fs';
-import path from 'node:path';
-
-import type * as Y from 'yjs';
-
-import { Y_TYPES } from '../collab/persistence.ts';
-import {
-  applyAnnotationsToDoc,
-  applyCommentsToDoc,
-  applyCssToDoc,
-  applyHtmlToDoc,
-  applyMetaToDoc,
-  Y_SYNC_TYPES,
-} from './codec.ts';
-import { ORIGINS } from './origins.ts';
-
-export interface MigrateSeedPaths {
-  html: string;
-  comments: string;
-  annotations: string;
-  meta?: string;
-  css?: string;
-}
-
-export interface MigrateSeedOptions {
-  slug: string;
-  doc: Y.Doc;
-  paths: MigrateSeedPaths;
-  /** `_history/<slug>/` (or any dir) — when set, local files are snapshotted
-   *  here before a hub-wins overwrite, for rollback. Best-effort. */
-  historyDir?: string;
-}
-
-export type MigrateSeedResult = 'hub-wins' | 'local-adopt' | 'empty';
-
-/** True when the shared doc holds no synced content for any of the five types. */
-export function docIsEmpty(doc: Y.Doc): boolean {
-  if (doc.getText(Y_SYNC_TYPES.html).length > 0) return false;
-  if (doc.getText(Y_SYNC_TYPES.css).length > 0) return false;
-  if (doc.getText(Y_SYNC_TYPES.meta).length > 0) return false;
-  if (doc.getArray(Y_TYPES.comments).length > 0) return false;
-  const svg = doc.getMap<unknown>(Y_TYPES.annotations).get('svg');
-  if (typeof svg === 'string' && svg.length > 0) return false;
-  return true;
-}
-
-/**
- * Run the one-time authoritative seed. Returns which source won so the caller
- * can log / surface a conflict. Safe to call on every boot (idempotent).
- */
-export function migrateSeed(opts: MigrateSeedOptions): MigrateSeedResult {
-  const { doc, paths } = opts;
-
-  // Hub had canonical state → it wins. The doc already holds it; the projection
-  // will materialize it to disk. Snapshot any divergent local files first so the
-  // overwrite is recoverable.
-  if (!docIsEmpty(doc)) {
-    snapshotLocal(opts);
-    return 'hub-wins';
-  }
-
-  // Hub was empty → adopt local. Build the doc from the local files ONCE, inside
-  // a single MIGRATION transaction. The apply* codecs delete-then-insert, so
-  // this is a clear+rebuild (re-running is a no-op once content matches).
-  const localHtml = readLocal(paths.html);
-  const localComments = readLocal(paths.comments);
-  const localAnnotations = readLocal(paths.annotations);
-  const localMeta = paths.meta ? readLocal(paths.meta) : null;
-  const localCss = paths.css ? readLocal(paths.css) : null;
-
-  const hasLocal =
-    !!localHtml || !!localComments || !!localAnnotations || !!localMeta || !!localCss;
-  if (!hasLocal) return 'empty';
-
-  doc.transact(() => {
-    if (localHtml) applyHtmlToDoc(doc, localHtml, ORIGINS.MIGRATION);
-    if (localComments) {
-      const parsed = tryParseJsonArray(localComments);
-      if (parsed) applyCommentsToDoc(doc, parsed, ORIGINS.MIGRATION);
-    }
-    if (localAnnotations) applyAnnotationsToDoc(doc, localAnnotations, ORIGINS.MIGRATION);
-    if (paths.meta && localMeta) applyMetaToDoc(doc, localMeta, ORIGINS.MIGRATION);
-    if (paths.css && localCss) applyCssToDoc(doc, localCss, ORIGINS.MIGRATION);
-  }, ORIGINS.MIGRATION);
-
-  return 'local-adopt';
-}
-
-/* ---------------------------------------------------------------- helpers */
-
-function snapshotLocal(opts: MigrateSeedOptions): void {
-  if (!opts.historyDir) return;
-  try {
-    const dir = path.join(opts.historyDir, 'pre-shared-doc-migration');
-    mkdirSync(dir, { recursive: true });
-    for (const p of [
-      opts.paths.html,
-      opts.paths.comments,
-      opts.paths.annotations,
-      opts.paths.meta,
-      opts.paths.css,
-    ]) {
-      if (p && existsSync(p)) {
-        copyFileSync(p, path.join(dir, path.basename(p)));
-      }
-    }
-  } catch {
-    /* best-effort — a snapshot failure must not block the cutover */
-  }
-}
-
-function readLocal(p: string): string | null {
-  if (!existsSync(p)) return null;
-  try {
-    return readFileSync(p, 'utf8');
-  } catch {
-    return null;
-  }
-}
-
-// Proto-pollution-safe (DDR-054 §2g), mirroring the agent + projection comments
-// parse — a planted local `_comments/<slug>.json` must not seed dangerous keys
-// during adopt.
-function tryParseJsonArray(s: string): unknown[] | null {
-  try {
-    const parsed = JSON.parse(s, (key, value) => {
-      if (key === '__proto__' || key === 'constructor' || key === 'prototype') return undefined;
-      return value;
-    });
-    return Array.isArray(parsed) ? parsed : null;
-  } catch {
-    return null;
-  }
-}

package/plugins/design/dev-server/test/canvas-edit.test.ts

@@ -1,139 +0,0 @@
-// canvas-edit.ts — Phase 3.6 Task 5. AST-aware single-attribute edits on TSX
-// canvases. The same component+jsxIndex bookkeeping as canvas-pipeline.ts
-// computes the target ID; tests round-trip an edit through transpileCanvasSource
-// to confirm the IDs line up.
-
-import { describe, expect, test } from 'bun:test';
-
-import { applyEdit, CanvasEditError, editAttribute } from '../canvas-edit.ts';
-import { transpileCanvasSource } from '../canvas-pipeline.ts';
-
-const CANVAS = '/abs/Canvas.tsx';
-
-function idsOf(source: string): Record<string, string> {
-  // Run the pipeline to learn what data-cd-id each JSX element gets, returning
-  // a map keyed by the element-type name so tests can ask "what id is the
-  // <span>?" without hardcoding the hash.
-  const { withIds } = transpileCanvasSource(CANVAS, source);
-  const out: Record<string, string> = {};
-  for (const m of withIds.matchAll(/<(\w+)([^>]*?)data-cd-id="([0-9a-f]{8})"/g)) {
-    if (!out[m[1] as string]) out[m[1] as string] = m[3] as string;
-  }
-  return out;
-}
-
-describe('canvas-edit / applyEdit', () => {
-  test('swaps an existing className literal value', () => {
-    const src = `function Demo() { return <button className="btn">OK</button>; }`;
-    const ids = idsOf(src);
-    const id = ids.button as string;
-    const out = applyEdit(CANVAS, src, id, 'className', 'btn btn--ghost');
-    expect(out.source).toBe(
-      `function Demo() { return <button className="btn btn--ghost">OK</button>; }`
-    );
-    // "btn" (3 chars) → "btn btn--ghost" (14 chars): +11 bytes inside the
-    // literal; quotes unchanged.
-    expect(out.delta).toBe(11);
-  });
-
-  test('inserts className when missing', () => {
-    const src = 'function Demo() { return <div>x</div>; }';
-    const ids = idsOf(src);
-    const id = ids.div as string;
-    const out = applyEdit(CANVAS, src, id, 'className', 'card');
-    expect(out.source).toBe(`function Demo() { return <div className="card">x</div>; }`);
-  });
-
-  test('rewrites a plain attribute (aria-label)', () => {
-    const src = `function Demo() { return <button aria-label="old">x</button>; }`;
-    const ids = idsOf(src);
-    const id = ids.button as string;
-    const out = applyEdit(CANVAS, src, id, 'aria-label', 'new');
-    expect(out.source).toBe(`function Demo() { return <button aria-label="new">x</button>; }`);
-  });
-
-  test('swaps a single style.<prop> value inside style={{...}}', () => {
-    const src = 'function Demo() { return <div style={{ padding: 8, margin: 4 }}>x</div>; }';
-    const ids = idsOf(src);
-    const id = ids.div as string;
-    const out = applyEdit(CANVAS, src, id, 'style.padding', '14');
-    expect(out.source).toBe(
-      'function Demo() { return <div style={{ padding: 14, margin: 4 }}>x</div>; }'
-    );
-  });
-
-  test('inserts a missing style key into existing style={{...}}', () => {
-    const src = 'function Demo() { return <div style={{ padding: 8 }}>x</div>; }';
-    const ids = idsOf(src);
-    const id = ids.div as string;
-    const out = applyEdit(CANVAS, src, id, 'style.color', '"#facc15"');
-    // The exact spacing of the appended key is implementation-defined; assert
-    // the substring lands without trampling the prior key.
-    expect(out.source).toContain('padding: 8');
-    expect(out.source).toContain('color: "#facc15"');
-  });
-
-  test('inserts a brand-new style={{}} prop when missing', () => {
-    const src = 'function Demo() { return <div>x</div>; }';
-    const ids = idsOf(src);
-    const id = ids.div as string;
-    const out = applyEdit(CANVAS, src, id, 'style.padding', '14');
-    expect(out.source).toContain('<div style={{ padding: 14 }}>x</div>');
-  });
-
-  test('throws CanvasEditError when id is not found', () => {
-    const src = 'function Demo() { return <div>x</div>; }';
-    expect(() => applyEdit(CANVAS, src, 'deadbeef', 'className', 'x')).toThrow(CanvasEditError);
-  });
-
-  test('refuses to edit data-cd-id (owned by pipeline)', () => {
-    const src = 'function Demo() { return <div>x</div>; }';
-    const ids = idsOf(src);
-    const id = ids.div as string;
-    expect(() => applyEdit(CANVAS, src, id, 'data-cd-id', 'beefcafe')).toThrow(CanvasEditError);
-  });
-
-  test('throws on malformed source', () => {
-    const src = 'function Demo() { return <not closing';
-    expect(() => applyEdit(CANVAS, src, 'aaaaaaaa', 'className', 'x')).toThrow();
-  });
-
-  test('targets the right element when multiple share a type', () => {
-    const src =
-      'function Demo() { return (<section>' +
-      `<button className="a">A</button>` +
-      `<button className="b">B</button>` +
-      '</section>); }';
-    // The pipeline visits in pre-order; expose all <button> ids by walking the
-    // withIds output and capturing each match.
-    const { withIds } = transpileCanvasSource(CANVAS, src);
-    const matches = [
-      ...withIds.matchAll(/<button[^>]*data-cd-id="([0-9a-f]{8})"[^>]*className="([ab])"/g),
-    ];
-    expect(matches.length).toBe(2);
-    const idA = matches.find((m) => m[2] === 'a')?.[1] as string;
-    expect(idA).toBeDefined();
-    const out = applyEdit(CANVAS, src, idA, 'className', 'A-edited');
-    expect(out.source).toContain('className="A-edited"');
-    expect(out.source).toContain('className="b"');
-  });
-});
-
-describe('canvas-edit / editAttribute (fs)', () => {
-  test('writes the edit atomically + is a no-op when source unchanged', async () => {
-    const tmp = `/tmp/canvas-edit-${Math.random().toString(36).slice(2)}.tsx`;
-    const src = `function Demo() { return <div className="x">y</div>; }`;
-    await Bun.write(tmp, src);
-    const ids = idsOf(src);
-    const id = ids.div as string;
-
-    const r1 = await editAttribute(tmp, id, 'className', 'longer-value');
-    expect(r1.delta).toBeGreaterThan(0);
-    const after = await Bun.file(tmp).text();
-    expect(after).toContain('className="longer-value"');
-
-    // Second pass with the same value — no-op (the source already has it).
-    const r2 = await editAttribute(tmp, id, 'className', 'longer-value');
-    expect(r2.delta).toBe(0);
-  });
-});

package/plugins/design/dev-server/test/shared-doc-migrate.test.ts

@@ -1,159 +0,0 @@
-// Phase 9.2 (DDR-064) Phase E — the one-time authoritative seed (Task 9).
-//
-// Pins the fix for Risk 1, the duplication-on-merge trap: a shared doc must be
-// populated from ONE source. These tests prove migrateSeed never duplicates
-// comments and is idempotent across re-runs (server restarts).
-
-import { afterEach, beforeEach, describe, expect, test } from 'bun:test';
-import { mkdtempSync, readdirSync, rmSync, writeFileSync } from 'node:fs';
-import { tmpdir } from 'node:os';
-import { join } from 'node:path';
-import * as Y from 'yjs';
-
-import { createPersistence, Y_TYPES } from '../collab/persistence.ts';
-import type { Context } from '../context.ts';
-import { applyCommentsToDoc, applyHtmlToDoc } from '../sync/codec.ts';
-import { docIsEmpty, migrateSeed } from '../sync/migrate-seed.ts';
-
-let dir: string;
-beforeEach(() => {
-  dir = mkdtempSync(join(tmpdir(), 'migrate-seed-'));
-});
-afterEach(() => {
-  rmSync(dir, { recursive: true, force: true });
-});
-
-function paths() {
-  return {
-    html: join(dir, 'screen.html'),
-    comments: join(dir, 'screen.comments.json'),
-    annotations: join(dir, 'screen.annotations.svg'),
-    meta: join(dir, 'screen.meta.json'),
-    css: join(dir, 'screen.css'),
-  };
-}
-
-describe('docIsEmpty', () => {
-  test('true for a fresh doc, false once any type is populated', () => {
-    const doc = new Y.Doc();
-    expect(docIsEmpty(doc)).toBe(true);
-    doc.getArray(Y_TYPES.comments).push([{ id: 'c1' }]);
-    expect(docIsEmpty(doc)).toBe(false);
-  });
-});
-
-describe('migrateSeed — adopt (hub was empty)', () => {
-  test('seeds an empty doc from local files and reports local-adopt', () => {
-    const p = paths();
-    writeFileSync(p.html, '<main>local body</main>');
-    writeFileSync(p.comments, JSON.stringify([{ id: 'c1', text: 'local' }]));
-
-    const doc = new Y.Doc();
-    const result = migrateSeed({ slug: 's', doc, paths: p });
-
-    expect(result).toBe('local-adopt');
-    expect(doc.getText('html').toString()).toBe('<main>local body</main>');
-    expect(doc.getArray(Y_TYPES.comments).toArray()).toEqual([{ id: 'c1', text: 'local' }]);
-  });
-
-  test('empty doc + no local files → "empty" (nothing to seed)', () => {
-    const doc = new Y.Doc();
-    expect(migrateSeed({ slug: 's', doc, paths: paths() })).toBe('empty');
-    expect(docIsEmpty(doc)).toBe(true);
-  });
-});
-
-describe('migrateSeed — idempotent (no duplication on re-run)', () => {
-  test('adopting twice leaves comments un-duplicated', () => {
-    const p = paths();
-    writeFileSync(
-      p.comments,
-      JSON.stringify([
-        { id: 'c1', text: 'x' },
-        { id: 'c2', text: 'y' },
-      ])
-    );
-
-    const doc = new Y.Doc();
-    expect(migrateSeed({ slug: 's', doc, paths: p })).toBe('local-adopt');
-    expect(doc.getArray(Y_TYPES.comments).length).toBe(2);
-
-    // Second run (e.g. server restart): the doc is now non-empty → hub-wins
-    // branch → no re-adopt → still exactly 2 comments, byte-identical.
-    expect(migrateSeed({ slug: 's', doc, paths: p })).toBe('hub-wins');
-    expect(doc.getArray(Y_TYPES.comments).toArray()).toEqual([
-      { id: 'c1', text: 'x' },
-      { id: 'c2', text: 'y' },
-    ]);
-  });
-});
-
-describe('room file-seed disable (the other half of the Risk-1 fix)', () => {
-  test('persistence.seed is a no-op for a pinned slug — never touches api or the doc', async () => {
-    let apiCalled = false;
-    // Minimal api stub; seed must short-circuit before reaching it.
-    const api = {
-      loadCommentsForFile: async () => {
-        apiCalled = true;
-        return [];
-      },
-      loadAnnotations: async () => {
-        apiCalled = true;
-        return null;
-      },
-    } as unknown as import('../api.ts').Api;
-    const ctx = { paths: { designRoot: dir } } as unknown as Context;
-
-    const persistence = createPersistence({
-      ctx,
-      api,
-      fileForSlug: async () => 'ui/x.html',
-      // Mirrors createCollab's predicate: don't seed a pinned (provider-attached)
-      // slug under sharedDoc — the migrate-seed + provider own population.
-      shouldSeed: (slug) => slug !== 'pinned',
-    });
-
-    const doc = new Y.Doc();
-    await persistence.seed('pinned', doc);
-    // Returned before touching the api or mutating the doc → no duplicate items
-    // can be introduced by the file-seed after the migrate-seed ran.
-    expect(apiCalled).toBe(false);
-    expect(docIsEmpty(doc)).toBe(true);
-  });
-});
-
-describe('migrateSeed — hub-wins (the duplication trap avoided)', () => {
-  test('a doc already holding hub comments is NOT merged with the local file → no dup', () => {
-    const p = paths();
-    // Local file (e.g. copied via git) holds the SAME logical comment id…
-    writeFileSync(p.comments, JSON.stringify([{ id: 'c1', text: 'from disk' }]));
-
-    // …but the doc already received it from the hub (a DIFFERENT CRDT item).
-    const doc = new Y.Doc();
-    applyCommentsToDoc(doc, [{ id: 'c1', text: 'from hub' }], 'hub');
-
-    const result = migrateSeed({ slug: 's', doc, paths: p });
-
-    // Hub wins; the local file is NOT pushed as a second item — exactly ONE c1.
-    expect(result).toBe('hub-wins');
-    expect(doc.getArray(Y_TYPES.comments).toArray()).toEqual([{ id: 'c1', text: 'from hub' }]);
-  });
-
-  test('snapshots divergent local files to historyDir before the hub-wins overwrite', () => {
-    const p = paths();
-    writeFileSync(p.html, '<main>LOCAL divergent</main>');
-    writeFileSync(p.comments, JSON.stringify([{ id: 'local' }]));
-
-    const doc = new Y.Doc();
-    applyHtmlToDoc(doc, '<main>HUB</main>', 'hub'); // doc holds hub state
-
-    const historyDir = join(dir, '_history', 's');
-    const result = migrateSeed({ slug: 's', doc, paths: p, historyDir });
-
-    expect(result).toBe('hub-wins');
-    // The pre-migration snapshot captured the local files for rollback.
-    const snapped = readdirSync(join(historyDir, 'pre-shared-doc-migration'));
-    expect(snapped).toContain('screen.html');
-    expect(snapped).toContain('screen.comments.json');
-  });
-});