stonyx 0.2.3-beta.77 → 0.2.3-beta.78

package/dist/modules.js

@@ -5,7 +5,7 @@
 import { readFile } from '@stonyx/utils/file';
 import { kebabCaseToCamelCase } from '@stonyx/utils/string';
 import { mergeObject } from '@stonyx/utils/object';
-import { importConfig } from './util/import-config.js';
+import { importModuleConfig } from './util/import-module-config.js';
 const modulePromises = {};
 // Configure module-specific logging
 function configureLog(chronicle, module, config) {
@@ -64,7 +64,10 @@ export default async function loadModules(config, rootPath, chronicle) {
         }
         try {
             // Load & Configure Async Modules
-            const moduleConfig = await importConfig(`${rootPath}/node_modules/${moduleName}/config/environment`);
+            // MODULE-OWNED: `.js` only, always. Routing this to the app-owned
+            // `importConfig` re-ships abofs/stonyx-orm#118 — Node refuses to
+            // type-strip inside `node_modules`. See abofs/stonyx#90.
+            const moduleConfig = await importModuleConfig(`${rootPath}/node_modules/${moduleName}/config/environment`);
             const module = kebabCaseToCamelCase(moduleName.split('/').pop() ?? moduleName);
             const userConfig = config[module] || {};
             const finalConfig = mergeObject(moduleConfig, userConfig);
@@ -77,7 +80,9 @@ export default async function loadModules(config, rootPath, chronicle) {
         }
         catch (error) {
             console.error(error);
-            throw new Error(`Stonyx modules with async loading must have a config/environment.js file with default configurations. Module "${moduleName}" failed to load.`);
+            throw new Error('Stonyx modules with async loading must ship a config/environment.js with default configurations. ' +
+                'Module configs resolve as .js only — never .ts, because Node refuses to type-strip inside node_modules. ' +
+                `Module "${moduleName}" failed to load.`);
         }
     }
     // Wait until all modules are initialized

package/dist/util/extension-resolution.d.ts

@@ -0,0 +1,22 @@
+/**
+ * Shared constants for APP-OWNED `{ts,js}` resolution.
+ *
+ * Two resolvers pick an extension for a file in the consuming project's own
+ * tree — `resolveEntryPoint` (`./resolve-entry-point.ts`, since
+ * abofs/stonyx#67) and `importConfig` (`./import-config.ts`, since
+ * abofs/stonyx#90). They must agree on the candidate list, on the preference
+ * order, and on the wording of the dual-extension warning.
+ *
+ * They previously agreed by copy. Each string was pinned by its own suite, so
+ * editing one file left BOTH suites green while the two messages diverged —
+ * which is why this lives here and not at the top of either consumer
+ * (`docs/conventions/index.md` § Universal Rules: constants shared across
+ * files go in a dedicated constants file).
+ *
+ * MODULE-OWNED configs are not in scope: `./import-module-config.ts` resolves
+ * `.js` only, forever, and deliberately shares nothing with this file.
+ */
+/** Candidate extensions in preference order — `.ts` wins over `.js`. */
+export declare const EXTENSIONS: readonly ["ts", "js"];
+/** The warning both app-owned resolvers emit when a `.ts`/`.js` pair exists. */
+export declare function dualExtensionWarning(basePath: string): string;

package/dist/util/extension-resolution.js

@@ -0,0 +1,24 @@
+/**
+ * Shared constants for APP-OWNED `{ts,js}` resolution.
+ *
+ * Two resolvers pick an extension for a file in the consuming project's own
+ * tree — `resolveEntryPoint` (`./resolve-entry-point.ts`, since
+ * abofs/stonyx#67) and `importConfig` (`./import-config.ts`, since
+ * abofs/stonyx#90). They must agree on the candidate list, on the preference
+ * order, and on the wording of the dual-extension warning.
+ *
+ * They previously agreed by copy. Each string was pinned by its own suite, so
+ * editing one file left BOTH suites green while the two messages diverged —
+ * which is why this lives here and not at the top of either consumer
+ * (`docs/conventions/index.md` § Universal Rules: constants shared across
+ * files go in a dedicated constants file).
+ *
+ * MODULE-OWNED configs are not in scope: `./import-module-config.ts` resolves
+ * `.js` only, forever, and deliberately shares nothing with this file.
+ */
+/** Candidate extensions in preference order — `.ts` wins over `.js`. */
+export const EXTENSIONS = ['ts', 'js'];
+/** The warning both app-owned resolvers emit when a `.ts`/`.js` pair exists. */
+export function dualExtensionWarning(basePath) {
+    return `Warning: both ${basePath}.ts and ${basePath}.js exist. Using .ts — delete the .js to silence this warning (it is likely a stale compiled artifact or postinstall stub).`;
+}

package/dist/util/import-config.d.ts

@@ -1 +1,22 @@
+/**
+ * Resolve an APP-OWNED config — `<cwd>/config/environment`,
+ * `<rootPath>/test/config/environment` — as `{ts,js}`, preferring `.ts`.
+ * Mirrors `src/util/resolve-entry-point.ts`, which has resolved app-owned
+ * entry points this way since abofs/stonyx#67 — the candidate list and the
+ * dual-extension warning are shared through `./extension-resolution.ts` so
+ * the two cannot drift.
+ *
+ * These paths live in the consuming project's own tree, never inside
+ * `node_modules`, so they load under the app's own loader and Node's type
+ * stripping applies to them.
+ *
+ * MODULE-OWNED configs are NOT this function's business: see
+ * `importModuleConfig` in `./import-module-config.ts`, which resolves `.js`
+ * only. Routing `src/modules.ts` here re-ships abofs/stonyx-orm#118 — Node
+ * refuses to type-strip inside `node_modules`, so a `.ts` module config
+ * crashes every consumer at parse time. See abofs/stonyx#90.
+ *
+ * The "Config not found:" prefix is load-bearing: `src/main.ts` matches on it
+ * to tell an absent optional config from a real import failure.
+ */
 export declare function importConfig<T = unknown>(basePath: string): Promise<T>;

package/dist/util/import-config.js

@@ -1,10 +1,36 @@
 import { existsSync } from 'fs';
 import { pathToFileURL } from 'url';
+import { EXTENSIONS, dualExtensionWarning } from './extension-resolution.js';
+/**
+ * Resolve an APP-OWNED config — `<cwd>/config/environment`,
+ * `<rootPath>/test/config/environment` — as `{ts,js}`, preferring `.ts`.
+ * Mirrors `src/util/resolve-entry-point.ts`, which has resolved app-owned
+ * entry points this way since abofs/stonyx#67 — the candidate list and the
+ * dual-extension warning are shared through `./extension-resolution.ts` so
+ * the two cannot drift.
+ *
+ * These paths live in the consuming project's own tree, never inside
+ * `node_modules`, so they load under the app's own loader and Node's type
+ * stripping applies to them.
+ *
+ * MODULE-OWNED configs are NOT this function's business: see
+ * `importModuleConfig` in `./import-module-config.ts`, which resolves `.js`
+ * only. Routing `src/modules.ts` here re-ships abofs/stonyx-orm#118 — Node
+ * refuses to type-strip inside `node_modules`, so a `.ts` module config
+ * crashes every consumer at parse time. See abofs/stonyx#90.
+ *
+ * The "Config not found:" prefix is load-bearing: `src/main.ts` matches on it
+ * to tell an absent optional config from a real import failure.
+ */
 export async function importConfig(basePath) {
-    const path = `${basePath}.js`;
-    if (!existsSync(path)) {
-        throw new Error(`Config not found: ${path}`);
+    const matches = EXTENSIONS.filter(ext => existsSync(`${basePath}.${ext}`));
+    if (matches.length === 0) {
+        throw new Error(`Config not found: ${basePath}.{ts,js}`);
     }
+    if (matches.length > 1) {
+        console.warn(dualExtensionWarning(basePath));
+    }
+    const path = `${basePath}.${matches[0]}`;
     const mod = await import(pathToFileURL(path).href);
     return mod.default;
 }

package/dist/util/import-module-config.d.ts

@@ -0,0 +1,16 @@
+/**
+ * Resolve a MODULE-OWNED config — `node_modules/<mod>/config/environment` — as
+ * `.js` ONLY. There is no `.ts` fallback and there must never be one.
+ *
+ * Node's type-strip loader refuses to process `.ts` files inside
+ * `node_modules`, so a `@stonyx/*` module that ships `config/environment.ts`
+ * crashes every consumer at parse time. That is abofs/stonyx-orm#118, the P0
+ * `4c80c87` was written to prevent, and this function preserves `4c80c87`'s
+ * behaviour byte-for-byte — including the exact `Config not found: <base>.js`
+ * message, which `test/unit/import-config-test.ts` pins as a regression guard.
+ *
+ * App-owned configs resolve `{ts,js}` instead: see `importConfig` in
+ * `./import-config.ts`. The boundary is OWNERSHIP, not extension — see
+ * abofs/stonyx#90 and `docs/conventions/framework-modules.md`.
+ */
+export declare function importModuleConfig<T = unknown>(basePath: string): Promise<T>;

package/dist/util/import-module-config.js

@@ -0,0 +1,25 @@
+import { existsSync } from 'fs';
+import { pathToFileURL } from 'url';
+/**
+ * Resolve a MODULE-OWNED config — `node_modules/<mod>/config/environment` — as
+ * `.js` ONLY. There is no `.ts` fallback and there must never be one.
+ *
+ * Node's type-strip loader refuses to process `.ts` files inside
+ * `node_modules`, so a `@stonyx/*` module that ships `config/environment.ts`
+ * crashes every consumer at parse time. That is abofs/stonyx-orm#118, the P0
+ * `4c80c87` was written to prevent, and this function preserves `4c80c87`'s
+ * behaviour byte-for-byte — including the exact `Config not found: <base>.js`
+ * message, which `test/unit/import-config-test.ts` pins as a regression guard.
+ *
+ * App-owned configs resolve `{ts,js}` instead: see `importConfig` in
+ * `./import-config.ts`. The boundary is OWNERSHIP, not extension — see
+ * abofs/stonyx#90 and `docs/conventions/framework-modules.md`.
+ */
+export async function importModuleConfig(basePath) {
+    const path = `${basePath}.js`;
+    if (!existsSync(path)) {
+        throw new Error(`Config not found: ${path}`);
+    }
+    const mod = await import(pathToFileURL(path).href);
+    return mod.default;
+}

package/dist/util/resolve-entry-point.js

@@ -1,13 +1,13 @@
 import { existsSync } from 'fs';
 import { resolve } from 'path';
-const EXTENSIONS = ['ts', 'js'];
+import { EXTENSIONS, dualExtensionWarning } from './extension-resolution.js';
 export function resolveEntryPoint(basePath) {
     const matches = EXTENSIONS.filter(ext => existsSync(resolve(`${basePath}.${ext}`)));
     if (matches.length === 0) {
         throw new Error(`Entry point not found: ${basePath}.{ts,js}`);
     }
     if (matches.length > 1) {
-        console.warn(`Warning: both ${basePath}.ts and ${basePath}.js exist. Using .ts — delete the .js to silence this warning (it is likely a stale compiled artifact or postinstall stub).`);
+        console.warn(dualExtensionWarning(basePath));
     }
     const ext = matches[0];
     return resolve(`${basePath}.${ext}`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "stonyx",
-  "version": "0.2.3-beta.77",
+  "version": "0.2.3-beta.78",
   "description": "Base stonyx framework module",
   "main": "dist/main.js",
   "types": "dist/main.d.ts",