@cosmicdrift/kumiko-framework 0.389.0 → 0.390.0

package/src/scripts/codemod/migrate-db-raw.ts

@@ -1,401 +0,0 @@
-#!/usr/bin/env bun
-// migrate-db-raw — rewrites standalone bun-db helper calls that manually pass
-// ctx.db.raw to their TenantDb method-form equivalent (db.global(table)... /
-// db.selectMany|fetchOne|insertOne(...)) wherever the rewrite is provably
-// safe, and reports every remaining ctx.db.raw-ish access for manual review.
-//
-// Usage:
-//   bun scripts/codemod/migrate-db-raw.ts [--dry-run] [--global-tables a,b] <path...>
-
-import { readFileSync, writeFileSync } from "node:fs";
-import { Node, Project, type SourceFile, SyntaxKind, VariableDeclarationKind } from "ts-morph";
-import { collectSourceFiles } from "./collect-source-files.ts";
-
-const DEFAULT_GLOBAL_TABLES: ReadonlySet<string> = new Set([
-  "userTable",
-  "globalFeatureStateTable",
-]);
-const PRUNABLE_FN_NAMES: ReadonlySet<string> = new Set(["selectMany", "fetchOne", "insertOne"]);
-const OWN_TENANT_FN_NAMES: ReadonlySet<string> = new Set(["selectMany", "fetchOne", "insertOne"]);
-const GLOBAL_TABLE_FN_NAMES: ReadonlySet<string> = new Set(["selectMany", "fetchOne"]);
-const TENANT_ID_SOURCES: ReadonlySet<string> = new Set([
-  "event.user.tenantId",
-  "query.user.tenantId",
-  "ctx.user.tenantId",
-]);
-const RAW_RECEIVER_IDENTIFIERS: ReadonlySet<string> = new Set([
-  "db",
-  "tdb",
-  "scopedDb",
-  "outsideTx",
-]);
-const MANUAL_TEXT_LIMIT = 100;
-
-export type RewriteRule = "global-table" | "own-tenant";
-
-export interface Rewrite {
-  readonly file: string;
-  readonly line: number;
-  readonly rule: RewriteRule;
-  readonly before: string;
-  readonly after: string;
-}
-
-export interface ManualSite {
-  readonly file: string;
-  readonly line: number;
-  readonly enclosing: string;
-  readonly text: string;
-}
-
-export interface MigrateDbRawResult {
-  readonly output: string;
-  readonly rewrites: Rewrite[];
-  readonly manual: ManualSite[];
-}
-
-function isRawOnCtxDb(argExpr: Node): boolean {
-  return argExpr.getText() === "ctx.db.raw";
-}
-
-function enclosingFunctionScope(node: Node): Node {
-  return (
-    node.getFirstAncestor(
-      (n) =>
-        Node.isFunctionDeclaration(n) ||
-        Node.isArrowFunction(n) ||
-        Node.isFunctionExpression(n) ||
-        Node.isMethodDeclaration(n),
-    ) ?? node.getSourceFile()
-  );
-}
-
-function isConstTenantIdBinding(scope: Node, varName: string): boolean {
-  return scope.getDescendantsOfKind(SyntaxKind.VariableDeclaration).some((decl) => {
-    if (decl.getName() !== varName) return false;
-    const statement = decl.getVariableStatement();
-    if (!statement || statement.getDeclarationKind() !== VariableDeclarationKind.Const)
-      return false;
-    const initializer = decl.getInitializer();
-    return initializer !== undefined && TENANT_ID_SOURCES.has(initializer.getText());
-  });
-}
-
-function isValidOwnTenantIdValue(valueNode: Node): boolean {
-  if (TENANT_ID_SOURCES.has(valueNode.getText())) return true;
-  if (!Node.isIdentifier(valueNode)) return false;
-  return isConstTenantIdBinding(enclosingFunctionScope(valueNode), valueNode.getText());
-}
-
-function objectHasValidOwnTenantId(obj: Node): boolean {
-  if (!Node.isObjectLiteralExpression(obj)) return false;
-  const props = obj.getProperties();
-  const tenantIdIndex = props.findIndex(
-    (prop) =>
-      (Node.isPropertyAssignment(prop) || Node.isShorthandPropertyAssignment(prop)) &&
-      prop.getName() === "tenantId",
-  );
-  if (tenantIdIndex === -1) return false;
-  // A later spread can override the literal tenantId at runtime.
-  if (props.slice(tenantIdIndex + 1).some((prop) => Node.isSpreadAssignment(prop))) return false;
-  const prop = props[tenantIdIndex];
-  if (Node.isPropertyAssignment(prop)) {
-    const init = prop.getInitializer();
-    return init !== undefined && isValidOwnTenantIdValue(init);
-  }
-  if (Node.isShorthandPropertyAssignment(prop)) {
-    return isConstTenantIdBinding(enclosingFunctionScope(prop), "tenantId");
-  }
-  return false;
-}
-
-interface PendingRewrite {
-  readonly callExpr: Node;
-  readonly rule: RewriteRule;
-  readonly before: string;
-  readonly after: string;
-  readonly line: number;
-}
-
-// The caller checks `args.length` immediately before every use — the index
-// is always in bounds; this replaces a non-null assertion with an explicit
-// invariant instead of silencing the compiler.
-function nthArg(args: Node[], index: number): Node {
-  const arg = args[index];
-  if (!arg) throw new Error(`planRewrite: expected an argument at index ${index}`);
-  return arg;
-}
-
-function planRewrite(
-  callExpr: Node,
-  globalTables: ReadonlySet<string>,
-): PendingRewrite | undefined {
-  if (!Node.isCallExpression(callExpr)) return undefined;
-  const calleeExpr = callExpr.getExpression();
-  if (!Node.isIdentifier(calleeExpr)) return undefined;
-  const fnName = calleeExpr.getText();
-  const args = callExpr.getArguments();
-  if (args.length === 0 || !isRawOnCtxDb(nthArg(args, 0))) return undefined;
-
-  const typeArgs = callExpr.getTypeArguments();
-  const typeArgsText =
-    typeArgs.length > 0 ? `<${typeArgs.map((t) => t.getText()).join(", ")}>` : "";
-
-  if (GLOBAL_TABLE_FN_NAMES.has(fnName) && args.length >= 2) {
-    const tableArg = nthArg(args, 1);
-    if (Node.isIdentifier(tableArg) && globalTables.has(tableArg.getText())) {
-      const rest = args.slice(2).map((a) => a.getText());
-      const after = `ctx.db.global(${tableArg.getText()}).${fnName}${typeArgsText}(${rest.join(", ")})`;
-      return {
-        callExpr,
-        rule: "global-table",
-        before: callExpr.getText(),
-        after,
-        line: callExpr.getStartLineNumber(),
-      };
-    }
-  }
-
-  if (OWN_TENANT_FN_NAMES.has(fnName) && args.length >= 3) {
-    const objArg = nthArg(args, 2);
-    if (objectHasValidOwnTenantId(objArg)) {
-      const rest = [nthArg(args, 1), objArg, ...args.slice(3)].map((a) => a.getText());
-      const after = `ctx.db.${fnName}${typeArgsText}(${rest.join(", ")})`;
-      return {
-        callExpr,
-        rule: "own-tenant",
-        before: callExpr.getText(),
-        after,
-        line: callExpr.getStartLineNumber(),
-      };
-    }
-  }
-
-  return undefined;
-}
-
-function isTenantDbRawAccess(pae: Node): boolean {
-  if (!Node.isPropertyAccessExpression(pae)) return false;
-  if (pae.getName() !== "raw") return false;
-  const receiver = pae.getExpression();
-  if (receiver.getText().endsWith(".db")) return true;
-  if (Node.isIdentifier(receiver) && RAW_RECEIVER_IDENTIFIERS.has(receiver.getText())) return true;
-  if (
-    Node.isCallExpression(receiver) &&
-    receiver.getExpression().getText().startsWith("ctx.systemDb.")
-  )
-    return true;
-  return false;
-}
-
-function enclosingName(node: Node): string {
-  let current: Node | undefined = node.getParent();
-  while (current) {
-    if (Node.isFunctionDeclaration(current)) {
-      const name = current.getName();
-      if (name) return name;
-    }
-    if (Node.isMethodDeclaration(current)) return current.getName();
-    if (Node.isPropertyAssignment(current)) return current.getName();
-    if (Node.isVariableDeclaration(current)) return current.getName();
-    current = current.getParent();
-  }
-  return "<module>";
-}
-
-function truncate(text: string, limit: number): string {
-  const collapsed = text.replace(/\s+/g, " ").trim();
-  return collapsed.length > limit ? `${collapsed.slice(0, limit)}…` : collapsed;
-}
-
-function directArgumentCall(pae: Node): Node | undefined {
-  const parent = pae.getParent();
-  if (parent && Node.isCallExpression(parent) && parent.getArguments().some((a) => a === pae)) {
-    return parent;
-  }
-  return undefined;
-}
-
-function collectManualSite(pae: Node, fileName: string): ManualSite {
-  const reportNode = directArgumentCall(pae) ?? pae;
-  return {
-    file: fileName,
-    line: reportNode.getStartLineNumber(),
-    enclosing: enclosingName(pae),
-    text: truncate(reportNode.getText(), MANUAL_TEXT_LIMIT),
-  };
-}
-
-function isPropertyAccessName(id: Node): boolean {
-  const parent = id.getParent();
-  return Node.isPropertyAccessExpression(parent) && parent.getNameNode() === id;
-}
-
-function pruneUnusedDbImports(sourceFile: SourceFile): void {
-  for (const importDecl of sourceFile.getImportDeclarations()) {
-    for (const spec of [...importDecl.getNamedImports()]) {
-      if (!PRUNABLE_FN_NAMES.has(spec.getName())) continue;
-      const localName = spec.getAliasNode()?.getText() ?? spec.getName();
-      const nameNode = spec.getAliasNode() ?? spec.getNameNode();
-      const stillUsed = sourceFile
-        .getDescendantsOfKind(SyntaxKind.Identifier)
-        .some((id) => id !== nameNode && id.getText() === localName && !isPropertyAccessName(id));
-      if (!stillUsed) spec.remove();
-    }
-    if (
-      importDecl.getNamedImports().length === 0 &&
-      !importDecl.getDefaultImport() &&
-      !importDecl.getNamespaceImport()
-    ) {
-      importDecl.remove();
-    }
-  }
-}
-
-export function migrateDbRawSource(
-  source: string,
-  fileName: string,
-  opts: { globalTables: ReadonlySet<string> },
-): MigrateDbRawResult {
-  const project = new Project({ useInMemoryFileSystem: true, skipFileDependencyResolution: true });
-  const sourceFile = project.createSourceFile(fileName, source);
-
-  const pending: PendingRewrite[] = [];
-  for (const callExpr of sourceFile.getDescendantsOfKind(SyntaxKind.CallExpression)) {
-    const plan = planRewrite(callExpr, opts.globalTables);
-    if (plan) pending.push(plan);
-  }
-
-  const rewrites: Rewrite[] = [];
-  for (const plan of pending) {
-    if (plan.callExpr.wasForgotten()) continue;
-    plan.callExpr.replaceWithText(plan.after);
-    rewrites.push({
-      file: fileName,
-      line: plan.line,
-      rule: plan.rule,
-      before: plan.before,
-      after: plan.after,
-    });
-  }
-
-  pruneUnusedDbImports(sourceFile);
-
-  const manual: ManualSite[] = [];
-  for (const pae of sourceFile.getDescendantsOfKind(SyntaxKind.PropertyAccessExpression)) {
-    if (isTenantDbRawAccess(pae)) manual.push(collectManualSite(pae, fileName));
-  }
-
-  return { output: sourceFile.getFullText(), rewrites, manual };
-}
-
-class CliError extends Error {}
-
-interface CliOptions {
-  readonly dryRun: boolean;
-  readonly globalTables: ReadonlySet<string>;
-  readonly paths: string[];
-}
-
-function parseArgs(argv: string[]): CliOptions {
-  let dryRun = false;
-  let globalTables: ReadonlySet<string> = DEFAULT_GLOBAL_TABLES;
-  const paths: string[] = [];
-
-  for (let i = 0; i < argv.length; i++) {
-    const arg = argv[i];
-    if (!arg) break;
-    if (arg === "--dry-run") {
-      dryRun = true;
-      continue;
-    }
-    if (arg === "--global-tables") {
-      const value = argv[++i];
-      if (!value) throw new CliError("--global-tables requires a comma-separated value");
-      globalTables = new Set(
-        value
-          .split(",")
-          .map((s) => s.trim())
-          .filter((s) => s.length > 0),
-      );
-      continue;
-    }
-    if (arg.startsWith("--")) throw new CliError(`unknown flag: ${arg}`);
-    paths.push(arg);
-  }
-
-  if (paths.length === 0) throw new CliError("at least one path is required");
-  return { dryRun, globalTables, paths };
-}
-
-function isMigratableFile(name: string): boolean {
-  if (!/\.(ts|tsx)$/.test(name)) return false;
-  if (name.endsWith(".test.ts") || name.endsWith(".test.tsx")) return false;
-  return true;
-}
-
-function collectFiles(paths: string[]): string[] {
-  return collectSourceFiles(paths, {
-    isMigratableFile,
-    extraSkippedDirNames: ["__tests__"],
-  });
-}
-
-async function main(): Promise<void> {
-  let options: CliOptions;
-  try {
-    options = parseArgs(process.argv.slice(2));
-  } catch (err) {
-    if (err instanceof CliError) {
-      console.error(`[migrate-db-raw] ${err.message}`);
-      process.exit(2);
-    }
-    throw err;
-  }
-
-  const files = collectFiles(options.paths);
-  const allRewrites: Rewrite[] = [];
-  const allManual: ManualSite[] = [];
-  let filesChanged = 0;
-
-  for (const file of files) {
-    const before = readFileSync(file, "utf8");
-    const result = migrateDbRawSource(before, file, { globalTables: options.globalTables });
-    allRewrites.push(...result.rewrites);
-    allManual.push(...result.manual);
-
-    if (result.rewrites.length === 0) continue;
-    filesChanged += 1;
-
-    if (options.dryRun) {
-      console.log(`\n=== ${file} (${result.rewrites.length} rewrite(s)) ===`);
-      for (const r of result.rewrites) {
-        console.log(`${file}:${r.line} [${r.rule}]`);
-        console.log(`- ${r.before}`);
-        console.log(`+ ${r.after}`);
-      }
-    } else {
-      writeFileSync(file, result.output);
-    }
-  }
-
-  const globalCount = allRewrites.filter((r) => r.rule === "global-table").length;
-  const ownTenantCount = allRewrites.filter((r) => r.rule === "own-tenant").length;
-  console.log(
-    `\n[migrate-db-raw] ${options.dryRun ? "would change" : "changed"} ${filesChanged} file(s); ` +
-      `rewrites: global-table=${globalCount}, own-tenant=${ownTenantCount}`,
-  );
-
-  if (allManual.length > 0) {
-    console.log(`\n[migrate-db-raw] manual review needed (${allManual.length}):`);
-    for (const m of allManual) {
-      console.log(`${m.file}:${m.line}  ${m.enclosing}  ${m.text}`);
-    }
-  }
-
-  process.exit(0);
-}
-
-if (import.meta.main) {
-  await main();
-}

package/src/scripts/codemod/migrate-open-to-all.ts

@@ -1,235 +0,0 @@
-#!/usr/bin/env bun
-// migrate-open-to-all — rewrites deprecated `openToAll: true` to the required
-// `{ reason }` form in test files, and reports every remaining non-test site
-// for a human to pick a reason.
-//
-// Usage:
-//   bun scripts/codemod/migrate-open-to-all.ts [--dry-run] [--test-reason "<text>"] <path...>
-
-import { readFileSync, writeFileSync } from "node:fs";
-import {
-  Node,
-  type ObjectLiteralExpression,
-  Project,
-  type PropertyAssignment,
-  SyntaxKind,
-} from "ts-morph";
-import { collectSourceFiles } from "./collect-source-files.ts";
-
-const DEFAULT_TEST_REASON = "test handler callable by any signed-in test user";
-
-export interface Rewrite {
-  readonly file: string;
-  readonly line: number;
-  readonly before: string;
-  readonly after: string;
-}
-
-export interface ManualSite {
-  readonly file: string;
-  readonly line: number;
-  readonly handlerName?: string;
-  readonly description?: string;
-}
-
-export interface MigrateOpenToAllResult {
-  readonly output: string;
-  readonly rewrites: Rewrite[];
-  readonly manual: ManualSite[];
-}
-
-export interface MigrateOpenToAllOptions {
-  readonly testReason: string;
-}
-
-function isTestFile(fileName: string): boolean {
-  return /(^|\/)__tests__\//.test(fileName) || /\.test\.(ts|tsx)$/.test(fileName);
-}
-
-function isOpenToAllTrueAssignment(node: Node): node is PropertyAssignment {
-  if (!Node.isPropertyAssignment(node)) return false;
-  if (node.getName() !== "openToAll") return false;
-  return node.getInitializer()?.getKind() === SyntaxKind.TrueKeyword;
-}
-
-function readStringLiteralProperty(obj: ObjectLiteralExpression, name: string): string | undefined {
-  const prop = obj.getProperty(name);
-  if (!prop || !Node.isPropertyAssignment(prop)) return undefined;
-  const init = prop.getInitializer();
-  return init && Node.isStringLiteral(init) ? init.getLiteralValue() : undefined;
-}
-
-function enclosingObjectLiterals(node: Node): ObjectLiteralExpression[] {
-  const result: ObjectLiteralExpression[] = [];
-  let current: Node | undefined = node.getParent();
-  while (current) {
-    if (Node.isObjectLiteralExpression(current)) result.push(current);
-    current = current.getParent();
-  }
-  return result;
-}
-
-// Best-effort human-triage hint (not the rewrite's source of truth): derive a handler name/description from the enclosing object literal, or the enclosing call's first string-literal argument for the positional form.
-function deriveHandlerContext(assignment: PropertyAssignment): {
-  readonly handlerName?: string;
-  readonly description?: string;
-} {
-  let handlerName: string | undefined;
-  let description: string | undefined;
-  for (const obj of enclosingObjectLiterals(assignment)) {
-    if (handlerName === undefined) handlerName = readStringLiteralProperty(obj, "name");
-    if (description === undefined) description = readStringLiteralProperty(obj, "description");
-    if (handlerName !== undefined && description !== undefined) break;
-  }
-  if (handlerName === undefined) {
-    const call = assignment.getFirstAncestorByKind(SyntaxKind.CallExpression);
-    const firstArg = call?.getArguments()[0];
-    if (firstArg && Node.isStringLiteral(firstArg)) handlerName = firstArg.getLiteralValue();
-  }
-  return {
-    ...(handlerName !== undefined && { handlerName }),
-    ...(description !== undefined && { description }),
-  };
-}
-
-export function migrateOpenToAllSource(
-  source: string,
-  fileName: string,
-  opts: MigrateOpenToAllOptions,
-): MigrateOpenToAllResult {
-  const project = new Project({ useInMemoryFileSystem: true, skipFileDependencyResolution: true });
-  const sourceFile = project.createSourceFile(fileName, source);
-  const testFile = isTestFile(fileName);
-
-  const targets = sourceFile
-    .getDescendantsOfKind(SyntaxKind.PropertyAssignment)
-    .filter(isOpenToAllTrueAssignment);
-
-  const rewrites: Rewrite[] = [];
-  const manual: ManualSite[] = [];
-
-  for (const assignment of targets) {
-    if (assignment.wasForgotten()) continue;
-    const line = assignment.getStartLineNumber();
-
-    if (testFile) {
-      const before = assignment.getText();
-      const after = `openToAll: { reason: ${JSON.stringify(opts.testReason)} }`;
-      assignment.replaceWithText(after);
-      rewrites.push({ file: fileName, line, before, after });
-      continue;
-    }
-
-    manual.push({ file: fileName, line, ...deriveHandlerContext(assignment) });
-  }
-
-  return { output: sourceFile.getFullText(), rewrites, manual };
-}
-
-class CliError extends Error {}
-
-interface CliOptions {
-  readonly dryRun: boolean;
-  readonly testReason: string;
-  readonly paths: string[];
-}
-
-function parseArgs(argv: string[]): CliOptions {
-  let dryRun = false;
-  let testReason = DEFAULT_TEST_REASON;
-  const paths: string[] = [];
-
-  for (let i = 0; i < argv.length; i++) {
-    const arg = argv[i];
-    if (!arg) break;
-    if (arg === "--dry-run") {
-      dryRun = true;
-      continue;
-    }
-    if (arg === "--test-reason") {
-      const value = argv[++i];
-      if (!value) throw new CliError("--test-reason requires a value");
-      testReason = value;
-      continue;
-    }
-    if (arg.startsWith("--")) throw new CliError(`unknown flag: ${arg}`);
-    paths.push(arg);
-  }
-
-  if (paths.length === 0) throw new CliError("at least one path is required");
-  return { dryRun, testReason, paths };
-}
-
-function isMigratableFile(name: string): boolean {
-  return /\.(ts|tsx)$/.test(name);
-}
-
-export function collectFiles(paths: string[]): string[] {
-  return collectSourceFiles(paths, { isMigratableFile });
-}
-
-// Remaining manual sites still fail boot validation, so upgrade-cli must not
-// treat this run as a completed migration.
-export function exitCodeForManualSites(manualCount: number): 0 | 1 {
-  return manualCount > 0 ? 1 : 0;
-}
-
-async function main(): Promise<void> {
-  let options: CliOptions;
-  try {
-    options = parseArgs(process.argv.slice(2));
-  } catch (err) {
-    if (err instanceof CliError) {
-      console.error(`[migrate-open-to-all] ${err.message}`);
-      process.exit(2);
-    }
-    throw err;
-  }
-
-  const files = collectFiles(options.paths);
-  const allManual: ManualSite[] = [];
-  let filesChanged = 0;
-  let totalRewrites = 0;
-
-  for (const file of files) {
-    const before = readFileSync(file, "utf8");
-    if (!before.includes("openToAll")) continue;
-    const result = migrateOpenToAllSource(before, file, { testReason: options.testReason });
-    allManual.push(...result.manual);
-
-    if (result.rewrites.length === 0) continue;
-    filesChanged += 1;
-    totalRewrites += result.rewrites.length;
-
-    if (options.dryRun) {
-      console.log(`\n=== ${file} (${result.rewrites.length} rewrite(s)) ===`);
-      for (const r of result.rewrites) {
-        console.log(`${file}:${r.line}`);
-        console.log(`- ${r.before}`);
-        console.log(`+ ${r.after}`);
-      }
-    } else {
-      writeFileSync(file, result.output);
-    }
-  }
-
-  console.log(
-    `\n[migrate-open-to-all] ${options.dryRun ? "would change" : "changed"} ${filesChanged} file(s); ` +
-      `rewrites: ${totalRewrites}`,
-  );
-
-  if (allManual.length > 0) {
-    console.log(`\n[migrate-open-to-all] manual review needed (${allManual.length}):`);
-    for (const m of allManual) {
-      console.log(
-        `${m.file}:${m.line}  ${m.handlerName ?? "<unknown>"}  ${m.description ?? "<no description>"}`,
-      );
-    }
-  }
-
-  process.exit(exitCodeForManualSites(allManual.length));
-}
-
-if (import.meta.main) {
-  await main();
-}