cleanup

Author: BioPhoton

package.json

@@ -15,8 +15,7 @@
   "scripts": {
     "prepare": "husky install",
     "commit": "git-cz",
-    "knip": "knip",
-    "postinstall": "npx nx run plugin-typescript:postinstall"
+    "knip": "knip"
   },
   "private": true,
   "engines": {

packages/plugin-typescript/project.json

@@ -35,11 +35,6 @@
       "options": {
         "configFile": "packages/plugin-typescript/vite.config.integration.ts"
       }
-    },
-    "postinstall": {
-      "dependsOn": ["build"],
-      "command": "npx tsx --tsconfig packages/plugin-typescript/tsconfig.lib.json packages/plugin-typescript/src/postinstall/create.bin.ts",
-      "cwd": ""
     }
   },
   "tags": ["scope:plugin", "type:feature", "publishable"]

packages/plugin-typescript/src/index.ts

@@ -11,4 +11,3 @@ export {
   type TypescriptPluginOptions,
 } from './lib/typescript-plugin.js';
 export default typescriptPlugin;
-export { getTsDefaultsFilename } from './lib/runner/constants.js';

packages/plugin-typescript/src/lib/runner/constants.ts

@@ -3,7 +3,7 @@ import {
   createProgram,
   getPreEmitDiagnostics,
 } from 'typescript';
-import { loadTargetConfig, validateDiagnostics } from './utils.js';
+import { loadTargetConfig } from './utils.js';
 
 export type DiagnosticsOptions = {
   tsConfigPath: string;
@@ -17,10 +17,7 @@ export async function getTypeScriptDiagnostics({
 
     const program = createProgram(fileNames, options);
     // @TODO use more fine-grained helpers like getSemanticDiagnostics instead of getPreEmitDiagnostics
-    const diagnostics = getPreEmitDiagnostics(program);
-    validateDiagnostics(diagnostics);
-
-    return diagnostics;
+    return getPreEmitDiagnostics(program);
   } catch (error) {
     throw new Error(
       `Can't create TS program in getDiagnostics. \n ${(error as Error).message}`,

packages/plugin-typescript/src/lib/runner/ts-runner.ts

@@ -10,7 +10,6 @@ import {
 } from 'typescript';
 import type { Issue } from '@code-pushup/models';
 import { readTextFile, truncateIssueMessage } from '@code-pushup/utils';
-import { AUDIT_LOOKUP } from './constants.js';
 import { TS_CODE_RANGE_NAMES } from './ts-error-codes.js';
 import type { CodeRangeName } from './types.js';
 
@@ -106,13 +105,3 @@ export async function loadTargetConfig(tsConfigPath: string) {
 
   return parsedConfig;
 }
-
-export function validateDiagnostics(diagnostics: readonly Diagnostic[]) {
-  diagnostics
-    .filter(({ code }) => !AUDIT_LOOKUP.has(code))
-    .forEach(({ code, messageText }) => {
-      console.warn(
-        `Diagnostic Warning: The code ${code} is not supported. ${messageText}`,
-      );
-    });
-}

packages/plugin-typescript/src/lib/runner/utils.ts

@@ -4,37 +4,8 @@ import {
   getIssueFromDiagnostic,
   getSeverity,
   tSCodeToAuditSlug,
-  validateDiagnostics,
 } from './utils.js';
 
-describe('validateDiagnostics', () => {
-  const consoleWarnSpy = vi.spyOn(console, 'warn');
-
-  it('should not log for known error codes', () => {
-    expect(() =>
-      validateDiagnostics([
-        {
-          code: 7005,
-          messageText: 'strich checks error',
-        } as Diagnostic,
-      ]),
-    ).not.toThrow();
-    expect(consoleWarnSpy).toHaveBeenCalledTimes(0);
-  });
-
-  it.todo('should log for known error codes', () => {
-    expect(() =>
-      validateDiagnostics([
-        {
-          code: 1337,
-          messageText: 'unknown error code',
-        } as Diagnostic,
-      ]),
-    ).not.toThrow();
-    expect(consoleWarnSpy).toHaveBeenCalledTimes(1);
-  });
-});
-
 describe('tSCodeToAuditSlug', () => {
   it('should transform supported code to readable audit', () => {
     expect(tSCodeToAuditSlug(Number.parseInt('2345', 10))).toBe(