Commit 1561479
test(objectql): drop the dangling debug helper in save-meta-response-conformance (#6431)
The `LOG` helper declared at :119 referenced two names that exist nowhere in
the file: `appendFileSync` (never imported from `node:fs`) and `OUT` (never
defined). `LOG` itself was never called, so it never threw at runtime — it was
dead debug scaffolding left behind when its import and constant were dropped.
A debug helper that looks usable but raises `ReferenceError` the moment someone
uncomments a call is worse than no helper at all.
Also passes the required `packageId` argument to `registry.registerObject` at
:115, which was called with one argument against a `(schema, packageId, ...)`
signature. Both defects are invisible to every gate because
`packages/objectql/tsconfig.json` excludes `**/*.test.ts` (#4311), so `tsc`
never reads this file; vitest does not type-check and ESLint does no
cross-symbol resolution.
Fixes #5924
Claude-Session: https://claude.ai/code/session_019Q7oc7ASjh8yxyS3Yz78We
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>1 parent 2a0d65e commit 1561479
1 file changed
Lines changed: 1 addition & 3 deletions
Lines changed: 1 addition & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
112 | 112 | | |
113 | 113 | | |
114 | 114 | | |
115 | | - | |
| 115 | + | |
116 | 116 | | |
117 | 117 | | |
118 | 118 | | |
119 | | - | |
120 | | - | |
121 | 119 | | |
122 | 120 | | |
123 | 121 | | |
| |||
0 commit comments