@nodefony/framework 10.0.0-alpha.9 → 10.0.0-beta.1

package/dist/nodefony/src/KernelAdminApi.js

@@ -2,7 +2,7 @@ import { CORE_PACKAGE, checkOutdated, countModuleDocs, listModuleDocs, listModul
 import { getResolvedPath, navigateSchemaNode, nodeFlags, notEditableReason, recipeFor, validateLeafValue } from "./configMutation.js";
 import { createRequire } from "node:module";
 import { join } from "node:path";
-import { GitService, Syslog, aggregateProvenance, applyResolvedPath, collectDevStatus, computeConfigProvenance, defaultAppConfig, extractJsonSchemaDefaults, extractMarkdownSection, flattenConfigSchema, getActiveLogDriver, listLogDrivers, outlineMarkdown, parseNfEnvOverrides, parseStoreManifest, readResolvedPath } from "nodefony";
+import { GitService, Syslog, aggregateProvenance, collectDevStatus, computeConfigProvenance, defaultAppConfig, extractJsonSchemaDefaults, extractMarkdownSection, flattenConfigSchema, getActiveLogDriver, listLogDrivers, outlineMarkdown, parseNfEnvOverrides, parseStoreManifest, readResolvedPath, withResolvedPath } from "nodefony";
 import { randomUUID } from "node:crypto";
 import { existsSync, readFileSync } from "node:fs";
 //#region nodefony/src/KernelAdminApi.ts
@@ -153,9 +153,8 @@ function flattenPaths(value, prefix, out) {
 function attributeOverrideSources(entries) {
 	const byTarget = /* @__PURE__ */ new Map();
 	for (const e of entries) for (const key of Object.keys(e.config)) {
-		const m = /^module-(.+)$/i.exec(key);
-		if (!m) continue;
-		const targetSeg = m[1].toLowerCase();
+		const targetSeg = /^module-(.+)$/i.exec(key)?.[1]?.toLowerCase();
+		if (targetSeg === void 0) continue;
 		const paths = /* @__PURE__ */ new Set();
 		flattenPaths(e.config[key], "", paths);
 		const arr = byTarget.get(targetSeg);
@@ -214,7 +213,7 @@ function computeConfigEntry(key, mod, runtimePaths) {
 		const envPaths = new Set(parseNfEnvOverrides(process.env).filter((o) => o.moduleSeg === seg).map((o) => o.path.join(".")));
 		provenance = computeConfigProvenance(opts, extractJsonSchemaDefaults(schema), envPaths);
 	}
-	if (provenance && runtimePaths && runtimePaths.size) {
+	if (provenance && runtimePaths?.size) {
 		for (const k of Object.keys(provenance)) if (runtimePaths.has(k.toLowerCase())) provenance[k] = "runtime";
 	}
 	return {
@@ -259,12 +258,13 @@ function auditConfigChange(kernel, request, moduleKey, path, before, after) {
 	});
 }
 /**
-* Journalise un changement de debug runtime (catégorie `log`). Même découplage
+* Journalise un changement de debug runtime (catégorie `config` : une verbosité
+* est une configuration runtime). Même découplage
 * que {@link auditConfigChange} (résolution par nom, no-op si pas d'audit).
 */
 function auditLogLevelChange(kernel, request, module, action, level, ttlMs) {
 	(kernel.container?.get("auditService"))?.record?.({
-		category: "log",
+		category: "config",
 		action: `log.debug.${action}`,
 		outcome: "success",
 		actor: actorLabel(request.user),
@@ -352,7 +352,7 @@ function createKernelAdminApi(kernel) {
 		if (!mod) return null;
 		return {
 			path: mod.path,
-			pkg: mod.getModuleName?.() ?? key
+			pkg: mod.getModuleName() ?? key
 		};
 	};
 	const resolvePackageDir = (name) => {
@@ -401,7 +401,7 @@ function createKernelAdminApi(kernel) {
 	const runtimeEdited = /* @__PURE__ */ new Map();
 	const devGuard = () => kernel.environment === "development" || Boolean(kernel.debug);
 	const repoRoot = process.cwd();
-	const relPath = (p) => p && p.startsWith(repoRoot) ? p.slice(repoRoot.length).replace(/^[/\\]+/, "") || "." : p ?? null;
+	const relPath = (p) => p?.startsWith(repoRoot) ? p.slice(repoRoot.length).replace(/^[/\\]+/, "") || "." : p ?? null;
 	const redactInfraUrl = (url) => url.replace(/\/\/[^/@]*@/, "//***@");
 	const buildConfigEntries = () => {
 		const modules = kernel.getModules();
@@ -581,16 +581,13 @@ function createKernelAdminApi(kernel) {
 					isApp: false,
 					path: relPath(core.path)
 				}];
-				for (const name of Object.keys(modules)) {
-					const mod = modules[name];
-					list.push({
-						key: name,
-						name: mod.getModuleName?.() ?? name,
-						version: mod.getModuleVersion?.() ?? null,
-						isApp: mod.isApp ?? false,
-						path: relPath(mod.path)
-					});
-				}
+				for (const [name, mod] of Object.entries(modules)) list.push({
+					key: name,
+					name: mod.getModuleName() ?? name,
+					version: mod.getModuleVersion() ?? null,
+					isApp: mod.isApp,
+					path: relPath(mod.path)
+				});
 				return list;
 			}
 		},
@@ -600,14 +597,11 @@ function createKernelAdminApi(kernel) {
 			handler: () => {
 				const modules = kernel.getModules();
 				const services = [];
-				for (const name of Object.keys(modules)) {
-					const mod = modules[name];
-					for (const service of mod.getServiceNames?.() ?? []) services.push({
-						name: service,
-						module: name,
-						class: mod.get(service)?.constructor?.name ?? null
-					});
-				}
+				for (const [name, mod] of Object.entries(modules)) for (const service of mod.getServiceNames()) services.push({
+					name: service,
+					module: name,
+					class: mod.get(service)?.constructor?.name ?? null
+				});
 				services.sort((a, b) => `${a.module}.${a.name}`.localeCompare(`${b.module}.${b.name}`));
 				return services;
 			}
@@ -627,8 +621,7 @@ function createKernelAdminApi(kernel) {
 				const entries = buildConfigEntries();
 				const matches = (name, key) => name.toLowerCase() === filter || key.toLowerCase() === filter;
 				if (filter !== null) {
-					const entry = entries.find((e) => matches(e.name, e.key));
-					if (!entry || entry.configSchema == null) {
+					if (entries.find((e) => matches(e.name, e.key))?.configSchema == null) {
 						const modules = kernel.getModules();
 						const nameOf = (k) => modules[k].getModuleName?.() ?? k;
 						const loadedKey = Object.keys(modules).find((k) => matches(nameOf(k), k));
@@ -658,7 +651,7 @@ function createKernelAdminApi(kernel) {
 					if (filter !== null && entry.name.toLowerCase() !== filter && entry.key.toLowerCase() !== filter) continue;
 					for (const leaf of flattenConfigSchema(entry.configSchema)) {
 						const path = leaf.key.split(".");
-						const note = SECRET_KEY.test(path[path.length - 1]) && !leaf.note.includes("secret") ? [leaf.note, "secret"].filter(Boolean).join(", ") : leaf.note;
+						const note = SECRET_KEY.test(path.at(-1) ?? "") && !leaf.note.includes("secret") ? [leaf.note, "secret"].filter(Boolean).join(", ") : leaf.note;
 						rows.push({
 							key: leaf.key,
 							module: entry.name,
@@ -750,7 +743,7 @@ function createKernelAdminApi(kernel) {
 						reason: "prod_immutable"
 					}
 				};
-				const key = request.params.module;
+				const key = request.params.module ?? "";
 				const mod = kernel.getModules()[key];
 				if (!mod) return {
 					status: 404,
@@ -811,17 +804,24 @@ function createKernelAdminApi(kernel) {
 				};
 				const opts = mod.options ?? {};
 				const before = getResolvedPath(opts, segments);
-				if (!applyResolvedPath(opts, segments, value)) return {
+				const nextOpts = withResolvedPath(opts, segments, value);
+				if (nextOpts === null) return {
 					status: 422,
 					body: {
 						error: "Path not present in module config",
 						path: body.path
 					}
 				};
+				const moduleInstance = kernel.getModules()[key];
+				if (moduleInstance) kernel.replaceModuleOptions(moduleInstance, nextOpts);
 				const svcMod = mod;
 				for (const sname of svcMod.getServiceNames?.() ?? []) {
 					const svc = svcMod.get?.(sname);
-					if (svc?.options && svc.options !== opts) applyResolvedPath(svc.options, segments, value);
+					if (svc?.options === opts) svc.options = nextOpts;
+					else if (svc?.options) {
+						const nextSvc = withResolvedPath(svc.options, segments, value);
+						if (nextSvc !== null) svc.options = nextSvc;
+					}
 					svc?.onConfigChanged?.(segments);
 				}
 				let edited = runtimeEdited.get(key);
@@ -914,7 +914,7 @@ function createKernelAdminApi(kernel) {
 			path: "module/{name}",
 			summary: "Detail of one module by key (http, framework, … or core)",
 			handler: async (request) => {
-				const key = request.params.name;
+				const key = request.params.name ?? "";
 				if (key === CORE_KEY) {
 					const core = await readCoreInfo();
 					return {
@@ -949,10 +949,10 @@ function createKernelAdminApi(kernel) {
 				return {
 					key,
 					name: cfg.name,
-					version: mod.getModuleVersion?.() ?? null,
+					version: mod.getModuleVersion() ?? null,
 					isApp: cfg.isApp,
 					path: relPath(mod.path),
-					dependencies: mod.getDependencies?.() ?? [],
+					dependencies: mod.getDependencies(),
 					services,
 					config: cfg.config,
 					configSchema: cfg.configSchema,
@@ -969,7 +969,7 @@ function createKernelAdminApi(kernel) {
 			path: "module/{name}/dependencies",
 			summary: "Module dependencies with installed versions",
 			handler: async (request) => {
-				const target = resolveTarget(request.params.name);
+				const target = resolveTarget(request.params.name ?? "");
 				if (!target) return {
 					status: 404,
 					body: {
@@ -988,7 +988,7 @@ function createKernelAdminApi(kernel) {
 			path: "module/{name}/dependencies/outdated",
 			summary: "Check external dependencies for updates (npm registry)",
 			handler: async (request) => {
-				const target = resolveTarget(request.params.name);
+				const target = resolveTarget(request.params.name ?? "");
 				if (!target) return {
 					status: 404,
 					body: {
@@ -1008,7 +1008,7 @@ function createKernelAdminApi(kernel) {
 			path: "module/{name}/docs",
 			summary: "Documentation index of one module (markdown in <module>/docs)",
 			handler: async (request) => {
-				const key = request.params.name;
+				const key = request.params.name ?? "";
 				const dir = resolveDocDir(key);
 				if (!dir) return {
 					status: 404,
@@ -1028,7 +1028,7 @@ function createKernelAdminApi(kernel) {
 			path: "module/{name}/docs/{slug}",
 			summary: "Raw markdown of one module doc by slug",
 			handler: async (request) => {
-				const key = request.params.name;
+				const key = request.params.name ?? "";
 				const dir = resolveDocDir(key);
 				if (!dir) return {
 					status: 404,
@@ -1038,7 +1038,7 @@ function createKernelAdminApi(kernel) {
 						available: docKeys()
 					}
 				};
-				const doc = await readModuleDoc(dir, request.params.slug);
+				const doc = await readModuleDoc(dir, request.params.slug ?? "");
 				if (!doc) return {
 					status: 404,
 					body: {
@@ -1067,7 +1067,7 @@ function createKernelAdminApi(kernel) {
 						markdown: section.markdown
 					};
 				}
-				if (request.query.outline !== void 0) {
+				if (Object.hasOwn(request.query, "outline")) {
 					const { markdown, ...rest } = doc;
 					return {
 						...rest,
@@ -1118,7 +1118,7 @@ function createKernelAdminApi(kernel) {
 			path: "module/{name}/symbol/{symbol}",
 			summary: "Declaration (signature + TSDoc) of one exported symbol",
 			handler: async (request) => {
-				const key = request.params.name;
+				const key = request.params.name ?? "";
 				const modulePath = resolveTarget(key)?.path ?? resolvePackageDir(key);
 				if (!modulePath) return {
 					status: 404,
@@ -1128,7 +1128,7 @@ function createKernelAdminApi(kernel) {
 						available: docKeys()
 					}
 				};
-				const symbol = request.params.symbol;
+				const symbol = request.params.symbol ?? "";
 				const found = await readSymbolDeclaration(modulePath, symbol);
 				if (!found) return {
 					status: 404,
@@ -1150,7 +1150,7 @@ function createKernelAdminApi(kernel) {
 			path: "module/{name}/symbols",
 			summary: "Exported TS symbols + TSDoc descriptions (.ai/symbols.json)",
 			handler: async (request) => {
-				const key = request.params.name;
+				const key = request.params.name ?? "";
 				const target = resolveTarget(key);
 				if (!target) return {
 					status: 404,
@@ -1171,7 +1171,7 @@ function createKernelAdminApi(kernel) {
 			path: "module/{name}/coverage",
 			summary: "Latest test coverage report (vitest json-summary)",
 			handler: async (request) => {
-				const key = request.params.name;
+				const key = request.params.name ?? "";
 				const target = resolveTarget(key);
 				if (!target) return {
 					status: 404,
@@ -1191,7 +1191,7 @@ function createKernelAdminApi(kernel) {
 			path: "module/{name}/tests",
 			summary: "List test files of one module",
 			handler: async (request) => {
-				const key = request.params.name;
+				const key = request.params.name ?? "";
 				const target = resolveTarget(key);
 				if (!target) return {
 					status: 404,
@@ -1218,7 +1218,7 @@ function createKernelAdminApi(kernel) {
 					status: 403,
 					body: { error: "Test runner disabled outside development" }
 				};
-				const key = request.params.name;
+				const key = request.params.name ?? "";
 				const target = resolveTarget(key);
 				if (!target) return {
 					status: 404,
@@ -1246,7 +1246,7 @@ function createKernelAdminApi(kernel) {
 					startedAt: Date.now()
 				});
 				if (testJobs.size > 16) {
-					const oldest = [...testJobs.entries()].sort((a, b) => a[1].startedAt - b[1].startedAt)[0];
+					const oldest = [...testJobs.entries()].sort((a, b) => a[1].startedAt - b[1].startedAt).at(0);
 					if (oldest) testJobs.delete(oldest[0]);
 				}
 				runModuleTests(target.path, file).then((result) => testJobs.set(jobId, {

package/dist/nodefony/src/PlaygroundAdminApi.js

@@ -4,8 +4,8 @@ import AdminApiController from "../controller/AdminApiController.js";
 //#region nodefony/src/PlaygroundAdminApi.ts
 /** Normalise `requirements.methods` (string | string[]) → tableau majuscule. */
 function methodsOf(route) {
-	const m = route.requirements?.methods ?? route.method;
-	if (Array.isArray(m)) return m.map((x) => String(x).toUpperCase());
+	const m = route.requirements.methods ?? route.method;
+	if (Array.isArray(m)) return m.map((x) => x.toUpperCase());
 	if (typeof m === "string") return m.split(",").map((s) => s.trim().toUpperCase()).filter(Boolean);
 	return ["ANY"];
 }
@@ -44,7 +44,7 @@ function serializeAction(route) {
 			csrfProtect: meta.csrfProtect,
 			csrfExempt: meta.csrfExempt,
 			session: meta.sessionIntent,
-			bypassFirewall: route.bypassFirewall === true
+			bypassFirewall: route.bypassFirewall
 		}
 	};
 }

package/dist/nodefony/src/RedisIdempotencyStore.js

@@ -173,7 +173,7 @@ var RedisIdempotencyStore = class {
 	}
 	/** Préfixe effectif des clés, cloisonné par application (mémoïsé). */
 	#prefix() {
-		if (this.#prefixCache === null) this.#prefixCache = this.#resolvePrefix();
+		this.#prefixCache ??= this.#resolvePrefix();
 		return this.#prefixCache;
 	}
 	#key(key) {
@@ -227,7 +227,7 @@ var RedisIdempotencyStore = class {
 		if (!client) return;
 		const k = this.#key(key);
 		const existing = this.#parse(await client.get(k));
-		if (existing === null || existing.s !== "if") {
+		if (existing?.s !== "if") {
 			this.#dec();
 			return;
 		}