Skip to content

Conversation

nornagon-openai
Copy link
Collaborator

/new, /init, /models, /approvals, etc. don't work correctly during a turn. disable them.

| SlashCommand::Status
| SlashCommand::Mcp
| SlashCommand::Quit
| SlashCommand::TestApproval => true,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why do we have a test approval even?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

it's for testing the approval dialogs, it's a debug-only command. easy way to make an approval dialog appear on the screen.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think we should remove it or render it only on Openai accounts but that's for later.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@aibrahim-oai it's already only rendered if you build in debug mode, release mode doesn't have it.

@nornagon-openai nornagon-openai merged commit e5611aa into main Aug 28, 2025
35 of 39 checks passed
@nornagon-openai nornagon-openai deleted the nornagon/slash-commands-while-running branch August 28, 2025 17:16
@github-actions github-actions bot locked and limited conversation to collaborators Aug 28, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants