@beignet/cli 0.0.37 → 0.0.39

package/src/inspect.ts

@@ -558,7 +558,7 @@ function inspectConvention(
   ]);
   const missingResourceGenerator = missingRequirements(files, [
     config.paths.appContext,
-    config.paths.infrastructurePorts,
+    config.paths.portWiring,
     config.paths.ports,
     config.paths.server,
     config.paths.useCaseBuilder,
@@ -1515,9 +1515,9 @@ async function inspectCanonicalConformance(
         "server/providers.ts should own provider registration, even when the provider list is small.",
     },
     {
-      file: config.paths.infrastructurePorts,
+      file: config.paths.portWiring,
       reason:
-        "infra/app-ports.ts should wire concrete app ports for the selected runtime.",
+        "infra/port-wiring.ts should wire concrete app ports for the selected runtime.",
     },
     {
       file: config.paths.routesBuilder,
@@ -2067,7 +2067,7 @@ async function inspectProductionReadiness(
   );
   const configFiles = operationalConfigFiles(files, config);
   const infrastructurePath = directoryPath(
-    path.dirname(config.paths.infrastructurePorts),
+    path.dirname(config.paths.portWiring),
   );
   const schemaDir = `${infrastructurePath}/db/schema`;
   let hasSecurityHeadersHook = false;
@@ -2115,7 +2115,13 @@ async function inspectProductionReadiness(
     if (
       isFeatureUploadDefinition(file, config) &&
       containsCallExpression(source, "defineUpload") &&
-      !/\bmaxSizeBytes\s*:/.test(source)
+      !(await uploadHasExplicitSizeLimit(
+        targetDir,
+        file,
+        source,
+        files,
+        sourceCache,
+      ))
     ) {
       diagnostics.push({
         severity: "warning",
@@ -2373,6 +2379,100 @@ async function inspectProductionReadiness(
   return diagnostics;
 }
 
+async function uploadHasExplicitSizeLimit(
+  targetDir: string,
+  file: string,
+  source: string,
+  files: string[],
+  sourceCache: Map<string, string>,
+): Promise<boolean> {
+  if (/\bmaxSizeBytes\s*:/.test(source)) return true;
+
+  const fileReference = source.match(
+    /\bfile\s*:\s*([A-Za-z_$][A-Za-z0-9_$]*)\b/,
+  )?.[1];
+  if (!fileReference) return false;
+
+  const importPattern = /import\s*\{([\s\S]*?)\}\s*from\s*["']([^"']+)["']/g;
+  for (const match of source.matchAll(importPattern)) {
+    const moduleName = match[2];
+    if (!moduleName?.startsWith(".")) continue;
+
+    const importedName = importedNameForLocalBinding(
+      match[1] ?? "",
+      fileReference,
+    );
+    if (!importedName) continue;
+
+    const importedFile = resolveLocalTypeScriptModule(file, moduleName, files);
+    if (!importedFile) continue;
+
+    const importedSource = await readCachedSource(
+      targetDir,
+      importedFile,
+      sourceCache,
+    );
+    if (
+      exportedObjectHasProperty(importedSource, importedName, "maxSizeBytes")
+    ) {
+      return true;
+    }
+  }
+
+  return false;
+}
+
+function importedNameForLocalBinding(
+  importList: string,
+  localName: string,
+): string | undefined {
+  for (const specifier of importList.split(",")) {
+    const [importedName, aliasedLocalName] = specifier
+      .trim()
+      .replace(/^type\s+/, "")
+      .split(/\s+as\s+/);
+    if (!importedName) continue;
+    if ((aliasedLocalName ?? importedName) === localName) return importedName;
+  }
+
+  return undefined;
+}
+
+function exportedObjectHasProperty(
+  source: string,
+  exportName: string,
+  propertyName: string,
+): boolean {
+  const declaration = new RegExp(
+    `\\bexport\\s+const\\s+${escapeRegExp(exportName)}\\s*=\\s*\\{`,
+  ).exec(source);
+  if (!declaration) return false;
+
+  const openIndex = source.indexOf(
+    "{",
+    declaration.index + declaration[0].length - 1,
+  );
+  const closeIndex = matchingDelimiterIndex(source, openIndex, "{", "}");
+  if (closeIndex < 0) return false;
+
+  const initializer = source.slice(openIndex, closeIndex + 1);
+  return new RegExp(`\\b${escapeRegExp(propertyName)}\\s*:`).test(initializer);
+}
+
+function resolveLocalTypeScriptModule(
+  importer: string,
+  moduleName: string,
+  files: string[],
+): string | undefined {
+  const base = normalizePath(
+    path.posix.normalize(
+      path.posix.join(path.posix.dirname(importer), moduleName),
+    ),
+  );
+  const candidates = [base, `${base}.ts`, `${base}.tsx`, `${base}/index.ts`];
+  return candidates.find((candidate) => files.includes(candidate));
+}
+
 async function hasReadinessCheck(
   targetDir: string,
   files: string[],
@@ -2567,7 +2667,7 @@ async function inspectPaymentsProductionReadiness(
     diagnostics.push({
       severity: "warning",
       code: "BEIGNET_BILLING_ENTITLEMENTS_PROVIDER_MISSING",
-      file: path.dirname(config.paths.infrastructurePorts),
+      file: path.dirname(config.paths.portWiring),
       message:
         "features/billing/entitlements.ts exists, but no infra or server provider appears to call createBillingEntitlements(...). Wire the billing-backed entitlements port into AppPorts so paid feature gates use app billing state.",
     });
@@ -3034,7 +3134,7 @@ async function hasBillingEntitlementsProviderWiring(
   config: ResolvedBeignetConfig,
   sourceCache: Map<string, string>,
 ): Promise<boolean> {
-  const infraPath = path.dirname(config.paths.infrastructurePorts);
+  const infraPath = path.dirname(config.paths.portWiring);
   const serverPath = path.dirname(config.paths.server);
   const candidateFiles = files.filter(
     (file) =>
@@ -3419,13 +3519,13 @@ function isInfraOrServerFile(
 ): boolean {
   const serverDir = directoryPath(path.dirname(config.paths.server));
   const infrastructureDir = directoryPath(
-    path.dirname(config.paths.infrastructurePorts),
+    path.dirname(config.paths.portWiring),
   );
 
   return (
     file === config.paths.server ||
     file.startsWith(`${serverDir}/`) ||
-    file === config.paths.infrastructurePorts ||
+    file === config.paths.portWiring ||
     file.startsWith(`${infrastructureDir}/`)
   );
 }
@@ -3550,9 +3650,7 @@ async function inspectWorkflowRegistrationDrift(
     }
   }
 
-  const infraDir = directoryPath(
-    path.dirname(config.paths.infrastructurePorts),
-  );
+  const infraDir = directoryPath(path.dirname(config.paths.portWiring));
   const listenerTarget = drift.listeners.wiringFile
     ? `${drift.listeners.wiringFile}, which already calls registerListeners(...)`
     : drift.listeners.eventBusFile
@@ -4382,7 +4480,7 @@ async function inspectDatabaseLifecycleDrift(
   const sourceCache = new Map<string, string>();
   const resources = await collectResourceNames(targetDir, files, config);
   const infrastructurePath = directoryPath(
-    path.dirname(config.paths.infrastructurePorts),
+    path.dirname(config.paths.portWiring),
   );
   const featuresPath = directoryPath(config.paths.features);
   const repositoriesPath = path.join(
@@ -7301,21 +7399,21 @@ async function inspectResourceSlices(
       "errors.ts",
     );
     const infrastructureDir = `${directoryPath(
-      path.dirname(config.paths.infrastructurePorts),
+      path.dirname(config.paths.portWiring),
     )}/${resource}/`;
     const drizzleSchemaFile = path.join(
-      path.dirname(config.paths.infrastructurePorts),
+      path.dirname(config.paths.portWiring),
       "db",
       "schema",
       `${resource}.ts`,
     );
     const drizzleRepositoryFile = path.join(
-      path.dirname(config.paths.infrastructurePorts),
+      path.dirname(config.paths.portWiring),
       resource,
       `drizzle-${singular}-repository.ts`,
     );
     const memoryRepositoryFile = path.join(
-      path.dirname(config.paths.infrastructurePorts),
+      path.dirname(config.paths.portWiring),
       resource,
       `in-memory-${singular}-repository.ts`,
     );
@@ -7653,7 +7751,7 @@ async function collectResourceNames(
   const portsPath = directoryPath(path.dirname(config.paths.ports));
   const featuresPath = directoryPath(config.paths.features);
   const infrastructurePath = directoryPath(
-    path.dirname(config.paths.infrastructurePorts),
+    path.dirname(config.paths.portWiring),
   );
 
   for (const file of files) {

package/src/lib.ts

@@ -43,4 +43,4 @@ export {
 export { runOutboxDrain } from "./outbox.js";
 export { runAppSchedule } from "./schedule.js";
 export { runAppTask } from "./task.js";
-export type { IntegrationName } from "./templates/index.js";
+export type { StarterProviderName } from "./templates/index.js";

package/src/lint.ts

@@ -1,5 +1,6 @@
 import { readdir, readFile, stat } from "node:fs/promises";
 import path from "node:path";
+import ts from "typescript";
 import { createPainter } from "./ansi.js";
 import {
   type BeignetConfig,
@@ -89,6 +90,7 @@ export async function lintApp(
   const diagnostics: LintDiagnostic[] = [];
   const sourceFiles = lintSourceFiles(files);
   const sourceFileSet = new Set(sourceFiles);
+  const atAliasRoot = resolveAtAliasRoot(targetDir);
   const sourceByFile = new Map<string, string>();
   const importGraph: ImportGraph = new Map();
 
@@ -96,7 +98,10 @@ export async function lintApp(
     const sourceLayer = classifyPath(file, config);
     const source = await readFile(path.join(targetDir, file), "utf8");
     sourceByFile.set(file, source);
-    importGraph.set(file, parseImports(source, file, sourceFileSet));
+    importGraph.set(
+      file,
+      parseImports(source, file, sourceFileSet, atAliasRoot),
+    );
 
     if (sourceLayer === "test" || sourceLayer === "unknown") continue;
 
@@ -281,15 +286,40 @@ function parseImports(
   source: string,
   importerFile: string,
   files: Set<string>,
+  atAliasRoot: string,
 ): ImportReference[] {
   const references = parseImportSpecifiers(source);
 
   return references.map((reference) => ({
     ...reference,
-    ...resolveImport(reference.importPath, importerFile, files),
+    ...resolveImport(reference.importPath, importerFile, files, atAliasRoot),
   }));
 }
 
+function resolveAtAliasRoot(targetDir: string): string {
+  const tsconfigPath = path.join(targetDir, "tsconfig.json");
+  const parsed = ts.getParsedCommandLineOfConfigFile(
+    tsconfigPath,
+    {},
+    {
+      ...ts.sys,
+      onUnRecoverableConfigFileDiagnostic: () => {},
+    },
+  );
+  const aliasTarget = parsed?.options.paths?.["@/*"]?.[0];
+  if (!aliasTarget) return "";
+
+  const targetWithoutWildcard = aliasTarget.replace(/[/\\]?\*$/, "");
+  const baseUrl = parsed.options.baseUrl ?? targetDir;
+  const absoluteTarget = path.resolve(baseUrl, targetWithoutWildcard);
+  const relativeTarget = normalizePath(
+    path.relative(targetDir, absoluteTarget),
+  );
+  if (relativeTarget === "" || relativeTarget === ".") return "";
+  if (relativeTarget === ".." || relativeTarget.startsWith("../")) return "";
+  return directoryPath(relativeTarget);
+}
+
 function parseImportSpecifiers(
   source: string,
 ): Array<{ importPath: string; kind: ImportKind; offset: number }> {
@@ -590,11 +620,12 @@ function resolveImport(
   importPath: string,
   importerFile: string,
   files: Set<string>,
+  atAliasRoot: string,
 ): Pick<ImportReference, "packageName" | "resolvedPath"> {
   if (importPath.startsWith("@/")) {
     return {
       resolvedPath: resolveLocalImportPath(
-        stripKnownExtension(importPath.slice(2)),
+        stripKnownExtension(path.posix.join(atAliasRoot, importPath.slice(2))),
         files,
       ),
     };
@@ -1171,7 +1202,7 @@ function classifyPath(
   if (
     isUnder(
       normalizedPath,
-      directoryPath(path.dirname(config.paths.infrastructurePorts)),
+      directoryPath(path.dirname(config.paths.portWiring)),
     )
   ) {
     return "infra";

package/src/make/inbox.ts

@@ -160,7 +160,7 @@ export async function makeInbox(
     .map((file) => file.path);
   const files = generatedFiles.map((file) => file.path);
   const dryRun = Boolean(options.dryRun);
-  const inboxSeedNames = seedNames("inbox/demo-inbox");
+  const inboxSeedNames = seedNames("inbox.demo-inbox");
   const seedScript = { "db:seed": `tsx ${seedEntrypointPath(config)}` };
 
   const recordSeedEntrypoint = (result: "created" | "updated" | "skipped") => {
@@ -248,8 +248,8 @@ export async function updateInboxWiring(
     await updatePortsIndex(targetDir, inboxNames, config, {
       ...options,
       generationOptions: {
-        auth: false,
-        tenant: false,
+        authorization: false,
+        tenantScoped: false,
         events: false,
         softDelete: false,
       },
@@ -265,7 +265,7 @@ export async function updateInboxWiring(
       options,
     )
   ) {
-    updated.add(config.paths.infrastructurePorts);
+    updated.add(config.paths.portWiring);
   }
   if (
     await updateDrizzleSchemaIndexExports(
@@ -1119,7 +1119,7 @@ import {
 \tcreateTestPorts,
 } from "@beignet/core/testing";
 import type { AppContext } from "${aliasModule(config.paths.appContext)}";
-import { appPorts } from "${aliasModule(config.paths.infrastructurePorts)}";
+import { initialPorts } from "${aliasModule(config.paths.portWiring)}";
 import type { AppTransactionPorts } from "${aliasModule(config.paths.ports)}";
 import { getUnreadCountUseCase } from "../use-cases/get-unread-count";
 import { listInboxUseCase } from "../use-cases/list-inbox";
@@ -1140,9 +1140,9 @@ function createHarness(
 \tconst inbox = options.inbox ?? createMemoryInboxRepository();
 \tconst actor = options.actor ?? createTestUserActor("user_test");
 \tconst fixture = createTestPorts<AppContext["ports"], AppTransactionPorts>({
-\t\tbase: appPorts,
+\t\tbase: initialPorts,
 \t\toverrides: {
-\t\t\tgate: appPorts.gate,
+\t\t\tgate: initialPorts.gate,
 \t\t\tinbox,
 \t\t},
 \t});
@@ -1305,7 +1305,7 @@ import {
 \tdefineInboxNotificationChannel,
 \ttype InboxNotificationInput,
 } from "${channelModule}";
-import { appPorts } from "${aliasModule(config.paths.infrastructurePorts)}";
+import { initialPorts } from "${aliasModule(config.paths.portWiring)}";
 import { defineNotification } from "${aliasModule(config.paths.notificationsBuilder)}";
 import type { AppTransactionPorts } from "${aliasModule(config.paths.ports)}";
 import { createMemoryInboxRepository } from "./factories/inbox-notification";
@@ -1340,9 +1340,9 @@ const skippingNotification = defineNotification("inbox.channel-skip-test", {
 function createHarness() {
 \tconst inbox = createMemoryInboxRepository();
 \tconst fixture = createTestPorts<AppContext["ports"], AppTransactionPorts>({
-\t\tbase: appPorts,
+\t\tbase: initialPorts,
 \t\toverrides: {
-\t\t\tgate: appPorts.gate,
+\t\t\tgate: initialPorts.gate,
 \t\t\tinbox,
 \t\t},
 \t});

package/src/make/payments.ts

@@ -82,12 +82,18 @@ export async function makePayments(
 
   const database = await detectResourceDatabase(targetDir, config);
   const ui = await detectUiShell(targetDir, config);
-  const generatedFiles = paymentsFiles(config, database, { ui });
+  const tenantScoped = await fileExists(
+    path.join(targetDir, config.paths.features, "workspaces/contracts.ts"),
+  );
+  const generatedFiles = paymentsFiles(config, database, {
+    ui,
+    idempotencyScope: tenantScoped ? "actor-tenant" : "actor",
+  });
   const plannedFiles = await planGeneratedFiles(targetDir, generatedFiles, {
     force: Boolean(options.force),
   });
   const billingNames = resourceNames("billing");
-  const billingSeedNames = seedNames("billing/demo-billing-account");
+  const billingSeedNames = seedNames("billing.demo-billing-account");
   const plannedBillingUpdates = await updatePaymentsWiring(
     targetDir,
     billingNames,
@@ -235,8 +241,8 @@ export async function updatePaymentsWiring(
     await updatePortsIndex(targetDir, billingNames, config, {
       ...options,
       generationOptions: {
-        auth: false,
-        tenant: false,
+        authorization: false,
+        tenantScoped: false,
         events: false,
         softDelete: false,
       },
@@ -246,7 +252,7 @@ export async function updatePaymentsWiring(
   }
   if (await updatePaymentsEntitlementsPort(targetDir, config, options)) {
     updated.add(config.paths.ports);
-    updated.add(config.paths.infrastructurePorts);
+    updated.add(config.paths.portWiring);
   }
   if (
     await updateInfrastructureDeferredPorts(
@@ -256,7 +262,7 @@ export async function updatePaymentsWiring(
       options,
     )
   ) {
-    updated.add(config.paths.infrastructurePorts);
+    updated.add(config.paths.portWiring);
   }
   if (
     await updateDrizzleSchemaIndexExports(
@@ -354,10 +360,7 @@ export async function updatePaymentsEntitlementsPort(
     if (!options.dryRun) await writeFile(portsFilePath, portsNext);
   }
 
-  const infrastructureFilePath = path.join(
-    targetDir,
-    config.paths.infrastructurePorts,
-  );
+  const infrastructureFilePath = path.join(targetDir, config.paths.portWiring);
   const infrastructureOriginal = await readFile(infrastructureFilePath, "utf8");
   const infrastructureNext = appendDeferredPortKey(
     infrastructureOriginal,
@@ -368,7 +371,7 @@ export async function updatePaymentsEntitlementsPort(
     !/["']entitlements["']/.test(infrastructureOriginal)
   ) {
     throw new Error(
-      `Could not find the deferred port list in ${config.paths.infrastructurePorts}. Add "entitlements" manually, or restore the generated infrastructure ports file before running make payments.`,
+      `Could not find the deferred port list in ${config.paths.portWiring}. Add "entitlements" manually, or restore the generated port wiring file before running make payments.`,
     );
   }
   if (infrastructureNext !== infrastructureOriginal) {
@@ -558,7 +561,10 @@ export async function updateBillingSettingsNav(
 export function paymentsFiles(
   config: ResolvedBeignetConfig,
   database: DatabaseName,
-  options: { ui: boolean } = { ui: false },
+  options: {
+    ui: boolean;
+    idempotencyScope?: "actor" | "actor-tenant";
+  } = { ui: false },
 ): GeneratedFile[] {
   const uiFiles: GeneratedFile[] = options.ui
     ? [
@@ -611,7 +617,7 @@ export function paymentsFiles(
     },
     {
       path: path.join(config.paths.features, "billing/contracts.ts"),
-      content: billingContractsFile(),
+      content: billingContractsFile(options.idempotencyScope ?? "actor"),
     },
     {
       path: path.join(config.paths.features, "billing/use-cases/index.ts"),
@@ -846,7 +852,9 @@ export function isBillingAccountActive(
 `;
 }
 
-export function billingContractsFile(): string {
+export function billingContractsFile(
+  idempotencyScope: "actor" | "actor-tenant" = "actor",
+): string {
   return `import { defineContractGroup } from "@beignet/core/contracts";
 import { z } from "zod";
 import {
@@ -883,7 +891,7 @@ export const createCheckoutSession = checkoutBilling
 \t\tidempotency: {
 \t\t\trequired: true,
 \t\t\theader: "idempotency-key",
-\t\t\tscope: "actor-tenant",
+\t\t\tscope: "${idempotencyScope}",
 \t\t\tttlSec: 86_400,
 \t\t},
 \t})
@@ -1410,7 +1418,7 @@ import {
 \tcreateTestPorts,
 } from "@beignet/core/testing";
 import type { AppContext } from "${aliasModule(config.paths.appContext)}";
-import { appPorts } from "@/infra/app-ports";
+import { initialPorts } from "@/infra/port-wiring";
 import type { AppTransactionPorts } from "@/ports";
 import { createBillingEntitlements } from "../entitlements";
 import type { BillingAccount, BillingRepository } from "../ports";
@@ -1451,11 +1459,11 @@ function createHarness(options: { actor?: AppContext["actor"] } = {}) {
 \tconst billing = createMemoryBillingRepository();
 \tconst actor = options.actor ?? createTestUserActor("user_test");
 \tconst fixture = createTestPorts<AppContext["ports"], AppTransactionPorts>({
-\t\tbase: appPorts,
+\t\tbase: initialPorts,
 \t\toverrides: {
 \t\t\tbilling,
 \t\t\tentitlements: createBillingEntitlements(billing),
-\t\t\tgate: appPorts.gate,
+\t\t\tgate: initialPorts.gate,
 \t\t},
 \t});
 \tconst createContext = createTestContextFactory<