chore(lint): add new oxlint rules and auto-fix violations (CopilotKit#3742)

## Summary
- Enable 9 new oxlint rules for better TypeScript/React code health
- Enforce `import type` for type-only imports
(`typescript/consistent-type-imports` + companions)
- Enforce self-closing JSX, `.slice()` over `.substring()`, optional
catch binding, and more
- Bump `typescript/no-unnecessary-type-assertion` from warn to error
- Auto-fix all 1,400+ existing violations across 55 files

## Test plan
- [ ] CI passes (lint + format)
- [ ] No runtime behavior changes (all fixes are syntactic)

Author: AlemTuzlak

.github/actions/changesets-action/src/run.ts

@@ -215,7 +215,7 @@ const requireChangesetsCliPkgJson = (cwd: string) => {
       err.code === "MODULE_NOT_FOUND"
     ) {
       throw new Error(
-        `Have you forgotten to install \`@changesets/cli\` in "${cwd}"?`,
+        `Have you forgotten to install \`@changesets/cli\` in "${cwd}"?`, { cause: err },
       );
     }
     throw err;

.oxlintrc.json

@@ -11,7 +11,21 @@
     "import/no-unassigned-import": "off",
     "unicorn/no-array-sort": "off",
     "copilotkit/require-cpk-prefix": "off",
-    "no-restricted-imports": "off"
+    "no-restricted-imports": "off",
+
+    "typescript/consistent-type-imports": [
+      "warn",
+      { "prefer": "type-imports", "fixStyle": "separate-type-imports" }
+    ],
+    "typescript/no-import-type-side-effects": "warn",
+    "import/consistent-type-specifier-style": ["warn", "prefer-top-level"],
+    "typescript/no-unnecessary-type-assertion": "error",
+
+    "react/self-closing-comp": "warn",
+    "unicorn/prefer-optional-catch-binding": "warn",
+    "eslint/no-useless-computed-key": "warn",
+    "unicorn/prefer-string-slice": "warn",
+    "unicorn/prefer-array-flat-map": "warn"
   },
   "overrides": [
     {
@@ -42,6 +56,17 @@
       "rules": {
         "copilotkit/require-cpk-prefix": "off"
       }
+    },
+    {
+      "files": [
+        "packages/angular/**/*.ts",
+        "packages/runtime/src/graphql/**/*.ts"
+      ],
+      "rules": {
+        "typescript/consistent-type-imports": "off",
+        "typescript/no-import-type-side-effects": "off",
+        "import/consistent-type-specifier-style": "off"
+      }
     }
   ],
   "ignorePatterns": [

examples/integrations/a2a-a2ui/app/theme.ts

@@ -413,16 +413,16 @@ export const theme: Types.Theme = {
     video,
   },
   markdown: {
-    p: [...Object.keys(pLight)],
-    h1: [...Object.keys(h1Light)],
-    h2: [...Object.keys(h2Light)],
-    h3: [...Object.keys(h3Light)],
-    h4: [...Object.keys(h4Light)],
-    h5: [...Object.keys(h5Light)],
-    ul: [...Object.keys(unorderedListLight)],
-    ol: [...Object.keys(orderedListLight)],
-    li: [...Object.keys(listItemLight)],
-    a: [...Object.keys(aLight)],
+    p: Object.keys(pLight),
+    h1: Object.keys(h1Light),
+    h2: Object.keys(h2Light),
+    h3: Object.keys(h3Light),
+    h4: Object.keys(h4Light),
+    h5: Object.keys(h5Light),
+    ul: Object.keys(unorderedListLight),
+    ol: Object.keys(orderedListLight),
+    li: Object.keys(listItemLight),
+    a: Object.keys(aLight),
     strong: [],
     em: [],
   },

examples/integrations/agent-spec/src/app/theme.ts

@@ -406,16 +406,16 @@ export const theme: Types.Theme = {
     video,
   },
   markdown: {
-    p: [...Object.keys(pLight)],
-    h1: [...Object.keys(h1Light)],
-    h2: [...Object.keys(h2Light)],
-    h3: [...Object.keys(h3Light)],
-    h4: [...Object.keys(h4Light)],
-    h5: [...Object.keys(h5Light)],
-    ul: [...Object.keys(unorderedListLight)],
-    ol: [...Object.keys(orderedListLight)],
-    li: [...Object.keys(listItemLight)],
-    a: [...Object.keys(aLight)],
+    p: Object.keys(pLight),
+    h1: Object.keys(h1Light),
+    h2: Object.keys(h2Light),
+    h3: Object.keys(h3Light),
+    h4: Object.keys(h4Light),
+    h5: Object.keys(h5Light),
+    ul: Object.keys(unorderedListLight),
+    ol: Object.keys(orderedListLight),
+    li: Object.keys(listItemLight),
+    a: Object.keys(aLight),
     strong: [],
     em: [],
   },

examples/integrations/agentcore/infra-cdk/lib/utils/config-manager.ts

@@ -118,6 +118,7 @@ export class ConfigManager {
     } catch (error) {
       throw new Error(
         `Failed to parse configuration file ${configPath}: ${error}`,
+        { cause: error },
       );
     }
   }

examples/showcases/generative-ui-playground/src/app/theme.ts

@@ -357,16 +357,16 @@ export const a2uiTheme: Types.Theme = {
     video: { "layout-el-cv": true },
   },
   markdown: {
-    p: [...Object.keys(p)],
-    h1: [...Object.keys(h1)],
-    h2: [...Object.keys(h2)],
-    h3: [...Object.keys(h3)],
-    h4: [...Object.keys(h4)],
-    h5: [...Object.keys(h5)],
-    ul: [...Object.keys(unorderedList)],
-    ol: [...Object.keys(orderedList)],
-    li: [...Object.keys(listItem)],
-    a: [...Object.keys(a)],
+    p: Object.keys(p),
+    h1: Object.keys(h1),
+    h2: Object.keys(h2),
+    h3: Object.keys(h3),
+    h4: Object.keys(h4),
+    h5: Object.keys(h5),
+    ul: Object.keys(unorderedList),
+    ol: Object.keys(orderedList),
+    li: Object.keys(listItem),
+    a: Object.keys(a),
     strong: [],
     em: [],
   },

examples/showcases/mcp-demo/src/components/Todo.tsx

@@ -78,7 +78,6 @@ export const TodoApp = () => {
       },
     ],
     handler: ({ id, subtask }) => {
-      debugger;
       addSubtask(id, subtask);
     },
   });

examples/showcases/mcp-demo/src/contexts/TodoContext.tsx

@@ -158,7 +158,6 @@ export const TodoProvider: React.FC<{ children: ReactNode }> = ({
   };
 
   const toggleSubtask = (parentId: number, subtaskId: number) => {
-    debugger;
     setTodos(
       todos.map((todo) => {
         if (todo.id === parentId) {

examples/showcases/orca/frontend/components/developer-dashboard.tsx

@@ -297,7 +297,6 @@ export function DeveloperDashboard() {
               <Select
                 value={filterParams.status}
                 onValueChange={(e) => {
-                  debugger;
                   console.log(ref2.current);
 
                   setFilterParams({ ...filterParams, status: e });
@@ -337,7 +336,6 @@ export function DeveloperDashboard() {
             <Select
               value={filterParams.author}
               onValueChange={(e) => {
-                debugger;
                 setFilterParams({ ...filterParams, author: e });
                 if (filterParams.status === "a") {
                   setFilteredData(

examples/showcases/orca/frontend/components/pr-line-chart-data.tsx

@@ -89,7 +89,6 @@ export function PRLineChartData({ args }: any) {
 
 const CustomPieTooltip = ({ active, payload }: any) => {
   if (active && payload && payload.length) {
-    debugger;
     const { week, count } = payload[0].payload;
     return (
       <div className="bg-white p-2 rounded shadow text-black">