dld-goal: work items, decision pinning, and selection - #31
Closed
jimutt wants to merge 1 commit into
Closed
Conversation
Implements DL-002: - decision-hash.sh hashes the intent-carrying fields (title, supersedes, amends, body) and ignores status/references/timestamp, so accepting a decision or refreshing its references does not invalidate a planned item - run-state.sh gains add-item, get-item, set-item-status, add-evidence, bump-attempt and repin-item; items are 1-indexed and currentItem tracks whichever item is in flight - next-item.sh selects the next item, preferring in-flight work, and exits 2 rather than stepping past a blocked item - verify-hashes.sh reports decisions that changed since planning, ignoring in-flight items by default because /dld-implement permits refining a still-proposed decision while implementing it DL-002 refined inline during implementation (still proposed, so mutable) to record the hash scope and the re-pinning rule. 42 new tests, suite 260. Co-Authored-By: Claude Opus 5 <noreply@pi.dev> Generated-By: pi 0.84.2
jimutt
force-pushed
the
feat/dld-goal-items
branch
from
August 21, 2026 11:49
d69f594 to
c4d8397
Compare
Owner
Author
|
Superseded by feat/pi-harness-extension — all commits from this branch are merged there. See #41 for the findings log on top. |
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.
Stacks on #30. Implements DL-002.
What
A work item is one decision by default, or several when genuinely coupled — same rule
/dld-implementalready applies. Each decision is pinned by an intent hash at planning time.New:
decision-hash.sh,next-item.sh,verify-hashes.sh, plus item subcommands onrun-state.sh(add-item,get-item,set-item-status,add-evidence,bump-attempt,repin-item).Notable
The hash covers intent only — title, supersedes, amends, body — not
status,references, ortimestamp. Hashing the whole file would fight the workflow:/dld-implementpermits refining a still-proposed decision while implementing it, and completion flips status and adds references. Drift checking is scoped by item status accordingly, withrepin-itemrefreshing hashes on completion. DL-002 was refined inline (while still proposed) to record this.next-item.shexits 2 rather than selecting past a blocked item, so a blocker can't be silently skipped. Trade-off recorded in the decision's consequences.42 new tests.