Skip to content

fix(dashboard): preserve profile unlock session - #251

Merged
GamerYuan merged 1 commit into
devfrom
fix/dashboard-profile-unlock-session
Oct 2, 2026
Merged

GamerYuan merged 1 commit into
devfrom
fix/dashboard-profile-unlock-session

Conversation

@GamerYuan

Copy link
Copy Markdown
Owner

Preserves the dashboard login session during command execution. Adds regression
coverage for builder and direct-DI paths. All 156 Dashboard tests passed; independent review approved.

@coderabbitai

coderabbitai Bot commented Oct 2, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Advanced

Run ID: 92355a65-af2e-4b6d-8a2a-a1cd34bd593a

  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


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.

@github-actions

github-actions Bot commented Oct 2, 2026

Copy link
Copy Markdown

No actionable code quality issues or bugs found.

The change correctly:

  • Forwards the current dashboard_session claim during command execution.
  • Supports both direct DI and builder-created executors.
  • Uses the existing IHttpContextAccessor registration.
  • Includes regression coverage for both paths.

dotnet build and tests could not complete because the required sixlabors.lic file is unavailable in the environment.

New%20session%20-%202026-10-02T15%3A08%3A40.683Z
opencode session  |  github run

@GamerYuan
GamerYuan merged commit ea9099a into dev Oct 2, 2026
6 checks passed
@GamerYuan
GamerYuan deleted the fix/dashboard-profile-unlock-session branch October 2, 2026 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant