Skip to content

docs: update CLAUDE.md#33

Merged
phernandez merged 1 commit intobasicmachines-co:mainfrom
eltociear:patch-1
Mar 24, 2025
Merged

docs: update CLAUDE.md#33
phernandez merged 1 commit intobasicmachines-co:mainfrom
eltociear:patch-1

Conversation

@eltociear
Copy link
Copy Markdown
Contributor

enviroment -> environment

enviroment -> environment

Signed-off-by: Ikko Eltociear Ashimine <eltociear@gmail.com>
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Mar 18, 2025

CLA assistant check
All committers have signed the CLA.

@phernandez phernandez merged commit dfaf0fe into basicmachines-co:main Mar 24, 2025
4 checks passed
phernandez added a commit that referenced this pull request Feb 28, 2026
Fixes issues #29 and #33 from openclaw-basic-memory.

🔧 Identifier resolution (#33):
- Router used get_by_permalink() which only matched exact permalinks.
  Replaced with link_resolver.resolve_link() so titles, paths, and
  fuzzy matches work consistently with other tools like read_note.
- Set total_entities=1 and total_notes=len(results) on single-note
  path for consistency with batch path.
- Guards for "no notes" and "no schema" now fire for identifier-based
  validation too, not just note_type-based.

🎨 Text rendering (#29):
- Tool returned raw Pydantic model which LLMs rendered as
  "undefined — invalid". Now returns pre-formatted markdown.
- Router uses entity.title (with permalink fallback) as note_identifier
  for human-readable output in both text and JSON modes.
- JSON output (output_format="json") unchanged for CLI compatibility.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: phernandez <paul@basicmachines.co>
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