knopersikcuo/Planner
Watch
1
0
Fork
You've already forked Planner
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Reduce MCP memory catalog overhead #126

Merged
kleb merged 1 commit from feature/mcp-memory-pruning into main 2026-08-30 13:51:48 +02:00
Conversation 1 Commits 1 Files changed 5 +152 -97
kleb commented 2026-08-30 13:49:51 +02:00
Collaborator
Copy link

Summary

  • remove MCP-only capture, prompt-save, and singular judgment registrations while keeping their REST, CLI, hook, and service paths
  • replace separate pause and resume tools with typed idempotent memory_session_set_paused
  • remove exact empty additionalProperties schema nodes and advertise tools.listChanged: false

Measured impact

  • remote catalog: 66 tools / 29,179 serialized characters to 62 / 26,982
  • local catalog: 68 tools / 30,133 serialized characters to 64 / 27,936

Verification

  • 69 focused MCP, memory, hook, CLI, REST, and work-protocol tests
  • bun run typecheck
  • bun run lint
  • bun run format:check
  • git diff --check
## Summary - remove MCP-only capture, prompt-save, and singular judgment registrations while keeping their REST, CLI, hook, and service paths - replace separate pause and resume tools with typed idempotent `memory_session_set_paused` - remove exact empty `additionalProperties` schema nodes and advertise `tools.listChanged: false` ## Measured impact - remote catalog: 66 tools / 29,179 serialized characters to 62 / 26,982 - local catalog: 68 tools / 30,133 serialized characters to 64 / 27,936 ## Verification - 69 focused MCP, memory, hook, CLI, REST, and work-protocol tests - `bun run typecheck` - `bun run lint` - `bun run format:check` - `git diff --check`
kAgent commented 2026-08-30 13:49:52 +02:00
Collaborator
Copy link

kReview review

Verdict: no findings

The visible changes present low risk, with no concrete correctness, security, or compatibility defects established by the diff and repository evidence.

Reviewed by kReview at 49efe4ea88. This comment is conservative and based only on the PR diff, metadata, and supplied repository context.

Est. cost ~$0.91 (439.6k in / 2.0k out / gpt-5.6-sol).

<!-- codex-forgejo-review --> <!-- codex-forgejo-review-head:49efe4ea881dbf784e7237ebd5bec247d40691f2 --> <!-- codex-forgejo-review-comparison:v1:a31318a80500fd8ee57e3fb51cafe26984c36c66aad43480f2f6f0586ac4c3a2 --> ## kReview review **Verdict:** no findings The visible changes present low risk, with no concrete correctness, security, or compatibility defects established by the diff and repository evidence. _Reviewed by kReview at `49efe4ea88`. This comment is conservative and based only on the PR diff, metadata, and supplied repository context._ _Est. cost ~$0.91 (439.6k in / 2.0k out / gpt-5.6-sol)._ <!-- codex-forgejo-review-state:eyJoZWFkU2hhIjoiNDllZmU0ZWE4ODFkYmY3ODRlNzIzN2ViZDViZWMyNDdkNDA2OTFmMiIsImNvbXBhcmlzb25LZXkiOiJ2MTphMzEzMThhODA1MDBmZDhlZTU3ZTNmYjUxY2FmZTI2OTg0YzM2YzY2YWFkNDM0ODBmMmY2ZjA1ODZhYzRjM2EyIiwic3VtbWFyeSI6IlRoZSB2aXNpYmxlIGNoYW5nZXMgcHJlc2VudCBsb3cgcmlzaywgd2l0aCBubyBjb25jcmV0ZSBjb3JyZWN0bmVzcywgc2VjdXJpdHksIG9yIGNvbXBhdGliaWxpdHkgZGVmZWN0cyBlc3RhYmxpc2hlZCBieSB0aGUgZGlmZiBhbmQgcmVwb3NpdG9yeSBldmlkZW5jZS4iLCJmaW5kaW5ncyI6W10sImN1bXVsYXRpdmVDb3N0Ijp7InVzZCI6MC45MTA3MDQsImlucHV0VG9rZW5zIjo0Mzk1NzksIm91dHB1dFRva2VucyI6MjAyNiwibW9kZWwiOiJncHQtNS42LXNvbCJ9fQ== -->
kleb merged commit 97c49a5435 into main 2026-08-30 13:51:48 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
knopersikcuo/Planner!126
No description provided.