ORB-85: Return user id from profile - #453
Conversation
|
I will add I will add focused handler unit tests under This shape reuses the profile query both clients already call and the user entity it already loads. It avoids a second database read, a new endpoint, auth response changes, JWT changes, and all client repository work reserved for ORB-77. |
|
|
@codex review |
|
Codex Review: Didn't find any major issues. Nice work! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
DEGRADED: same-vendor review Review verdict: clean after round 1. No Blocking or Non-blocking findings against the frozen rubric. This used a fresh Sol xhigh session; same-family bias is not eliminated and its magnitude is unmeasured. |
|
@codex review |
|
Codex Review: Didn't find any major issues. Bravo. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
@codex review |
|
Codex Review: Didn't find any major issues. Keep them coming! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |



DEGRADED: same-vendor review
Summary
userIdas the first member ofProfileResponse.GetProfileQueryHandler.Issue: ORB-85
Verification
dotnet build Orbit.slnx: 0 errorsdotnet test Orbit.slnx: 5,595 passed, 0 failedContract evidence
The handler test proves
ProfileResponse.UserIdequals the caller GUID and that its default string form is lowercase, hyphenated GuidDformat. The serializer test confirms the web JSON property isuserIdand that a consumer model without that property still deserializes the unchanged name and email fields.The generated OpenAPI document includes
userIdas a UUID string onProfileResponse. This worker did not capture a tokenized liveGET /api/profilebody because the work order explicitly forbids starting or connecting to a development server.No external CLI or HTTP response field is read by the implementation.