refactor(di)!: replace Provider by Factory

Provider has been deprecated by inversify

Author: c100k

CHANGELOG.md

@@ -1,5 +1,11 @@
 # CHANGELOG
 
+## v0.28.0 (unreleased)
+
+Replace `Provider` by `Factory` in Dependency Injection (`inversify`). If you're using `bindProvider`, simply replace it by `bindFactory`.
+
+See all the changes here : https://github.com/c100k/libmodulor/compare/v0.27.0...master
+
 ## v0.27.0 (2026-02-09)
 
 **Highlights**

dist/esm/target/lib/server/AuthenticationChecker.d.ts

@@ -1,4 +1,4 @@
-import { type Provider } from 'inversify';
+import { type Factory } from 'inversify';
 import type { AppManifest } from '../../../app/index.js';
 import type { Logger, Worker } from '../../../std/index.js';
 import { type UC, type UCAuth, type UCDef, type UCInput, type UCOPIBase, type UCPolicy } from '../../../uc/index.js';
@@ -23,8 +23,8 @@ export declare class AuthenticationChecker implements Worker<Input, Promise<Outp
     private jwtAuthenticationChecker;
     private privateApiKeyAuthenticationChecker;
     private logger;
-    private ucPolicyProvider;
-    constructor(basicAuthenticationChecker: BasicAuthenticationChecker, jwtAuthenticationChecker: JWTAuthenticationChecker, privateApiKeyAuthenticationChecker: PrivateApiKeyAuthenticationChecker, logger: Logger, ucPolicyProvider: Provider<UCPolicy>);
+    private ucPolicyFactory;
+    constructor(basicAuthenticationChecker: BasicAuthenticationChecker, jwtAuthenticationChecker: JWTAuthenticationChecker, privateApiKeyAuthenticationChecker: PrivateApiKeyAuthenticationChecker, logger: Logger, ucPolicyFactory: Factory<UCPolicy>);
     exec<I extends UCInput | undefined = undefined, OPI0 extends UCOPIBase | undefined = undefined, OPI1 extends UCOPIBase | undefined = undefined>({ authCookie, authorizationHeader, uc, }: Input<I, OPI0, OPI1>): Promise<Output>;
 }
 export {};

dist/esm/target/lib/server/AuthenticationChecker.js

