mirror of
https://github.com/abhigyanpatwari/GitNexus.git
synced 2026-10-11 03:38:07 +00:00
Review finding on the mixed-import handler choice: a route registered on a group (`api := e.Group(...); api.GET(...)`) bound its receiver to the Group() call rather than a constructor, so constructor provenance failed and the conservative last-argument fallback picked echo's middleware as the handler name. receiverBindsToEchoConstructor now walks enclosing Group() calls on the way to the constructor (depth-capped by MAX_GROUP_DEPTH, mirroring groupPrefix), so an echo group chain resolves to echo.New()/echo.Default() and takes the first-argument handler, while a gin chain still fails the echo-alias check and keeps the last-argument rule. Provenance must still END at a constructor: parameters and unrelated New() remain fallback. Tests: 39 in the go-gin file (new: grouped echo + gin chains in one mixed file); group suite 1302 green; prettier/eslint/tsc clean. Co-Authored-By: Claude Code <noreply@anthropic.com> |
||
|---|---|---|
| .. | ||
| fixtures | ||
| helpers | ||
| integration | ||
| unit | ||
| utils | ||