fix(tasks): seed text view with T{displayNumber} instead of P#T##277
Merged
Conversation
seedTextFromProject was generating phase-relative IDs like [P1T1].
Now uses the stable server-assigned displayNumber so IDs are
permanent, cross-phase, and consistent with the guide text and
.tasks/my-tasks.md sync format.
Deliverables and checkpoints use a globally sequential D{n}/C{n}
counter instead of per-phase P#D#/P#C# to match the guide text.
Closes #275
…torage
Existing users with old P#T# content persisted in localStorage would
not benefit from the seedTextFromProject fix. Adding P#T# to the
legacy-format detection clears stale storage and triggers a fresh
reseed with T{displayNumber} IDs on next load.
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
1 participant
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
seedTextFromProjectwas generating phase-relative[P1T1]IDs in the text view — PR chore(tasks): migrate task identifiers from P#T# to T# #254 fixed the guide text but left this function unchanged[T{displayNumber}]using the stable server-assigned display number (e.g.[T22]), consistent with the guide text and.tasks/my-tasks.mdsync formatD{n}/C{n}counter instead of per-phaseP#D#/P#C#P#T#content in localStorage are auto-migrated: the legacy format detector now also matches[P\d+[TDC]\d+]and triggers a fresh reseed on next loadTest plan
[T22]not[P1T1]P#T#content in localStorage — auto-reseeds toT#format[trigger —nextShortIdassigns the correct nextT#npm run typecheckpassesCloses #275