fix(telos): remove reference to missing update-telos.ts script#764
Closed
nbost130 wants to merge 1 commit into
Closed
fix(telos): remove reference to missing update-telos.ts script#764nbost130 wants to merge 1 commit into
nbost130 wants to merge 1 commit into
Conversation
The Telos Update workflow referenced a non-existent script at ~/.claude/commands/update-telos.ts. The actual implementation lives at ~/.claude/skills/Telos/Tools/UpdateTelos.ts. Updated all three path references in the workflow to point to the correct location. Closes danielmiessler#752 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Owner
|
Thanks for this PR! The issue this addresses has been fixed in the v4.0.x releases:
Closing as addressed. If there are gaps not covered by the releases, please feel free to resubmit against the latest release. We appreciate the contribution! |
nbost130
added a commit
to nbost130/PAI
that referenced
this pull request
Mar 2, 2026
The Update workflow references `~/.claude/commands/update-telos.ts` which does not exist. The actual script lives at `~/.claude/skills/Telos/Tools/UpdateTelos.ts`. Updated all three references (command invocation, example, and documentation footer) in v4.0.2 and v4.0.3. Follow-up to danielmiessler#764 which was closed as superseded but the fix was not applied to the v4.x releases. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
3 tasks
Author
|
You are welcome. Enjoying this project. Doesn't appear to be fixed...opened #852 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Releases/v3.0/.claude/skills/Telos/Workflows/Update.md) references~/.claude/commands/update-telos.ts, but that path does not exist~/.claude/skills/Telos/Tools/UpdateTelos.tswithin the same skill directoryTools/UpdateTelos.tslocationCloses #752
Test plan
buncommand invocation points to the correctTools/UpdateTelos.tspath~/.claude/commands/update-telos.tsTools/UpdateTelos.tsmatches what the workflow expects (file validation, backup, changelog)