feat(eslint-rules): add base-hook-no-forbidden-runtime rule (#36253)

Author: Hotell

tools/eslint-rules/index.ts

@@ -5,6 +5,10 @@ import {
   rule as consistentCallbackType,
 } from './rules/consistent-callback-type';
 import { RULE_NAME as baseHookSignatureName, rule as baseHookSignature } from './rules/base-hook-signature';
+import {
+  RULE_NAME as baseHookNoForbiddenRuntimeName,
+  rule as baseHookNoForbiddenRuntime,
+} from './rules/base-hook-no-forbidden-runtime';
 
 /**
  * Import your custom workspace rules at the top of this file.
@@ -34,6 +38,7 @@ module.exports = {
   rules: {
     [consistentCallbackTypeName]: consistentCallbackType,
     [baseHookSignatureName]: baseHookSignature,
+    [baseHookNoForbiddenRuntimeName]: baseHookNoForbiddenRuntime,
     [noRestrictedGlobalsName]: noRestrictedGlobals,
     [noMissingJsxPragmaName]: noMissingJsxPragma,
   },

tools/eslint-rules/rules/__fixtures__/base-hook-no-forbidden-runtime/src/dummy.ts

@@ -0,0 +1,4 @@
+// Placeholder so the tsconfig has an actual file to anchor the project.
+// RuleTester test cases reference filenames inside this directory but their
+// content comes from the inline `code` field.
+export const dummy = 1;

tools/eslint-rules/rules/__fixtures__/base-hook-no-forbidden-runtime/src/test.ts

@@ -0,0 +1,4 @@
+// Placeholder anchor for typed RuleTester cases. The actual code being linted
+// comes from each test's inline `code` field; this file just needs to exist so
+// that the fixture tsconfig's Program contains the filename used by tests.
+export {};

tools/eslint-rules/rules/__fixtures__/base-hook-no-forbidden-runtime/stubs/cyclic-heavy-pkg/a.ts

@@ -0,0 +1,7 @@
+import { useB } from './b';
+import { runHeavy } from 'heavy-runtime';
+
+export function useA(): number {
+  runHeavy();
+  return useB() + 1;
+}

tools/eslint-rules/rules/__fixtures__/base-hook-no-forbidden-runtime/stubs/cyclic-heavy-pkg/b.ts

@@ -0,0 +1,6 @@
+import { useA } from './a';
+
+export function useB(): number {
+  // Keep the static import cycle while avoiding direct execution recursion.
+  return (useA as unknown as () => number).length;
+}

tools/eslint-rules/rules/__fixtures__/base-hook-no-forbidden-runtime/stubs/cyclic-heavy-pkg/index.ts

@@ -0,0 +1,3 @@
+// Cyclic graph where `a` imports a forbidden runtime and `b` reaches it only transitively.
+export { useA } from './a';
+export { useB } from './b';

tools/eslint-rules/rules/__fixtures__/base-hook-no-forbidden-runtime/stubs/cyclic-pkg/a.ts

@@ -0,0 +1,5 @@
+import { useB } from './b';
+
+export function useA(): number {
+  return useB() + 1;
+}

tools/eslint-rules/rules/__fixtures__/base-hook-no-forbidden-runtime/stubs/cyclic-pkg/b.ts

@@ -0,0 +1,6 @@
+import { useA } from './a';
+
+export function useB(): number {
+  // Pretend lazy ref to break true cycle at runtime; the static graph is cyclic.
+  return (useA as unknown as () => number).length;
+}

tools/eslint-rules/rules/__fixtures__/base-hook-no-forbidden-runtime/stubs/cyclic-pkg/index.ts

@@ -0,0 +1,3 @@
+// Cyclic re-export — exercises the cycle-safety of the transitive walk.
+export { useA } from './a';
+export { useB } from './b';

tools/eslint-rules/rules/__fixtures__/base-hook-no-forbidden-runtime/stubs/heavy-runtime/index.ts

@@ -0,0 +1,5 @@
+export function runHeavy(): { tag: 'heavy' } {
+  return { tag: 'heavy' };
+}
+
+export type HeavyOptions = { kind: 'heavy' };