mirror of
https://gitee.com/wanwujie/sub2api
synced 2026-05-05 21:50:44 +08:00
docs: add account bulk edit implementation plan
This commit is contained in:
KnowSky404
@@ -0,0 +1,359 @@
|
|||||||
|
# Account Bulk Edit Scope And Compact Implementation Plan
|
||||||
|
|
||||||
|
> **For agentic workers:** REQUIRED SUB-SKILL: Use superpowers:subagent-driven-development (recommended) or superpowers:executing-plans to implement this plan task-by-task. Steps use checkbox (`- [ ]`) syntax for tracking.
|
||||||
|
|
||||||
|
**Goal:** Add filter-result bulk edit to admin accounts, unify the table-level bulk-edit entry, and align OpenAI bulk-edit controls with the existing compact-related single-account settings.
|
||||||
|
|
||||||
|
**Architecture:** Extend the existing `/admin/accounts/bulk-update` flow to accept either explicit account IDs or a server-resolved filter target. Reuse the current account-list filter contract for scope resolution, then update the accounts view and bulk-edit modal so the UI can launch either selected-account edits or current-filter-result edits from one compact dropdown. Keep the existing bulk-edit form, but expand its target contract and OpenAI-specific field coverage.
|
||||||
|
|
||||||
|
**Tech Stack:** Vue 3, TypeScript, Vitest, Gin, Go service/repository layer, existing admin accounts API.
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
### Task 1: Add backend test coverage for filter-target bulk update
|
||||||
|
|
||||||
|
**Files:**
|
||||||
|
- Modify: `backend/internal/handler/admin/account_handler_mixed_channel_test.go`
|
||||||
|
- Modify: `backend/internal/service/admin_service_bulk_update_test.go`
|
||||||
|
- Test: `backend/internal/handler/admin/account_handler_mixed_channel_test.go`
|
||||||
|
- Test: `backend/internal/service/admin_service_bulk_update_test.go`
|
||||||
|
|
||||||
|
- [ ] **Step 1: Write the failing handler test for filter-target request acceptance**
|
||||||
|
|
||||||
|
```go
|
||||||
|
func TestBulkUpdateAcceptsFilterTargetRequest(t *testing.T) {
|
||||||
|
// add a request body that omits account_ids and submits filters instead
|
||||||
|
// assert the route does not reject the request as malformed once service stubs are wired
|
||||||
|
}
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 2: Run test to verify it fails**
|
||||||
|
|
||||||
|
Run: `GOCACHE=/tmp/go-build GOMODCACHE=/tmp/go-mod go test ./backend/internal/handler/admin -run TestBulkUpdateAcceptsFilterTargetRequest -count=1`
|
||||||
|
Expected: FAIL because `BulkUpdateAccountsRequest` does not yet support `filters`.
|
||||||
|
|
||||||
|
- [ ] **Step 3: Write the failing service test for resolving IDs from filters**
|
||||||
|
|
||||||
|
```go
|
||||||
|
func TestAdminServiceBulkUpdateAccounts_ResolvesIDsFromFilters(t *testing.T) {
|
||||||
|
// construct BulkUpdateAccountsInput with Filters and no AccountIDs
|
||||||
|
// stub repository list/search path to return matching IDs
|
||||||
|
// assert BulkUpdate is called with all matching account IDs
|
||||||
|
}
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 4: Run test to verify it fails**
|
||||||
|
|
||||||
|
Run: `GOCACHE=/tmp/go-build GOMODCACHE=/tmp/go-mod go test ./backend/internal/service -run TestAdminServiceBulkUpdateAccounts_ResolvesIDsFromFilters -count=1`
|
||||||
|
Expected: FAIL because `BulkUpdateAccountsInput` and service logic only use explicit `AccountIDs`.
|
||||||
|
|
||||||
|
- [ ] **Step 5: Commit**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
git add backend/internal/handler/admin/account_handler_mixed_channel_test.go backend/internal/service/admin_service_bulk_update_test.go
|
||||||
|
git commit -m "test: cover filter-target account bulk update"
|
||||||
|
```
|
||||||
|
|
||||||
|
### Task 2: Implement backend filter-target bulk update
|
||||||
|
|
||||||
|
**Files:**
|
||||||
|
- Modify: `backend/internal/handler/admin/account_handler.go`
|
||||||
|
- Modify: `backend/internal/service/admin_service.go`
|
||||||
|
- Modify: `backend/internal/repository/account_repo.go`
|
||||||
|
- Modify: `backend/internal/service/account_service.go`
|
||||||
|
- Test: `backend/internal/handler/admin/account_handler_mixed_channel_test.go`
|
||||||
|
- Test: `backend/internal/service/admin_service_bulk_update_test.go`
|
||||||
|
|
||||||
|
- [ ] **Step 1: Implement request structs and validation for filter targets**
|
||||||
|
|
||||||
|
```go
|
||||||
|
type BulkUpdateAccountFilters struct {
|
||||||
|
Platform string `json:"platform"`
|
||||||
|
Type string `json:"type"`
|
||||||
|
Status string `json:"status"`
|
||||||
|
Group string `json:"group"`
|
||||||
|
Search string `json:"search"`
|
||||||
|
PrivacyMode string `json:"privacy_mode"`
|
||||||
|
}
|
||||||
|
|
||||||
|
type BulkUpdateAccountsRequest struct {
|
||||||
|
AccountIDs []int64 `json:"account_ids"`
|
||||||
|
Filters *BulkUpdateAccountFilters `json:"filters"`
|
||||||
|
// existing fields remain unchanged
|
||||||
|
}
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 2: Resolve filter targets in the service layer with one canonical path**
|
||||||
|
|
||||||
|
```go
|
||||||
|
type BulkUpdateAccountsInput struct {
|
||||||
|
AccountIDs []int64
|
||||||
|
Filters *BulkUpdateAccountFilters
|
||||||
|
// existing fields remain unchanged
|
||||||
|
}
|
||||||
|
|
||||||
|
if len(input.AccountIDs) == 0 && input.Filters != nil {
|
||||||
|
ids, err := s.resolveBulkUpdateTargetIDs(ctx, input.Filters)
|
||||||
|
if err != nil {
|
||||||
|
return nil, err
|
||||||
|
}
|
||||||
|
input.AccountIDs = ids
|
||||||
|
}
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 3: Reuse existing account-search/repository logic to resolve all matching IDs**
|
||||||
|
|
||||||
|
```go
|
||||||
|
func (s *AdminService) resolveBulkUpdateTargetIDs(ctx context.Context, filters *BulkUpdateAccountFilters) ([]int64, error) {
|
||||||
|
// call the existing repository list/search path with the submitted filters
|
||||||
|
// page through all matching rows or use a dedicated ID-only query helper
|
||||||
|
// return unique IDs in stable order
|
||||||
|
}
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 4: Run targeted backend tests**
|
||||||
|
|
||||||
|
Run: `GOCACHE=/tmp/go-build GOMODCACHE=/tmp/go-mod go test ./backend/internal/handler/admin ./backend/internal/service -run 'TestBulkUpdateAcceptsFilterTargetRequest|TestAdminServiceBulkUpdateAccounts_ResolvesIDsFromFilters' -count=1`
|
||||||
|
Expected: PASS
|
||||||
|
|
||||||
|
- [ ] **Step 5: Commit**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
git add backend/internal/handler/admin/account_handler.go backend/internal/service/admin_service.go backend/internal/repository/account_repo.go backend/internal/service/account_service.go backend/internal/handler/admin/account_handler_mixed_channel_test.go backend/internal/service/admin_service_bulk_update_test.go
|
||||||
|
git commit -m "feat: support filter-target account bulk update"
|
||||||
|
```
|
||||||
|
|
||||||
|
### Task 3: Add frontend API and modal tests for target scope
|
||||||
|
|
||||||
|
**Files:**
|
||||||
|
- Modify: `frontend/src/components/account/__tests__/BulkEditAccountModal.spec.ts`
|
||||||
|
- Create: `frontend/src/views/admin/__tests__/AccountsView.bulkEdit.spec.ts`
|
||||||
|
- Modify: `frontend/src/api/admin/accounts.ts`
|
||||||
|
- Test: `frontend/src/components/account/__tests__/BulkEditAccountModal.spec.ts`
|
||||||
|
- Test: `frontend/src/views/admin/__tests__/AccountsView.bulkEdit.spec.ts`
|
||||||
|
|
||||||
|
- [ ] **Step 1: Write the failing modal test for filter-target payload submission**
|
||||||
|
|
||||||
|
```ts
|
||||||
|
it('submits bulk edit using current filters when target mode is filtered-results', async () => {
|
||||||
|
// mount BulkEditAccountModal with targetMode='filtered'
|
||||||
|
// submit a minimal change
|
||||||
|
// expect adminAPI.accounts.bulkUpdate to receive { filters: ... } rather than account_ids
|
||||||
|
})
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 2: Run test to verify it fails**
|
||||||
|
|
||||||
|
Run: `pnpm -C frontend test:run src/components/account/__tests__/BulkEditAccountModal.spec.ts -t "filtered-results"`
|
||||||
|
Expected: FAIL because the modal only accepts `accountIds`.
|
||||||
|
|
||||||
|
- [ ] **Step 3: Write the failing accounts-view test for dropdown launch actions**
|
||||||
|
|
||||||
|
```ts
|
||||||
|
it('opens bulk edit for current filtered results from the table action dropdown', async () => {
|
||||||
|
// mount AccountsView with filters set
|
||||||
|
// click Bulk edit > current filtered results
|
||||||
|
// assert modal props contain filter target metadata
|
||||||
|
})
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 4: Run test to verify it fails**
|
||||||
|
|
||||||
|
Run: `pnpm -C frontend test:run src/views/admin/__tests__/AccountsView.bulkEdit.spec.ts`
|
||||||
|
Expected: FAIL because the dropdown action and target scope state do not exist yet.
|
||||||
|
|
||||||
|
- [ ] **Step 5: Commit**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
git add frontend/src/components/account/__tests__/BulkEditAccountModal.spec.ts frontend/src/views/admin/__tests__/AccountsView.bulkEdit.spec.ts frontend/src/api/admin/accounts.ts
|
||||||
|
git commit -m "test: cover account bulk edit target scopes"
|
||||||
|
```
|
||||||
|
|
||||||
|
### Task 4: Implement unified frontend bulk-edit target scope flow
|
||||||
|
|
||||||
|
**Files:**
|
||||||
|
- Modify: `frontend/src/views/admin/AccountsView.vue`
|
||||||
|
- Modify: `frontend/src/components/admin/account/AccountBulkActionsBar.vue`
|
||||||
|
- Modify: `frontend/src/components/account/BulkEditAccountModal.vue`
|
||||||
|
- Modify: `frontend/src/api/admin/accounts.ts`
|
||||||
|
- Modify: `frontend/src/i18n/locales/zh.ts`
|
||||||
|
- Modify: `frontend/src/i18n/locales/en.ts`
|
||||||
|
- Test: `frontend/src/components/account/__tests__/BulkEditAccountModal.spec.ts`
|
||||||
|
- Test: `frontend/src/views/admin/__tests__/AccountsView.bulkEdit.spec.ts`
|
||||||
|
|
||||||
|
- [ ] **Step 1: Add a typed frontend target contract for bulk edit**
|
||||||
|
|
||||||
|
```ts
|
||||||
|
export type AccountBulkEditTarget =
|
||||||
|
| { mode: 'selected'; accountIds: number[]; selectedPlatforms: AccountPlatform[]; selectedTypes: AccountType[] }
|
||||||
|
| { mode: 'filtered'; filters: AccountListFilters; previewCount: number; selectedPlatforms: AccountPlatform[]; selectedTypes: AccountType[] }
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 2: Replace the single selected-row edit button with one dropdown**
|
||||||
|
|
||||||
|
```vue
|
||||||
|
<BulkEditDropdown
|
||||||
|
:has-selection="selectedIds.length > 0"
|
||||||
|
@edit-selected="openBulkEditSelected"
|
||||||
|
@edit-filtered="openBulkEditFiltered"
|
||||||
|
/>
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 3: Snapshot current filters and preview count when launching filtered mode**
|
||||||
|
|
||||||
|
```ts
|
||||||
|
const openBulkEditFiltered = async () => {
|
||||||
|
const filters = toBulkEditFilterSnapshot(params)
|
||||||
|
const preview = await adminAPI.accounts.list(1, 1, filters)
|
||||||
|
bulkEditTarget.value = {
|
||||||
|
mode: 'filtered',
|
||||||
|
filters,
|
||||||
|
previewCount: preview.pagination.total,
|
||||||
|
selectedPlatforms: collectPlatforms(preview.data),
|
||||||
|
selectedTypes: collectTypes(preview.data)
|
||||||
|
}
|
||||||
|
showBulkEdit.value = true
|
||||||
|
}
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 4: Update modal submission to call `bulkUpdate` with either `account_ids` or `filters`**
|
||||||
|
|
||||||
|
```ts
|
||||||
|
if (props.target.mode === 'selected') {
|
||||||
|
await adminAPI.accounts.bulkUpdate({ account_ids: props.target.accountIds, ...updates })
|
||||||
|
} else {
|
||||||
|
await adminAPI.accounts.bulkUpdate({ filters: props.target.filters, ...updates })
|
||||||
|
}
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 5: Run targeted frontend tests**
|
||||||
|
|
||||||
|
Run: `pnpm -C frontend test:run src/components/account/__tests__/BulkEditAccountModal.spec.ts src/views/admin/__tests__/AccountsView.bulkEdit.spec.ts`
|
||||||
|
Expected: PASS
|
||||||
|
|
||||||
|
- [ ] **Step 6: Commit**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
git add frontend/src/views/admin/AccountsView.vue frontend/src/components/admin/account/AccountBulkActionsBar.vue frontend/src/components/account/BulkEditAccountModal.vue frontend/src/api/admin/accounts.ts frontend/src/i18n/locales/zh.ts frontend/src/i18n/locales/en.ts frontend/src/components/account/__tests__/BulkEditAccountModal.spec.ts frontend/src/views/admin/__tests__/AccountsView.bulkEdit.spec.ts
|
||||||
|
git commit -m "feat: add filtered-result account bulk edit"
|
||||||
|
```
|
||||||
|
|
||||||
|
### Task 5: Add failing tests for missing OpenAI bulk-edit fields
|
||||||
|
|
||||||
|
**Files:**
|
||||||
|
- Modify: `frontend/src/components/account/__tests__/BulkEditAccountModal.spec.ts`
|
||||||
|
- Test: `frontend/src/components/account/__tests__/BulkEditAccountModal.spec.ts`
|
||||||
|
|
||||||
|
- [ ] **Step 1: Write the failing OAuth test for `codex_cli_only`**
|
||||||
|
|
||||||
|
```ts
|
||||||
|
it('OpenAI OAuth bulk edit can submit codex_cli_only', async () => {
|
||||||
|
// enable the toggle and submit
|
||||||
|
// expect extra.codex_cli_only to be sent
|
||||||
|
})
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 2: Run test to verify it fails**
|
||||||
|
|
||||||
|
Run: `pnpm -C frontend test:run src/components/account/__tests__/BulkEditAccountModal.spec.ts -t "codex_cli_only"`
|
||||||
|
Expected: FAIL because the modal has no such control or payload mapping.
|
||||||
|
|
||||||
|
- [ ] **Step 3: Write the failing API key test for API key WS mode**
|
||||||
|
|
||||||
|
```ts
|
||||||
|
it('OpenAI API key bulk edit submits API key WS mode fields', async () => {
|
||||||
|
// enable the API key WS mode selector and submit
|
||||||
|
// expect openai_apikey_responses_websockets_v2_mode and enabled flag
|
||||||
|
})
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 4: Run test to verify it fails**
|
||||||
|
|
||||||
|
Run: `pnpm -C frontend test:run src/components/account/__tests__/BulkEditAccountModal.spec.ts -t "API key WS mode"`
|
||||||
|
Expected: FAIL because the modal only submits OAuth WS mode.
|
||||||
|
|
||||||
|
- [ ] **Step 5: Commit**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
git add frontend/src/components/account/__tests__/BulkEditAccountModal.spec.ts
|
||||||
|
git commit -m "test: cover missing OpenAI bulk edit fields"
|
||||||
|
```
|
||||||
|
|
||||||
|
### Task 6: Implement missing OpenAI bulk-edit controls and payload wiring
|
||||||
|
|
||||||
|
**Files:**
|
||||||
|
- Modify: `frontend/src/components/account/BulkEditAccountModal.vue`
|
||||||
|
- Modify: `frontend/src/i18n/locales/zh.ts`
|
||||||
|
- Modify: `frontend/src/i18n/locales/en.ts`
|
||||||
|
- Test: `frontend/src/components/account/__tests__/BulkEditAccountModal.spec.ts`
|
||||||
|
|
||||||
|
- [ ] **Step 1: Add UI controls for OAuth `codex_cli_only` and API key WS mode**
|
||||||
|
|
||||||
|
```vue
|
||||||
|
<div v-if="allOpenAIOAuth">
|
||||||
|
<!-- existing OAuth WS mode -->
|
||||||
|
<!-- add codex_cli_only toggle -->
|
||||||
|
</div>
|
||||||
|
|
||||||
|
<div v-if="allOpenAIAPIKey">
|
||||||
|
<!-- add API key WS mode selector -->
|
||||||
|
</div>
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 2: Mirror single-account payload semantics in the bulk-edit submit builder**
|
||||||
|
|
||||||
|
```ts
|
||||||
|
if (enableCodexCLIOnly.value) {
|
||||||
|
const extra = ensureExtra()
|
||||||
|
extra.codex_cli_only = codexCLIOnlyEnabled.value
|
||||||
|
}
|
||||||
|
|
||||||
|
if (enableOpenAIAPIKeyWSMode.value) {
|
||||||
|
const extra = ensureExtra()
|
||||||
|
extra.openai_apikey_responses_websockets_v2_mode = openaiAPIKeyResponsesWebSocketV2Mode.value
|
||||||
|
extra.openai_apikey_responses_websockets_v2_enabled = isOpenAIWSModeEnabled(openaiAPIKeyResponsesWebSocketV2Mode.value)
|
||||||
|
}
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 3: Run focused modal tests**
|
||||||
|
|
||||||
|
Run: `pnpm -C frontend test:run src/components/account/__tests__/BulkEditAccountModal.spec.ts`
|
||||||
|
Expected: PASS
|
||||||
|
|
||||||
|
- [ ] **Step 4: Commit**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
git add frontend/src/components/account/BulkEditAccountModal.vue frontend/src/i18n/locales/zh.ts frontend/src/i18n/locales/en.ts frontend/src/components/account/__tests__/BulkEditAccountModal.spec.ts
|
||||||
|
git commit -m "feat: align OpenAI bulk edit compact settings"
|
||||||
|
```
|
||||||
|
|
||||||
|
### Task 7: Final regression verification
|
||||||
|
|
||||||
|
**Files:**
|
||||||
|
- Modify: none expected
|
||||||
|
- Test: `frontend/src/components/account/__tests__/BulkEditAccountModal.spec.ts`
|
||||||
|
- Test: `frontend/src/views/admin/__tests__/AccountsView.bulkEdit.spec.ts`
|
||||||
|
- Test: `backend/internal/handler/admin/account_handler_mixed_channel_test.go`
|
||||||
|
- Test: `backend/internal/service/admin_service_bulk_update_test.go`
|
||||||
|
|
||||||
|
- [ ] **Step 1: Run frontend typecheck**
|
||||||
|
|
||||||
|
Run: `pnpm -C frontend typecheck`
|
||||||
|
Expected: PASS
|
||||||
|
|
||||||
|
- [ ] **Step 2: Run focused frontend test suite**
|
||||||
|
|
||||||
|
Run: `pnpm -C frontend test:run src/components/account/__tests__/BulkEditAccountModal.spec.ts src/views/admin/__tests__/AccountsView.bulkEdit.spec.ts`
|
||||||
|
Expected: PASS
|
||||||
|
|
||||||
|
- [ ] **Step 3: Run focused backend test suite**
|
||||||
|
|
||||||
|
Run: `GOCACHE=/tmp/go-build GOMODCACHE=/tmp/go-mod go test ./backend/internal/handler/admin ./backend/internal/service -run 'BulkUpdate|bulk update' -count=1`
|
||||||
|
Expected: PASS
|
||||||
|
|
||||||
|
- [ ] **Step 4: Commit final integration fixes if needed**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
git add frontend/src/components/account/BulkEditAccountModal.vue frontend/src/views/admin/AccountsView.vue frontend/src/components/admin/account/AccountBulkActionsBar.vue frontend/src/api/admin/accounts.ts frontend/src/i18n/locales/zh.ts frontend/src/i18n/locales/en.ts backend/internal/handler/admin/account_handler.go backend/internal/service/admin_service.go backend/internal/repository/account_repo.go backend/internal/service/account_service.go frontend/src/components/account/__tests__/BulkEditAccountModal.spec.ts frontend/src/views/admin/__tests__/AccountsView.bulkEdit.spec.ts backend/internal/handler/admin/account_handler_mixed_channel_test.go backend/internal/service/admin_service_bulk_update_test.go
|
||||||
|
git commit -m "feat: finish account bulk edit scope and compact support"
|
||||||
|
```
|
||||||
Reference in New Issue
Block a user