Skip to content

feat: route accounts through sidecar sqlite store#4

Merged
linhay merged 4 commits into
masterfrom
账号与凭证统一存储方案
May 29, 2026

Hidden character warning

The head ref may contain hidden characters: "\u8d26\u53f7\u4e0e\u51ed\u8bc1\u7edf\u4e00\u5b58\u50a8\u65b9\u6848"
Merged

feat: route accounts through sidecar sqlite store#4
linhay merged 4 commits into
masterfrom
账号与凭证统一存储方案

Conversation

@linhay
Copy link
Copy Markdown
Contributor

@linhay linhay commented May 29, 2026

Summary

  • add sidecar-owned SQLite account store and unified management APIs
  • route GetTokens account CRUD through sidecar unified account APIs
  • record execution plan and migration/delete-legacy checklist in docs-linhay space

Tests

  • sidecar: go test ./internal/...
  • GetTokens: go test ./internal/...
  • frontend: npm run typecheck
  • frontend: npm run test:unit (614 passed)
  • frontend: npm run build (passes with existing Vite chunk size warning)

Remaining rollout checks

  • real desktop validation for account list/create/edit/disable/priority/delete/restart
  • real profile migration dry-run, commit, credential update validation, then delete legacy sources

Copilot AI review requested due to automatic review settings May 29, 2026 07:54
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@linhay linhay merged commit 5c5f31a into master May 29, 2026
@linhay linhay deleted the 账号与凭证统一存储方案 branch June 3, 2026 09:05
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.

2 participants