bazis 0.111.1 → 0.112.0

package/cli/main.ts

@@ -210,7 +210,10 @@ async function runGenerate(cwd: string, args: ParsedGenerateArgs, codegen: CliRu
   if (result.registered) s.line(`  ${s.strong("↗", PALETTE.export)}  ${s.text(t.module.registered(host ?? "App.module.ts"))}`);
   else s.warning(t.module.notRegistered);
   // What the module needs from the application to run.
-  if (args.generator === "module" && args.profile !== "empty") s.line(`  ${s.strong("◆", PALETTE.service)}  ${s.muted(t.module.needsDb)}`);
+  if (args.generator === "module" && args.profile !== "empty") {
+    s.line(`  ${s.strong("◆", PALETTE.service)}  ${s.muted(t.module.needsDb)}`);
+    s.line(`  ${s.strong("◆", PALETTE.service)}  ${s.muted(t.module.needsTable(`${args.name}.module.ts`))}`);
+  }
   if (args.generator === "module" && args.profile === "full") s.line(`  ${s.strong("◆", PALETTE.service)}  ${s.muted(t.module.needsCache)}`);
   s.line();
   if (result.dryRun) {

package/core/grpc/GrpcSetupError.ts

@@ -0,0 +1,8 @@
+import { KernelError } from "../kernel/errors";
+
+/**
+ * A gRPC controller that cannot be served as declared (an RPC without
+ * @GrpcMethod, a duplicate RPC). A configuration error of the code: the start
+ * prints it as one line and exits with code 1.
+ */
+export class GrpcSetupError extends KernelError {}

package/core/grpc/decorators.ts

@@ -1,4 +1,5 @@
 import type { ServiceDefinition } from "./serviceDefinition";
+import { GrpcSetupError } from "./GrpcSetupError";
 import type { Class } from "../di";
 import type { GrpcControllerDefinition } from "./contracts";
 import type { RequestModelClass } from "../http/Binding/requestModelRegistry";
@@ -62,21 +63,21 @@ export function GrpcMethod(rpcName?: string, requestModel?: RequestModelClass) {
 export function grpcControllerDefinition(controller: Class<object>): GrpcControllerDefinition {
   const carrier = (controller as unknown as { [Symbol.metadata]?: Carrier })[Symbol.metadata];
   const meta = carrier?.[GRPC_META];
-  if (!meta?.service) throw new TypeError(`${controller.name} must use @GrpcController().`);
+  if (!meta?.service) throw new GrpcSetupError(`${controller.name} must use @GrpcController().`);
   const methods = new Map<string, string | symbol>();
   for (const [methodName, rpcName] of meta.methods) {
     const keys = Object.keys(meta.service).filter((key) => {
       const rpc = meta.service![key]!;
       return key === rpcName || rpc.originalName === rpcName || rpc.path.split("/").at(-1) === rpcName;
     });
-    if (keys.length !== 1) throw new TypeError(`${controller.name}.${String(methodName)}: unknown or ambiguous RPC ${rpcName}.`);
+    if (keys.length !== 1) throw new GrpcSetupError(`${controller.name}.${String(methodName)}: unknown or ambiguous RPC ${rpcName}.`);
     const key = keys[0]!;
-    if (methods.has(key)) throw new TypeError(`${controller.name}: duplicate RPC ${rpcName}.`);
-    if (typeof controller.prototype[methodName] !== "function") throw new TypeError(`Missing RPC handler ${String(methodName)}.`);
+    if (methods.has(key)) throw new GrpcSetupError(`${controller.name}: duplicate RPC ${rpcName}.`);
+    if (typeof controller.prototype[methodName] !== "function") throw new GrpcSetupError(`Missing RPC handler ${String(methodName)}.`);
     methods.set(key, methodName);
   }
   for (const key of Object.keys(meta.service)) {
-    if (!methods.has(key)) throw new TypeError(`${controller.name}: RPC ${key} is missing @GrpcMethod().`);
+    if (!methods.has(key)) throw new GrpcSetupError(`${controller.name}: RPC ${key} is missing @GrpcMethod().`);
   }
   return Object.freeze({ controller, service: meta.service, methods, requestModels: new Map(meta.requestModels) });
 }

package/core/grpc/index.ts

@@ -1,5 +1,6 @@
 export { GrpcController, GrpcMethod } from "./decorators";
 export { GrpcError } from "./GrpcError";
+export { GrpcSetupError } from "./GrpcSetupError";
 export { grpcModule } from "./Grpc.module";
 export { GrpcServer } from "./GrpcServer";
 export { GrpcClient } from "./GrpcClient";

package/core/infra/connectors/codex/CodexAppServerClient.ts

@@ -232,7 +232,7 @@ export class CodexAppServerClient implements CodexClient {
         if (method === "account/updated") this.modelCache = undefined;
       }, () => { this.clearLogin(); this.modelCache = undefined; });
       const initialized = await client.request("initialize", {
-        clientInfo: { name: "bazis", title: "Bazis", version: "0.111.1" }, capabilities: { experimentalApi: true },
+        clientInfo: { name: "bazis", title: "Bazis", version: "0.112.0" }, capabilities: { experimentalApi: true },
       }, 15_000, signal);
       if (resolve(text(initialized.codexHome, 4096)) !== this.paths.home) throw new CodexError("PROTOCOL_ERROR");
       client.notify("initialized");

package/core/kernel/LifecycleCoordinator.ts

@@ -174,8 +174,10 @@ export class LifecycleCoordinator {
     const names = batch.map(serviceName);
     for (const name of names) this.pending.add(name);
     const startedAt = performance.now();
-    const results = await Promise.allSettled(batch.map((service) => this.startService(service, signal)));
-    for (const name of names) this.pending.delete(name);
+    // Each service leaves the pending list as soon as it has started, so a
+    // startup timeout names only the ones still starting.
+    const results = await Promise.allSettled(batch.map((service, index) =>
+      this.startService(service, signal).finally(() => this.pending.delete(names[index] as string))));
     this.timings.push(Object.freeze({ names: Object.freeze(names), phase: batch[0]?.phase ?? 0, ms: performance.now() - startedAt }));
     const failures: StartupFailure[] = [];
     results.forEach((result, index) => {

package/core/kernel/config/ConfigValidationError.ts

@@ -1,6 +1,6 @@
 import { KernelError } from "../errors";
 
-export type ConfigIssueKind = "missing-secret" | "development-secret" | "number" | "boolean" | "string" | "enum" | "validator" | "conflict";
+export type ConfigIssueKind = "missing-secret" | "empty-secret" | "development-secret" | "number" | "boolean" | "string" | "enum" | "validator" | "conflict";
 
 /** One invalid configuration key, with what can change it. */
 export interface ConfigIssue {

package/core/kernel/config/defineConfig.ts

@@ -202,7 +202,10 @@ export function defineConfig(prefixOrSchema: string | ConfigSchema<Defaults>, ma
       const raw = selected?.raw ?? (isSecret(cell) || isEnum(cell) ? cell.default : cell);
       let value: ConfigValueType | undefined;
       if (type === "secret") {
-        if (typeof raw !== "string" || !raw.trim()) issue(key, "missing-secret", `required non-empty secret is not set (${names.get(key)!.join(", ")})`);
+        // secret("") reads like an empty default but would mean "required":
+        // the declaration is ambiguous, so it is refused with the fix.
+        if (isSecret(cell) && cell.default !== undefined && !cell.default.trim()) issue(key, "empty-secret", 'secret("") is not a value: a secret cannot be empty. Write secret() for a required secret, or give a non-empty development value');
+        else if (typeof raw !== "string" || !raw.trim()) issue(key, "missing-secret", `required non-empty secret is not set (${names.get(key)!.join(", ")})`);
         // A secret default in the code is a development value; production must
         // get the secret from a source or declare its own value explicitly.
         // No "apiKey: …" in the hint: the console redaction would hide it.

package/core/kernel/report/startupProblems.ts

@@ -84,6 +84,7 @@ function configProblem(issue: ConfigIssue, screen: Screen): Problem {
   const variables = issue.env.map((name) => screen.code(name)).join(" / ");
   const causes: Record<ConfigIssue["kind"], string> = {
     "missing-secret": t.config.issue.missingSecret,
+    "empty-secret": t.config.issue.emptySecret,
     "development-secret": t.config.issue.devSecret,
     number: t.config.issue.number,
     boolean: t.config.issue.boolean,
@@ -94,6 +95,7 @@ function configProblem(issue: ConfigIssue, screen: Screen): Problem {
   };
   const fixes: string[] = [];
   if (issue.kind === "missing-secret") fixes.push(`${t.config.fix.setVariable(variables)}, ${t.config.fix.orSecretFile}`);
+  else if (issue.kind === "empty-secret") fixes.push(t.config.fix.emptySecret);
   else if (issue.kind === "development-secret") fixes.push(t.config.fix.setVariable(variables), t.config.fix.orProduction(screen.code(issue.key.split(".").at(-1) ?? issue.key)));
   else fixes.push(t.config.fix.change(issue.source === undefined ? variables : screen.code(issue.source)));
   // The wrong value and the variable that brought it.

package/library/orm/Schema/MigrationRunner.ts

@@ -101,8 +101,11 @@ export class MigrationRunner {
 
     for (const id of toRollback) {
       const migration = this.migrations.find((m) => m.id === id);
-      if (!migration?.down) {
-        throw new Error(`Migration "${id}" has no down() method.`);
+      if (!migration) {
+        throw new OrmError(`Migration "${id}" is applied but is not in the migrations passed to rollbackVersioned(); pass the full list.`);
+      }
+      if (!migration.down) {
+        throw new OrmError(`Migration "${id}" has no down() method.`);
       }
       await this.provider.transaction(async (tx) => {
         await migration.down!(this.context(tx));

package/library/orm/Transactions/TransactionScopeCoordinator.ts

@@ -198,7 +198,7 @@ export async function transactionScope<TResult>(context: DbContext, provider: Da
 function assertRetryable(context: DbContext): void {
   const tracked = trackedEntryCount(context.changeTracker);
   if (tracked === 0) return;
-  throw new OrmTransactionScopeError(`transactionScope({ retry }) needs ${contextName(context)} without tracked entities: every attempt starts from a clean context, and the ${tracked} entities tracked before the scope would lose their changes. Start the scope before loading anything, or create a new scope for it.`);
+  throw new OrmTransactionScopeError(`transactionScope({ retry }) needs ${contextName(context)} without tracked entities: every attempt starts from a clean context, and the ${tracked === 1 ? "1 entity" : `${tracked} entities`} tracked before the scope would lose ${tracked === 1 ? "its" : "their"} changes. Start the scope before loading anything, or create a new scope for it.`);
 }
 
 async function runScope<TResult>(context: DbContext, provider: DatabaseProvider, work: (transaction: OrmTransaction) => Promise<TResult>, options: OrmTransactionScopeOptions | undefined, retryContexts?: Set<DbContext>): Promise<TResult> {

package/library/terminal/messages.ts

@@ -111,6 +111,7 @@ const en = {
     invalid: (total: number) => `of ${total} sections invalid`,
     issue: {
       missingSecret: "required secret is not set",
+      emptySecret: 'secret("") is not a value: a secret cannot be empty',
       devSecret: "production would use the development default secret from the code",
       number: "expected a finite number",
       boolean: "expected a boolean (true, false, 1, 0)",
@@ -124,6 +125,7 @@ const en = {
       setVariable: (names: string) => `set ${names}`,
       orSecretFile: "or a file in a secretFilesSource directory",
       orProduction: (key: string) => `or declare ${key}: secret() with a value source in the production section`,
+      emptySecret: "write secret() for a required secret, or give a non-empty development value",
       change: (names: string) => `change ${names}`,
     },
   },
@@ -233,6 +235,7 @@ const en = {
     notRegistered: "not connected — codegen skips it until a module imports it",
     codegenSkipped: "codegen skipped (--no-codegen) — run bazis codegen",
     needsDb: "this module uses a database: connect one with @Infra({ db: ormBazisConnect(dbConfig) })",
+    needsTable: (moduleFile: string) => `its table is not created: add ensureCreated: true to ormBazis in ${moduleFile}, or write a migration`,
     needsCache: "this module caches: pass runApp(AppModule, { cache: memory() })",
     keptOnFailure: "scaffold files were kept — fix the error and run bazis codegen",
   },
@@ -273,6 +276,7 @@ const ru: Messages = {
     invalid: (total) => `из ${total} с ошибками`,
     issue: {
       missingSecret: "обязательный секрет не задан",
+      emptySecret: 'secret("") — не значение: секрет не может быть пустым',
       devSecret: "в production попадёт секрет для разработки из кода",
       number: "нужно конечное число",
       boolean: "нужно логическое значение (true, false, 1, 0)",
@@ -286,6 +290,7 @@ const ru: Messages = {
       setVariable: (names) => `задайте ${names}`,
       orSecretFile: "или файл в каталоге secretFilesSource",
       orProduction: (key) => `или объявите ${key}: secret() с источником значения в разделе production`,
+      emptySecret: "обязательный секрет пишется secret(), а для разработки нужно непустое значение",
       change: (names) => `исправьте ${names}`,
     },
   },
@@ -339,10 +344,10 @@ const ru: Messages = {
     typecheck: (count) => `проверка типов не прошла · ${count} ${formRu(count, NOUNS_RU.error)}`,
     tests: "тесты не прошли",
     command: "команда не выполнена",
-    skipConfig: "не проверялось — конфигурация с ошибками",
-    skipInfra: "не запускалось — инфраструктура недоступна",
-    skipCodegen: "не запускалось — кодогенерация не удалась",
-    skipEarlier: "не запускалось — предыдущий этап не удался",
+    skipConfig: "без проверки — конфигурация с ошибками",
+    skipInfra: "без запуска — инфраструктура недоступна",
+    skipCodegen: "без запуска — кодогенерация не удалась",
+    skipEarlier: "без запуска — предыдущий этап не удался",
     exitNothing: (code) => `код выхода ${code} · ничего не запущено`,
     exitWatch: (code, src) => `код выхода ${code} · слежу за ${src} — исправьте и сохраните для перезапуска`,
     exit: (code) => `код выхода ${code}`,
@@ -395,6 +400,7 @@ const ru: Messages = {
     notRegistered: "не подключён — кодогенерация пропускает его, пока модуль не импортирован",
     codegenSkipped: "кодогенерация пропущена (--no-codegen) — запустите bazis codegen",
     needsDb: "модулю нужна база данных: подключите её через @Infra({ db: ormBazisConnect(dbConfig) })",
+    needsTable: (moduleFile) => `таблицу шаблон не создаёт: добавьте ensureCreated: true в ormBazis в ${moduleFile} или напишите миграцию`,
     needsCache: "модуль кэширует: передайте runApp(AppModule, { cache: memory() })",
     keptOnFailure: "файлы модуля оставлены — исправьте ошибку и запустите bazis codegen",
   },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bazis",
-  "version": "0.111.1",
+  "version": "0.112.0",
   "description": "Modular backend framework for Bun in TypeScript: DI wired by codegen, HTTP, a PostgreSQL ORM, JWT, WebSocket, agents and a CLI.",
   "license": "MIT",
   "author": "Alexander Alenin",