Skip to content

Commit 1854943

Browse files
op7418claude
andcommitted
fix(lint): use block-level eslint-disable for multi-line require()
CI lint treats @typescript-eslint/no-require-imports as an error. eslint-disable-next-line only covers the first line of a multi-line destructuring; the closing require() on a later line is still flagged. Switch to block-level disable/enable to cover the full statement. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent e59f74e commit 1854943

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

src/__tests__/unit/provider-resolver.test.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1442,13 +1442,14 @@ describe('resolveAuxiliaryModel (live wrapper)', () => {
14421442
// The `source` field is the unambiguous discriminator. Asserting
14431443
// source !== 'main_small' catches the exact pre-fix regression.
14441444

1445-
// eslint-disable-next-line @typescript-eslint/no-require-imports
1445+
/* eslint-disable @typescript-eslint/no-require-imports */
14461446
const {
14471447
createProvider,
14481448
deleteProvider,
14491449
getSetting,
14501450
setSetting,
14511451
} = require('../../lib/db');
1452+
/* eslint-enable @typescript-eslint/no-require-imports */
14521453

14531454
const savedDefaultId = getSetting('default_provider_id') || '';
14541455

0 commit comments

Comments
 (0)