quiver-cli 1.3.1 → 2.0.0

package/template/.agents/skills/design/impeccable/scripts/live/browser-script-parts.mjs

@@ -1,84 +0,0 @@
-import fs from 'node:fs';
-import path from 'node:path';
-
-import { LIVE_CHROME_MOUNT_CONTRACT, LIVE_UI_SURFACES } from './ui-surfaces.mjs';
-
-export const LIVE_BROWSER_SCRIPT_PARTS = Object.freeze([
-  Object.freeze({ name: 'session-state', file: 'live-browser-session.js' }),
-  Object.freeze({ name: 'dom-helpers', file: 'live-browser-dom.js' }),
-  Object.freeze({ name: 'project-ignores', file: 'live-browser-ignores.js' }),
-  Object.freeze({ name: 'browser-ui', file: 'live-browser.js' }),
-]);
-
-export function resolveLiveBrowserScriptParts(scriptsDir, parts = LIVE_BROWSER_SCRIPT_PARTS) {
-  if (!scriptsDir) throw new Error('scriptsDir is required');
-  return parts.map((part, index) => ({
-    ...part,
-    index,
-    path: path.join(scriptsDir, part.file),
-  }));
-}
-
-export function assertLiveBrowserScriptParts(parts, exists = fs.existsSync) {
-  for (const part of parts) {
-    if (!exists(part.path)) {
-      throw new Error(`Live browser script part missing: ${part.name} (${part.path})`);
-    }
-  }
-  return parts;
-}
-
-export function readLiveBrowserScriptParts(parts, readFile = (filePath) => fs.readFileSync(filePath, 'utf-8')) {
-  return parts.map((part) => ({
-    ...part,
-    source: readFile(part.path),
-  }));
-}
-
-export function assembleLiveBrowserScript({
-  token,
-  port,
-  vocabulary,
-  commandPrefix = '/',
-  appRoot = null,
-  parts,
-  // Defaulted rather than threaded through live-server.mjs: the browser bundle
-  // must always carry the canonical inventory, and a default makes that true by
-  // construction instead of by every caller remembering to pass it. Overridable
-  // so tests can assemble with a stand-in.
-  uiSurfaces = LIVE_UI_SURFACES,
-  mountContract = LIVE_CHROME_MOUNT_CONTRACT,
-  // Project detector waivers ({ ignoreRules, ignoreValues, roots }), read from
-  // .impeccable config by live-server.mjs. live-browser-ignores.js resolves
-  // them against the page when a detect scan starts, so the overlay filters
-  // the same findings the CLI and the edit hook do (issue #639).
-  projectIgnores = null,
-}) {
-  const prelude =
-    `window.__IMPECCABLE_TOKEN__ = '${token}';\n` +
-    `window.__IMPECCABLE_PORT__ = ${port};\n` +
-    // Project identity for browser-side session storage. localStorage is
-    // keyed by ORIGIN, and two projects routinely share a localhost port
-    // across time; saved sessions carry this value so a resume can tell a
-    // foreign project's leftovers from its own.
-    `window.__IMPECCABLE_APP_ROOT__ = ${JSON.stringify(appRoot)};\n` +
-    `window.__IMPECCABLE_COMMAND_PREFIX__ = ${JSON.stringify(commandPrefix)};\n` +
-    // Canonical command vocabulary (values + labels + icons). live-browser.js
-    // builds its action picker from this instead of an inline copy.
-    `window.__IMPECCABLE_VOCAB__ = ${JSON.stringify(vocabulary)};\n` +
-    // Canonical Live chrome inventory from live/ui-surfaces.mjs. live-browser.js
-    // is a classic script and cannot import an ES module at runtime, so the list
-    // is serialized here and read off the global there. Node consumers (this
-    // repo's tests, the impeccable-site Live UI lab) import the module directly,
-    // which is what keeps the two from drifting.
-    `window.__IMPECCABLE_LIVE_UI_SURFACES__ = ${JSON.stringify(uiSurfaces)};\n` +
-    `window.__IMPECCABLE_LIVE_MOUNT_CONTRACT__ = ${JSON.stringify(mountContract)};\n` +
-    `window.__IMPECCABLE_PROJECT_IGNORES__ = ${JSON.stringify(projectIgnores)};\n`;
-
-  const body = parts.map((part) => {
-    const file = part.file || path.basename(part.path || '');
-    return `// --- impeccable live script part: ${part.name} (${file}) ---\n${part.source}`;
-  }).join('\n');
-
-  return prelude + body;
-}

package/template/.agents/skills/design/impeccable/scripts/live/completion.mjs

@@ -1,28 +0,0 @@
-// A preview whose variants live in component modules rather than in the user's
-// source. These leave no markers in the real file, so a failed accept gives the
-// agent nothing to hand-edit and must be reported as a failure rather than
-// reference/live.md's manual-cleanup handoff. Kept as a set: any future
-// component-module preview mode belongs here the day it lands.
-const PREVIEW_MODES_WITHOUT_SOURCE_MARKERS = new Set([
-  'svelte-component',
-]);
-
-export function completionTypeForAcceptResult(eventType, acceptResult) {
-  if (eventType === 'discard') return acceptResult?.handled === true ? 'discarded' : 'error';
-  if (acceptResult?.handled === true && acceptResult?.carbonize === true) return 'agent_done';
-  if (acceptResult?.handled === true) return 'complete';
-  if (acceptResult?.mode === 'error') return 'error';
-  if (eventType === 'accept' && PREVIEW_MODES_WITHOUT_SOURCE_MARKERS.has(acceptResult?.previewMode)) return 'error';
-  return 'agent_done';
-}
-
-export function completionAckForAcceptResult(eventId, completionType, acceptResult) {
-  const ack = { ok: true, type: completionType };
-  if (acceptResult?.handled === true && acceptResult?.carbonize === true) {
-    ack.final = false;
-    ack.requiresComplete = true;
-    ack.nextCommand = `live-complete.mjs --id ${eventId}`;
-    ack.message = 'Carbonize cleanup must be verified, then the session must be completed explicitly before polling again.';
-  }
-  return ack;
-}

package/template/.agents/skills/design/impeccable/scripts/live/event-validation.mjs

@@ -1,199 +0,0 @@
-/**
- * Shared event validation for the live helper server.
- * Extracted for unit testing (insert mode rules).
- */
-
-import { canCreateInsert } from './insert-ui.mjs';
-
-// The accepted protocol values come from the canonical vocabulary so the
-// validator, the store, the server, and the picker UI never drift. Imported
-// (not just re-exported) so they are also in scope for the validators below.
-import { AGENT_PHASES, CLIENT_EVENT_TYPES, VISUAL_ACTIONS } from './vocabulary.mjs';
-export { AGENT_PHASES, CLIENT_EVENT_TYPES, VISUAL_ACTIONS };
-
-const AGENT_PHASE_SET = new Set(AGENT_PHASES);
-
-const ID_PATTERN = /^[0-9a-f]{8}$/;
-const VARIANT_ID_PATTERN = /^[0-9]{1,3}$/;
-const INSERT_POSITIONS = new Set(['before', 'after']);
-const FORBIDDEN_MANUAL_EDIT_TEXT_CHARS = ['<', '{', '}', '`'];
-
-// Mount acknowledgements carry a module URL and a raw exception message from
-// the page. Both are attacker-adjacent (any script on the page can POST them
-// with the token it can already read), so they are length-capped before they
-// reach the journal.
-export const MOUNT_URL_MAX_LENGTH = 2000;
-export const MOUNT_ERROR_MAX_LENGTH = 1000;
-
-function isValidId(v) { return typeof v === 'string' && ID_PATTERN.test(v); }
-function isValidVariantId(v) { return typeof v === 'string' && VARIANT_ID_PATTERN.test(v); }
-
-function validateManualEditText(newText) {
-  if (typeof newText !== 'string') return null;
-  const hits = FORBIDDEN_MANUAL_EDIT_TEXT_CHARS.filter((char) => newText.includes(char));
-  return hits.length > 0 ? hits : null;
-}
-
-function validateAnnotationFields(msg) {
-  if (msg.screenshotPath !== undefined && typeof msg.screenshotPath !== 'string') {
-    return 'generate: screenshotPath must be string';
-  }
-  if (msg.comments !== undefined && !Array.isArray(msg.comments)) {
-    return 'generate: comments must be array';
-  }
-  if (msg.strokes !== undefined && !Array.isArray(msg.strokes)) {
-    return 'generate: strokes must be array';
-  }
-  return null;
-}
-
-function validateInsertGenerate(msg) {
-  if (!msg.insert || typeof msg.insert !== 'object') return 'generate: insert mode requires insert object';
-  if (!INSERT_POSITIONS.has(msg.insert.position)) return 'generate: insert.position must be before or after';
-  const anchor = msg.insert.anchor;
-  if (!anchor || typeof anchor !== 'object') return 'generate: insert.anchor required';
-  if (!anchor.tagName && !anchor.outerHTML && !(Array.isArray(anchor.classes) && anchor.classes.length)) {
-    return 'generate: insert.anchor needs tagName, classes, or outerHTML';
-  }
-  if (!msg.placeholder || typeof msg.placeholder !== 'object') return 'generate: insert mode requires placeholder dimensions';
-  if (!Number.isFinite(msg.placeholder.width) || !Number.isFinite(msg.placeholder.height)) {
-    return 'generate: placeholder width and height must be numbers';
-  }
-  if (!canCreateInsert({
-    prompt: msg.freeformPrompt,
-    comments: msg.comments,
-    strokes: msg.strokes,
-  })) {
-    return 'generate: insert requires freeformPrompt or annotations';
-  }
-  return validateAnnotationFields(msg);
-}
-
-function validateReplaceGenerate(msg) {
-  if (!msg.action || !VISUAL_ACTIONS.includes(msg.action)) return 'generate: invalid action';
-  if (!msg.element || !msg.element.outerHTML) return 'generate: missing element context';
-  return validateAnnotationFields(msg);
-}
-
-function validateManualEditEvent(msg, label) {
-  if (!isValidId(msg.id)) return label + ': missing or malformed id';
-  if (!msg.pageUrl || typeof msg.pageUrl !== 'string') return label + ': missing pageUrl';
-  if (!msg.element || typeof msg.element !== 'object') return label + ': missing element';
-  if (!Array.isArray(msg.ops) || msg.ops.length === 0) return label + ': ops must be non-empty array';
-  if (msg.ops.length > 100) return label + ': too many ops (max 100)';
-  for (const op of msg.ops) {
-    if (typeof op.ref !== 'string') return label + ': op.ref required';
-    if (typeof op.tag !== 'string') return label + ': op.tag required';
-    if (typeof op.originalText !== 'string') return label + ': op.originalText required';
-    if (op.deleted !== true && typeof op.newText !== 'string') {
-      return label + ': text op requires newText';
-    }
-    if (typeof op.newText === 'string') {
-      if (op.deleted !== true && op.newText.trim().length === 0) {
-        return label + ': newText cannot be empty';
-      }
-      const forbidden = validateManualEditText(op.newText);
-      if (forbidden) {
-        return label + ': newText cannot contain ' + forbidden.join(' ') + ' (plain text only; ask the AI to insert markup)';
-      }
-    }
-  }
-  return null;
-}
-
-function isValidMountVariant(value) {
-  return Number.isInteger(value) && value >= 1 && value <= 999;
-}
-
-/**
- * Mount acknowledgements are the browser's answer to "did the thing you
- * published actually render". They are validated strictly because the render
- * truth in the session snapshot is built from them: a malformed ack that slid
- * through would report a variant as mounted that never was.
- */
-function validateMountAck(msg) {
-  if (!isValidId(msg.id)) return 'variant_mounted: missing or malformed id';
-  if (!isValidMountVariant(msg.variant)) return 'variant_mounted: variant must be an integer 1-999';
-  if (msg.url !== undefined) {
-    if (typeof msg.url !== 'string') return 'variant_mounted: url must be string';
-    if (msg.url.length > MOUNT_URL_MAX_LENGTH) return 'variant_mounted: url too long';
-  }
-  return null;
-}
-
-function validateMountFailure(msg) {
-  if (!isValidId(msg.id)) return 'variant_mount_failed: missing or malformed id';
-  if (!isValidMountVariant(msg.variant)) return 'variant_mount_failed: variant must be an integer 1-999';
-  if (typeof msg.url !== 'string' || !msg.url.trim()) return 'variant_mount_failed: url required';
-  if (msg.url.length > MOUNT_URL_MAX_LENGTH) return 'variant_mount_failed: url too long';
-  if (typeof msg.error !== 'string' || !msg.error.trim()) return 'variant_mount_failed: error required';
-  if (msg.error.length > MOUNT_ERROR_MAX_LENGTH) return 'variant_mount_failed: error too long';
-  return null;
-}
-
-export function validateEvent(msg) {
-  if (!msg || typeof msg !== 'object' || !msg.type) return 'Missing or invalid message';
-  switch (msg.type) {
-    case 'generate':
-      if (!isValidId(msg.id)) return 'generate: missing or malformed id';
-      if (!Number.isInteger(msg.count) || msg.count < 1 || msg.count > 8) return 'generate: count must be 1-8';
-      if (msg.mode === 'insert') return validateInsertGenerate(msg);
-      return validateReplaceGenerate(msg);
-    case 'accept':
-      if (!isValidId(msg.id)) return 'accept: missing or malformed id';
-      if (!isValidVariantId(msg.variantId)) return 'accept: missing or malformed variantId';
-      if (msg.paramValues !== undefined) {
-        if (typeof msg.paramValues !== 'object' || msg.paramValues === null || Array.isArray(msg.paramValues)) {
-          return 'accept: paramValues must be an object';
-        }
-      }
-      return null;
-    case 'discard':
-      return isValidId(msg.id) ? null : 'discard: missing or malformed id';
-    case 'checkpoint':
-      if (!isValidId(msg.id)) return 'checkpoint: missing or malformed id';
-      if (!Number.isInteger(msg.revision) || msg.revision < 0) return 'checkpoint: revision must be a non-negative integer';
-      if (msg.paramValues !== undefined && (typeof msg.paramValues !== 'object' || msg.paramValues === null || Array.isArray(msg.paramValues))) {
-        return 'checkpoint: paramValues must be an object';
-      }
-      return null;
-    case 'agent_phase':
-      if (!isValidId(msg.id)) return 'agent_phase: missing or malformed id';
-      if (typeof msg.phase !== 'string' || !msg.phase) return 'agent_phase: missing phase';
-      // The enum, not a shape pattern. A phase the browser cannot rank is a
-      // phase the progress bar cannot show, so accepting an arbitrary
-      // lowercase word only defers the failure to the UI.
-      if (!AGENT_PHASE_SET.has(msg.phase)) {
-        return 'agent_phase: unknown phase ' + msg.phase + ' (expected one of ' + AGENT_PHASES.join(', ') + ')';
-      }
-      if (msg.durationMs !== undefined && (!Number.isFinite(msg.durationMs) || msg.durationMs < 0)) {
-        return 'agent_phase: durationMs must be a non-negative number';
-      }
-      return null;
-    case 'variant_mounted':
-      return validateMountAck(msg);
-    case 'variant_mount_failed':
-      return validateMountFailure(msg);
-    case 'exit':
-      return null;
-    case 'prefetch':
-      if (!msg.pageUrl || typeof msg.pageUrl !== 'string') return 'prefetch: missing pageUrl';
-      return null;
-    case 'manual_edits':
-      return validateManualEditEvent(msg, 'manual_edits');
-    case 'steer':
-      if (!isValidId(msg.id)) return 'steer: missing or malformed id';
-      if (typeof msg.message !== 'string' || !msg.message.trim()) return 'steer: message required';
-      if (msg.message.length > 4000) return 'steer: message too long';
-      if (msg.pageUrl !== undefined && typeof msg.pageUrl !== 'string') return 'steer: pageUrl must be string';
-      return null;
-    case 'carbonize_cleanup':
-      if (!isValidId(msg.id)) return 'carbonize_cleanup: missing or malformed id';
-      if (!isValidId(msg.sessionId)) return 'carbonize_cleanup: missing or malformed sessionId';
-      if (!msg.file || typeof msg.file !== 'string') return 'carbonize_cleanup: missing file';
-      if (!isValidVariantId(String(msg.variantId))) return 'carbonize_cleanup: missing or malformed variantId';
-      return null;
-    default:
-      return 'Unknown event type: ' + msg.type;
-  }
-}

package/template/.agents/skills/design/impeccable/scripts/live/frameworks/astro.mjs

@@ -1,47 +0,0 @@
-/**
- * Astro registry entry.
- *
- * Astro takes the generic tag strategy, with two Astro-specific values that
- * used to sit as inline `endsWith('.astro')` branches in live-inject.mjs and
- * live-wrap.mjs:
- *
- *   injectScriptAttrs  Astro processes <script> tags by default and rewrites
- *                      src to its own bundled URL; is:inline opts out.
- *   styleMode          Astro scopes component styles, which strips preview CSS
- *                      off the generated variant wrappers, so preview rules are
- *                      authored global and prefixed instead of @scope'd.
- */
-
-import { findConfigFile, hasAnyDependency, literalConfigFiles } from './detect-utils.mjs';
-
-const ASTRO_CONFIG_RE = /^astro\.config\.(?:js|mjs|cjs|ts|mts|cts)$/;
-
-export function detectAstroProject(cwd = process.cwd(), config = null) {
-  const configFile = findConfigFile(cwd, ASTRO_CONFIG_RE);
-  if (configFile) return { configFile, via: 'config' };
-  if (hasAnyDependency(cwd, ['astro'])) return { configFile: null, via: 'package' };
-  // A tree of .astro entry templates with no astro.config still belongs to
-  // Astro; the configured injection target names it.
-  const entry = literalConfigFiles(cwd, config).find((rel) => rel.endsWith('.astro'));
-  if (entry) return { configFile: null, via: 'config-files', entry };
-  return null;
-}
-
-export const astro = {
-  name: 'astro',
-
-  detect(cwd, config) {
-    return detectAstroProject(cwd, config);
-  },
-
-  inject: { kind: 'tag' },
-
-  source: {
-    extensions: ['.astro'],
-    preview: 'source',
-    styleMode: 'astro-global-prefixed',
-    styleTag: '<style is:inline data-impeccable-css="SESSION_ID">',
-    commentSyntax: 'html',
-    injectScriptAttrs: 'is:inline ',
-  },
-};

package/template/.agents/skills/design/impeccable/scripts/live/frameworks/detect-utils.mjs

@@ -1,73 +0,0 @@
-/**
- * Small read-only probes the framework entries share.
- *
- * Every helper here is cheap and failure-tolerant: detection runs on every
- * inject, against project trees that may be half-installed, so a missing or
- * malformed file means "not this framework", never a throw.
- */
-
-import fs from 'node:fs';
-import path from 'node:path';
-
-/** Merged dependency names from package.json, or an empty object. */
-export function readPackageDeps(cwd) {
-  const file = path.join(cwd, 'package.json');
-  try {
-    const pkg = JSON.parse(fs.readFileSync(file, 'utf-8'));
-    return {
-      ...(pkg.dependencies || {}),
-      ...(pkg.devDependencies || {}),
-      ...(pkg.peerDependencies || {}),
-    };
-  } catch {
-    return {};
-  }
-}
-
-export function hasAnyDependency(cwd, names) {
-  const deps = readPackageDeps(cwd);
-  return names.some((name) => Boolean(deps[name]));
-}
-
-/** First top-level file name matching `re`, or null. */
-export function findConfigFile(cwd, re) {
-  try {
-    return fs.readdirSync(cwd, { withFileTypes: true })
-      .find((entry) => entry.isFile() && re.test(entry.name))
-      ?.name ?? null;
-  } catch {
-    return null;
-  }
-}
-
-export function fileExists(cwd, rel) {
-  try {
-    return fs.existsSync(path.join(cwd, rel));
-  } catch {
-    return false;
-  }
-}
-
-export function firstExistingFile(cwd, candidates) {
-  for (const rel of candidates) {
-    if (fileExists(cwd, rel)) return rel;
-  }
-  return null;
-}
-
-/**
- * Literal (non-glob) entries of `config.files` that exist on disk. Several
- * detectors read the configured injection target as a signal, which is how the
- * bare fixtures — a tree of `.astro` files with no astro.config — still resolve
- * to the framework that authored them.
- */
-export function literalConfigFiles(cwd, config) {
-  const files = Array.isArray(config?.files) ? config.files : [];
-  const out = [];
-  for (const rel of files) {
-    if (typeof rel !== 'string' || rel.includes('*') || rel.includes('?')) continue;
-    const normalized = rel.split(path.sep).join('/');
-    if (fileExists(cwd, normalized)) out.push(normalized);
-  }
-  return out;
-}

