Repository navigation
Add account commands and si models - #10
Merged
Merged
Conversation
si auth signup creates an account with a confirmed password prompt, si auth resend sends the confirmation email again (the hint in the unverified-email message already pointed at it), si auth reset-password emails reset instructions, and si auth delete-account asks you to type your email before deleting, or takes --yes. si models lists the models a session can use. The client crate gets reset_password and models. Co-Authored-By: Claude Opus 5.5 <[email protected]>
![sigit-code-review[bot]](https://avatars.githubusercontent.com/in/3492899?s=60&v=4){kind=small}
There was a problem hiding this comment.
This change adds new account management commands to the CLI: signup, resend confirmation, reset password, and delete account, each with appropriate prompts and flows. It also introduces the si models command to list available models, with supporting API and rendering code. The client API gains reset_password and models methods, and a new Model struct is defined. Tests are added for email normalization and model deserialization. The changes are well-structured and integrate with existing patterns. No previous findings are present or reintroduced.
Automated review by siGit Code Review · commit 04c82c5 · see the review dashboard
paydii
approved these changes
Oct 28, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds
si auth signup,resend,reset-passwordanddelete-account, andsi models.signupasks for the password twice.resendfixes the existing "runsi auth resend" hint, which pointed at a command that didn't exist.reset-passwordandresendsay "if the address has an account" because the server answers the same either way.delete-accountasks you to type your email, or takes--yes, then deletes the account and clears the stored token.si modelsneeds no sign-in.The client crate gets
reset_passwordandmodels, plus theModeltype (defaultfalls back to false).Checked with
cargo fmt --check,cargo clippy -D warnings,cargo test --workspaceandcargo check -p sigit-si-api --no-default-features.