chore(eslint): enforce no-explicit-any with bulk suppressions for existing violations

Author: edelauna

packages/config-eslint/base.js

@@ -2,8 +2,6 @@ import js from "@eslint/js"
 import eslintConfigPrettier from "eslint-config-prettier"
 import turboPlugin from "eslint-plugin-turbo"
 import tseslint from "typescript-eslint"
-import onlyWarn from "eslint-plugin-only-warn"
-
 /**
  * A shared ESLint configuration for the repository.
  *
@@ -21,11 +19,6 @@ export const config = [
 			"turbo/no-undeclared-env-vars": "off",
 		},
 	},
-	{
-		plugins: {
-			onlyWarn,
-		},
-	},
 	{
 		ignores: ["dist/**"],
 	},

packages/config-eslint/package.json

@@ -12,7 +12,6 @@
 		"@next/eslint-plugin-next": "15.5.20",
 		"eslint": "9.39.4",
 		"eslint-config-prettier": "10.1.8",
-		"eslint-plugin-only-warn": "1.2.1",
 		"eslint-plugin-react": "7.37.5",
 		"eslint-plugin-react-hooks": "5.2.0",
 		"eslint-plugin-turbo": "2.10.0",

pnpm-lock.yaml

@@ -12,7 +12,7 @@ import { type Task } from "../../core/task/Task"
  * Pass `tasks` (array of Task mocks) to pre-seed the registry in stack order.
  * The legacy `clineStack` key is accepted and converted automatically.
  */
-export function makeProviderStub<T extends object>(stub: T): T {
+export function makeProviderStub<T extends object>(stub: T): ClineProvider {
 	const s = stub as any
 	const proto = ClineProvider.prototype as any
 	s.delegationTransitionLocks ??= new Map()
@@ -29,8 +29,8 @@ export function makeProviderStub<T extends object>(stub: T): T {
 	}
 	delete s.clineStack
 
-	s.runDelegationTransition = proto.runDelegationTransition.bind(s)
+	s.runDelegationTransition ??= proto.runDelegationTransition.bind(s)
 	s.removeClineFromStack ??= proto.removeClineFromStack.bind(s)
 	s.evictCurrentTask ??= proto.evictCurrentTask.bind(s)
-	return s
+	return s as unknown as ClineProvider
 }

src/__tests__/helpers/provider-stub.ts

@@ -301,6 +301,9 @@ describe("ClineProvider.markDelegatedChildInterrupted() — live eviction path",
 			return { historyItem: { id: childTaskId, status: "interrupted", parentTaskId } }
 		})
 
+		const realRunDelegation = ClineProvider.prototype["runDelegationTransition"].bind({
+			delegationTransitionLocks: new Map(),
+		})
 		const provider = makeProviderStub({
 			clineStack: [] as any[],
 			taskEventListeners: new Map(),
@@ -320,15 +323,13 @@ describe("ClineProvider.markDelegatedChildInterrupted() — live eviction path",
 					return undefined
 				}),
 			},
+			// Wrap the real runDelegationTransition to set lockAcquired before the lock fires
+			runDelegationTransition: async (parentId: string, fn: () => Promise<void>) => {
+				lockAcquired = true
+				return realRunDelegation(parentId, fn)
+			},
 		})
 
-		// Patch runDelegationTransition to set lockAcquired before calling fn
-		const realRunDelegation = provider["runDelegationTransition"].bind(provider)
-		provider["runDelegationTransition"] = async (_parentId: string, fn: () => Promise<void>) => {
-			lockAcquired = true
-			return realRunDelegation(_parentId, fn)
-		}
-
 		await (ClineProvider.prototype as any).markDelegatedChildInterrupted.call(provider, {
 			childTaskId,
 			parentTaskId,
@@ -642,7 +643,7 @@ describe("onTaskCompleted callback — writes completed status before re-emittin
 					`[onTaskCompleted] Failed to write completed status for ${taskId}: ${err instanceof Error ? err.message : String(err)}`,
 				)
 			}
-			provider["emit"]("TaskCompleted", taskId, {}, {})
+			emit("TaskCompleted", taskId, {}, {})
 		}
 
 		return { onTaskCompleted, updateTaskHistory, emit, log }