Skip to content
This repository was archived by the owner on Jul 24, 2026. It is now read-only.

Fix switching users on client, properly changes stdiod config#40

Merged
dimitriosGX merged 1 commit into
mainfrom
dk-stdio-switch
Jul 20, 2026
Merged

Fix switching users on client, properly changes stdiod config#40
dimitriosGX merged 1 commit into
mainfrom
dk-stdio-switch

Conversation

@dimitriosGX

@dimitriosGX dimitriosGX commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Summary by cubic

Ensure switching users and signing out correctly reconfigures the stdiod daemon so it doesn't keep tunneling under the previous account. Adds safe teardown on sign-out and reprovisions the daemon on account switch.

  • Bug Fixes
    • On in-app sign-out (setup:reset), unload stdiod with purge:false to stop the tunnel but keep config.toml; errors are swallowed so sign-out never blocks.
    • On account switch, re-point stdiod to the active account if installed and credentials/backend exist; otherwise stop it. No-op if not installed; skipped under EDISON_DRY_RUN=1.
    • Wire-ups in IPC: call the teardown on sign-out and reprovision after switching accounts. Added unit tests covering all paths.

Written for commit e45f0a5. Summary will update on new commits.

Review in cubic

@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

All reported issues were addressed across 3 files

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread src/main/stdiod/accountSwitch.ts
Comment thread src/main/stdiod/accountSwitch.ts
@dimitriosGX
dimitriosGX merged commit 8e980e2 into main Jul 20, 2026
5 checks passed
@dimitriosGX
dimitriosGX deleted the dk-stdio-switch branch July 20, 2026 12:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant