@rtrentjones/greenlight 0.2.25 → 0.2.26

package/assets/skills/provider-neon/SKILL.md

@@ -34,11 +34,27 @@ prod and beta hit **different branches**. Pin the provider `kislerdm/neon ~> 0.1
 Do **not** add a Neon tool to `module.keepalive.targets_json`. Neon resumes on connect — a request
 just wakes it. (`doctor` does not flag `data: neon` for keepalive; that exemption is intentional.)
 
-## Migrations
+## Schema as code / migrations
 
-Run migrations against the env's **branch** (`DIRECT_URL`). A PR's ephemeral branch is the safe place
-to test a migration before it touches prod. Gate them with `greenlight migrations scan` (the
-dangerous-SQL pre-apply check) in the tool's CI.
+**Greenlight does NOT run migrations — by design.** The split:
+- **Schema** lives in the tool (an ORM — Drizzle/Prisma — or plain `.sql` migrations).
+- **Branch-per-env**: the TF module owns stable `prod`/`beta`; the **native Neon↔Vercel integration**
+  owns ephemeral per-PR preview branches (+ auto-injects `DATABASE_URL`). Don't put ephemeral branches
+  in Terraform.
+- **Execution**: the app's own build runs its migrate (`drizzle-kit migrate` / `prisma migrate deploy`)
+  against the wired **`DIRECT_URL`** — prod build → prod branch, preview build → preview branch. A
+  failed migrate fails the build = a natural gate.
+- **Greenlight's role**: the **dangerous-SQL gate**. Run `greenlight migrations scan` (no `<dir>` →
+  it auto-detects `supabase/migrations | migrations | drizzle/migrations | …`) in CI before the migrate.
+
+See [docs/migrations.md](../../../docs/migrations.md).
+
+## Sharing one DB + multi-account
+
+- **One DB, many services**: a second tool sets `dataShareWith: '<owner>'` (or `add … --share <owner>`)
+  — it creates no project and wires the owner's connection strings.
+- **A second Neon account**: `tokenOverrides: { NEON_API_KEY: 'NEON_API_KEY_X' }` → an aliased `neon`
+  provider authenticates that account. (A sharer can't also override — it uses the owner's account.)
 
 ## MCP
 

package/dist/bin.js

@@ -6,7 +6,7 @@ import {
   resolveUrl,
   scanSqlFiles,
   verifyAll
-} from "./chunk-GPPUZ6Z5.js";
+} from "./chunk-HMU7D7R2.js";
 import "./chunk-HX7VA25D.js";
 import "./chunk-N3IKUCSF.js";
 import "./chunk-KP3Y6WRU.js";
@@ -64,6 +64,7 @@ function serializeTool(t) {
     const ov = Object.entries(t.tokenOverrides).map(([k, v]) => `${k}: ${q(v)}`).join(", ");
     parts.push(`tokenOverrides: { ${ov} }`);
   }
+  if (t.dataShareWith !== void 0) parts.push(`dataShareWith: ${q(t.dataShareWith)}`);
   return `    { ${parts.join(", ")} },`;
 }
 function serializeConfig(c) {
@@ -111,7 +112,8 @@ function addTool(config, t) {
         ...t.port !== void 0 ? { port: t.port } : {},
         ...t.preview ? { preview: t.preview } : {},
         ...t.tokens?.length ? { tokens: t.tokens } : {},
-        ...t.tokenOverrides && Object.keys(t.tokenOverrides).length ? { tokenOverrides: t.tokenOverrides } : {}
+        ...t.tokenOverrides && Object.keys(t.tokenOverrides).length ? { tokenOverrides: t.tokenOverrides } : {},
+        ...t.dataShareWith ? { dataShareWith: t.dataShareWith } : {}
       }
     ]
   };
@@ -139,7 +141,8 @@ function upsertTool(config, t) {
     ...t.port !== void 0 ? { port: t.port } : {},
     ...t.preview ? { preview: t.preview } : {},
     ...t.tokens?.length ? { tokens: t.tokens } : {},
-    ...t.tokenOverrides && Object.keys(t.tokenOverrides).length ? { tokenOverrides: t.tokenOverrides } : {}
+    ...t.tokenOverrides && Object.keys(t.tokenOverrides).length ? { tokenOverrides: t.tokenOverrides } : {},
+    ...t.dataShareWith ? { dataShareWith: t.dataShareWith } : {}
   };
   const tools = config.tools.some((x) => x.name === t.name) ? config.tools.map((x) => x.name === t.name ? entry : x) : [...config.tools, entry];
   const result = ConfigSchema.safeParse({ ...config, tools });
@@ -475,7 +478,7 @@ function tokensForTool(tool) {
 }
 
 // src/version.ts
-var MODULE_REF = "v0.2.25";
+var MODULE_REF = "v0.2.26";
 var MODULE_SOURCE_BASE = "git::https://github.com/RTrentJones/greenlight.git//infra/modules";
 function moduleSource(module, ref = MODULE_REF) {
   return `${MODULE_SOURCE_BASE}/${module}?ref=${ref}`;
@@ -492,6 +495,8 @@ function emitToolTf(opts) {
   const useVercel = target === "vercel";
   const useOci = target === "oci";
   const supabaseOverride = opts.tokenOverrides?.SUPABASE_ACCESS_TOKEN;
+  const neonOverride = opts.tokenOverrides?.NEON_API_KEY;
+  const neonOwner = opts.dataShareWith ?? name;
   const envList = envs.map((e) => `"${e}"`).join(", ");
   const blocks = [];
   const assumes = ["var.cloudflare_zone_id"];
@@ -542,18 +547,37 @@ variable "${name}_supabase_database_password" {
   default   = "import-placeholder" # ignored when importing an existing project
 }${overrideBlock}`);
   }
-  if (useNeon) {
+  if (useNeon && opts.dataShareWith) {
+    blocks.push(`# Shares the Neon project owned by "${opts.dataShareWith}" (one DB, many services).
+# No neon module here \u2014 the env wiring below reads module.${neonOwner}_neon.* (its prod/beta branches).`);
+  } else if (useNeon) {
+    const providersLine = neonOverride ? `
+  providers = { neon = neon.${name} }` : "";
+    const overrideBlock = neonOverride ? `
+
+# Multi-account: ${name}'s Neon lives in a SECOND account \u2014 an aliased provider authenticates with
+# its own token. In infra.yml: TF_VAR_${name}_neon_api_key: \${{ secrets.${neonOverride} }}
+provider "neon" {
+  alias   = "${name}"
+  api_key = var.${name}_neon_api_key
+}
+
+variable "${name}_neon_api_key" {
+  type        = string
+  sensitive   = true
+  description = "Neon API key for ${name}'s account (scoped secret ${neonOverride})."
+}` : "";
     blocks.push(`# One Neon project, a branch per env (prod = the project's default branch; beta = a child
 # branch \u2014 copy-on-write, instant). Compute scales to zero and auto-resumes on the next connection,
 # so a Neon tool needs NO keepalive (the reason Neon is the default Postgres). NEON_API_KEY configures
 # the provider in main.tf; the connection strings are module OUTPUTS \u2014 no per-tool secret to gather.
 module "${name}_neon" {
-  source = "${moduleSource("neon", ref)}"
+  source = "${moduleSource("neon", ref)}"${providersLine}
 
   name   = "${name}"
   region = "aws-us-east-1" # Neon region id, e.g. aws-us-east-1 / aws-us-west-2
   envs   = [${envList}]
-}`);
+}${overrideBlock}`);
   }
   if (useVercel) {
     const env = useSupabase ? `
@@ -590,10 +614,10 @@ module "${name}_neon" {
   environment_values = {
     site_url_prod  = "https://${name}.${domain}"
     site_url_beta  = "https://beta.${name}.${domain}"
-    db_url_prod    = module.${name}_neon.database_url["prod"]
-    db_direct_prod = module.${name}_neon.direct_url["prod"]
-    db_url_beta    = module.${name}_neon.database_url["beta"]
-    db_direct_beta = module.${name}_neon.direct_url["beta"]
+    db_url_prod    = module.${neonOwner}_neon.database_url["prod"]
+    db_direct_prod = module.${neonOwner}_neon.direct_url["prod"]
+    db_url_beta    = module.${neonOwner}_neon.database_url["beta"]
+    db_direct_beta = module.${neonOwner}_neon.direct_url["beta"]
   }` : `
   # No managed data store \u2014 add environment/environment_values if the app needs vars.
   environment        = {}
@@ -1127,7 +1151,7 @@ async function addCommand(args) {
   const name = args[0];
   if (!name || name.startsWith("-")) {
     throw new Error(
-      "usage: greenlight add <name> --lane <lane> --target <target> [--data <d>] [--auth <a>] [--envs beta,prod] [--port 8000]"
+      "usage: greenlight add <name> --lane <lane> --target <target> [--data <d>] [--auth <a>] [--envs beta,prod] [--port 8000] [--share <owner>]"
     );
   }
   const lane = flag2(args, "--lane");
@@ -1145,7 +1169,9 @@ async function addCommand(args) {
     data: flag2(args, "--data"),
     auth: flag2(args, "--auth"),
     envs: flag2(args, "--envs")?.split(","),
-    port: portFlag ? Number(portFlag) : void 0
+    port: portFlag ? Number(portFlag) : void 0,
+    // --share <owner>: this tool reads the owner's Neon DB instead of creating its own (one DB, many services).
+    dataShareWith: flag2(args, "--share")
   });
   const entry = next.tools.find((t) => t.name === name);
   const data = entry?.data ?? "none";
@@ -1194,7 +1220,8 @@ async function addCommand(args) {
         data,
         envs,
         port: entry?.port,
-        tokenOverrides: entry?.tokenOverrides
+        tokenOverrides: entry?.tokenOverrides,
+        dataShareWith: entry?.dataShareWith
       })
     );
     console.log(`\u2714 wrote infra/${name}.tf (modules: ${providers.join(", ")})`);
@@ -2527,19 +2554,30 @@ Next:
 }
 
 // src/commands/migrations.ts
-import { readFileSync as readFileSync7, readdirSync as readdirSync3 } from "fs";
+import { existsSync as existsSync10, readFileSync as readFileSync7, readdirSync as readdirSync3 } from "fs";
 import { join as join5 } from "path";
 var DEFAULT_DIR = "supabase/migrations";
+var CANDIDATE_DIRS = [
+  DEFAULT_DIR,
+  "migrations",
+  "drizzle/migrations",
+  "drizzle",
+  "db/migrations"
+];
+function resolveMigrationsDir(explicit, root = process.cwd()) {
+  if (explicit) return explicit;
+  return CANDIDATE_DIRS.find((d) => existsSync10(join5(root, d))) ?? DEFAULT_DIR;
+}
 async function migrationsCommand(args) {
   if (args[0] !== "scan") {
     console.log(
       `usage: greenlight migrations scan [<dir>] [--strict]
   scan SQL migrations for data-destroying / lock-heavy statements (the pre-apply gate).
-  default dir: ${DEFAULT_DIR}. Acknowledge an intentional op with \`-- greenlight:allow\`.`
+  no <dir> \u2192 auto-detects ${CANDIDATE_DIRS.join(" | ")}. Acknowledge an intentional op with \`-- greenlight:allow\`.`
     );
     process.exit(args[0] ? 1 : 0);
   }
-  const dir = args.slice(1).find((a) => !a.startsWith("-")) ?? DEFAULT_DIR;
+  const dir = resolveMigrationsDir(args.slice(1).find((a) => !a.startsWith("-")));
   const strict = args.includes("--strict");
   let names;
   try {

package/dist/{chunk-GPPUZ6Z5.js → chunk-HMU7D7R2.js}

@@ -65,7 +65,12 @@ var ToolSchema = z.object({
   // var to an alternate secret name, so this tool authenticates that provider with a SECOND account
   // — e.g. { SUPABASE_ACCESS_TOKEN: 'SUPABASE_ACCESS_TOKEN_HEISTMIND' }. Absent ⇒ unchanged (the
   // default token). `add`/`adopt` emit an aliased provider + scoped var/secret for an overridden token.
-  tokenOverrides: z.record(z.string(), z.string()).optional()
+  tokenOverrides: z.record(z.string(), z.string()).optional(),
+  // Share another tool's data store instead of creating one (multiple services on one Neon DB).
+  // The value is the OWNER tool's name; this tool emits no data module and wires the owner's
+  // connection strings. Cross-tool validity (owner exists, same data, no chains) is checked on
+  // the whole config below.
+  dataShareWith: z.string().optional()
 }).superRefine((tool, ctx) => {
   const rule = MATRIX[tool.lane];
   if (!rule.targets.includes(tool.target)) {
@@ -89,6 +94,13 @@ var ToolSchema = z.object({
       message: 'private tools must set auth to "bearer" or "oauth", never "none"'
     });
   }
+  if (tool.dataShareWith && tool.data !== "neon") {
+    ctx.addIssue({
+      code: z.ZodIssueCode.custom,
+      path: ["dataShareWith"],
+      message: 'dataShareWith currently supports data: "neon" only'
+    });
+  }
 });
 var BlogSchema = z.object({
   lane: z.literal("astro"),
@@ -104,6 +116,35 @@ var ConfigSchema = z.object({
   // Optional: a tool-only repo (a poly-repo consumer) has no blog.
   blog: BlogSchema.optional(),
   tools: z.array(ToolSchema).default([])
+}).superRefine((config, ctx) => {
+  for (const [i, tool] of config.tools.entries()) {
+    if (!tool.dataShareWith) continue;
+    const issue = (message) => ctx.addIssue({ code: z.ZodIssueCode.custom, path: ["tools", i, "dataShareWith"], message });
+    if (tool.dataShareWith === tool.name) {
+      issue(`"${tool.name}" cannot share a data store with itself`);
+      continue;
+    }
+    const owner = config.tools.find((t) => t.name === tool.dataShareWith);
+    if (!owner) {
+      issue(`dataShareWith "${tool.dataShareWith}" is not a tool in this manifest`);
+      continue;
+    }
+    if (owner.data !== tool.data) {
+      issue(
+        `"${tool.name}" (data: ${tool.data}) must share a tool with the same data \u2014 "${owner.name}" is ${owner.data}`
+      );
+    }
+    if (owner.dataShareWith) {
+      issue(
+        `cannot share with "${owner.name}" \u2014 it is itself a sharer (no chains); point at the owner`
+      );
+    }
+    if (tool.tokenOverrides?.NEON_API_KEY) {
+      issue(
+        `a sharer uses the owner's Neon account \u2014 remove the NEON_API_KEY override from "${tool.name}"`
+      );
+    }
+  }
 });
 
 // ../packages/shared/src/load.ts

package/dist/index.js

@@ -2,7 +2,7 @@ import {
   defineConfig,
   defineVerify,
   loadConfig
-} from "./chunk-GPPUZ6Z5.js";
+} from "./chunk-HMU7D7R2.js";
 import "./chunk-HX7VA25D.js";
 import "./chunk-N3IKUCSF.js";
 import "./chunk-KP3Y6WRU.js";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rtrentjones/greenlight",
-  "version": "0.2.25",
+  "version": "0.2.26",
   "description": "Greenlight CLI — setup and lifecycle for the harness.",
   "license": "MIT",
   "repository": {
@@ -31,10 +31,10 @@
     "@anthropic-ai/sdk": "^0.69.0"
   },
   "devDependencies": {
-    "@rtrentjones/greenlight-adapters": "0.2.25",
-    "@rtrentjones/greenlight-shared": "0.2.25",
-    "@rtrentjones/greenlight-verify": "0.2.25",
-    "@rtrentjones/greenlight-loop": "0.2.25"
+    "@rtrentjones/greenlight-adapters": "0.2.26",
+    "@rtrentjones/greenlight-loop": "0.2.26",
+    "@rtrentjones/greenlight-shared": "0.2.26",
+    "@rtrentjones/greenlight-verify": "0.2.26"
   },
   "scripts": {
     "build": "node scripts/copy-assets.mjs && tsup",