@beignet/cli 0.0.38 → 0.0.39

package/src/inspect.ts

@@ -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[],

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/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,
       },

package/src/make/payments.ts

@@ -93,7 +93,7 @@ export async function makePayments(
     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,
@@ -241,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,
       },

package/src/make/shared.ts

@@ -45,8 +45,8 @@ export type ResourceNames = {
 export type ResourceGenerationMode = "feature" | "resource";
 
 export type ResourceGenerationOptions = {
-  auth: boolean;
-  tenant: boolean;
+  authorization: boolean;
+  tenantScoped: boolean;
   events: boolean;
   softDelete: boolean;
 };
@@ -220,6 +220,13 @@ export type UploadNames = FeatureArtifactNames & {
   uploadExportName: string;
   metadataSchemaName: string;
   metadataTypeName: string;
+  fileConstraintsExportName: string;
+  manifestExportName: string;
+  manifestFileName: string;
+  reactUploadsExportName: string;
+  clientFileName: string;
+  componentExportName: string;
+  componentFileName: string;
 };
 
 export type FeatureUiNames = ResourceNames & {
@@ -502,7 +509,7 @@ export function wireListenersProviderSource(
  *
  * Each wiring is independent and skipped silently when its port key already
  * appears in the ports file, so apps that wired their own adapter (or scaffolds
- * whose integrations already contribute the port) are left untouched. All
+ * whose providers already contribute the port) are left untouched. All
  * anchors are computed before any file is written; an anchor miss aborts with a
  * copy-pasteable manual instruction so the generator never leaves partial
  * wiring behind. Callers validate anchors with `dryRun: true` before writing
@@ -529,7 +536,7 @@ export async function wirePortProviders(
   }
 
   // A port key that already appears in the ports file means the app wired its
-  // own adapter or an integration already contributes the port.
+  // own adapter or a provider already contributes the port.
   const active = wirings.filter(
     (wiring) => !new RegExp(`\\b${wiring.portKey}\\b`).test(portsOriginal),
   );
@@ -1179,7 +1186,7 @@ export async function updatePortsIndex(
     next = insertAfterImports(next, importLine);
   }
 
-  if (options.generationOptions?.auth) {
+  if (options.generationOptions?.authorization) {
     const policyPath = path.join(
       config.paths.features,
       names.pluralKebab,
@@ -2911,13 +2918,12 @@ export function artifactNameParts(
 ): [string, string] {
   const parts = input
     .trim()
-    .split("/")
-    .map((part) => part.trim())
-    .filter(Boolean);
+    .split(".")
+    .map((part) => part.trim());
 
-  if (parts.length !== 2) {
+  if (parts.length !== 2 || parts.some((part) => part.length === 0)) {
     throw new Error(
-      `${label} name must use feature/name format, for example posts/published.`,
+      `${label} name must use feature.name format, for example posts.published.`,
     );
   }