fix(vitest): preserve subproperties on chained access through it proxy

.changeset/fix-vitest-it-proxy-subproperties.md

@@ -0,0 +1,5 @@
+---
+"@effect/vitest": patch
+---
+
+Fix `it.describe.each`, `it.skip.each`, `it.only.each` (and similar chained access on the `it` proxy) by removing an unnecessary `value.bind(target)` from `makeItProxy`'s `get` trap. The bind stripped each function's own properties (`.each`, `.skip`, `.only`, ...), so `it.describe.each` resolved to `undefined` and threw `TypeError: it.describe.each is not a function`.

packages/vitest/src/internal/internal.ts

@@ -64,8 +64,7 @@ const makeItProxy = <Methods extends object>(
       if (property in overrides) {
         return Reflect.get(overrides, property)
       }
-      const value = Reflect.get(target, property, receiver)
-      return typeof value === "function" ? value.bind(target) : value
+      return Reflect.get(target, property, receiver)
     }
   })
 

packages/vitest/test/index.test.ts

@@ -33,6 +33,25 @@ it.effect.skip(
   () => Effect.die("skipped anyway")
 )
 
+// chained access through proxy preserves subproperties (regression for
+// https://github.com/Effect-TS/effect-smol/issues/2301)
+
+it.describe("it.describe.each", () => {
+  it.describe.each(["a", "b"] as const)("%s", (text) => {
+    it("matches the case label", () => {
+      assert.include(["a", "b"], text)
+    })
+  })
+})
+
+it.skip.each([1, 2])("it.skip.each is skipped (%s)", (n) => {
+  assert.fail(`should never run for ${n}`)
+})
+
+it.only.each([] as Array<number>)("it.only.each has no cases", () => {
+  assert.fail("no cases registered")
+})
+
 // skipIf
 
 it.effect.skipIf(true)("effect skipIf (true)", () => Effect.die("skipped anyway"))