Ayuriel/openclaw
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

revert(test): restore upstream jiti test — Node 24.13 breakage is upstream issue

Browse Source 
The loader.git-path-regression test fails on Node 24.13+ due to require
being sealed as a read-only getter on ESM module objects. Both sync and
async Jiti paths are affected. This is an upstream issue that needs to be
fixed in the test infrastructure, not in our feature branches.
This commit is contained in:
Antonio 2026-03-20 10:30:21 -03:00
parent 41a5875715
commit 429364f01c
2 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/plugins/loader.git-path-regression.test.ts
View File

@ -77,9 +77,9 @@ export const copiedRuntimeMarker = {
...__testing.buildPluginLoaderJitiOptions({}),
tryNative: false,
});
// Node 24.13+ sealed `require` on ESM module objects, breaking Jiti's sync
// CJS path when tryNative is false. Use the async `.import()` helper instead.
await expect(async () => await withoutAlias.import(copiedChannelRuntime)).rejects.toThrow();
// The production loader uses sync Jiti evaluation, so this regression test
// should exercise the same seam instead of Jiti's async import helper.
expect(() => withoutAlias(copiedChannelRuntime)).toThrow();
const withAlias = createJiti(jitiBaseUrl, {
...__testing.buildPluginLoaderJitiOptions({
@ -87,7 +87,7 @@ export const copiedRuntimeMarker = {
}),
tryNative: false,
});
expect(await withAlias.import(copiedChannelRuntime)).toMatchObject({
expect(withAlias(copiedChannelRuntime)).toMatchObject({
copiedRuntimeMarker: {
PAIRING_APPROVED_MESSAGE: "paired",
resolveOutboundSendDep: expect.any(Function),

8
src/plugins/loader.test.ts
View File

@ -3595,9 +3595,9 @@ export const syntheticRuntimeMarker = {
...__testing.buildPluginLoaderJitiOptions({}),
tryNative: false,
});
// Node 24.13+ sealed `require` on ESM module objects, breaking Jiti's sync
// CJS path when tryNative is false. Use the async `.import()` helper instead.
await expect(async () => await withoutAlias.import(copiedChannelRuntime)).rejects.toThrow();
// The production loader uses sync Jiti evaluation, so this boundary should
// follow the same path instead of the async import helper.
expect(() => withoutAlias(copiedChannelRuntime)).toThrow();
const withAlias = createJiti(jitiBaseUrl, {
...__testing.buildPluginLoaderJitiOptions({
@ -3605,7 +3605,7 @@ export const syntheticRuntimeMarker = {
}),
tryNative: false,
});
expect(await withAlias.import(copiedChannelRuntime)).toMatchObject({
expect(withAlias(copiedChannelRuntime)).toMatchObject({
syntheticRuntimeMarker: {
resolveOutboundSendDep: expect.any(Function),
},