feat(frontend): add additional debug information when actor inspector fails to load#4346
Conversation
|
This change is part of the following stack: Change managed by git-spice. |
|
🚅 Deployed to the rivet-pr-4346 environment in rivet-frontend
|
PR ReviewThis PR improves Actor Inspector error handling and debug UX. The core feature changes are solid; a few issues worth addressing before merge. BugsSentry events fire on every render (
When both versions have pre-release tags the function returns Missing backend changeThe PR description says Minor issues
Once fetched, the token query will never be refetched. If tokens can expire or be rotated, callers will silently use a stale value. If create-once semantics are intentional, a comment would help future maintainers.
The function has version-comparison edge cases and would benefit from unit tests, especially since it gates the "upgrade RivetKit" message shown to users.
ScopeThis PR mixes the Inspector error improvements with substantial unrelated additions: managed pools query options, image repositories, Docker tags, API token creation, the new Rivet Cloud deployment option, and UI component updates. Bundling these makes the PR harder to review and revert in isolation. |
9312097 to
e97a31a
Compare
69e7656 to
d1a420b
Compare
e97a31a to
a10371b
Compare
d1a420b to
f20ed36
Compare
* feat: add deployments route * new endpoints * feat: deployments * feat: deployments * fix ts issues * fix * fix * fixes * fixes * fixes * fixes * fixes
* fix/firefox-modals * feat(frontend): add additional debug information when actor inspector fails to load (#4346) * feat/debug-inspector-info * feat: add deployments route (#4357) * feat: add deployments route * new endpoints * feat: deployments * feat: deployments * fix ts issues * fix * fix * fixes * fixes * fixes * fixes * fixes
* fix/runner-configs-outdated * fix cache management * fix: modals being stuck opaque on firefox (#4344) * fix/firefox-modals * feat(frontend): add additional debug information when actor inspector fails to load (#4346) * feat/debug-inspector-info * feat: add deployments route (#4357) * feat: add deployments route * new endpoints * feat: deployments * feat: deployments * fix ts issues * fix * fix * fixes * fixes * fixes * fixes * fixes
* fix/docs-button * fix: invalid docs link * fix: runner configs being outdated when editing again (#4343) * fix/runner-configs-outdated * fix cache management * fix: modals being stuck opaque on firefox (#4344) * fix/firefox-modals * feat(frontend): add additional debug information when actor inspector fails to load (#4346) * feat/debug-inspector-info * feat: add deployments route (#4357) * feat: add deployments route * new endpoints * feat: deployments * feat: deployments * fix ts issues * fix * fix * fixes * fixes * fixes * fixes * fixes
* fix/new-project-incorrect-redirect * fix: incorrect redirection after project creation in cloud onboarding * fix: docs-button (#4342) * fix/docs-button * fix: invalid docs link * fix: runner configs being outdated when editing again (#4343) * fix/runner-configs-outdated * fix cache management * fix: modals being stuck opaque on firefox (#4344) * fix/firefox-modals * feat(frontend): add additional debug information when actor inspector fails to load (#4346) * feat/debug-inspector-info * feat: add deployments route (#4357) * feat: add deployments route * new endpoints * feat: deployments * feat: deployments * fix ts issues * fix * fix * fixes * fixes * fixes * fixes * fixes
Closes RVT-6021
This pull request enhances error handling and version checking for the Actor Inspector in both the frontend and RivetKit backend. The main improvements include more detailed Inspector error messages, better detection of outdated RivetKit versions (including pre-release versions), and propagation of environment type (local vs deployed) through metadata.
Frontend improvements to Inspector error handling and version checks:
isVersionOutdatedto accurately detect outdated RivetKit versions, including pre-release tags (e.g., "rc", "alpha") (frontend/src/components/actors/guard-connectable-inspector.tsx).frontend/src/components/actors/guard-connectable-inspector.tsx) [1] [2].typefield ("local" or "deployed") in addition to the version, enabling environment-specific error handling (frontend/src/components/actors/actor-inspector-context.tsx).frontend/src/components/actors/guard-connectable-inspector.tsx).Backend (RivetKit) metadata enhancements:
MetadataResponsetype now includes atypefield to indicate whether the environment is "local" or "deployed", and the router endpoint sets this field accordingly (rivetkit-typescript/packages/rivetkit/src/actor/router.ts) [1] [2].