-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Composable Extensions #61
Comments
With this in mind, should we rename the repo? I liked your description on stream, vscode-line-highlighter. |
Yes we'll need to do some significant refactoring to pull this off I think. I've been looking into how the Azure suite of VS Code extensions work since there's a main extension "Azure Account" and then other extensions depend upon it (like Azure Functions). Here's where Azure Functions extension declares its dependency on another extension (Azure Account): https://github.com/Microsoft/vscode-azurefunctions/blob/master/package.json#L864 I've not found how it's being used yet within the Azure Functions extension at this time thought. |
It appears they call commands in Azure Account from Azure Functions. |
Oh, nice find! I overlooked that in the code! That's great to know for understanding how to call other extension commands. Thinking about how we could modularize this extension, it seems like breaking out the language server (Twitch chat listener) from the UI components (commands, tree view, and status bar icon) into separate extensions would be best. What do you think? With that in mind I suppose the integration point between UI based extensions and the language server extension would be to listen to the IPC messages on the connection, right? I wonder if those This was me sharing some thoughts to the discussion. Let me know if you have any as well. |
I was thinking about that as well and I wondered, why do we need the language server? I agree that we need to separate the UI components from the chat service components. I was thinking we could create an interface that chat extensions (i.e. our twitch client) would implement that includes the |
Removed milestone 0.2.0 since we already baked that stone... |
…-param lowercase command as well as param. Closes clarkio#61
It would be nice to break out components within this extension (such as the twitch chat listener component) into their own extension to allow other extensions to depend upon it and interact with it.
This would allow us to do things like letting the Highlighter UI display highlights regardless of which streaming platform chat system is connected.
The text was updated successfully, but these errors were encountered: