ReviewOS

also looking at this

stacks/bun-query-builder

fix(orm): make createModel models findable, so relations read the declared table

#1104
Merged glennmichael123 wants to merge fix/1093-relation-declared-table into main
3 files +244 -3
packages/bun-query-builder/src/model.tsmodified+6-1
Changes to packages/bun-query-builder/src/model.ts
@@ -41,7 +41,7 @@
4141 */
4242
4343import { createBrowserModel, isBrowser, type BrowserModelDefinition } from './browser'
44import { createModel, type ModelDefinition as OrmModelDefinition } from './orm'
44import { clearLocalModels, createModel, type ModelDefinition as OrmModelDefinition } from './orm'
4545
4646// Re-export the browser model types for convenience
4747export type { BrowserModelDefinition as ModelDefinition }
@@ -119,9 +119,14 @@ export function hasModel(name: string): boolean {
119119
120120/**
121121 * Clear all registered models. Primarily useful for testing.
122 *
123 * Also drops the models `createModel` registered internally, and the memoized
124 * relation resolutions derived from them. Leaving either behind lets a stale
125 * resolution from one test file decide a relation's table in the next.
122126 */
123127export function clearModelRegistry(): void {
124128 modelRegistry.clear()
129 clearLocalModels()
125130}
126131
127132/**