Skip to content

Rename some things#2

Merged
murrayju merged 3 commits intomainfrom
murrayju/rename
Sep 19, 2025
Merged

Rename some things#2
murrayju merged 3 commits intomainfrom
murrayju/rename

Conversation

@murrayju
Copy link
Member

@murrayju murrayju commented Sep 4, 2025

Wanted to rename two things to avoid confusion and be a bit more consistent.

  • getMemories -> recall - to be consistent with the other tool names
  • key -> scope - as some expected a key to be for an individual memory

Copy link
Contributor

@MasterOdin MasterOdin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You'll need to make PRs against slack-db and progress-tracking-agent to update the guard for memory mcp to use scope instead of key. Will also need to update the direct_tool_call I added in slack-db as well to the new tool name.

@murrayju
Copy link
Member Author

murrayju commented Sep 8, 2025

You'll need to make PRs against slack-db and progress-tracking-agent to update the guard for memory mcp to use scope instead of key. Will also need to update the direct_tool_call I added in slack-db as well to the new tool name.

Yeah, I mentioned that here. @MasterOdin would you mind handling that and getting this merged (while I'm still on vacation)?

@CLAassistant
Copy link

CLAassistant commented Sep 19, 2025

CLA assistant check
All committers have signed the CLA.

@murrayju
Copy link
Member Author

I'm going to go ahead and merge now, since all the deploys are broken anyways. I created timescale/tiger-eon#17 to implement the needed changes.

@murrayju murrayju merged commit 34b69f9 into main Sep 19, 2025
2 of 4 checks passed
@murrayju murrayju deleted the murrayju/rename branch September 19, 2025 17:22
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.

3 participants