@zudojs/container 1.2.0 → 1.2.1

package/README.md

@@ -162,9 +162,13 @@ container.register(API, {
 });
 ```
 
-Classes without an `inject` list are constructed with zero arguments. There
-is no reflection/decorator magic — dependencies are exactly the tokens you
-list, resolved in order.
+Classes without an `inject` list are constructed with zero arguments, so
+only a constructor that declares no parameters may omit it (parameters with
+a default value do not count). Registering such a class without `inject`
+is allowed, but resolving it throws a `ProviderResolutionError` naming the
+class and the `inject: [...]` fix, instead of building it with `undefined`
+dependencies. There is no reflection/decorator magic — dependencies are
+exactly the tokens you list, resolved in order.
 
 ## Duplicate registrations
 

package/dist/containerResolution/containerResolution.autoRegister.d.ts

@@ -1,9 +1,9 @@
 /**
  * @zudojs/container/containerResolution/containerResolution.autoRegister
  *
- * Guards for `autoRegisterClasses`: resolving an unregistered class builds
- * it with no arguments, which is only correct for a constructor that
- * declares none.
+ * Guards against building a class with no arguments when its constructor
+ * declares some: an unregistered class under `autoRegisterClasses`, and a
+ * class registration with no `inject` list.
  */
 /**
  * Whether an unregistered token may be auto-registered: a class whose
@@ -24,4 +24,19 @@ export declare function isAutoRegistrable(token: unknown): boolean;
 export declare function assertAutoRegistrable(token: {
     readonly length: number;
 }): void;
+/**
+ * Throws when a class registration has no `inject` list but its constructor
+ * declares parameters. Without one the class is built with no arguments,
+ * so every dependency would be `undefined`. Registering it is allowed;
+ * resolving it is what fails.
+ *
+ * Parameters with a default value do not count (`Function.length` stops at
+ * the first one).
+ *
+ * @throws {ProviderResolutionError} naming the class and the `inject` fix.
+ */
+export declare function assertClassInjectable(token: string, ctor: {
+    readonly length: number;
+    readonly name: string;
+}, inject: readonly unknown[]): void;
 //# sourceMappingURL=containerResolution.autoRegister.d.ts.map

package/dist/containerResolution/containerResolution.autoRegister.js

@@ -1,11 +1,11 @@
 /**
  * @zudojs/container/containerResolution/containerResolution.autoRegister
  *
- * Guards for `autoRegisterClasses`: resolving an unregistered class builds
- * it with no arguments, which is only correct for a constructor that
- * declares none.
+ * Guards against building a class with no arguments when its constructor
+ * declares some: an unregistered class under `autoRegisterClasses`, and a
+ * class registration with no `inject` list.
  */
-import { RegistrationNotFoundError } from "@zudojs/errors";
+import { ProviderResolutionError, RegistrationNotFoundError, } from "@zudojs/errors";
 import { describeToken } from "../containerToken/containerToken.type.js";
 /**
  * Whether an unregistered token may be auto-registered: a class whose
@@ -37,4 +37,26 @@ export function assertAutoRegistrable(token) {
         `{ inject: [/* one token per parameter */] }) or ` +
         `container.registerFactory(${name}, (...deps) => new ${name}(...deps), [/* tokens */]).`);
 }
+/**
+ * Throws when a class registration has no `inject` list but its constructor
+ * declares parameters. Without one the class is built with no arguments,
+ * so every dependency would be `undefined`. Registering it is allowed;
+ * resolving it is what fails.
+ *
+ * Parameters with a default value do not count (`Function.length` stops at
+ * the first one).
+ *
+ * @throws {ProviderResolutionError} naming the class and the `inject` fix.
+ */
+export function assertClassInjectable(token, ctor, inject) {
+    const count = ctor.length;
+    if (inject.length > 0 || count === 0)
+        return;
+    const name = ctor.name || "anonymous class";
+    throw new ProviderResolutionError(token, `Cannot build class "${name}" for token "${token}": its constructor ` +
+        `declares ${count} parameter${count === 1 ? "" : "s"} and the ` +
+        `registration has no inject list, so it would be built with undefined ` +
+        `dependencies. Register it with container.registerClass(${token}, ` +
+        `${name}, { inject: [/* one token per parameter */] }).`);
+}
 //# sourceMappingURL=containerResolution.autoRegister.js.map

package/dist/containerResolution/containerResolution.core.js

@@ -35,7 +35,7 @@ import { unwrapToken } from "../containerToken/containerToken.type.js";
 import { CircularDependencyError, ContainerError, ProviderResolutionError, RegistrationNotFoundError, } from "@zudojs/errors";
 import { AsyncProviderError, CaptiveDependencyError, DependencyResolutionError, MaxResolutionDepthError, ScopedResolutionError, } from "./containerResolution.error.js";
 import { describeToken } from "../containerToken/containerToken.type.js";
-import { assertAutoRegistrable, isAutoRegistrable, } from "./containerResolution.autoRegister.js";
+import { assertAutoRegistrable, assertClassInjectable, isAutoRegistrable, } from "./containerResolution.autoRegister.js";
 import { DependentIndex } from "./containerResolution.dependents.js";
 /**
  * Scope cache that falls back to its parent scope's cache for lookups while
@@ -239,6 +239,7 @@ export class ContainerResolver {
             }
             if (isClassProvider(provider)) {
                 const deps = provider.inject ?? [];
+                assertClassInjectable(describeToken(token), provider.useClass, deps);
                 const args = deps.map((d) => this.resolveInternal(unwrapToken(d), state, singletonAncestor)
                     .value);
                 const ctor = provider.useClass;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zudojs/container",
-  "version": "1.2.0",
+  "version": "1.2.1",
   "description": "Token-based dependency injection container for managing application dependencies and service lifetimes.",
   "type": "module",
   "main": "./dist/index.js",