@o3co/auth-provider-foundation 0.3.0 → 0.4.0

package/README.ja.md

@@ -23,8 +23,8 @@ npm install redis
 
 ```typescript
 function registerBuiltinAdapters(factories: {
-  userFactory: RepositoryFactory<UserRepository>;
-  codeFactory: RepositoryFactory<CodeRepository>;
+  userFactory: AdapterFactory<UserRepository>;
+  codeFactory: AdapterFactory<CodeRepository>;
   pathResolver?: PathResolver; // 任意 — "redis" モジュールのパス解決に使用
 }): void;
 ```
@@ -110,5 +110,5 @@ const codeRepo = await codeFactory.create({
 
 ## 関連
 
-- [`@o3co/auth-provider-core`](../core/README.md) — コアインターフェース（`UserRepository`、`CodeRepository`、`RepositoryFactory`、`PathResolver`）
+- [`@o3co/auth-provider-core`](../core/README.md) — コアインターフェース（`UserRepository`、`CodeRepository`、`AdapterFactory`、`createAdapterFactory`、`BuilderContext`、`PathResolver`）
 - [auth.provider](../../README.md) — リポジトリ全体のドキュメント

package/README.md

@@ -23,8 +23,8 @@ Registers the built-in adapter factories into the provided factory instances.
 
 ```typescript
 function registerBuiltinAdapters(factories: {
-  userFactory: RepositoryFactory<UserRepository>;
-  codeFactory: RepositoryFactory<CodeRepository>;
+  userFactory: AdapterFactory<UserRepository>;
+  codeFactory: AdapterFactory<CodeRepository>;
   pathResolver?: PathResolver; // optional — used to resolve the "redis" module path
 }): void;
 ```
@@ -110,5 +110,5 @@ const codeRepo = await codeFactory.create({
 
 ## See Also
 
-- [`@o3co/auth-provider-core`](../core/README.md) — Core interfaces (`UserRepository`, `CodeRepository`, `RepositoryFactory`, `PathResolver`)
+- [`@o3co/auth-provider-core`](../core/README.md) — Core interfaces (`UserRepository`, `CodeRepository`, `AdapterFactory`, `createAdapterFactory`, `BuilderContext`, `PathResolver`)
 - [auth.provider](../../README.md) — Top-level repository documentation

package/dist/index.d.mts

@@ -1,7 +1,7 @@
-import type { CodeRepository, PathResolver, RepositoryFactory, UserRepository } from "@o3co/auth-provider-core";
+import type { AdapterFactory, CodeRepository, PathResolver, UserRepository } from "@o3co/auth-provider-core";
 export declare const registerBuiltinAdapters: (factories: {
-    userFactory: RepositoryFactory<UserRepository>;
-    codeFactory: RepositoryFactory<CodeRepository>;
+    userFactory: AdapterFactory<UserRepository>;
+    codeFactory: AdapterFactory<CodeRepository>;
     pathResolver?: PathResolver;
 }) => void;
 export { HttpUserRepository } from "./repositories/HttpUserRepository.mjs";

package/dist/index.d.mts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.mts","sourceRoot":"","sources":["../src/index.mts"],"names":[],"mappings":"AAgBA,OAAO,KAAK,EACX,cAAc,EACd,YAAY,EACZ,iBAAiB,EACjB,cAAc,EACd,MAAM,0BAA0B,CAAC;AAIlC,eAAO,MAAM,uBAAuB,GAAI,WAAW;IAClD,WAAW,EAAE,iBAAiB,CAAC,cAAc,CAAC,CAAC;IAC/C,WAAW,EAAE,iBAAiB,CAAC,cAAc,CAAC,CAAC;IAC/C,YAAY,CAAC,EAAE,YAAY,CAAC;CAC5B,KAAG,IAsBH,CAAC;AAEF,OAAO,EAAE,kBAAkB,EAAE,MAAM,uCAAuC,CAAC;AAC3E,OAAO,EAAE,mBAAmB,EAAE,MAAM,wCAAwC,CAAC"}
+{"version":3,"file":"index.d.mts","sourceRoot":"","sources":["../src/index.mts"],"names":[],"mappings":"AAgBA,OAAO,KAAK,EACX,cAAc,EACd,cAAc,EACd,YAAY,EACZ,cAAc,EACd,MAAM,0BAA0B,CAAC;AAIlC,eAAO,MAAM,uBAAuB,GAAI,WAAW;IAClD,WAAW,EAAE,cAAc,CAAC,cAAc,CAAC,CAAC;IAC5C,WAAW,EAAE,cAAc,CAAC,cAAc,CAAC,CAAC;IAC5C,YAAY,CAAC,EAAE,YAAY,CAAC;CAC5B,KAAG,IAiCH,CAAC;AAEF,OAAO,EAAE,kBAAkB,EAAE,MAAM,uCAAuC,CAAC;AAC3E,OAAO,EAAE,mBAAmB,EAAE,MAAM,wCAAwC,CAAC"}

package/dist/index.mjs

@@ -27,7 +27,18 @@ export const registerBuiltinAdapters = (factories) => {
         return new HttpUserRepository({
             authenticateUrl: config.authenticateUrl,
             authenticateByTokenUrl: config.authenticateByTokenUrl,
-            timeout: typeof config.timeout === "number" ? config.timeout : 5000,
+            timeout: (() => {
+                if (typeof config.timeout === "number" && Number.isFinite(config.timeout)) {
+                    return config.timeout;
+                }
+                if (typeof config.timeout === "string") {
+                    const n = Number(config.timeout);
+                    if (Number.isFinite(n) && n > 0) {
+                        return n;
+                    }
+                }
+                return 5000;
+            })(),
         });
     });
     factories.codeFactory.register("redis", (config) => {

package/dist/repositories/__tests__/registerBuiltinAdapters.test.mjs

@@ -13,7 +13,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-import { RepositoryFactory } from "@o3co/auth-provider-core";
+import { AdapterFactoryError, createAdapterFactory, } from "@o3co/auth-provider-core";
 import { HttpResponse, http } from "msw";
 import { setupServer } from "msw/node";
 import { afterAll, afterEach, beforeAll, describe, expect, it } from "vitest";
@@ -38,20 +38,40 @@ afterEach(() => server.resetHandlers());
 afterAll(() => server.close());
 describe("registerBuiltinAdapters", () => {
     it("registers 'http' type in userFactory", async () => {
-        const userFactory = new RepositoryFactory("user");
-        const codeFactory = new RepositoryFactory("code");
+        const userFactory = createAdapterFactory("UserRepository");
+        const codeFactory = createAdapterFactory("CodeRepository");
         registerBuiltinAdapters({ userFactory, codeFactory });
-        await expect(userFactory.create({ type: "unknown" })).rejects.toThrow(/http/);
+        try {
+            await userFactory.create({ type: "unknown" });
+            throw new Error("should have thrown");
+        }
+        catch (err) {
+            expect(err).toBeInstanceOf(AdapterFactoryError);
+            const e = err;
+            expect(e.reason).toBe("unknown");
+            expect(e.kind).toBe("UserRepository");
+            expect(e.registered).toContain("http");
+        }
     });
     it("registers 'redis' type in codeFactory", async () => {
-        const userFactory = new RepositoryFactory("user");
-        const codeFactory = new RepositoryFactory("code");
+        const userFactory = createAdapterFactory("UserRepository");
+        const codeFactory = createAdapterFactory("CodeRepository");
         registerBuiltinAdapters({ userFactory, codeFactory });
-        await expect(codeFactory.create({ type: "unknown" })).rejects.toThrow(/redis/);
+        try {
+            await codeFactory.create({ type: "unknown" });
+            throw new Error("should have thrown");
+        }
+        catch (err) {
+            expect(err).toBeInstanceOf(AdapterFactoryError);
+            const e = err;
+            expect(e.reason).toBe("unknown");
+            expect(e.kind).toBe("CodeRepository");
+            expect(e.registered).toContain("redis");
+        }
     });
     it("http builder creates a working HttpUserRepository", async () => {
-        const userFactory = new RepositoryFactory("user");
-        const codeFactory = new RepositoryFactory("code");
+        const userFactory = createAdapterFactory("UserRepository");
+        const codeFactory = createAdapterFactory("CodeRepository");
         registerBuiltinAdapters({ userFactory, codeFactory });
         const repo = await userFactory.create({
             type: "http",
@@ -67,15 +87,31 @@ describe("registerBuiltinAdapters", () => {
         const invalid = await repo.authenticate("alice", "wrong");
         expect(invalid).toBeNull();
     });
+    it("http builder coerces string timeout to number (env-override path)", async () => {
+        const userFactory = createAdapterFactory("UserRepository");
+        const codeFactory = createAdapterFactory("CodeRepository");
+        registerBuiltinAdapters({ userFactory, codeFactory });
+        const repo = await userFactory.create({
+            type: "http",
+            authenticateUrl: `${BASE_URL}/auth`,
+            authenticateByTokenUrl: `${BASE_URL}/auth/token`,
+            timeout: "1234", // string, as HOCON env override produces
+        });
+        expect(repo).toBeDefined();
+        // HttpUserRepository doesn't expose timeout publicly; we verify it didn't throw
+        // and works via the MSW fixture. The coercion path is internal.
+        const user = await repo.authenticate("alice", "pass");
+        expect(user).not.toBeNull();
+    });
     it("http builder throws when authenticateUrl is missing", async () => {
-        const userFactory = new RepositoryFactory("user");
-        const codeFactory = new RepositoryFactory("code");
+        const userFactory = createAdapterFactory("UserRepository");
+        const codeFactory = createAdapterFactory("CodeRepository");
         registerBuiltinAdapters({ userFactory, codeFactory });
         await expect(userFactory.create({ type: "http", timeout: 5000 })).rejects.toThrow(/authenticateUrl/);
     });
     it("redis builder throws when endpointUri is missing", async () => {
-        const userFactory = new RepositoryFactory("user");
-        const codeFactory = new RepositoryFactory("code");
+        const userFactory = createAdapterFactory("UserRepository");
+        const codeFactory = createAdapterFactory("CodeRepository");
         registerBuiltinAdapters({ userFactory, codeFactory });
         await expect(codeFactory.create({ type: "redis" })).rejects.toThrow(/endpointUri/);
     });

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@o3co/auth-provider-foundation",
   "description": "Production adapters for auth.provider",
-  "version": "0.3.0",
+  "version": "0.4.0",
   "license": "Apache-2.0",
   "type": "module",
   "main": "./dist/index.mjs",
@@ -43,7 +43,7 @@
     "redis": "^5.10.0",
     "pino": "^10.3.1",
     "vitest": "^4.1.2",
-    "@o3co/auth-provider-core": "0.3.0"
+    "@o3co/auth-provider-core": "0.4.0"
   },
   "scripts": {
     "prebuild": "rimraf dist",