← Back to PRs

#23247: fix(cli): removes --query from memory cmd options

by stuhorsman open 2026-02-22 04:28 View on GitHub →
cli size: XS
## Summary Describe the problem and fix in 2–5 bullets: - Problem: Incorrect syntax is cmd line help for `openclaw memory search...` - Why it matters: cmd options should be correct - What changed: updated cmd options - What did NOT change (scope boundary): ## Change Type (select all) - [X] Bug fix - [ ] Feature - [ ] Refactor - [ ] Docs - [ ] Security hardening - [ ] Chore/infra ## Scope (select all touched areas) - [ ] Gateway / orchestration - [ ] Skills / tool execution - [ ] Auth / tokens - [ ] Memory / storage - [ ] Integrations - [ ] API / contracts - [ ] UI / DX - [ ] CI/CD / infra ## Linked Issue/PR - Closes # https://github.com/openclaw/openclaw/issues/23231 - Related # ## User-visible / Behavior Changes Updated cli options ## Security Impact (required) - New permissions/capabilities? (No) - Secrets/tokens handling changed? (No) - New/changed network calls? (No) - Command/tool execution surface changed? (No) - Data access scope changed? (No) - If any `Yes`, explain risk + mitigation: ## Repro + Verification ### Environment - OS: - Runtime/container: - Model/provider: - Integration/channel (if any): - Relevant config (redacted): ### Steps ``` ⏺ Build succeeded. Now let's run the tests for the changed file. ⏺ Bash(source ~/.zshrc 2>/dev/null; pnpm vitest run src/cli/memory-cli.test.ts 2>&1) ⎿   RUN v4.0.18 /Users/shorsman/Documents/openclaw ✓ src/cli/memory-cli.test.ts (17 tests) 68ms … +5 lines (ctrl+o to expand) ⏺ Build and tests both passed: - Build: Successful — 279 files compiled - Tests: 17/17 passed in src/cli/memory-cli.test.ts Your change (removing --query from the search example in the help text) is working correctly and all existing tests pass. ``` ## Evidence Attach at least one: - [ X] Failing test/log before + passing after - [ ] Trace/log snippets - [ ] Screenshot/recording - [ ] Perf numbers (if relevant) ## Human Verification (required) What you personally verified (not just CI), and how: - Verified scenarios: - Edge cases checked: - What you did **not** verify: ## Compatibility / Migration - Backward compatible? (`Yes/No`) - Config/env changes? (`Yes/No`) - Migration needed? (`Yes/No`) - If yes, exact upgrade steps: ## Failure Recovery (if this breaks) - How to disable/revert this change quickly: - Files/config to restore: - Known bad symptoms reviewers should watch for: ## Risks and Mitigations List only real risks for this PR. Add/remove entries as needed. If none, write `None`. None

Most Similar PRs