Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 11, 2025

The update-memory tool required content even for tags-only updates, forcing users to fetch and re-provide unchanged content.

Changes

Tool schema (src/tools/update-memory/index.ts)

  • Removed content from required fields; only hash is required
  • Updated descriptions to clarify optional nature

Executor (src/tools/update-memory/executor.ts)

  • Made content optional in UpdateMemoryArgs interface
  • Fetch existing content when not provided: args.content ?? existing.content
  • Enhanced messages to indicate update type ("tags updated only" vs "content and tags updated")

Tests (src/tests/memory-server-tests.ts)

  • Updated testUpdateMemoryTags() to verify tags-only updates without content parameter

Usage

# Before: required content even for tags-only updates
update-memory --hash abc123 --content "unchanged" --tags "new,tags"

# After: content is optional
update-memory --hash abc123 --tags "new,tags"

Backward compatible - existing calls with content continue to work.

Original prompt

This section details on the original issue you should resolve

<issue_title>update-memory tool</issue_title>
<issue_description>Tool does not allow a user to only update tags, it requires "content" even though no changes are required to it.
User should be able to update the tags in an efficient manner.</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits November 11, 2025 14:12
Co-authored-by: chrisribe <1999791+chrisribe@users.noreply.github.com>
Co-authored-by: chrisribe <1999791+chrisribe@users.noreply.github.com>
Copilot AI changed the title [WIP] Update memory tool to allow tag-only updates Make content optional in update-memory tool Nov 11, 2025
Copilot AI requested a review from chrisribe November 11, 2025 14:21
Copilot finished work on behalf of chrisribe November 11, 2025 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

update-memory tool

2 participants