mirror of
https://github.com/abhigyanpatwari/GitNexus.git
synced 2026-08-28 05:25:25 +00:00
1415bd5c2f
The module.registerHooks compat seam and the onnxruntime resolvers cited the old '>=22.0.0' floor as the reason their sub-22.15 fallback was reachable. With the floor now ^22.18.0 || >=24.11.0 (all >=22.15), every supported runtime exposes the API; the fallback stays as defensive handling for below-floor runtimes (engines is advisory, not engine-strict). Comments only - no behavior change. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
60 lines
2.5 KiB
TypeScript
60 lines
2.5 KiB
TypeScript
import { describe, it, expect, vi, afterEach } from 'vitest';
|
||
|
||
/**
|
||
* Tests for the #2372 `node:module` compat seam. `module.registerHooks` was
|
||
* added in Node 22.15 / 23.5. The engines floor is ^22.18.0 || >=24.11.0 (all
|
||
* >=22.15), but engines is advisory, so a below-floor 22.0–22.14 / 23.0–23.4
|
||
* runtime can still run, where the export is absent. `getRegisterHooks()` must
|
||
* hand back the real function when present and `undefined` when not — the value
|
||
* the resolver guards degrade on. `isPrefixRuntimeLoadable()` (exported from
|
||
* runtime-install.ts so CLI code never imports the compat module) is the
|
||
* boolean the truthful-messaging gates consume; it is tested here un-mocked,
|
||
* through the same `node:module` doMock seam, because a polarity bug in that
|
||
* thin wrapper would otherwise ship green (every consumer mocks it wholesale).
|
||
*
|
||
* Absence is simulated by passing an explicit `registerHooks: undefined` over
|
||
* the `importOriginal` spread — a bare omission would keep the real function.
|
||
*/
|
||
|
||
const COMPAT = '../../src/core/embeddings/node-module-compat.js';
|
||
const RUNTIME_INSTALL = '../../src/core/embeddings/runtime-install.js';
|
||
|
||
async function loadWithRegisterHooks(registerHooks: unknown) {
|
||
vi.resetModules();
|
||
vi.doMock('node:module', async (importOriginal) => {
|
||
const orig = await importOriginal<typeof import('node:module')>();
|
||
return { ...orig, registerHooks };
|
||
});
|
||
const compat = await import(COMPAT);
|
||
const runtimeInstall = await import(RUNTIME_INSTALL);
|
||
return { compat, runtimeInstall };
|
||
}
|
||
|
||
afterEach(() => {
|
||
vi.doUnmock('node:module');
|
||
});
|
||
|
||
describe('getRegisterHooks', () => {
|
||
it('returns the real function when node:module exposes registerHooks', async () => {
|
||
const fn = vi.fn();
|
||
const { compat } = await loadWithRegisterHooks(fn);
|
||
expect(compat.getRegisterHooks()).toBe(fn);
|
||
});
|
||
|
||
it('returns undefined when registerHooks is absent (Node < 22.15 / < 23.5)', async () => {
|
||
const { compat } = await loadWithRegisterHooks(undefined);
|
||
expect(compat.getRegisterHooks()).toBeUndefined();
|
||
});
|
||
});
|
||
|
||
describe('isPrefixRuntimeLoadable', () => {
|
||
it('is true when registerHooks is a function', async () => {
|
||
const { runtimeInstall } = await loadWithRegisterHooks(vi.fn());
|
||
expect(runtimeInstall.isPrefixRuntimeLoadable()).toBe(true);
|
||
});
|
||
|
||
it('is false when registerHooks is absent', async () => {
|
||
const { runtimeInstall } = await loadWithRegisterHooks(undefined);
|
||
expect(runtimeInstall.isPrefixRuntimeLoadable()).toBe(false);
|
||
});
|
||
});
|