Commita288c96

authored

Merge pull request#14 from code-found/bugfix/context-null-error

Bugfix/context null error

2 parents9478090 +46694bc commita288c96Copy full SHA for a288c96

File tree

+18

-1

lines changed

+18

-1

lines changed

Lines changed: 8 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,8 @@`
	`1`	`+---`
	`2`	`+"rts.js":patch`
	`3`	`+---`
	`4`	`+ Fixed`
	`5`	`+- resolver: handle null resolve context to avoid crash on Node 18`
	`6`	+- Guard destructuring with`context \|\| {}` to prevent runtime error when`context` is null.
	`7`	+- Affects:`src/resolver/index.ts`
	`8`	`+- Environment: reproducible on macOS with Node 18; Windows with Node 23 unaffected.`

Lines changed: 10 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -226,7 +226,16 @@ export class ModuleResolver extends ModuleTransformer {`
`226`	`226`	`specifier:string,`
`227`	`227`	`context:ResolveHookContext,`
`228`	`228`	`):{url?:string}{`
`229`		`-const{ parentURL=""}=context;`
	`229`	`+/**`
	`230`	`+ * bugfix: fix the bug that context is null on node 18.`
	`231`	`+ *`
	`232`	`+ * on node 18,context will be null.`
	`233`	`+ * not very clear know if only node 18 is like this.`
	`234`	`+ * on my windows with node 23 it sis ok.`
	`235`	`+ * but on mac with node 18 the bug come out.`
	`236`	`+ *`
	`237`	`+ */`
	`238`	`+const{ parentURL=""}=context\|\|{};`
`230`	`239`	constcacheKey=`${specifier}:${parentURL}`;
`231`	`240`	`leturl=this.cache.get(cacheKey);`
`232`	`241`

Comments

(0)