Add "Custom hydrator type casters" article

[vjik]

No description provided.

[Copilot]

🟡 Changes recommended

The request-input example imports RouteArgument from a nonexistent namespace.

1 open finding

• Import RouteArgument from the router package · New

What changed in this PR

Adds a cookbook article explaining custom Yii Hydrator type casters.

Changes:

• Documents caster creation and registration.
• Demonstrates UUID value objects with request and form models.
• Adds cookbook navigation entries.

File	Description
src/​cookbook/​index.md	Adds the article to the cookbook index.
src/​cookbook/​custom-hydrator-type-casters.md	Adds the new tutorial and examples.
src/​.vitepress/​config.js	Adds the article to VitePress navigation.

🧠 Review effort: Balanced

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

🟡 Changes recommended

The article contains incomplete DI guidance and edge cases that can make examples fail or produce an incorrect value-object type.

3 open findings

• Ambiguous UUID unions silently select the wrong subtype · New
• Missing ContainerInterface import breaks factory invocation · New
• FormHydrator bypasses the configured type-caster chain · New

1 resolved since last review

• Import RouteArgument from the router package

🧠 Review effort: Balanced

[Copilot]

🔵 Needs a closer look

The article incorrectly describes failed casts as universally silent, although required constructor arguments cause object creation to fail.

0 open findings

3 resolved since last review

• Ambiguous UUID unions silently select the wrong subtype
• FormHydrator bypasses the configured type-caster chain
• Missing ContainerInterface import breaks factory invocation

Previously missed (1)

In code that hasn't changed since last review

Clarify caster behavior for required constructor arguments

src/​cookbook/​custom-hydrator-type-casters.md:91

This is only silent when hydrating a property. If the same caster rejects a value for a required constructor parameter, ConstructorArgumentsExtractor omits that argument and ReflectionObjectFactory throws WrongConstructorArgumentsCountException, so the article's earlier promise that the caster applies to constructor parameters makes this general statement misleading. Please distinguish property hydration from required constructor arguments.

🧠 Review effort: Balanced