Skip to content

fix: remove dead user_id field from CreateSessionRequest #36

@tumberger

Description

@tumberger

Problem

The CLI populates CreateSessionRequest.user_id with the user's email/name, but the API completely ignores it — user identity is always derived from the OIDC bearer token.

This field is dead bytes on the wire and misleading in the proto definition (it implies the CLI controls user attribution, but it doesn't).

Options

  1. Remove from proto — delete the field, CLI stops sending it. Clean break.
  2. Keep as advisory — document that it's informational only, API may ignore it. Useful for logging/debugging but not authoritative.

Recommend option 1 — the bearer token is the source of truth and having a dead field invites confusion.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions