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
This commit is contained in:
parent
e64d3634a9
commit
55c216e069
3 changed files with 8 additions and 8 deletions
|
|
@ -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
|
||||
|
|
|
|||
|
|
@ -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}"
|
||||
|
|
|
|||
|
|
@ -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
|
||||
|
|
|
|||
Loading…
Reference in a new issue