Skip to content

feat(memory): make tags optional with auto-generation fallback#2

Merged
tooury merged 1 commit intomainfrom
Pre-ToolCallSupression
Mar 25, 2026
Merged

feat(memory): make tags optional with auto-generation fallback#2
tooury merged 1 commit intomainfrom
Pre-ToolCallSupression

Conversation

@tooury
Copy link
Copy Markdown
Collaborator

@tooury tooury commented Mar 25, 2026

  • Change tags field in create_memory tool from required to optional
  • Update description to clarify tags are auto-generated from content when omitted or blank
  • Update system prompt to document blank tag (||) auto-generation behavior
  • Add usage example demonstrating automatic semantic tag generation
  • Refactor AttachmentMimePolicyService to extract resolvePolicy as a testable static method with explicit parameters

Summary

Describe the change and its motivation.

Type of Change

  • Bug fix
  • New feature
  • Refactor / code cleanup
  • Documentation
  • Build / CI / tooling

Testing

Describe how this was tested.

Checklist

  • I have read CONTRIBUTING.md
  • My code follows the project's Swift style conventions
  • I have not committed secrets, API keys, or personal information
  • I have updated documentation if needed
  • New/changed Swift code compiles without warnings

Notes for Reviewer

Any context that would help the reviewer.

- Change `tags` field in create_memory tool from required to optional
- Update description to clarify tags are auto-generated from content when omitted or blank
- Update system prompt to document blank tag (`||`) auto-generation behavior
- Add usage example demonstrating automatic semantic tag generation
- Refactor `AttachmentMimePolicyService` to extract `resolvePolicy` as a testable static method with explicit parameters
@tooury tooury merged commit 1a0021b into main Mar 25, 2026
2 of 3 checks passed
@tooury tooury deleted the Pre-ToolCallSupression branch March 25, 2026 11:13
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.

1 participant