Expose provider to Devtools and show correct message when Transitions are not supported#628
Closed
henri-hulski wants to merge 2 commits intonextfrom
Closed
Expose provider to Devtools and show correct message when Transitions are not supported#628henri-hulski wants to merge 2 commits intonextfrom
henri-hulski wants to merge 2 commits intonextfrom
Conversation
Review or Edit in CodeSandboxOpen the branch in Web Editor • VS Code • Insiders |
There was a problem hiding this comment.
Pull Request Overview
This PR enhances the devtools integration by exposing provider details (name and version) and updating the messaging in the Transitions tab based on Overmind’s feature support. It also updates dependency versions across multiple packages and adjusts CI workflow configurations.
- Exposes provider information to devtools in Overmind and devtools client.
- Updates dependency versions and removes unneeded devDependencies in several package.json files.
- Adjusts messaging in the StateMachines component and updates CI workflows.
Reviewed Changes
Copilot reviewed 23 out of 23 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| packages/overmind/src/Overmind.ts | Introduces provider info; note potential misassignment of version from package.json |
| packages/overmind/package.json | Updates dependency versions |
| packages/overmind-vue/package.json | Removes devDependencies section |
| packages/overmind-svelte/package.json | Removes devDependencies entries |
| packages/overmind-statechart/package.json | Removes devDependencies entries |
| packages/overmind-react/package.json | Updates dependency versions and test libraries |
| packages/overmind-graphql/package.json | Updates graphql dependency version |
| packages/overmind-devtools/package.json | Updates electron and ws version |
| packages/overmind-devtools-vscode/package.json | Updates dependencies for babel-loader and webpack |
| packages/overmind-devtools-client/src/overmind/utils.ts | Adds provider default values in createApp |
| packages/overmind-devtools-client/src/overmind/types.ts | Extends type definitions with the provider property |
| packages/overmind-devtools-client/src/overmind/operators.ts | Propagates provider info from incoming messages |
| packages/overmind-devtools-client/src/components/StateMachines/index.tsx | Adds conditional messaging based on provider details |
| packages/overmind-devtools-client/package.json | Updates several dependency versions |
| packages/betsy/package.json | Removes unnecessary devDependencies |
| package.json | Updates versions for electron, graphql, ws, and devDependencies |
| README.md | Updates token expiration date in documentation |
| Procfile | Removes the Procfile |
| .github/workflows/test.yml | Adds Node 24.x to test matrix |
| .github/workflows/release.yml | Updates Node version to 22 for release workflow |
5395c49 to
684cea2
Compare
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.
This PR exposes devtools provider name and version to Devtools, which makes devtools more generic.
It allows to handle things differently depending which provider (and with which version) is sending messages to devtools.
Beside Overmind this could be e.g. Mobx Lite.
The PR also shows the correct message in the Transitions tab when the Overmind version doesn't support transitions.