Problem
In cli/index_list.go:
- Line 26:
Short: "Shows the list indexes in the database..." — ungrammatical, inconsistent with other CLI commands that start with an imperative verb (e.g. "Show ...").
- Line 27:
Long: Shows the list indexes... — same issue.
- Line 62: example label
"show all index for 'Users' collection" — should be plural "indexes".
Also in cli/errors.go:37:
ErrInvalidLensConfig = errors.New("invalid lens configuration") hard-codes the literal instead of reusing errInvalidLensConfig (line 20), unlike ErrEmptyCollectionSDL which correctly reuses its const. Behavior-preserving cleanup to prevent future drift.
Expected
- "Show the list of indexes ..." for Short/Long.
- Example: "show all indexes for 'Users' collection".
ErrInvalidLensConfig = errors.New(errInvalidLensConfig).
Location
cli/index_list.go:26-27,62cli/errors.go:20,37
Verified on current develop.
Problem
In
cli/index_list.go:Short: "Shows the list indexes in the database..."— ungrammatical, inconsistent with other CLI commands that start with an imperative verb (e.g. "Show ...").Long: Shows the list indexes...— same issue."show all index for 'Users' collection"— should be plural "indexes".Also in
cli/errors.go:37:ErrInvalidLensConfig = errors.New("invalid lens configuration")hard-codes the literal instead of reusingerrInvalidLensConfig(line 20), unlikeErrEmptyCollectionSDLwhich correctly reuses its const. Behavior-preserving cleanup to prevent future drift.Expected
ErrInvalidLensConfig = errors.New(errInvalidLensConfig).Location
cli/index_list.go:26-27,62cli/errors.go:20,37Verified on current
develop.