refactor(lint): replace typecheck with oxlint --type-check (#200)

* refactor(lint): replace typecheck with oxlint --type-check

- Replace typecheck scripts with oxlint --type-check in lint and format
- Remove dedicated typecheck, typecheck:root, typecheck:submodules scripts
- Add lint:submodule and format:submodule for workspace linting/formatting
- Update examples to use lint and format with oxlint directly
- Simplify lefthook configuration: remove nested groups, add type-check to hook

This consolidates type checking into oxlint, eliminating the need for
separate tsgo --noEmit invocations. Introduces explicit submodule commands
for workspace-level operations.

* chore(knip): ignore @typescript/native-preview

Add @typescript/native-preview to knip's ignoreDependencies list to
prevent false positive unused dependency warnings. This package is
required for TypeScript native preview features and is not directly
imported in source code.

* ci: remove obsolete typecheck step from CI workflow

The typecheck functionality is now integrated into the lint step via
oxlint --type-check, so the dedicated typecheck step is no longer needed.

* chore(hooks): add knip check to push hook

Add knip dependency check to the push hook to catch unused dependencies
before they are pushed to the repository.

Author: ryoppippi

.github/workflows/ci.yaml

@@ -31,9 +31,6 @@ jobs:
       - name: Run Lint
         run: nix develop --command pnpm run lint
 
-      - name: Run Type Check
-        run: nix develop --command pnpm run typecheck
-
       - name: Run Build
         run: nix develop --command pnpm run build
 

examples/package.json

@@ -4,7 +4,8 @@
 	"private": true,
 	"type": "module",
 	"scripts": {
-		"typecheck": "tsgo --noEmit"
+		"lint": "oxlint --max-warnings=0 --type-aware --type-check",
+		"format": "oxlint --max-warnings=0 --type-aware --type-check --fix"
 	},
 	"dependencies": {
 		"@stackone/ai": "workspace:*"

knip.config.ts

@@ -13,7 +13,9 @@ export default {
 	},
 	ignore: ['**/*.test.ts', '**/*.spec.ts', '**/*.test-d.ts'],
 	ignoreBinaries: ['only-allow'],
+	ignoreDependencies: ['@typescript/native-preview', 'lefthook'],
 	rules: {
 		optionalPeerDependencies: 'off',
+		devDependencies: 'warn',
 	},
 } satisfies KnipConfig;

lefthook.yaml

@@ -1,19 +1,16 @@
 pre-commit:
-  parallel: true
+  piped: true
   jobs:
-    - name: lint-fmt
-      group:
-        piped: true
-        jobs:
-          - name: oxlint
-            glob: '*.{ts,tsx,js,jsx,mts,cts}'
-            run: bun oxlint --max-warnings=0 --type-aware --fix {staged_files}
-            stage_fixed: true
-          - name: oxfmt
-            glob: '*'
-            run: bun oxfmt --no-error-on-unmatched-pattern {staged_files}
-            stage_fixed: true
+    - name: oxlint
+      glob: '*.{ts,tsx,js,jsx,mts,cts}'
+      run: bun oxlint --max-warnings=0 --type-aware --type-check --fix {staged_files}
+      stage_fixed: true
+    - name: oxfmt
+      glob: '*'
+      run: bun oxfmt --no-error-on-unmatched-pattern {staged_files}
+      stage_fixed: true
 
-    - name: typecheck
-      glob: '*.{ts,tsx,mts,cts}'
-      run: bun run typecheck
+push:
+  jobs:
+    - name: knip
+      run: pnpm run lint:knip

package.json

@@ -21,19 +21,18 @@
 		"build": "tsdown",
 		"format": "pnpm --no-bail --aggregate-output run '/^format:/'",
 		"format:oxfmt": "oxfmt --no-error-on-unmatched-pattern .",
-		"format:oxlint": "oxlint --max-warnings=0 --type-aware --fix",
+		"format:oxlint": "oxlint --max-warnings=0 --type-aware --type-check --fix",
 		"format:knip": "knip --fix --no-exit-code",
-		"lint": "pnpm --no-bail --aggregate-output run '/^lint:/'",
+		"format:submodule": "pnpm --parallel -r --aggregate-output format",
+		"lint": "pnpm --aggregate-output run '/^lint:/'",
 		"lint:oxfmt": "oxfmt --no-error-on-unmatched-pattern --check .",
-		"lint:oxlint": "oxlint --max-warnings=0 --type-aware",
+		"lint:oxlint": "oxlint --max-warnings=0 --type-aware --type-check",
+		"lint:submodule": "pnpm --parallel -r --aggregate-output lint",
 		"lint:knip": "knip",
 		"preinstall": "npx only-allow pnpm",
 		"prepack": "npm pkg delete scripts.preinstall && pnpm run build",
 		"test": "vitest",
-		"coverage": "vitest run --coverage",
-		"typecheck": "pnpm --aggregate-output run '/^typecheck:/'",
-		"typecheck:submodules": "pnpm --parallel -r --aggregate-output typecheck",
-		"typecheck:root": "tsgo --noEmit"
+		"coverage": "vitest run --coverage"
 	},
 	"dependencies": {
 		"@modelcontextprotocol/sdk": "catalog:prod",