From 55c216e0693dfad0efd524fc89b857dde5556c29 Mon Sep 17 00:00:00 2001 From: Chris Bartholomew Date: Wed, 14 Jan 2026 12:41:04 -0500 Subject: [PATCH] Fix skill installer test examples to use meaningful content (#160) The "Test memory" example is too short for the LLM to extract meaningful facts from, causing the test to silently fail (0 memories created). Replace with "Alice works at Google as a software engineer" which has enough context for fact extraction. Fixes test examples in: - get-skill installer (local and cloud modes) - hindsight-embed configure output - skills.md documentation --- hindsight-docs/docs/sdks/integrations/skills.md | 4 ++-- hindsight-docs/static/get-skill | 8 ++++---- hindsight-embed/hindsight_embed/cli.py | 4 ++-- 3 files changed, 8 insertions(+), 8 deletions(-) diff --git a/hindsight-docs/docs/sdks/integrations/skills.md b/hindsight-docs/docs/sdks/integrations/skills.md index 9af37e1e..ee99dda0 100644 --- a/hindsight-docs/docs/sdks/integrations/skills.md +++ b/hindsight-docs/docs/sdks/integrations/skills.md @@ -220,10 +220,10 @@ After installation, verify the connection: ```bash # Store a test memory -hindsight memory retain team-acme-frontend "Test memory from setup" +hindsight memory retain team-acme-frontend "Alice works at Google as a software engineer" # Recall it -hindsight memory recall team-acme-frontend "test" +hindsight memory recall team-acme-frontend "Alice" ``` ### Switching Between Banks diff --git a/hindsight-docs/static/get-skill b/hindsight-docs/static/get-skill index fbac63c9..e717603a 100755 --- a/hindsight-docs/static/get-skill +++ b/hindsight-docs/static/get-skill @@ -414,8 +414,8 @@ if [ "$MODE" = "local" ]; then echo -e " The Hindsight skill is now available in ${BOLD}$APP_NAME${NC}." echo "" echo -e " ${DIM}Test the CLI:${NC}" - echo -e " ${CYAN}uvx hindsight-embed memory retain default \"Test memory\"${NC}" - echo -e " ${CYAN}uvx hindsight-embed memory recall default \"test\"${NC}" + echo -e " ${CYAN}uvx hindsight-embed memory retain default \"Alice works at Google as a software engineer\"${NC}" + echo -e " ${CYAN}uvx hindsight-embed memory recall default \"Alice\"${NC}" echo "" echo -e " ${DIM}$APP_NAME will automatically use the skill when relevant.${NC}" echo "" @@ -505,8 +505,8 @@ EOF echo -e " Memory bank: ${CYAN}$CLOUD_BANK_ID${NC}" echo "" echo -e " ${DIM}Test the CLI:${NC}" - echo -e " ${CYAN}hindsight memory retain $CLOUD_BANK_ID \"Test memory\"${NC}" - echo -e " ${CYAN}hindsight memory recall $CLOUD_BANK_ID \"test\"${NC}" + echo -e " ${CYAN}hindsight memory retain $CLOUD_BANK_ID \"Alice works at Google as a software engineer\"${NC}" + echo -e " ${CYAN}hindsight memory recall $CLOUD_BANK_ID \"Alice\"${NC}" echo "" echo -e " ${DIM}Share this bank ID with your team for shared memories.${NC}" echo -e " ${DIM}$APP_NAME will automatically use the skill when relevant.${NC}" diff --git a/hindsight-embed/hindsight_embed/cli.py b/hindsight-embed/hindsight_embed/cli.py index d32ff1f5..bda475b8 100644 --- a/hindsight-embed/hindsight_embed/cli.py +++ b/hindsight-embed/hindsight_embed/cli.py @@ -353,8 +353,8 @@ def _do_configure_interactive(): print(f" \033[2mConfig:\033[0m {CONFIG_FILE}") print() print(" \033[2mTest with:\033[0m") - print(' \033[36mhindsight-embed retain "Test memory"\033[0m') - print(' \033[36mhindsight-embed recall "test"\033[0m') + print(' \033[36mhindsight-embed retain "Alice works at Google as a software engineer"\033[0m') + print(' \033[36mhindsight-embed recall "Alice"\033[0m') print() return 0