Skip to content

Commit 3b2edfc

Browse files
committed
fix(formula): use the static import in the #6133 test (NodeNext extension)
`await import('./cel-engine')` needs an explicit `.js` extension under `moduleResolution: nodenext`, so the new test added one TS2835 to the package's TEST_DEBT ledger (17 -> 18) and tripped `check:type-check-debt`. The ledger is a shrink-only ratchet (#5278), and the error is trivially removable rather than irreducible: `parseCelToAst` is exported from the module the file already imports statically. Re-measured back to 17, the recorded value, with zero errors attributable to this PR's files. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_019Q7oc7ASjh8yxyS3Yz78We
1 parent fb7cabf commit 3b2edfc

1 file changed

Lines changed: 2 additions & 3 deletions

File tree

packages/formula/src/cel-error-classification.test.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515
*/
1616
import { describe, expect, it } from 'vitest';
1717

18-
import { celEngine } from './cel-engine';
18+
import { celEngine, parseCelToAst } from './cel-engine';
1919
import type { Expression } from '@objectstack/spec';
2020

2121
const cel = (source: string): Expression => ({ dialect: 'cel', source });
@@ -157,12 +157,11 @@ describe('celEngine error classification (#6133)', () => {
157157
});
158158
});
159159

160-
it('parseCelToAst never reaches the classifier — it returns null (#4812)', async () => {
160+
it('parseCelToAst never reaches the classifier — it returns null (#4812)', () => {
161161
// Recorded because the classification fix has a natural blast radius
162162
// question: does the #4812 canonical parse entry re-emit `kind`? It does
163163
// not — it swallows the fault and answers `null`, leaving the verdict to
164164
// compile()/validateExpression. Nothing here changes for it.
165-
const { parseCelToAst } = await import('./cel-engine');
166165
expect(parseCelToAst('((record.a)')).toBeNull();
167166
expect(parseCelToAst('record.a > 1')).not.toBeNull();
168167
});

0 commit comments

Comments
 (0)