Skip to content

fix: align settings workspace shell#222

Merged
shuxueshuxue merged 2 commits intodevfrom
code-killer-phase-11
Apr 7, 2026
Merged

fix: align settings workspace shell#222
shuxueshuxue merged 2 commits intodevfrom
code-killer-phase-11

Conversation

@shuxueshuxue
Copy link
Copy Markdown
Collaborator

@shuxueshuxue shuxueshuxue commented Apr 6, 2026

Summary

  • add router-local helpers for workspace path normalization and recent-workspace list updates
  • reuse them from set_default_workspace and add_recent_workspace without flattening their route-specific semantics
  • keep repo/local persistence ownership inside each route and leave add_recent_workspace unable to mutate default_workspace

Verification

  • python3 -m py_compile backend/web/routers/settings.py
  • uv run ruff check backend/web/routers/settings.py

@shuxueshuxue shuxueshuxue marked this pull request as ready for review April 7, 2026 01:24
@shuxueshuxue shuxueshuxue merged commit d13f84d into dev Apr 7, 2026
5 of 6 checks passed
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