fix scripts

Author: millotp

oxlintrc.json

@@ -1,5 +1,8 @@
 {
   "env": {
     "es6": true
+  },
+  "rules": {
+    "@typescript-eslint/consistent-type-imports": "error"
   }
 }

scripts/ci/codegen/__tests__/codegen.test.ts

@@ -3,6 +3,7 @@ import { describe, expect, it, vi } from 'vitest';
 import { pushGeneratedCode } from '../pushGeneratedCode.ts';
 
 vi.mock('../../../common.ts', async (importOriginal) => {
+  // eslint-disable-next-line
   const mod = await importOriginal<typeof import('../../../common.ts')>();
   return {
     ...mod,

scripts/ci/codegen/pushToRepository.ts

@@ -15,7 +15,8 @@ import {
   toAbsolutePath,
 } from '../../common.ts';
 import { getNbGitDiff } from '../utils.ts';
-import { GuidesToPush, pushToRepositoryConfiguration, RepositoryConfiguration, SpecsToPush } from './types.ts';
+import type { GuidesToPush, RepositoryConfiguration, SpecsToPush } from './types.ts';
+import { pushToRepositoryConfiguration } from './types.ts';
 
 import { getClientsConfigField } from '../../config.ts';
 import { commitStartRelease } from './text.ts';

scripts/cts/testServer/index.ts

@@ -61,13 +61,13 @@ export async function setupServer(name: string, port: number, addRoutes: (app: E
   addRoutes(app);
 
   // 404 handler
-  app.use((req, res) => {
+  app.use((req, _) => {
     console.error(`[PORT ${port}] endpoint not implemented for`, req.method, req.url);
     expect.fail('endpoint not implemented');
   });
 
   // catch all error handler
-  app.use((err, _req, res, _) => {
+  app.use((err, _req, _res, _) => {
     console.error(err.message);
     expect.fail(err.message);
   });

scripts/release/__tests__/createReleasePR.test.ts

@@ -30,6 +30,7 @@ const buildTestCommit = (
 
 // Mock `getOctokit` to bypass the API call and credential requirements
 vi.mock('../../common.ts', async (importOriginal) => {
+  // eslint-disable-next-line
   const mod = await importOriginal<typeof import('../../common.ts')>();
   return {
     ...mod,
@@ -48,6 +49,7 @@ vi.mock('../../common.ts', async (importOriginal) => {
 });
 
 vi.mock('../../ci/utils.ts', async (importOriginal) => {
+  // eslint-disable-next-line
   const mod = await importOriginal<typeof import('../../ci/utils.ts')>();
   return {
     ...mod,
@@ -56,6 +58,7 @@ vi.mock('../../ci/utils.ts', async (importOriginal) => {
 });
 
 vi.mock('../common.ts', async (importOriginal) => {
+  // eslint-disable-next-line
   const mod = await importOriginal<typeof import('../common.ts')>();
   const getFileChangesMockFn = vi.fn();
 
@@ -68,6 +71,7 @@ vi.mock('../common.ts', async (importOriginal) => {
 });
 
 vi.mock('../../config.ts', async (importOriginal) => {
+  // eslint-disable-next-line
   const mod = await importOriginal<typeof import('../../config.ts')>();
   return {
     ...mod,

scripts/specs/__tests__/snippets.test.ts

@@ -1,7 +1,7 @@
 import { describe, expect, it } from 'vitest';
 
 import { parseCodeSamples } from '../snippets.ts';
-import { CodeSamples } from '../types.ts';
+import type { CodeSamples } from '../types.ts';
 
 describe('init', () => {
   it('parses a multi line import', () => {