Context
PR LeanerCloud/cloud-commitments-cli#947 renumbered the Purchaser group migration from 000058 to 000059 to avoid a collision.
The backend (internal/auth/types.go:320) and the SQL files were updated, but two frontend
comment references were not:
frontend/src/permissions.ts:81 -- JSDoc for PURCHASER_GROUP_ID still says migration 000058
frontend/src/users/userActions.ts:101 -- inline comment still says migration 000058
These are comment-only changes and carry no runtime risk, but they create a misleading paper trail
for future developers reading the code.
Fix
In each file, replace 000058 with 000059 in the migration-reference comment only.
Discovered in
CodeRabbit review of PR LeanerCloud/cloud-commitments-cli#947 (anchored to internal/auth/types.go:320). The files were not in
scope for that PR so the fix was deferred here per scope-discipline rules.
Context
PR LeanerCloud/cloud-commitments-cli#947 renumbered the Purchaser group migration from
000058to000059to avoid a collision.The backend (
internal/auth/types.go:320) and the SQL files were updated, but two frontendcomment references were not:
frontend/src/permissions.ts:81-- JSDoc forPURCHASER_GROUP_IDstill saysmigration 000058frontend/src/users/userActions.ts:101-- inline comment still saysmigration 000058These are comment-only changes and carry no runtime risk, but they create a misleading paper trail
for future developers reading the code.
Fix
In each file, replace
000058with000059in the migration-reference comment only.Discovered in
CodeRabbit review of PR LeanerCloud/cloud-commitments-cli#947 (anchored to
internal/auth/types.go:320). The files were not inscope for that PR so the fix was deferred here per scope-discipline rules.