IgorBabkin / ts-ioc-container / 24984514781

97%

master: 94%

push

github

feat(injector)!: require explicit @inject for every constructor param

Constructor parameters without @inject metadata now resolve to `undefined`
instead of being implicitly filled from positional `args`. Annotate every
parameter that should come from the container or runtime args list
(`@inject(args(0))`, `@inject('key')`, etc.).

Removes `resolveByArgs` — the inject-time resolver already auto-resolves
`InjectionToken` values passed via `token.args(...)` / `setArgs(...)`,
so the helper is redundant. Bare constructors are not auto-resolved;
consumers must wrap them in `ClassToken` to opt into resolution.

Also drops the now-unused `fillEmptyIndexes` array helper.

BREAKING CHANGE: constructor parameters without `@inject` no longer pick up
positional arguments — they resolve to `undefined`. `resolveByArgs` is removed
from the public API.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

232 of 243 branches covered (95.47%)

Branch coverage included in aggregate %.

4 of 4 new or added lines in 1 file covered. (100.0%)

556 of 574 relevant lines covered (96.86%)

107.4 hits per line