feat(prinfo): expand author field to include account type

[javirln]

Summary

• Expand author field in PR info from a plain string to an object with login and type (User/Bot/unknown), enabling policies to distinguish bot-authored PRs without relying on naming conventions
• Add JSON schema v1.3 with oneOf supporting both string (backwards compat) and object formats
• Custom UnmarshalJSON on Data handles both formats at the Go struct level, converting legacy string authors to Author{Login: str, Type: "unknown"}
• GitHub: extracts author type from the PR event payload (user.type field)
• GitLab: refactors fetchGitLabReviewers into fetchGitLabMRDetails to extract author bot status from the MR API (author.bot field) in a single call, with fallback to env var if API unavailable
• GitLab reviewers also get bot detection from the same API response

[cubic-dev-ai]

1 issue found across 7 files

Prompt for AI agents (unresolved issues)

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="internal/prinfo/prinfo.go">

<violation number="1" location="internal/prinfo/prinfo.go:49">
P1: This change breaks runtime backward compatibility for old PR info payloads with string `author`: decoding fails before schema validation can apply the v1.3 `oneOf` rule.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[cubic-dev-ai]

1 issue found across 4 files (changes from recent commits).

Prompt for AI agents (unresolved issues)

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="internal/prinfo/prinfo.go">

<violation number="1" location="internal/prinfo/prinfo.go:80">
P2: Unmarshaling JSON `null` into a struct value leaves it as an empty struct rather than `nil`. Unmarshal into a pointer instead to properly handle nulls.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[cubic-dev-ai]

No issues found across 8 files