From 8f34131f01258dae6ba980336089f18c7429c804 Mon Sep 17 00:00:00 2001 From: Tianyi Cui <53024+tianyicui@users.noreply.github.com> Date: Wed, 29 Jul 2026 23:35:10 +0800 Subject: [PATCH] fix: restore unused suppression reports --- .oxlintrc.json | 1 + scripts/oxlint-contract.spec.ts | 27 +++++++++++++++++++++++++++ 2 files changed, 28 insertions(+) diff --git a/.oxlintrc.json b/.oxlintrc.json index 048ba7f0de..1dee47f9ee 100644 --- a/.oxlintrc.json +++ b/.oxlintrc.json @@ -5,6 +5,7 @@ "correctness": "off" }, "options": { + "reportUnusedDisableDirectives": "warn", "typeAware": true }, "env": { diff --git a/scripts/oxlint-contract.spec.ts b/scripts/oxlint-contract.spec.ts index 530e5bf29f..ce91e771a8 100644 --- a/scripts/oxlint-contract.spec.ts +++ b/scripts/oxlint-contract.spec.ts @@ -183,6 +183,33 @@ export const longProbe = 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + expect(maxLen).toStrictEqual(['error', { code: 140, ignoreUrls: true, ignoreStrings: true, ignoreTemplateLiterals: true }]) }) + it('reports an unused suppression', async () => { + const suffix = randomUUID() + const configPath = await writeContractConfig(suffix) + const path = join(repositoryRoot, 'scripts', `oxlint-contract-${suffix}.ts`) + + try { + await writeFile(path, '// oxlint-disable-next-line no-console\nexport const value = 1\n') + const result = runOxlint([ + '--config', + relative(repositoryRoot, configPath), + '--format', + 'unix', + relative(repositoryRoot, path), + ]) + const output = normalizedOutput(result) + + expect(result.error).toBeUndefined() + expect(result.status, output).toBe(0) + expect(output).toContain('Unused oxlint-disable directive') + } finally { + await Promise.all([ + rm(path, { force: true }), + rm(configPath, { force: true }), + ]) + } + }) + it('applies staged stylistic fixes before Oxlint validation', async () => { const suffix = randomUUID() const configPath = await writeContractConfig(suffix)