Skip to content

Conversation

@dlqqq
Copy link
Contributor

@dlqqq dlqqq commented Oct 7, 2025

Description

  • Removes the slash command listener from this package.

  • Removes backend handling of the /refresh-personas command as it will be implemented in Add existing chat commands jupyter-ai-chat-commands#1.

  • Makes PersonaManager instances accessible to other extensions by binding the dictionary to self.settings['jupyter-ai']['persona-managers'].

  • Makes PersonaManager._refresh_personas() a public method as it will be called by jupyter_ai_chat_commands.

Technical details

jupyter_ai_chat_commands will do the following in its extension to handle /refresh-personas:

  1. Asynchronously poll until self.settings['jupyter-ai']['router'] is available.
  2. Add a slash command listener. On each slash command call:
  3. Asynchronously poll until persona_manager := self.settings['jupyter-ai']['persona-managers'][room_id] are accessible.
  4. Call persona_manager._refresh_personas() in a background task.

@dlqqq dlqqq added the bug Something isn't working label Oct 7, 2025
@dlqqq dlqqq changed the title Remove slash command listener Remove slash command listener, make PersonaManagers accessible Oct 7, 2025
Copy link
Contributor

@3coins 3coins left a comment

Choose a reason for hiding this comment

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

LGTM!

@dlqqq
Copy link
Contributor Author

dlqqq commented Oct 7, 2025

@3coins Thanks! Merging & releasing as v0.0.2.

@dlqqq dlqqq merged commit 2ec15b6 into jupyter-ai-contrib:main Oct 7, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants