# NousResearch/hermes-agent · Memory tool missing or invalid action field fixes

## Open now

- Nothing open on GitHub.

Multiple issues and fixes addressing the memory tool's handling of missing, null, or unrecognized 'action' parameters, including schema validation gaps and unclear runtime error messages.

## TL;DR

- 9 member(s): 0 open issue(s), 2 closed issue(s), 0 open PR(s), 1 merged PR(s), 6 PR(s) closed without merging.
- A fix was merged to main in PR #131887. Verify it addresses your case.
- **Recommended:** read PR #131887 (and any competing PRs below) before filing. Confirm the merged change covers your symptom.

## Maintainer resolution verdict

- Internal-tracker verdict over this cluster's issue members: `partially_resolved` · deciding PR(s): `pr-131887`. This is distinct from the fix status above. Weigh it yourself.

## How this cluster was grouped

- Link evidence (`STRUCTURAL_PARTIAL`): some members are connected by structural evidence (shared changed files, or a closing/superseded reference); the rest are joined by an automated relevance judgement over their text.
- Members outside that structural core may belong to a different underlying problem.
- Pairwise link density `0.20`: the share of member pairs that carry a link, counting a structural link in full and a relevance-only link at 0.36. `1.00` means every pair is linked.

## Closed / duplicate issues

- #55755: tool_executor: 'Unknown action None' error when memory tool receives invalid action string
- #64291: \[Bug\]: memory tool schema missing "action" from top-level required fields

## Prior & competing PRs

- **#131887: fix: guard memory tool against missing action and operations \[risk 0.20\]**. merged to main · fix tier `verified`
  - maintainer direction: category `merged`; implication `superseded`
  - files: `tests/tools/test_memory_tool.py`, `tools/memory_tool.py`
- **#55768: fix(memory): validate action parameter early with clear error messages**. closed without merge
  - maintainer direction: category `implemented_elsewhere`; implication `superseded`; refs `d682f320b35a13084371a541a835e1d988c982b8`, `#55755`
- **#64311: fix(tools): require 'action' in memory tool schema validation**. closed without merge
  - maintainer direction: category `superseded_by_pr`; implication `superseded`; refs `#64330`, `#64330`
- **#64330: fix: guard memory tool against missing action and operations (#64291)**. closed without merge
  - maintainer direction: category `implemented_elsewhere`; implication `superseded`; refs `#131887`
- **#64670: fix(memory): reject missing action at runtime with clear error message**. closed without merge
  - maintainer direction: category `duplicate_of`; implication `duplicate`; refs `#64330`, `#64330`
- **#66099: fix(memory): add action to required parameters in tool schema**. closed without merge
  - maintainer direction: category `partial_fix`; implication `needs_llm`; refs `1fcc10a59497`, `#64291`, `#64330`, `#64670`, `#66866`, `#64311`, `#66099`, `#64311`, `#64330`, `#64330`, `#64670`, `#64330`, `#64330`, `#66099`, `#64311`, `#66866`, `#64330`, `#64311`, `#66099`, `#64330`, `#64670`, `#66866`, `#64670`, `#66866`, `#64330`, `#64330`, `#64311`, `#66099`, `#64330`, `#64670`, `#66866`, `#64330`, `#374151`, `#64291`, `#64311`, `#66099`
- **#66866: fix(memory): clear error when both 'action' and 'operations' are missing (#64291)**. closed without merge
  - maintainer direction: category `duplicate_of`; implication `duplicate`; refs `#64330`, `#64330`

## Fix status

- Merged to main in PR #131887 (cited above). No release data is tracked. Verify downstream.
- Fix last verified (merged) `2026-10-03`.
- Subsystem: `agent`