fix(factory): handle undefined id in file name

Akryum · Akryum · commit ec31bdde639c · 2024-09-02T17:10:10.000+02:00
diff --git a/packages/core/src/factory/storage.ts b/packages/core/src/factory/storage.ts
@@ -15,7 +15,7 @@ import { serializeFactory } from './serialize.js'
 export function getFactoryFilename(mq: MoquerieInstance, resourceName: string, id: string, name: string, location: DBLocation) {
   // eslint-disable-next-line regexp/no-super-linear-backtracking
   const [, idPart] = /.+?(@@.+)?$/.exec(id) ?? []
-  return `${resourceName}/${name}${location === 'local' ? idPart : ''}.${projectHasTypescript(mq) ? 'ts' : 'js'}`
+  return `${resourceName}/${name}${location === 'local' ? idPart ?? '' : ''}.${projectHasTypescript(mq) ? 'ts' : 'js'}`
 }
 
 let storage: MergedStorage<ResourceFactory>

Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,7 @@ import { serializeFactory } from './serialize.js'`
`15`	`15`	`export function getFactoryFilename(mq: MoquerieInstance, resourceName: string, id: string, name: string, location: DBLocation) {`
`16`	`16`	`// eslint-disable-next-line regexp/no-super-linear-backtracking`
`17`	`17`	`const [, idPart] = /.+?(@@.+)?$/.exec(id) ?? []`
`18`		- return `${resourceName}/${name}${location === 'local' ? idPart : ''}.${projectHasTypescript(mq) ? 'ts' : 'js'}`
	`18`	+ return `${resourceName}/${name}${location === 'local' ? idPart ?? '' : ''}.${projectHasTypescript(mq) ? 'ts' : 'js'}`
`19`	`19`	`}`
`20`	`20`
`21`	`21`	`let storage: MergedStorage<ResourceFactory>`