Skip to content

[backport cloud/1.37] Feat/workspaces 5 auth gate check#8357

Merged
AustinMroz merged 2 commits intocloud/1.37from
backport-8350-to-cloud-1.37
Jan 28, 2026
Merged

[backport cloud/1.37] Feat/workspaces 5 auth gate check#8357
AustinMroz merged 2 commits intocloud/1.37from
backport-8350-to-cloud-1.37

Conversation

@simula-r
Copy link
Contributor

@simula-r simula-r commented Jan 28, 2026

Summary

Backport of #8350 to cloud/1.37

  • Fix auth related race conditions with a new WorkspaceAuthGate in App.vue
  • De dup initialization calls
  • Add state machine to track state of refreshRemoteConfig
  • Fix websocket not using new workspace jwt
  • Misc improvements

Changes

Cherry-picked from commit 34fc28a

Resolved conflict in src/views/GraphView.vue:

  • Kept workspace store initialization from cloud/1.37 branch
  • Applied PR's refactored event listener pattern using useEventListener from VueUse
  • Applied PR's useIntervalFn for tab count tracking

┆Issue is synchronized with this Notion page by Unito

## Summary

- Fix auth related race conditions with a new WorkspaceAuthGate in App.vue
- De dup initialization calls
- Add state machine to track state of refreshRemoteConfig
- Fix websocket not using new workspace jwt
- Misc improvments

(cherry picked from commit 34fc28a)
@simula-r simula-r added the backport Backporting a PR onto a release candidate label Jan 28, 2026
@simula-r simula-r requested a review from a team as a code owner January 28, 2026 04:48
@dosubot dosubot bot added the size:XL This PR changes 500-999 lines, ignoring generated files. label Jan 28, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 28, 2026

Important

Review skipped

Auto reviews are limited based on label configuration.

🚫 Excluded labels (none allowed) (1)
  • backport

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • 🔍 Trigger a full review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link

github-actions bot commented Jan 28, 2026

🎨 Storybook Build Status

Build completed successfully!

⏰ Completed at: 01/28/2026, 06:17:56 AM UTC

🔗 Links


🎉 Your Storybook is ready for review!

@github-actions
Copy link

github-actions bot commented Jan 28, 2026

🎭 Playwright Tests: ❌ Failed

Results: 502 passed, 1 failed, 1 flaky, 8 skipped (Total: 512)

❌ Failed Tests

📊 Browser Reports
  • chromium: View Report (✅ 492 / ❌ 1 / ⚠️ 0 / ⏭️ 8)
  • chromium-2x: View Report (✅ 2 / ❌ 0 / ⚠️ 0 / ⏭️ 0)
  • chromium-0.5x: View Report (✅ 1 / ❌ 0 / ⚠️ 0 / ⏭️ 0)
  • mobile-chrome: View Report (✅ 7 / ❌ 0 / ⚠️ 1 / ⏭️ 0)

The extensionManager assignment was accidentally dropped when resolving
conflicts during cherry-pick. This caused all Playwright tests to fail
waiting for window['app'].extensionManager to be defined.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@simula-r
Copy link
Contributor Author

simula-r commented Jan 28, 2026

Playwright failure unrelated to these changes

@AustinMroz AustinMroz merged commit 8e5a037 into cloud/1.37 Jan 28, 2026
24 of 25 checks passed
@AustinMroz AustinMroz deleted the backport-8350-to-cloud-1.37 branch January 28, 2026 06:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport Backporting a PR onto a release candidate size:XL This PR changes 500-999 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants