Hide recharge entry from account menu
This commit is contained in:
1 parent
10da39341e
commit
e9171208e6
6 files changed
+112
-61
No files matched your search
@@ -0,0 +1,75 @@
|
||||
# Task: Hide point wallet recharge entry from account menu
|
||||
|
||||
## Identity
|
||||
|
||||
- Task ID: 20261013-hide-wallet-entry-8c3eaf71
|
||||
- Mode: Feature
|
||||
- Branch: codex/20261013-hide-wallet-entry-8c3eaf71-hide-wallet-entry
|
||||
- Worktree: /Users/chillishark/Makelore 麦洛/.codex-worktrees/makelore/20261013-hide-wallet-entry-8c3eaf71
|
||||
- Base commit: 10da39341ead9e3db17ea4395d66181f6fe4433d
|
||||
- Owner: codex
|
||||
- Status: Ready for Integration
|
||||
|
||||
## Scope
|
||||
|
||||
- Hide the screenshot-marked “词元点数与充值” menu item. Retain personal balance, reserved points, retries and existing recharge implementation.
|
||||
|
||||
## Intent And Constraints
|
||||
|
||||
- User authorizes hiding this menu entry. Preserve billing contracts and all account controls. Primary checkout is occupied and remains untouched; changes and verification stay in the owned isolated worktree. No live app update or main integration is included.
|
||||
|
||||
## Outcome
|
||||
|
||||
- Sidebar hides “词元点数与充值” via PointWallet.showRechargeEntry=false. Own balance/reservations, retry, shared-funding privacy, account controls and refresh remain. Existing recharge/dialog/Main/API implementation is retained. README and the account-menu unit/Electron expectations match.
|
||||
- Independent read-only review found no actionable issue; visual screenshot confirms the entry is absent and other controls remain visible. Primary checkout and current running application are unchanged.
|
||||
|
||||
## Verification
|
||||
|
||||
- Pinned pnpm 10.33.4 (npx wrapper): typecheck passed; focused Sidebar/PointWallet unit files passed (14 tests); scoped ESLint passed; full build:vite passed (Renderer/Main/Preload/utility); permanent-point-wallet Electron spec passed (1 test).
|
||||
- Electron verifies absent entry/dialog, preserved account controls, refreshed 0→500 fixture balance after reopen and reload, and zero recharge/history requests. No real payment or model call.
|
||||
- Screenshot: test-results/permanent-point-wallet-Per-f6563--and-refreshing-the-balance/account-menu-recharge-hidden.png (ignored local QA output).
|
||||
- git diff --check passed; task-aware documentation drift check passed.
|
||||
|
||||
## Follow-ups
|
||||
|
||||
- Integrate this feature branch and activate the intended client when requested. No running-trial update, main merge or push performed.
|
||||
|
||||
## Promotion Candidates
|
||||
|
||||
- Target: current-state and permanent-points UI architecture at serialized integration. Proposal: keep account balance/reservations visible while hiding “词元点数与充值”. Evidence: user-marked screenshot, focused source diff, unit/Electron/visual verification. Future impact: account-menu work should preserve this visibility decision. Semantic conflict: earlier visible-entry description is superseded only for this entry by the explicit request; billing rules and API remain unchanged. Further human confirmation is not required for the accepted visibility change.
|
||||
|
||||
## Project Context Loaded
|
||||
|
||||
Task context:
|
||||
- Task ID: 20261013-hide-wallet-entry-8c3eaf71
|
||||
- Mode: feature
|
||||
- Branch: codex/20261013-hide-wallet-entry-8c3eaf71-hide-wallet-entry
|
||||
- Worktree: /Users/chillishark/Makelore 麦洛/.codex-worktrees/makelore/20261013-hide-wallet-entry-8c3eaf71
|
||||
- Base commit: 10da39341ead9e3db17ea4395d66181f6fe4433d
|
||||
- Ownership: official start exited 0; status matches task, branch, path and mode.
|
||||
- Other active local tasks: registry peer records reviewed read-only through Scope, Intent And Constraints and Promotion Candidates. Peers concern teacher, Agent configuration, knowledge, channel and trial work.
|
||||
- Overlap: legacy Sidebar styling/account-control work is separate and completed; no peer requests a conflicting recharge entry. Child-language record retains undefined scope (unknown coordination state) without a concrete dependency here.
|
||||
|
||||
Read:
|
||||
- AGENTS.md; 05-agent-entry/{read-before-planning,memory-index,planning-gate,read-before-coding}.md
|
||||
- Own task record; 00-brief/{project-positioning,success-criteria}.md
|
||||
- 30-worklog/current-state.md (integrated metadata and task-relevant account sections)
|
||||
- 10-decisions/decision-index.md; 20-architecture/{system-overview,community-collaboration}.md (task-relevant sections)
|
||||
- 40-domain/business-rules.md (permanent token points); evidence/reflection/commitment indexes (relevant sections)
|
||||
|
||||
Relevant understanding:
|
||||
- Goal: creation client with Main-owned authentication and server-authoritative permanent points.
|
||||
- Integrated focus: current-account recharge; no course client.
|
||||
- Scope: hide the recharge menu item while showing current balance.
|
||||
- Constraints: account identity, precise own balance, shared-wallet privacy, billing and payment-result safety remain.
|
||||
- Decision: the explicit current user request supersedes the earlier visible-entry presentation only.
|
||||
- Evidence/commitments: real payment/install smoke remains separate; this task needs no payment or model invocation.
|
||||
- Files: Sidebar, PointWallet, focused unit/Electron specs, README.
|
||||
- Unknowns: running trial may differ from main; no trial activation is requested in this turn.
|
||||
- Concurrent Task Gate: Passed. Planning Gate: Passed.
|
||||
|
||||
## Implementation Plan
|
||||
|
||||
- Add an explicit recharge-entry visibility prop and disable it in Sidebar, retaining existing wallet behavior for direct component coverage.
|
||||
- Update Sidebar/Electron expectations and README to match the hidden entry.
|
||||
- Run focused units, typecheck, scoped lint, Vite build and the account-menu Electron scenario; record review and handoff.
|
||||
Reference in new issue
Block a user