Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In
`@codemod/zod-to-valibot/src/transform/schemas-and-links/methods/brand/brand.ts`:
- Around line 4-20: `transformBrand` is a pure exported helper that is missing
the required documentation and side-effect marker. Add JSDoc directly above the
exported `transformBrand` function, and place the `// `@__NO_SIDE_EFFECTS__``
annotation before it so it follows the codemod guidelines for pure factory
helpers. Keep the annotations attached to `transformBrand` so they remain
discoverable even if the implementation shifts.
- Line 2: The relative import in brand.ts should use an explicit .ts extension
to satisfy the repo’s ESM import rule. Update the import for addToPipe from the
local helpers module so it references the .ts file directly, keeping the
existing path structure and symbol name intact.

In
`@codemod/zod-to-valibot/src/transform/schemas-and-links/methods/brand/index.ts`:
- Line 1: The barrel re-export in the methods/brand entrypoint uses a
extensionless path, which violates the ESM import rule used throughout the
codemod sources. Update the export in the brand index barrel to use an explicit
.ts re-export path so it matches the rest of the codebase and satisfies the
ESLint import extension requirement.

In `@codemod/zod-to-valibot/src/transform/schemas-and-links/methods/index.ts`:
- Line 2: The barrel export in the methods index is using an extensionless path,
which violates the ESM import rules in this codebase. Update the export in the
index module to reference the explicit TypeScript entry point for brand, such as
./brand/index.ts or the concrete module it re-exports, so the new barrel remains
ESLint-compliant and ESM-safe. Use the existing index barrel and brand module
names to locate the change.

In
`@codemod/zod-to-valibot/src/transform/schemas-and-links/methods/partial/partial.ts`:
- Around line 25-39: The standalone partial transformation in partial.ts is
being detected too broadly by checking inputArgs[0].type instead of the actual
call shape, which causes method-chain cases like Schema.partial(mask) to lose
the schema receiver and emit the wrong output. Update the branch in the partial
transformer to only match true standalone z.partial(...) calls using the
call/method structure around inputArgs and schemaArg, then keep method-chain
forms flowing through the schemaExp path. Add a fixture covering
Foo.partial(mask) to lock in the correct behavior for identifier-based
selectors.

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="codemod/zod-to-valibot/__testfixtures__/brand-schema/input.ts">

<violation number="1" location="codemod/zod-to-valibot/__testfixtures__/brand-schema/input.ts:3">
P2: Brand test fixture only covers runtime-string-argument brands, missing the common `.brand<"Name">()` and `.brand()` (no-argument) forms. Since `transformBrand` only copies runtime `args` and drops TypeScript type parameters, the dominant Zod brand pattern is untested and its type information would be lost in transformation.</violation>
</file>

<file name="codemod/zod-to-valibot/src/transform/schemas-and-links/methods/brand/brand.ts">

<violation number="1" location="codemod/zod-to-valibot/src/transform/schemas-and-links/methods/brand/brand.ts:7">
P1: `transformBrand` drops TypeScript generic brand names by only forwarding runtime `args`, producing invalid `v.brand()` for the canonical `z.string().brand<'Name'>()` pattern.</violation>
</file>

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@codemod/zod-to-valibot/src/transform/schemas-and-links/schemas-and-links.ts`:
- Around line 589-595: The standalone-call check in schemas-and-links.ts is too
broad because `transformedExp === null` also matches linked alias chains like
`Foo.partial(mask)`, causing `toValibotMethodExp` and `transformPartial` to
treat `mask` as the schema argument. Update the `isStandaloneCall` logic in this
recursive link path to detect only the واقعی standalone API shape for `partial`
rather than relying on `transformedExp` alone, and keep the linked call flowing
through the normal method-call branch with the correct schema/mask argument
order.

---

Nitpick comments:
In `@codemod/zod-to-valibot/src/transform/schemas-and-links/schemas-and-links.ts`:
- Line 14: The new methods import in schemas-and-links.ts uses a bare ./methods
path, which violates the repo’s TS ESM import convention. Update the import in
the schemas-and-links module to use the .ts specifier for the methods file, and
make sure any related import statements in this transform path follow the same
ESM extension rule.

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="codemod/zod-to-valibot/__testfixtures__/brand-schema/input.ts">

<violation number="1" location="codemod/zod-to-valibot/__testfixtures__/brand-schema/input.ts:3">
P2: Brand test fixture only covers runtime-string-argument brands, missing the common `.brand<"Name">()` and `.brand()` (no-argument) forms. Since `transformBrand` only copies runtime `args` and drops TypeScript type parameters, the dominant Zod brand pattern is untested and its type information would be lost in transformation.</violation>
</file>

<file name="codemod/zod-to-valibot/src/transform/schemas-and-links/methods/brand/brand.ts">

<violation number="1" location="codemod/zod-to-valibot/src/transform/schemas-and-links/methods/brand/brand.ts:7">
P1: `transformBrand` drops TypeScript generic brand names by only forwarding runtime `args`, producing invalid `v.brand()` for the canonical `z.string().brand<'Name'>()` pattern.</violation>
</file>

<file name="codemod/zod-to-valibot/__testfixtures__/object-partial-variable-mask/input.ts">

<violation number="1" location="codemod/zod-to-valibot/__testfixtures__/object-partial-variable-mask/input.ts:1">
P2: The `object-partial-variable-mask` test fixture is present with input and output files, but it is not registered in `test-setup.test.ts`. As a result, this edge case for `.partial(mask)` with a variable mask is never executed. Add `'object-partial-variable-mask'` to the `defineTests` array in `codemod/zod-to-valibot/src/test-setup.test.ts`.</violation>
</file>

<file name="codemod/zod-to-valibot/src/transform/schemas-and-links/schemas-and-links.ts">

<violation number="1" location="codemod/zod-to-valibot/src/transform/schemas-and-links/schemas-and-links.ts:589">
P1: Standalone-call detection misclassifies linked-schema method chains as function-form calls, causing incorrect argument semantics in transformPartial.</violation>
</file>