Skip to content

feat(auth): add session-less sign-up - #9

Merged
swkeever merged 1 commit into
mainfrom
skeever/sdk-auth-sign-up
Aug 31, 2026
Merged

swkeever merged 1 commit into
mainfrom
skeever/sdk-auth-sign-up

Conversation

@swkeever

Copy link
Copy Markdown
Collaborator

Summary

  • add Python-native client.auth.sign_up(...) over the existing signup endpoint
  • return an immutable SignUpResult without creating or replacing session state
  • forward optional user metadata through the generated internal client and preserve typed errors
  • document the flow and cover the facade, transport boundary, and public export

Verification

  • uv run python scripts/check_openapi.py
  • uv run ruff check .
  • uv run ruff format --check .
  • uv run mypy
  • uv run pyright
  • uv run pytest tests/unit -q
  • uv run python -m build
  • npm test --prefix tests/sdk-contract (volcano-hosting)
  • bash scripts/ci/run-sdk-contract-tests_test.sh (volcano-hosting)

Jira: VOL-941

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Aug 31, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-08-31T19:08:09.511930Z 2cd4e52 Manual request
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Keep them coming!

Reviewed commit: 2cd4e52006

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

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".

@swkeever
swkeever marked this pull request as ready for review August 31, 2026 19:08
@swkeever
swkeever merged commit 67f7c07 into main Aug 31, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant