fix(telos): update UpdateTelos.ts paths for v3.0 directory structure#746
Closed
wojteksbt wants to merge 1 commit intodanielmiessler:mainfrom
Closed
fix(telos): update UpdateTelos.ts paths for v3.0 directory structure#746wojteksbt wants to merge 1 commit intodanielmiessler:mainfrom
wojteksbt wants to merge 1 commit intodanielmiessler:mainfrom
Conversation
The script still referenced the v2.x path `context/life/telos` which
no longer exists in v3.0. Updated to use `paiPath()` from paths.ts
for portable PAI_DIR-aware path resolution.
Changes:
- Import `paiPath` from hooks/lib/paths (supports PAI_DIR env var)
- TELOS_DIR: hardcoded HOME path → paiPath('skills', 'PAI', 'USER', 'TELOS')
- BACKUPS_DIR: `backups/` → `Backups/` (matching actual directory name)
- VALID_FILES + docstring: `LESSONS.md` → `LEARNED.md` (actual filename)
wojteksbt
force-pushed
the
fix/telos-update-path-migration
branch
from
a8b4108 to
93aa23e
Compare
Owner
|
Thanks for this contribution! PAI v4.0 significantly restructured the architecture — new directory layout, Algorithm v3.6.0 (up from v1.x), hooks/lib/paths.ts for path resolution, and many of the underlying systems this PR targets have been rewritten. This PR targets the v3.0 architecture and can't be cleanly applied to v4.0.x. Closing as superseded. The latest release is v4.0.2. We're actively working on platform compatibility and other bigger items. If there's still a gap not covered by the new releases, we'd welcome a fresh PR against the current codebase. Thanks again! |
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.
Problem
`UpdateTelos.ts` references `.claude/context/life/telos` — a path that has never matched the actual Telos location. This causes the backup system to silently fail with file-not-found errors.
This bug exists in all releases (v2.3 through v3.0). In v3.0, Telos lives at `skills/PAI/USER/TELOS/` (relative to PAI_DIR).
Changes
File: `Releases/v3.0/.claude/skills/Telos/Tools/UpdateTelos.ts`
Why use paiPath()?
`paths.ts` provides `paiPath()` which resolves paths relative to `PAI_DIR` env var (with `~/.claude` fallback). Using it instead of `process.env.HOME + '.claude'` makes the script portable and consistent with the rest of the codebase.
Why only v3.0?
Earlier releases (v2.3–v2.5) also have the broken path but fixing them would change historical releases. v3.0 is the current release users install — that's where the fix matters.
Testing
Verified locally — `bun UpdateTelos.ts` correctly creates backups and appends to `updates.md`.