@@ -21,13 +21,13 @@ let AuthenticationChecker = class AuthenticationChecker {
     jwtAuthenticationChecker;
     privateApiKeyAuthenticationChecker;
     logger;
-    ucPolicyProvider;
-    constructor(basicAuthenticationChecker, jwtAuthenticationChecker, privateApiKeyAuthenticationChecker, logger, ucPolicyProvider) {
+    ucPolicyFactory;
+    constructor(basicAuthenticationChecker, jwtAuthenticationChecker, privateApiKeyAuthenticationChecker, logger, ucPolicyFactory) {
         this.basicAuthenticationChecker = basicAuthenticationChecker;
         this.jwtAuthenticationChecker = jwtAuthenticationChecker;
         this.privateApiKeyAuthenticationChecker = privateApiKeyAuthenticationChecker;
         this.logger = logger;
-        this.ucPolicyProvider = ucPolicyProvider;
+        this.ucPolicyFactory = ucPolicyFactory;
     }
     async exec({ authCookie, authorizationHeader, uc, }) {
         this.logger.trace('Checking auth', {
@@ -43,7 +43,7 @@ let AuthenticationChecker = class AuthenticationChecker {
             return output;
         }
         const authType = sec?.authType ?? DEFAULT_UC_SEC_AT;
-        const policy = (await this.ucPolicyProvider(server.policy));
+        const policy = (await this.ucPolicyFactory(server.policy));
         const canBeExecutedPreAuth = await policy.canBeExecutedPreAuth();
         if (canBeExecutedPreAuth) {
             const { allowed } = await policy.exec({
@@ -101,7 +101,7 @@ AuthenticationChecker = __decorate([
     __param(1, inject(JWTAuthenticationChecker)),
     __param(2, inject(PrivateApiKeyAuthenticationChecker)),
     __param(3, inject('Logger')),
-    __param(4, inject('Provider<UCPolicy>')),
+    __param(4, inject('Factory<UCPolicy>')),
     __metadata("design:paramtypes", [BasicAuthenticationChecker,
         JWTAuthenticationChecker,
         PrivateApiKeyAuthenticationChecker, Object, Function])

dist/esm/uc/impl/SimpleUCManager.d.ts

@@ -1,4 +1,4 @@
-import { type Provider } from 'inversify';
+import { type Factory } from 'inversify';
 import type { ClockManager, CryptoManager, Logger } from '../../std/index.js';
 import type { UCClientConfirmManager } from '../client.js';
 import type { UCData } from '../data.js';
@@ -23,10 +23,10 @@ export declare class SimpleUCManager implements UCManager {
     private ucExecChecker;
     private ucInputFilesProcessor;
     private ucInputValidator;
-    private ucInitProvider;
-    private ucMainProvider;
+    private ucInitFactory;
+    private ucMainFactory;
     private tx?;
-    constructor(ucClientConfirmManager: UCClientConfirmManager, clockManager: ClockManager, cryptoManager: CryptoManager, logger: Logger, ucDataStore: UCDataStore, ucExecChecker: UCExecChecker, ucInputFilesProcessor: UCInputFilesProcessor, ucInputValidator: UCInputValidator, ucInitProvider: Provider<UCInit>, ucMainProvider: Provider<UCMain>);
+    constructor(ucClientConfirmManager: UCClientConfirmManager, clockManager: ClockManager, cryptoManager: CryptoManager, logger: Logger, ucDataStore: UCDataStore, ucExecChecker: UCExecChecker, ucInputFilesProcessor: UCInputFilesProcessor, ucInputValidator: UCInputValidator, ucInitFactory: Factory<UCInit>, ucMainFactory: Factory<UCMain>);
     commitTx(): Promise<void>;
     confirmClient<I extends UCInput | undefined = undefined, OPI0 extends UCOPIBase | undefined = undefined, OPI1 extends UCOPIBase | undefined = undefined>(uc: UC<I, OPI0, OPI1>): Promise<boolean>;
     execClient<I extends UCInput | undefined = undefined, OPI0 extends UCOPIBase | undefined = undefined, OPI1 extends UCOPIBase | undefined = undefined>(uc: UC<I, OPI0, OPI1>, opts?: UCManagerExecClientOpts<I, OPI0, OPI1>): Promise<UCOutputReader<I, OPI0, OPI1>>;

dist/esm/uc/impl/SimpleUCManager.js

@@ -27,11 +27,11 @@ let SimpleUCManager = class SimpleUCManager {
     ucExecChecker;
     ucInputFilesProcessor;
     ucInputValidator;
-    ucInitProvider;
-    ucMainProvider;
+    ucInitFactory;
+    ucMainFactory;
     // WARNING : This property makes this class "thread unsafe". Be careful how you inject it into your components.
     tx;
-    constructor(ucClientConfirmManager, clockManager, cryptoManager, logger, ucDataStore, ucExecChecker, ucInputFilesProcessor, ucInputValidator, ucInitProvider, ucMainProvider) {
+    constructor(ucClientConfirmManager, clockManager, cryptoManager, logger, ucDataStore, ucExecChecker, ucInputFilesProcessor, ucInputValidator, ucInitFactory, ucMainFactory) {
         this.ucClientConfirmManager = ucClientConfirmManager;
         this.clockManager = clockManager;
         this.cryptoManager = cryptoManager;
@@ -40,8 +40,8 @@ let SimpleUCManager = class SimpleUCManager {
         this.ucExecChecker = ucExecChecker;
         this.ucInputFilesProcessor = ucInputFilesProcessor;
         this.ucInputValidator = ucInputValidator;
-        this.ucInitProvider = ucInitProvider;
-        this.ucMainProvider = ucMainProvider;
+        this.ucInitFactory = ucInitFactory;
+        this.ucMainFactory = ucMainFactory;
         this.tx = undefined;
     }
     async commitTx() {
@@ -80,7 +80,7 @@ let SimpleUCManager = class SimpleUCManager {
         if (server === undefined) {
             await this.ucInputFilesProcessor.exec({ uc });
         }
-        const main = (await this.ucMainProvider(client.main));
+        const main = (await this.ucMainFactory(client.main));
         const streamOpts = opts?.stream;
         const output = await main.exec({
             opts: {
@@ -113,7 +113,7 @@ let SimpleUCManager = class SimpleUCManager {
         }
         this.ucInputValidator.exec({ uc });
         await this.ucInputFilesProcessor.exec({ uc });
-        const main = (await this.ucMainProvider(server.main));
+        const main = (await this.ucMainFactory(server.main));
         const output = main.exec({
             opts: {
                 stream: opts?.stream,
@@ -132,7 +132,7 @@ let SimpleUCManager = class SimpleUCManager {
             return;
         }
         this.logger.info('Initializing ucd', { name: metadata.name });
-        const init = (await this.ucInitProvider(server.init));
+        const init = (await this.ucInitFactory(server.init));
         await init.exec();
     }
     async startTx() {
@@ -179,8 +179,8 @@ SimpleUCManager = __decorate([
     __param(5, inject(UCExecChecker)),
     __param(6, inject(UCInputFilesProcessor)),
     __param(7, inject(UCInputValidator)),
-    __param(8, inject('Provider<UCInit>')),
-    __param(9, inject('Provider<UCMain>')),
+    __param(8, inject('Factory<UCInit>')),
+    __param(9, inject('Factory<UCMain>')),
     __metadata("design:paramtypes", [Object, Object, Object, Object, Object, UCExecChecker,
         UCInputFilesProcessor,
         UCInputValidator, Function, Function])

dist/esm/uc/workers/UCExecChecker.d.ts

@@ -1,4 +1,4 @@
-import { type Provider } from 'inversify';
+import { type Factory } from 'inversify';
 import type { ProductManifest } from '../../product/index.js';
 import type { Configurable, SettingsManager, Worker } from '../../std/index.js';
 import type { UCDefLifecycle } from '../def.js';
@@ -16,8 +16,8 @@ type S = Pick<UCSettings, 'uc_disabled_use_cases'>;
 export declare class UCExecChecker implements Configurable<S>, Worker<Input, Promise<Output>> {
     private productManifest;
     private settingsManager;
-    private ucPolicyProvider;
-    constructor(productManifest: ProductManifest, settingsManager: SettingsManager<S>, ucPolicyProvider: Provider<UCPolicy>);
+    private ucPolicyFactory;
+    constructor(productManifest: ProductManifest, settingsManager: SettingsManager<S>, ucPolicyFactory: Factory<UCPolicy>);
     s(): S;
     exec<I extends UCInput | undefined = undefined, OPI0 extends UCOPIBase | undefined = undefined, OPI1 extends UCOPIBase | undefined = undefined>({ lifecycle, uc }: Input<I, OPI0, OPI1>): Promise<Output>;
 }

dist/esm/uc/workers/UCExecChecker.js

@@ -14,11 +14,11 @@ import { inject, injectable } from 'inversify';
 let UCExecChecker = class UCExecChecker {
     productManifest;
     settingsManager;
-    ucPolicyProvider;
-    constructor(productManifest, settingsManager, ucPolicyProvider) {
+    ucPolicyFactory;
+    constructor(productManifest, settingsManager, ucPolicyFactory) {
         this.productManifest = productManifest;
         this.settingsManager = settingsManager;
-        this.ucPolicyProvider = ucPolicyProvider;
+        this.ucPolicyFactory = ucPolicyFactory;
     }
     s() {
         return {
@@ -47,7 +47,7 @@ let UCExecChecker = class UCExecChecker {
                 const ucds = app.ucds?.exclude ?? [];
                 output.allowed = !ucds.includes(metadata.name);
                 if (output.allowed) {
-                    const policy = (await this.ucPolicyProvider(client.policy));
+                    const policy = (await this.ucPolicyFactory(client.policy));
                     const { allowed } = await policy.exec({ uc });
                     output.allowed = allowed;
                 }
@@ -68,7 +68,7 @@ UCExecChecker = __decorate([
     injectable(),
     __param(0, inject('ProductManifest')),
     __param(1, inject('SettingsManager')),
-    __param(2, inject('Provider<UCPolicy>')),
+    __param(2, inject('Factory<UCPolicy>')),
     __metadata("design:paramtypes", [Object, Object, Function])
 ], UCExecChecker);
 export { UCExecChecker };

dist/esm/utils/index.d.ts

@@ -12,7 +12,7 @@ export { SSEStreamManager } from './http/SSEStreamManager.js';
 export { fmtSingleDataMsg, fmtSSEError, isSSEError, parseDataLine, SSE_HEADERS, } from './http/sse.js';
 export { isClientError, isError, isServerError } from './http/status.js';
 export type { HTTPDataEnvelope, HTTPReqData } from './http/types.js';
-export { bindProvider } from './ioc/bindProvider.js';
+export { bindFactory } from './ioc/bindFactory.js';
 export { CONTAINER_OPTS } from './ioc/container.js';
 export type { Class } from './ioc/types.js';
 export { fmt as fmtNumber } from './numbers/fmt.js';

dist/esm/utils/index.js

@@ -7,7 +7,7 @@ export { fromQueryParams, toQueryParams } from './http/query-params.js';
 export { SSEStreamManager } from './http/SSEStreamManager.js';
 export { fmtSingleDataMsg, fmtSSEError, isSSEError, parseDataLine, SSE_HEADERS, } from './http/sse.js';
 export { isClientError, isError, isServerError } from './http/status.js';
-export { bindProvider } from './ioc/bindProvider.js';
+export { bindFactory } from './ioc/bindFactory.js';
 export { CONTAINER_OPTS } from './ioc/container.js';
 export { fmt as fmtNumber } from './numbers/fmt.js';
 export { baseFromSquareUnit, isSquareUnit, } from './numbers/units.js';

dist/esm/utils/ioc/bindCommon.js

@@ -16,7 +16,7 @@ import { InMemoryUCDataStore } from '../../uc/impl/InMemoryUCDataStore.js';
 import { SimpleUCManager } from '../../uc/impl/SimpleUCManager.js';
 import { StaticUCClientConfirmManager } from '../../uc/impl/StaticUCClientConfirmManager.js';
 import { UC_DEFAULT_SETTINGS, } from '../../uc/index.js';
-import { bindProvider } from './bindProvider.js';
+import { bindFactory } from './bindFactory.js';
 export function bindCommon(container) {
     // settings
     container.bind('Settings').toConstantValue({
@@ -55,9 +55,9 @@ export function bindCommon(container) {
         .to(SettingsServerClientManager);
     container.bind('XMLManager').to(NoopXMLManager);
     // uc
-    bindProvider(container, 'UCInit');
-    bindProvider(container, 'UCMain');
-    bindProvider(container, 'UCPolicy');
+    bindFactory(container, 'UCInit');
+    bindFactory(container, 'UCMain');
+    bindFactory(container, 'UCPolicy');
     container
         .bind('UCClientConfirmManager')
         .to(StaticUCClientConfirmManager);