Skip to content

Improve connector reload authentication UX#2370

Open
alexyaang wants to merge 1 commit into
github:mainfrom
alexyaang:alexyaang-connector-auth-ux
Open

Improve connector reload authentication UX#2370
alexyaang wants to merge 1 commit into
github:mainfrom
alexyaang:alexyaang-connector-auth-ux

Conversation

@alexyaang

Copy link
Copy Markdown
Member

Summary

  • keep Azure access and refresh tokens in process memory instead of using the persistent native cache
  • retain the selected connector namespace across reloads and show a focused sign-in state when authentication is missing
  • return directly to the linked namespace catalog after browser sign-in
  • update authentication, renderer, server regression coverage, and documentation

Validation

  • 109 connector tests pass
  • plugin validation passes
  • marketplace generation produces no additional changes

Keep Azure tokens in memory and preserve linked namespace context across extension reloads.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@alexyaang
alexyaang requested a review from aaronpowell as a code owner July 21, 2026 01:44
Copilot AI review requested due to automatic review settings July 21, 2026 01:44
@github-actions github-actions Bot added the canvas-extension PR touches canvas extensions label Jul 21, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Moves Azure authentication to process memory while preserving namespace selection across reloads.

Changes:

  • Removes persistent Azure credential caching.
  • Adds linked-namespace sign-in and catalog return flow.
  • Updates regression tests and documentation.
Show a summary per file
File Description
extensions/connector-namespaces/auth.mjs Uses in-memory credentials and removes legacy artifacts.
extensions/connector-namespaces/auth.test.mjs Tests in-memory authentication behavior.
extensions/connector-namespaces/package.json Removes the persistence dependency.
extensions/connector-namespaces/renderer.mjs Adds focused linked-namespace sign-in UI.
extensions/connector-namespaces/renderer.test.mjs Covers rendering, redirects, and escaping.
extensions/connector-namespaces/server.mjs Passes saved namespace context to the renderer.
extensions/connector-namespaces/server.test.mjs Verifies reload sign-in messaging.
extensions/connector-namespaces/review-fixes.test.mjs Guards against persistent-cache usage.
extensions/connector-namespaces/README.md Documents process-memory authentication.
extensions/connector-namespaces/test/README.md Updates smoke-test prerequisites.

Review details

  • Files reviewed: 10/10 changed files
  • Comments generated: 0
  • Review effort level: Medium

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

canvas-extension PR touches canvas extensions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants