@everystack/cli 0.4.50 → 0.4.51

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@everystack/cli",
-  "version": "0.4.50",
+  "version": "0.4.51",
   "description": "CLI and OTA updates for Expo apps on everystack",
   "license": "AGPL-3.0-only",
   "author": "Scalable Technology, Inc. <licensing@scalable.technology>",

package/src/cli/commands/db-check.ts

@@ -37,7 +37,8 @@ import { compileDrizzleSource } from '../schema-source.js';
 import { currentGitRef } from '../state-apply.js';
 import { createDatabase, dropDatabase, buildIntoDatabase, withDatabase } from '../db-build.js';
 import { resolveModelsPath } from '../models-path.js';
-import { loadModels } from './db-generate.js';
+import { loadModels, loadModules } from './db-generate.js';
+import { findUnexportedModels, scanModelDirectory, type ModelFileScan } from '../model-barrel-lint.js';
 import { loadDeclaredDerived, retiredSqlDirAnywhere, retiredSqlDirFlagRefusal, type DeclaredDerived } from '../declared-derived.js';
 import type { SequenceDescriptor } from '@everystack/model';
 import type { SourceObject } from '../derived-source.js';
@@ -67,6 +68,10 @@ export interface CheckFinding {
 
 export interface StaticCheckInput {
   modelsPath: string;
+  /** Barrel siblings and what each declares — the "on disk but ungoverned" scan. */
+  modelFileScans?: ModelFileScan[];
+  /** The module's extensions — the bootstrap the from-scratch compose needs. */
+  extensions?: string[];
   /** null = the barrel did not load — the gate's first failure. */
   models: ModelDescriptor[] | null;
   modelsError?: string;
@@ -104,6 +109,17 @@ export function runStaticChecks(input: StaticCheckInput): CheckFinding[] {
   }
   findings.push({ level: 'ok', area: 'models', message: `${input.models.length} model(s) loaded from ${input.modelsPath}` });
 
+  // A file on disk the barrel never exported is outside governance while looking declared.
+  // FAIL, not warn: it is absent from the fingerprint, so nothing downstream will ever
+  // notice, and the only other signal is a model COUNT nobody can compare against.
+  const barrelGaps = findUnexportedModels(
+    input.modelFileScans ?? [],
+    new Set(input.models.map((m) => `${m.schema}.${m.table}`)),
+  );
+  for (const gap of barrelGaps) {
+    findings.push({ level: 'fail', area: 'models', message: gap.message });
+  }
+
   // The semantic merge conflict git can't see: two branches declaring the
   // same table both merge cleanly — the merged state must still compose.
   const seen = new Map<string, number>();
@@ -144,7 +160,7 @@ export function runStaticChecks(input: StaticCheckInput): CheckFinding[] {
 
   try {
     compileDeclaredState(input.models);
-    const statements = compileMigration(input.models, { sequences: input.sequences });
+    const statements = compileMigration(input.models, { sequences: input.sequences, extensions: input.extensions });
     findings.push({ level: 'ok', area: 'compose', message: `declared state composes — ${statements.length} statement(s) from scratch` });
   } catch (err: any) {
     findings.push({ level: 'fail', area: 'compose', message: `the merged declared state does NOT compose: ${err.message}` });
@@ -281,7 +297,7 @@ export interface EphemeralComposeResult {
 export async function executeEphemeralCompose(
   adminUrl: string,
   models: ModelDescriptor[],
-  opts: { actor?: string | null; gitRef?: string | null; declared?: SourceObject[]; sequences?: SequenceDescriptor[] } = {},
+  opts: { actor?: string | null; gitRef?: string | null; declared?: SourceObject[]; sequences?: SequenceDescriptor[]; extensions?: string[] } = {},
 ): Promise<EphemeralComposeResult> {
   const database = `escheck_${process.pid}_${Date.now()}`;
   await createDatabase(adminUrl, database);
@@ -289,6 +305,7 @@ export async function executeEphemeralCompose(
     const built = await buildIntoDatabase(withDatabase(adminUrl, database), models, {
       declared: opts.declared,
       sequences: opts.sequences,
+      extensions: opts.extensions,
       actor: opts.actor ?? 'db:check',
       gitRef: opts.gitRef ?? null,
     });
@@ -315,6 +332,19 @@ export async function dbCheckCommand(flags: Record<string, string>): Promise<voi
     modelsError = err.message;
   }
 
+  // Only meaningful once the barrel itself loaded — otherwise every sibling reads as
+  // ungoverned and the real error is buried under the noise.
+  const modelFileScans = models ? await scanModelDirectory(modelsPath) : [];
+
+  // The module carries the extensions; `models` alone cannot. Without them the from-scratch
+  // compose builds a schema whose column types do not exist.
+  let extensions: string[] = [];
+  if (models) {
+    try {
+      extensions = [...new Set((await loadModules(modelsPath)).flatMap((m) => m.extensions ?? []))];
+    } catch { /* a models-only barrel has no modules export — nothing to bootstrap */ }
+  }
+
   const artifactPath = flags['schema-out'] || DEFAULT_SCHEMA_OUT;
   let artifactSource: string | null = null;
   try {
@@ -349,7 +379,7 @@ export async function dbCheckCommand(flags: Record<string, string>): Promise<voi
   } catch { /* no baseline — see above */ }
 
   const findings = runStaticChecks({
-    modelsPath, models, modelsError, artifactPath, artifactSource, sqlDirRetired,
+    modelsPath, models, modelsError, modelFileScans, extensions, artifactPath, artifactSource, sqlDirRetired,
     sequences: declaredDb?.sequences, derived: declaredDb?.derived, derivedError,
     moduleModels: declaredDb?.models ?? null,
     baselineSource,
@@ -374,6 +404,7 @@ export async function dbCheckCommand(flags: Record<string, string>): Promise<voi
         gitRef: currentGitRef(),
         declared: declaredDb?.objects,
         sequences: declaredDb?.sequences,
+        extensions,
       });
       for (const line of ephemeral.report) info(`  ${line}`);
     } catch (err: any) {

package/src/cli/commands/db-generate.ts

@@ -72,7 +72,7 @@ export async function loadModels(modelsPath: string): Promise<ModelDescriptor[]>
  * COMPLETE migration. Falls back to wrapping a bare `models` array in one module (an app on the
  * older models-only barrel still gets a schema+authz init, just no package functions).
  */
-async function loadModules(modelsPath: string): Promise<Module[]> {
+export async function loadModules(modelsPath: string): Promise<Module[]> {
   const abs = path.resolve(modelsPath);
   let mod: any;
   try {

package/src/cli/commands/db-plan.ts

@@ -221,6 +221,22 @@ export async function dbPlanCommand(flags: Record<string, string>): Promise<void
       }
     }
 
+    // The class rides on the ARTIFACT, not only the terminal — a reviewer reading
+    // db.plan.json had the aggregate and no way to reach the statements behind it, so
+    // `unclassified: 5` was unreadable by the only person positioned to catch what it meant.
+    //
+    // Attached HERE, before the plan is serialized AND before planHash is taken, because
+    // both must see the same object. It used to be assigned after the write, onto an
+    // instance nobody read again: the artifact carried no adoption at all, and the summary
+    // counted statements the file could not show. Hashing after attaching also keeps the
+    // ref db:plan prints equal to the one db:apply computes for that file — `planHash` is
+    // sha256 over the WHOLE plan, so a field added later silently changes the identity.
+    //
+    // Re-DERIVED from live every mint, never stored across mints: a recorded claim about
+    // what the models fail to capture goes stale the moment someone closes the gap.
+    const adoption = classifyAdoption(declaredAuthz, contract, { governedRoles: governedRoleSet(declaredAuthz, declaredGovernedRoles) });
+    plan.adoption = adoption.statements;
+
     const body = JSON.stringify(plan, null, 2) + '\n';
     if (out === '-') {
       console.log(body);
@@ -238,16 +254,11 @@ export async function dbPlanCommand(flags: Record<string, string>): Promise<void
     // fail to capture goes stale the moment someone closes the gap, and a stale note is worse
     // than none. `capability` is the count that says "this plan removes something the model
     // cannot express" — the operator reading this is the last one who can catch it.
-    const adoption = classifyAdoption(declaredAuthz, contract, { governedRoles: governedRoleSet(declaredAuthz, declaredGovernedRoles) });
     const adoptionTotal = Object.values(adoption.counts).reduce((a, b) => a + b, 0);
     if (adoptionTotal > 0) {
       info(`${adoptionTotal} authorization statement(s), by why they exist:`);
       for (const line of renderAdoptionReport(adoption.counts, adoption.statements)) info(line);
     }
-    // The class rides on the ARTIFACT too, not only the terminal. A reviewer reading
-    // db.plan.json had the aggregate and no way to reach the statements behind it, so
-    // `unclassified: 5` was unreadable by the only person positioned to catch what it meant.
-    plan.adoption = adoption.statements;
 
     // WHO owns the tables this plan authorizes, and does that owner obey the policies it
     // is about to write? Re-derived live every mint, never stored — the owner is a fact

package/src/cli/commands/db-pull.ts

@@ -108,7 +108,12 @@ export function derivedImportSpecifier(out: string, derivedOut: string): string
 }
 
 export async function dbPullCommand(flags: Record<string, string>): Promise<void> {
-  const schema = flags.schema || 'public';
+  // A LIST: `--schema public,auth,metrics`. One value still works. Without this there was
+  // no way to adopt a multi-schema database — `--out <dir>` regenerates the barrel, so a
+  // second pull replaced the first schema's models rather than joining them.
+  const schemas = (flags.schema || 'public').split(',').map((s) => s.trim()).filter(Boolean);
+  if (schemas.length === 0) fail('--schema needs at least one schema name.');
+  const schema = schemas.length === 1 ? schemas[0] : schemas;
 
   // --derived-out: the brownfield splice, first-class. A consumer with an existing
   // hand-maintained barrel wants the derived layer as its own file — not codemodded
@@ -167,7 +172,7 @@ export async function dbPullCommand(flags: Record<string, string>): Promise<void
       runner = lambdaQueryRunner(config.region, opsFunction(config));
       session = lambdaSessionRunner(config.region, opsFunction(config));
     }
-    note(`Introspecting live database (schema: ${schema})...`);
+    note(`Introspecting live database (schema: ${schemas.join(", ")})...`);
     current = await introspectSchema(session);
     // The derived layer rides the same pull (B5) — views/matviews/functions/sequences
     // render as descriptors; adoption is pull → commit → --baseline → clean reconcile.

package/src/cli/db-build.ts

@@ -17,6 +17,7 @@ import type { SequenceDescriptor } from '@everystack/model';
 import { compileDeclaredState } from './declared-diff.js';
 import { createUrlRunner } from './db-source.js';
 import { executeSync, buildSyncReport } from './commands/db-sync.js';
+import { executeReconcile } from './commands/db-reconcile.js';
 
 const SAFE_NAME = /^[a-z_][a-z0-9_$]*$/;
 
@@ -93,6 +94,10 @@ export interface BuildOptions {
   declared?: SourceObject[];
   /** Standalone sequences (state — created before tables, in the fingerprint bar). */
   sequences?: SequenceDescriptor[];
+  /** Extensions the declared state needs. Applied BEFORE anything else — a column typed
+   *  `hstore` cannot be created until the type exists, and the sync path is a DIFF, which
+   *  has no bootstrap phase of its own. `IF NOT EXISTS`, so re-running is free. */
+  extensions?: string[];
   actor?: string | null;
   gitRef?: string | null;
 }
@@ -109,7 +114,62 @@ export async function buildIntoDatabase(
 ): Promise<BuildResult> {
   const { runner, session, end } = await createUrlRunner(url);
   try {
+    // Bootstrap FIRST: roles and tables both come after the types they use exist.
+    for (const ext of [...new Set(options.extensions ?? [])].sort()) {
+      await runner(`CREATE EXTENSION IF NOT EXISTS "${ext}"`);
+    }
+    // Schemas, for the same reason. This path is a DIFF (executeSync), and a diff has no
+    // bootstrap phase: `compileMigration` emits CREATE SCHEMA for every non-public schema a
+    // model lives in, but nothing on the sync path did — so a multi-schema declared state
+    // failed with `schema "auth" does not exist` before a single table was created. Taken
+    // from the MODELS and from the declared derived objects, because a schema can hold only
+    // functions (an `auth` of nothing but SECURITY DEFINER functions is a real shape).
+    const modelSchemas = models.map((m) => m.schema ?? 'public');
+    const derivedSchemas = (options.declared ?? []).map((o) => o.schema);
+    for (const schema of [...new Set([...modelSchemas, ...derivedSchemas])].filter((s) => s && s !== 'public').sort()) {
+      await runner(`CREATE SCHEMA IF NOT EXISTS "${schema}"`);
+    }
     const createdRoles = await ensureContractRoles(runner, models);
+
+    // FUNCTIONS BEFORE STATE. An RLS policy's predicate is resolved when the policy is
+    // created, so `USING (user_id = auth.user_id())` cannot be created before
+    // `auth.user_id()` exists — and the sync path applies state (tables, RLS, policies)
+    // before the derived layer. On a real brownfield schema that is not an edge case: 18 of
+    // one consumer's policies call an `auth.*` function, so the from-scratch build failed
+    // with `function auth.user_id() does not exist` before any of them could be created.
+    //
+    // Only FUNCTIONS move: views and triggers depend on TABLES, so the layers genuinely
+    // interleave and "derived before state" would just fail the other way round.
+    //
+    // `check_function_bodies = off` for this window — the pg_dump restore idiom. A
+    // SQL-language function whose body reads a table that does not exist yet is a forward
+    // reference, not an error; plpgsql bodies are never validated at creation, so only the
+    // SQL-language ones need it. MEASURED, not assumed: dropping this SET puts the reference
+    // brownfield schema straight back to failing, so the escape is load-bearing.
+    //
+    // It is a BARE set, not SET LOCAL, and that is deliberate: reconcile opens its own
+    // transaction, so this pass has none to scope to. It is safe under the rule the repo's
+    // own guard documents — `createUrlRunner` gives this command a dedicated max:1
+    // connection it owns for the run and closes in a finally, never a pooled or shared one.
+    // RESET in a finally so the window closes even on failure.
+    //
+    // Provenance is recorded by this pass, so executeSync's own reconcile below sees the
+    // functions already managed and unchanged, and plans nothing for them.
+    const declaredFunctions = (options.declared ?? []).filter((o) => o.kind === 'function');
+    if (declaredFunctions.length > 0) {
+      await runner('SET check_function_bodies = off');
+      try {
+        const pre = await executeReconcile(runner, session, {
+          declared: declaredFunctions,
+          apply: true,
+          actor: options.actor ?? 'db-build',
+          gitRef: options.gitRef ?? null,
+        });
+        if (pre.refusal) throw new Error(`could not create declared functions first: ${pre.refusal}`);
+      } finally {
+        await runner('RESET check_function_bodies');
+      }
+    }
     const run = await executeSync(runner, session, models, {
       declared: options.declared,
       sequences: options.sequences,

package/src/cli/derived-render.ts

@@ -16,7 +16,7 @@
 import type { DerivedCatalog, LiveObject } from './derived-introspect.js';
 import type { ColumnSchema, SequenceSchema, TableSchema } from './schema-introspect.js';
 import { parseIndexDefinition } from './schema-introspect.js';
-import { modelFileName, renderFieldLines } from './model-render.js';
+import { modelImportPath, renderFieldLines } from './model-render.js';
 import { splitFunctionIdentity } from './pg-argtypes.js';
 import { GOVERNED_VOCABULARY, KNOWN_ROLES } from './authz-derive.js';
 
@@ -676,7 +676,7 @@ export function renderDerivedFile(result: DerivedRenderResult, knownTables: Map<
     lines.push(`import { ${result.imports.join(', ')} } from '@everystack/model';`);
   }
   const refs = result.modelRefs
-    .map((table) => ({ varName: knownTables.get(table)!, path: `./${modelFileName(table).replace(/\.ts$/, '')}` }))
+    .map((table) => ({ varName: knownTables.get(table)!, path: modelImportPath(table) }))
     .sort((a, b) => a.path.localeCompare(b.path));
   for (const ref of refs) lines.push(`import { ${ref.varName} } from '${ref.path}';`);
   lines.push('', result.block, '');

package/src/cli/migration-compile.ts

@@ -38,10 +38,21 @@ function emptyTable(table: string): TableContract {
  * schema). Tables, then foreign keys, then authz — the order a fresh database must
  * apply them in.
  */
-export function compileMigration(models: ModelDescriptor[], opts: CompileTableOptions & { sequences?: SequenceDescriptor[] } = {}): string[] {
+export function compileMigration(
+  models: ModelDescriptor[],
+  opts: CompileTableOptions & { sequences?: SequenceDescriptor[]; extensions?: string[] } = {},
+): string[] {
   const sql: string[] = [];
   const schemaOf = (m: ModelDescriptor): string => m.schema ?? 'public';
 
+  // 0. Extensions — the BOOTSTRAP, before any table that might use one of their types.
+  //    compileModuleMigration has always emitted these; compileMigration had no way to,
+  //    so db:check's from-scratch ring compiled a state whose column types did not exist
+  //    and failed with `type "hstore" does not exist` on an otherwise clean schema.
+  //    Deduped + sorted, quoted so a hyphenated name (`uuid-ossp`) stays valid.
+  const extensions = [...new Set(opts.extensions ?? [])].sort();
+  sql.push(...extensions.map((e) => `CREATE EXTENSION IF NOT EXISTS "${e}";`));
+
   // 0a. Non-public schemas — `CREATE SCHEMA` for each distinct one a model lives in,
   //     before any table is created in it. public is the implicit default, never emitted.
   const schemas = [...new Set(models.map(schemaOf))].filter((s) => s !== 'public').sort();
@@ -137,12 +148,14 @@ export function compileMigration(models: ModelDescriptor[], opts: CompileTableOp
 export function compileModuleMigration(modules: Module[], opts: CompileTableOptions = {}): string[] {
   const sql: string[] = [];
 
-  // 1. Extensions — deduped + sorted, quoted so a hyphenated name (`uuid-ossp`) is valid.
-  const extensions = [...new Set(modules.flatMap((m) => m.extensions))].sort();
-  sql.push(...extensions.map((e) => `CREATE EXTENSION IF NOT EXISTS "${e}";`));
-
-  // 2. Schema + authz for every modeled table (plus the modules' standalone sequences).
-  sql.push(...compileMigration(modules.flatMap((m) => m.models), { ...opts, sequences: modules.flatMap((m) => m.sequences) }));
+  // 1+2. Extensions (the bootstrap) then schema + authz for every modeled table, plus the
+  //       modules' standalone sequences. compileMigration owns the extension emission now, so
+  //       the two entry points cannot disagree about the order or the quoting.
+  sql.push(...compileMigration(modules.flatMap((m) => m.models), {
+    ...opts,
+    sequences: modules.flatMap((m) => m.sequences),
+    extensions: modules.flatMap((m) => m.extensions),
+  }));
 
   // 3. Package SQL — functions + triggers, after the tables they reference. Each thunk's
   //    output is a self-contained multi-statement block applied as one unit.

package/src/cli/model-barrel-lint.ts

@@ -0,0 +1,115 @@
+/**
+ * model-barrel-lint — the file that is on disk and outside governance.
+ *
+ * Every verb reads the BARREL: `db:check`, `db:plan` and `db:fingerprint` all call
+ * `loadModels(modelsPath)` and see exactly what it exports. So a model file sitting in
+ * `db/models/` that `index.ts` never imports does not exist as far as any of them is
+ * concerned — it is absent from the fingerprint, absent from the plan, and absent from the
+ * check — while looking entirely declared to a human reading the directory.
+ *
+ * That is the failure the declared-state model exists to prevent: a table quietly outside
+ * governance, with a file that says otherwise. It is also silent by construction, because
+ * the only signal is a COUNT ("30 model(s)") that matches nothing the reader can compare it
+ * to. A consumer lost an hour to it and drew a false stage-drift conclusion from the
+ * resulting one-statement plan.
+ *
+ * The scan is deliberately shallow: the barrel's OWN directory, non-recursive. A models
+ * directory is a flat directory of models by convention, and walking deeper would start
+ * flagging fixtures and generated output.
+ */
+
+import fs from 'node:fs/promises';
+import path from 'node:path';
+import { pathToFileURL } from 'node:url';
+import type { ModelDescriptor } from '@everystack/model';
+
+/** One file's declared tables, as read off its own exports. */
+export interface ModelFileScan {
+  /** Basename, relative to the barrel's directory — what the operator has to go open. */
+  file: string;
+  /** Qualified tables (`schema.table`) the file declares, in any order. */
+  tables: string[];
+}
+
+export interface UnexportedModelGap {
+  file: string;
+  /** The declared tables no verb can see, sorted. */
+  tables: string[];
+  message: string;
+}
+
+/**
+ * Files declaring a table the barrel never exported. Pure — the caller does the reading, so
+ * the rule is testable without a filesystem.
+ *
+ * Findings are per FILE, because the file is the unit that gets fixed (add it to the
+ * barrel, or delete it). Everything is sorted: import order is not a contract, and a lint
+ * whose output reorders between runs is one nobody can diff.
+ */
+export function findUnexportedModels(
+  scans: readonly ModelFileScan[],
+  loadedTables: ReadonlySet<string>,
+): UnexportedModelGap[] {
+  const gaps: UnexportedModelGap[] = [];
+  for (const scan of scans) {
+    const missing = scan.tables.filter((t) => !loadedTables.has(t)).sort();
+    if (missing.length === 0) continue;
+    gaps.push({
+      file: scan.file,
+      tables: missing,
+      message:
+        `${scan.file} declares ${missing.join(', ')} but the barrel does not export it — so no verb can see it. ` +
+        `db:check, db:plan and db:fingerprint all read the barrel, which means the table is outside governance ` +
+        `while the file makes it look declared. Export it from the barrel, or delete the file.`,
+    });
+  }
+  return gaps.sort((a, b) => a.file.localeCompare(b.file));
+}
+
+/** A qualified table identity, matching the form the loaded models are keyed by. */
+export function modelIdentity(m: ModelDescriptor): string {
+  return `${m.schema}.${m.table}`;
+}
+
+/**
+ * Read the barrel's sibling files and report what each one declares.
+ *
+ * Import failures are SKIPPED, not raised: a sibling that does not compile is a different
+ * problem with its own error path, and a lint that turns an unrelated broken file into a
+ * governance finding would send the operator to the wrong place. A file exporting no model
+ * simply scans as zero tables.
+ */
+export async function scanModelDirectory(modelsPath: string): Promise<ModelFileScan[]> {
+  const abs = path.resolve(modelsPath);
+  const dir = path.dirname(abs);
+  const barrel = path.basename(abs);
+
+  let entries: string[];
+  try {
+    entries = await fs.readdir(dir);
+  } catch {
+    return []; // Not a directory-shaped barrel (a single-module app); nothing to scan.
+  }
+
+  const scans: ModelFileScan[] = [];
+  for (const entry of entries.sort()) {
+    if (entry === barrel) continue;
+    if (!entry.endsWith('.ts') || entry.endsWith('.d.ts')) continue;
+    let mod: Record<string, unknown>;
+    try {
+      mod = (await import(pathToFileURL(path.join(dir, entry)).href)) as Record<string, unknown>;
+    } catch {
+      continue;
+    }
+    const tables = Object.values(mod)
+      .filter((v): v is ModelDescriptor => (
+        typeof v === 'object' && v !== null
+        && typeof (v as ModelDescriptor).table === 'string'
+        && typeof (v as ModelDescriptor).schema === 'string'
+        && Array.isArray((v as ModelDescriptor).abilities)
+      ))
+      .map(modelIdentity);
+    scans.push({ file: entry, tables: [...new Set(tables)] });
+  }
+  return scans;
+}

package/src/cli/model-render.ts

@@ -61,7 +61,8 @@ export function checkToValidate(expr: string): { column: string; zod: string } |
 
 export interface RenderOptions {
   /** Only pull tables in this Postgres schema (others are framework-managed). Default: `public`. */
-  schema?: string;
+  /** One schema, or several — a database is not always one schema. Default `'public'`. */
+  schema?: string | string[];
   /**
    * The read-model scaffold. Default `'commented'`: every model carries the authz decision
    * as a commented stanza (same guidance as the db:check gate — the model fails the gate
@@ -241,8 +242,25 @@ const INFRASTRUCTURE_TABLES = new Set([
  * database can carry `public.__drizzle_migrations` — the journal is tooling state,
  * never an app model. Exported so the command shell counts the same set it writes.
  */
-export function pullableTables(snapshot: SchemaSnapshot, schema: string): TableSchema[] {
-  return snapshot.tables.filter((t) => t.table.startsWith(`${schema}.`) && !INFRASTRUCTURE_TABLES.has(bareName(t.table)));
+export function pullableTables(snapshot: SchemaSnapshot, schema: string | string[]): TableSchema[] {
+  // A LIST, because a database is not one schema. `--schema` took a single value and
+  // `--out <dir>` regenerates the barrel, so pulling a second schema overwrote the first —
+  // there was no way to land a multi-schema database in one declared state at all.
+  //
+  // Matched on the exact prefix, never `startsWith(schema)` alone: `pub` must not match
+  // `public.users`.
+  const schemas = new Set(Array.isArray(schema) ? schema : [schema]);
+  return snapshot.tables.filter((t) => {
+    const dot = t.table.indexOf('.');
+    const owner = dot === -1 ? 'public' : t.table.slice(0, dot);
+    return schemas.has(owner) && !INFRASTRUCTURE_TABLES.has(bareName(t.table));
+  });
+}
+
+/** The schema a qualified (or bare) table lives in — bare means public. */
+function schemaOf(table: string): string {
+  const dot = table.indexOf('.');
+  return dot === -1 ? 'public' : table.slice(0, dot);
 }
 
 /**
@@ -258,7 +276,14 @@ export function pullableTables(snapshot: SchemaSnapshot, schema: string): TableS
 export function modelVarName(table: string): string {
   const bare = bareName(table);
   const singular = bare.endsWith('s') ? bare.slice(0, -1) : bare;
-  return singular.split('_').filter(Boolean).map((w) => w.charAt(0).toUpperCase() + w.slice(1)).join('');
+  const pascal = (word: string): string =>
+    word.split('_').filter(Boolean).map((w) => w.charAt(0).toUpperCase() + w.slice(1)).join('');
+  // Non-public schemas are QUALIFIED, so `public.users` and `auth.users` can live in one
+  // barrel. Public stays bare, so an existing single-schema barrel does not churn and
+  // nothing renames when a second schema arrives. Same rule the derived renderer already
+  // uses for its own symbols.
+  const owner = schemaOf(table);
+  return owner === 'public' ? pascal(singular) : pascal(owner) + pascal(singular);
 }
 
 /**
@@ -511,7 +536,15 @@ export function renderModelBlock(table: TableSchema, known: Set<string>, enums:
   const writtenBy = writtenByStanza(table, liveAuthz);
   const softDelete = softDeleteStanza(table, stanza.publicRead);
 
-  return `export const ${modelVarName(table.table)} = defineModel('${bareName(table.table)}', {\n${stanza.text}\n${writtenBy}${softDelete}  fields: {\n${fields}\n  },${constraints}\n});`;
+  // A non-public table carries `schema:` — defineModel stores the name VERBATIM, so the
+  // qualification cannot ride in the first argument (that would make the table literally
+  // named `metrics.impressions`). Without it a multi-schema pull rendered
+  // `defineModel('impressions')`, which declares `public.impressions`, and the matview that
+  // selects FROM metrics.impressions then failed to build. Same idiom the derived renderer
+  // already uses for defineMaterializedTable.
+  const owner = schemaOf(table.table);
+  const schemaProp = owner === 'public' ? '' : `  schema: '${owner}',\n`;
+  return `export const ${modelVarName(table.table)} = defineModel('${bareName(table.table)}', {\n${stanza.text}\n${schemaProp}${writtenBy}${softDelete}  fields: {\n${fields}\n  },${constraints}\n});`;
 }
 
 /** The `import` lines a rendered body needs — only what it actually uses, so the file reads clean. */
@@ -548,6 +581,7 @@ function moduleFooter(
   derived?: DerivedRenderResult,
   multiline = false,
   external?: ExternalDerived,
+  extensions?: string[],
 ): string {
   // A materialized table (--matviews-as-tables) is a MODEL — its block rides the derived
   // render (topo-ordered with the objects around it) but its name belongs in `models`.
@@ -570,6 +604,22 @@ function moduleFooter(
     if (!external) parts.push(`export const derived = [${derived!.names.join(', ')}];`);
     keys.push('derived');
   }
+  // Extensions are the BOOTSTRAP half of a declared state: `field.pgType('hstore')` compiles
+  // to a column whose type does not exist unless something ran CREATE EXTENSION first.
+  // `defineModule` has always accepted the key and migration-compile has always emitted
+  // `CREATE EXTENSION IF NOT EXISTS` from it — the renderer was the only missing link, and a
+  // consumer found it when db:check's from-scratch ring failed with `type "hstore" does not
+  // exist` on a state that was otherwise clean.
+  //
+  // ALL installed extensions, not only those reachable from a declared column type: an
+  // adopter's `pg_stat_statements` and `unaccent` reach no column, and a from-scratch
+  // database without them is not their database.
+  if (extensions?.length) {
+    // Sorted HERE, not just at the introspection: the renderer owns byte-stability, and a
+    // caller passing an unsorted list must not produce a diff against an identical database.
+    parts.push(`export const extensions = [${[...extensions].sort().map((e) => `'${e}'`).join(', ')}];`);
+    keys.push('extensions');
+  }
   parts.push(`export const appModule = defineModule({ ${keys.join(', ')} });`);
   parts.push(`export const modules = [appModule];`);
   return parts.join('\n\n');
@@ -611,7 +661,7 @@ export function renderModelSource(snapshot: SchemaSnapshot, opts: RenderOptions
 
   // Module composition is the norm: the pulled barrel exports `modules` alongside `models`,
   // so a fresh brownfield project lands on the same shape the framework composes.
-  const footer = moduleFooter(tables.map((t) => modelVarName(t.table)), opts.derived);
+  const footer = moduleFooter(tables.map((t) => modelVarName(t.table)), opts.derived, false, undefined, snapshot.extensions);
 
   const header = importHeader([...blocks, footer].join('\n\n'), opts.derived?.imports ?? []);
 
@@ -626,8 +676,18 @@ export function renderModelSource(snapshot: SchemaSnapshot, opts: RenderOptions
  * NAMING CONTRACT — changing this rule is a BREAKING CHANGE: index.ts and cross-file
  * FK imports reference these paths. Pinned by naming-contract.test.ts.
  */
+/** The import specifier a barrel (or a sibling file) uses for a model — the FILENAME rule
+ *  minus the extension. Derived from modelFileName so the two can never disagree; they did,
+ *  and a multi-schema pull emitted `import … from './refresh-tokens'` next to a file called
+ *  `auth-refresh-tokens.ts`. */
+export function modelImportPath(table: string): string {
+  return `./${modelFileName(table).replace(/\.ts$/, '')}`;
+}
+
 export function modelFileName(table: string): string {
-  return `${bareName(table).replace(/_/g, '-')}.ts`;
+  const owner = schemaOf(table);
+  const bare = bareName(table).replace(/_/g, '-');
+  return owner === 'public' ? `${bare}.ts` : `${owner.replace(/_/g, '-')}-${bare}.ts`;
 }
 
 /** The bare tables (other than itself) a table's rendered FKs reference within the pulled set. */
@@ -664,7 +724,7 @@ export function renderModelFiles(snapshot: SchemaSnapshot, opts: RenderOptions =
   const files: RenderedModelFile[] = tables.map((t) => {
     const block = renderModelBlock(t, known, enums, opts.abilities ?? 'commented', opts.liveAuthz);
     const crossImports = referencedTables(t, known).map(
-      (target) => `import { ${modelVarName(target)} } from './${bareName(target).replace(/_/g, '-')}';`,
+      (target) => `import { ${modelVarName(target)} } from '${modelImportPath(target)}';`,
     );
     const header = [importHeader(block), ...crossImports].join('\n');
     return { file: modelFileName(t.table), source: `${header}\n\n${block}\n` };
@@ -683,13 +743,13 @@ export function renderModelFiles(snapshot: SchemaSnapshot, opts: RenderOptions =
       // The --derived-out layer, imported rather than re-declared — the barrel must WIRE it
       // or db:plan silently compares against models only.
       ...(extNames.length ? [`import { ${extNames.join(', ')} } from '${ext!.specifier}';`] : []),
-      ...names.map((n, i) => `import { ${n} } from './${bareName(tables[i].table).replace(/_/g, '-')}';`),
+      ...names.map((n, i) => `import { ${n} } from '${modelImportPath(tables[i].table)}';`),
     ].join('\n'),
     `export {\n${names.map((n) => `  ${n},`).join('\n')}\n};`,
     // Re-exported so the barrel remains the one place that describes the database.
     ...(extNames.length ? [`export { ${extNames.join(', ')} };`] : []),
     ...(opts.derived?.block ? [opts.derived.block] : []),
-    moduleFooter(names, opts.derived, true, ext),
+    moduleFooter(names, opts.derived, true, ext, snapshot.extensions),
   ].join('\n\n');
   files.push({ file: 'index.ts', source: index + '\n' });
 

package/src/cli/schema-introspect.ts

@@ -105,6 +105,8 @@ export interface SchemaSnapshot {
   enums?: EnumType[];
   /** Standalone sequences (serial-owned ones excluded). Absent/empty when none. */
   sequences?: SequenceSchema[];
+  /** Installed extensions, `plpgsql` excluded. Absent when the read predates this. */
+  extensions?: string[];
 }
 
 // ---------------------------------------------------------------------------
@@ -216,6 +218,27 @@ ORDER BY n.nspname, t.typname;
  * has an 'a'/'i' pg_depend link to its column and stays represented BY that column).
  * These hold state, so they are base-schema: migrated, diffed, fingerprinted.
  */
+/**
+ * Installed extensions — the bootstrap a declared state needs before anything else.
+ *
+ * Database-wide, not per-schema: `pg_extension` has no schema filter worth applying, and an
+ * extension installed into `public` is just as required by a model in `auth`. So this read
+ * ignores `--schema` deliberately.
+ *
+ * `plpgsql` is excluded because PostgreSQL installs it into every database by default —
+ * emitting `CREATE EXTENSION IF NOT EXISTS plpgsql` is noise, not bootstrap.
+ *
+ * Without this, a declared state that renders `field.pgType('hstore')` compiles to a CREATE
+ * TABLE whose type does not exist, and `db:check`'s from-scratch ring fails with
+ * `type "hstore" does not exist` — the state describes a database it cannot build.
+ */
+export const EXTENSIONS_SQL = `
+SELECT extname AS name
+FROM pg_extension
+WHERE extname <> 'plpgsql'
+ORDER BY extname;
+`.trim();
+
 export const SEQUENCES_SQL = `
 SELECT
   n.nspname AS schema,
@@ -540,6 +563,7 @@ export interface SchemaRows {
   enums?: EnumRow[];
   indexes?: IndexRow[];
   sequences?: SequenceRow[];
+  extensions?: ExtensionRow[];
 }
 
 /**
@@ -618,10 +642,18 @@ export function assembleSchema(rows: SchemaRows): SchemaSnapshot {
     .map(sequenceRowToDescriptor)
     .sort((a, b) => a.name.localeCompare(b.name));
 
+  // ABSENT vs EMPTY matters: a caller that did not run the extensions query leaves the key
+  // off entirely, and the renderer must not read that as "this database has none" and emit
+  // an empty list that later looks authoritative.
+  const extensions = rows.extensions === undefined
+    ? undefined
+    : rows.extensions.map((r) => r.name).sort();
+
   return {
     tables: [...tables.values()].sort((a, b) => a.table.localeCompare(b.table)),
     enums,
     ...(sequences.length ? { sequences } : {}),
+    ...(extensions !== undefined ? { extensions } : {}),
   };
 }
 
@@ -631,13 +663,15 @@ export function assembleSchema(rows: SchemaRows): SchemaSnapshot {
  * (the ops Lambda `db:query` in production, a fake in tests) and folds the rows with
  * `assembleSchema`. This is the `current` side `db:generate` diffs the compiled Models against.
  */
+export interface ExtensionRow { name: string }
+
 export async function introspectSchema(session: SessionRunner): Promise<SchemaSnapshot> {
   // ONE session, so all five queries see one database at one moment, under one pinned
   // search_path. Column defaults, CHECK constraints and index expressions all deparse
   // relative to that path — spread across separate connections these five rows can render
   // the same schema two ways, and a fingerprint minted from the mix describes nothing.
-  const [columns, constraints, enums, indexes, sequences] = await session(
-    [COLUMNS_SQL, CONSTRAINTS_SQL, ENUMS_SQL, INDEXES_SQL, SEQUENCES_SQL],
+  const [columns, constraints, enums, indexes, sequences, extensions] = await session(
+    [COLUMNS_SQL, CONSTRAINTS_SQL, ENUMS_SQL, INDEXES_SQL, SEQUENCES_SQL, EXTENSIONS_SQL],
     INTROSPECTION_SESSION,
   );
   return assembleSchema({
@@ -646,5 +680,6 @@ export async function introspectSchema(session: SessionRunner): Promise<SchemaSn
     enums: enums as EnumRow[],
     indexes: indexes as IndexRow[],
     sequences: sequences as SequenceRow[],
+    extensions: extensions as ExtensionRow[],
   });
 }