Skip to content

feat: add login, logout, and custom command support to zsh plugin - #1859

Merged
tusharmath merged 11 commits into
mainfrom
feat/add-login-logout-commands
Nov 9, 2025
Merged

tusharmath merged 11 commits into
mainfrom
feat/add-login-logout-commands

Conversation

@amitksingh1490

@amitksingh1490 amitksingh1490 commented Nov 7, 2025 •

Copy link
Copy Markdown
Contributor

Summary

This PR extends the zsh plugin with comprehensive command support, enabling login/logout operations.

- Added :login command to call forge provider login
- Added :logout command to call forge provider logout
- Added both commands to list commands output for tab completion
- Commands now appear in forge list commands and zsh autocomplete

Co-Authored-By: ForgeCode <noreply@forgecode.dev>
@github-actions github-actions Bot added the type: feature Brand new functionality, features, pages, workflows, endpoints, etc. label Nov 7, 2025
Comment thread shell-plugin/forge.plugin.zsh Outdated

if [[ -n "$output" ]]; then
local selected
selected=$(echo "$output" | _forge_fzf --delimiter="$_FORGE_DELIMITER" --with-nth="1,3" --prompt="Provider ❯ ")

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lets reuse the :provider UI. They both are selecting a provider.

@amitksingh1490 amitksingh1490 changed the title feat: add :login and :logout commands feat: add login, logout, and custom command support to zsh plugin Nov 7, 2025

@amitksingh1490 amitksingh1490 left a comment •

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

""

@amitksingh1490 amitksingh1490 left a comment •

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

""

)))?;
}
return Ok(());
}

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we move it to api?

@tusharmath
tusharmath enabled auto-merge (squash) November 9, 2025 05:47
@tusharmath
tusharmath merged commit 0b9824b into main Nov 9, 2025
9 checks passed
@tusharmath
tusharmath deleted the feat/add-login-logout-commands branch November 9, 2025 05:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: feature Brand new functionality, features, pages, workflows, endpoints, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants