Skip to content

fix(telos): rename LESSONS.md to LEARNED.md and fix Update.md script path#1024

Open
badosanjos wants to merge 2 commits intodanielmiessler:mainfrom
badosanjos:fix/telos-learned-filename-and-command-path
Open

fix(telos): rename LESSONS.md to LEARNED.md and fix Update.md script path#1024
badosanjos wants to merge 2 commits intodanielmiessler:mainfrom
badosanjos:fix/telos-learned-filename-and-command-path

Conversation

@badosanjos
Copy link
Copy Markdown

Summary

Two bugs in the Telos skill that cause the TELOS update command to silently fail:

Bug 1 — Wrong filename: LESSONS.md vs LEARNED.md

SKILL.md consistently documents the lessons file as LEARNED.md:

- **LEARNED.md** - Lessons learned over time

But Tools/UpdateTelos.ts VALID_FILES and Workflows/Update.md both reference LESSONS.md.

Effect: The file on disk is LEARNED.md. If a user tries to update it:

  • Via the tool directly (update-telos LEARNED.md ...) → ❌ Invalid file: LEARNED.md
  • Via the workflow examples (update-telos LESSONS.md ...) → ❌ File does not exist

Both paths fail. Lessons cannot be updated at all.

Bug 2 — Stale script path in Update.md

Workflows/Update.md invokes the update script as:

bun ~/.claude/commands/update-telos.ts

But ~/.claude/commands/ does not exist. The correct installed path is:

bun ~/.claude/skills/Telos/Tools/UpdateTelos.ts

This was partially addressed in #930/#929 for UpdateTelos.ts but not for Update.md.

Changes

  • Packs/Telos/src/Tools/UpdateTelos.ts — renamed 'LESSONS.md''LEARNED.md' in VALID_FILES array and JSDoc comment
  • Packs/Telos/src/Workflows/Update.md — renamed all LESSONS.mdLEARNED.md references; fixed script invocation path from ~/.claude/commands/update-telos.ts to ~/.claude/skills/Telos/Tools/UpdateTelos.ts

Test plan

  • update-telos LEARNED.md "- New lesson" "Test lesson" succeeds (was rejected as invalid file)
  • update-telos LESSONS.md "..." "..." correctly rejected (file doesn't exist)
  • LEARNED.md appears in the valid files list printed by the workflow

🤖 Generated with Claude Code

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