package/template/.agents/skills/design/impeccable/scripts/live/frameworks/index.mjs

@@ -1,143 +0,0 @@
-/**
- * The live-mode framework registry.
- *
- * Before this existed, framework knowledge was smeared across live-inject.mjs
- * (detection order, the Nuxt adapter, the Astro `is:inline` branch), the two
- * adapter modules, and live-wrap.mjs (which extension gets component preview,
- * which gets Astro's global-prefixed CSS, which gets JSX comments). Adding or
- * fixing a framework meant reading all of them.
- *
- * One entry per framework now declares everything the live scripts need:
- *
- *   name        stable identifier; also the `adapter` value in inject JSON.
- *   detect      (cwd, config) → falsy when this is not the project, otherwise
- *               a truthy project descriptor that apply/remove/artifacts read.
- *               Order in FRAMEWORKS is priority order; first truthy wins.
- *   inject      { kind: 'adapter', apply, remove, ignorePatterns, artifacts,
- *               unpatch } for frameworks that server-render their document
- *               shell, or { kind: 'tag' } for the generic marker-wrapped
- *               <script src> block.
- *   source      how live-wrap treats files this framework authors:
- *               extensions, preview ('source' | 'component'), styleMode,
- *               styleTag, commentSyntax, injectScriptAttrs. Anything omitted
- *               falls back to SOURCE_TRAIT_DEFAULTS.
- *
- * Two rules hold the thing together:
- *
- * 1. **Detection order is injection priority.** SvelteKit → Nuxt → TanStack
- *    Start → Astro → Next → Vite → static HTML, exactly the order
- *    live-inject.mjs used to hard-code. static-html always matches, so
- *    resolveFramework never returns null.
- * 2. **Source traits resolve by file extension, not by project.** A SvelteKit
- *    project's injection target is `src/app.html`; a Vite app can contain
- *    `.astro` partials. live-wrap has always keyed these off the target file,
- *    and resolveSourceTraits keeps it that way. Several entries may claim the
- *    same extension (`.tsx` belongs to three); when they do, the values must
- *    agree, which tests/live-frameworks.test.mjs asserts.
- */
-
-import path from 'node:path';
-
-import { sveltekit } from './sveltekit.mjs';
-import { nuxt } from './nuxt.mjs';
-import { tanstackStart } from './tanstack-start.mjs';
-import { astro } from './astro.mjs';
-import { nextjs } from './nextjs.mjs';
-import { viteGeneric } from './vite-generic.mjs';
-import { staticHtml } from './static-html.mjs';
-import { TAG_PATCH_MARKERS, unpatchTagFile } from './tag-strategy.mjs';
-
-/** Priority order. Do not reorder without re-reading rule 1 above. */
-export const FRAMEWORKS = Object.freeze([
-  sveltekit,
-  nuxt,
-  tanstackStart,
-  astro,
-  nextjs,
-  viteGeneric,
-  staticHtml,
-]);
-
-export const PREVIEW_MODES = Object.freeze(['source', 'component']);
-export const STYLE_MODES = Object.freeze(['scoped', 'astro-global-prefixed']);
-export const COMMENT_SYNTAXES = Object.freeze(['html', 'jsx']);
-export const INJECT_KINDS = Object.freeze(['adapter', 'tag']);
-
-export const SOURCE_TRAIT_DEFAULTS = Object.freeze({
-  preview: 'source',
-  styleMode: 'scoped',
-  styleTag: '<style data-impeccable-css="SESSION_ID">',
-  commentSyntax: 'html',
-  injectScriptAttrs: '',
-});
-
-/** The patch kind the generic tag strategy records in the journal. */
-export const TAG_PATCH_KIND = 'live-tag';
-
-/**
- * Undo functions keyed by the `patch` value an artifact carries. Built from
- * the entries so a new adapter registers its own undo alongside its apply.
- */
-export const PATCH_UNDOERS = Object.freeze(Object.assign(
-  { [TAG_PATCH_KIND]: unpatchTagFile },
-  ...FRAMEWORKS.map((framework) => framework.inject.unpatch || {}),
-));
-
-/**
- * First entry whose detect() matches. Returns { framework, project } where
- * project is the detector's descriptor (adapters read it; tag frameworks
- * mostly ignore it).
- */
-export function resolveFramework(cwd = process.cwd(), config = null) {
-  for (const framework of FRAMEWORKS) {
-    const project = framework.detect(cwd, config);
-    if (project) return { framework, project };
-  }
-  // Unreachable while static-html stays terminal, but a caller that reorders
-  // the array should get a diagnosable null rather than a silent tag inject.
-  return null;
-}
-
-/**
- * Source-authoring traits for one file, merged over SOURCE_TRAIT_DEFAULTS.
- * `framework` names the entry that claimed the extension, or null.
- */
-export function resolveSourceTraits(filePath) {
-  const ext = path.extname(String(filePath || '')).toLowerCase();
-  for (const framework of FRAMEWORKS) {
-    const source = framework.source;
-    if (!source || !source.extensions.includes(ext)) continue;
-    const { extensions, ...traits } = source;
-    return { framework: framework.name, ...SOURCE_TRAIT_DEFAULTS, ...traits };
-  }
-  return { framework: null, ...SOURCE_TRAIT_DEFAULTS };
-}
-
-/**
- * Extra gitignore patterns the resolved framework needs beyond the static
- * LIVE_IGNORE_PATTERNS list (paths that depend on a detected srcDir or file
- * extension and so cannot be written down ahead of time).
- */
-export function frameworkIgnorePatterns(resolved) {
-  const fn = resolved?.framework?.inject?.ignorePatterns;
-  return typeof fn === 'function' ? (fn(resolved.project) || []) : [];
-}
-
-/**
- * The files this injection will create or patch, in journal-artifact form.
- * Adapters declare their own; the tag strategy patches exactly the resolved
- * config files.
- */
-export function describeInjectArtifacts(resolved, { cwd = process.cwd(), files = [] } = {}) {
-  if (!resolved) return [];
-  const { framework, project } = resolved;
-  if (framework.inject.kind === 'adapter') {
-    return (framework.inject.artifacts?.({ cwd, project }) || []).filter((a) => a && a.path);
-  }
-  return files.map((file) => ({
-    kind: 'patched',
-    path: file,
-    patch: TAG_PATCH_KIND,
-    markers: [...TAG_PATCH_MARKERS],
-  }));
-}