fix(uncheck): review the whole package: safer hooks, tsc selection like tsc, faster runs, clearer docs

package.json

@@ -37,6 +37,6 @@
     }
   },
   "engines": {
-    "node": ">=22.20 <23 || >=24.8"
+    "node": ">=22.20"
   }
 }

packages/uncheck/package.json

@@ -55,15 +55,16 @@
     "@effect/platform-node": "^4.0.0-rc.116",
     "effect": "^4.0.0-rc.116",
     "jsonc-parser": "^3.3.1",
+    "minimatch": "^10.2.6",
     "oxfmt": "^0.68.0",
     "oxlint": "^1.83.0",
     "sherif": "^1.13.0",
     "typescript": "^7.0.2"
   },
   "peerDependencies": {
     "oxfmt": "*",
-    "oxlint": "*",
-    "sherif": "*",
+    "oxlint": ">=1.60.0",
+    "sherif": ">=1.10.0",
     "typescript": "*"
   },
   "peerDependenciesMeta": {
@@ -81,6 +82,6 @@
     }
   },
   "engines": {
-    "node": ">=22.20 <23 || >=24.8"
+    "node": ">=22.20"
   }
 }

packages/uncheck/src/bin.ts

@@ -1,37 +1,8 @@
 #!/usr/bin/env node
 
-import process from 'node:process'
+import { enableCompileCache } from 'node:module'
 
-import * as NodeRuntime from '@effect/platform-node/NodeRuntime'
-import * as NodeServices from '@effect/platform-node/NodeServices'
-import { Effect } from 'effect'
-import { Command } from 'effect/unstable/cli'
+// The cache only covers modules compiled after it is on, hence the import that follows it.
+enableCompileCache()
 
-import pkg from '../package.json'
-import { hooks } from './commands/hooks'
-import { prepare } from './commands/prepare'
-import { staged } from './commands/staged'
-import { uncheck } from './commands/uncheck'
-
-// Any failed write (a reader gone after `| head`, a closed terminal, a full disk) would otherwise end
-// the run before `staged` puts unstaged changes back, so output errors are ignored.
-for (const stream of [process.stdout, process.stderr]) {
-  stream.on('error', () => {})
-}
-
-Command.run(uncheck.pipe(Command.withSubcommands([staged, prepare, hooks])), {
-  version: pkg.version,
-}).pipe(
-  Effect.catchTag('CheckFailed', () =>
-    Effect.sync(() => {
-      process.exitCode = 1
-    }),
-  ),
-  Effect.catchTag('StopBlocked', () =>
-    Effect.sync(() => {
-      process.exitCode = 2
-    }),
-  ),
-  Effect.provide(NodeServices.layer),
-  NodeRuntime.runMain,
-)
+await import('./cli')

packages/uncheck/src/checks/oxfmt.ts

@@ -6,7 +6,7 @@ import type { Check } from '../types'
 
 export const oxfmt: Check = {
   name: 'oxfmt',
-  fixes: true,
+  fixes: 'files',
   plan: Effect.fn(function* ({ cwd, fix, files }) {
     const bin = yield* resolveBin('oxfmt', cwd)
 
@@ -20,7 +20,7 @@ export const oxfmt: Check = {
       return [{ bin, args }]
     }
 
-    // Given files may include ones oxfmt does not handle (a Markdown file), which is not a failure.
+    // Given files may include ones oxfmt does not handle (a .txt file), which is not a failure.
     return argvBatches(files).map((batch) => ({
       bin,
       args: [...args, '--no-error-on-unmatched-pattern'],

packages/uncheck/src/checks/oxlint.ts

@@ -6,7 +6,7 @@ import type { Check } from '../types'
 
 export const oxlint: Check = {
   name: 'oxlint',
-  fixes: true,
+  fixes: 'files',
   plan: Effect.fn(function* ({ cwd, fix, files }) {
     const bin = yield* resolveBin('oxlint', cwd)
 

packages/uncheck/src/checks/sherif.ts

@@ -11,7 +11,7 @@ const WORKSPACE_FILES = new Set(['package.json', 'pnpm-workspace.yaml'])
 
 export const sherif: Check = {
   name: 'sherif',
-  fixes: true,
+  fixes: 'workspace',
   plan: Effect.fn(function* ({ cwd, fix, files }) {
     const fs = yield* FileSystem.FileSystem
     const path = yield* Path.Path