feat: backlog lifecycle sync (030), transactional pack refresh (031), and agent YAGNI directives - #9
Merged
Merged
Conversation
Adds a dedicated 'compose refresh <pack>' subcommand that updates an installed pack's .github/ projection from its current authoritative source in one all-or-restored transaction. Editing a pack source previously made that pack unrefreshable: 'remove' refuses on the Feature 028 recorded-source digest guard, and 'add --force' early-returns as already installed. The documented remove-then-add path was unusable in exactly the case it documented. Refresh proves installed-side authority (exact profile bytes, whole-profile currency, files-to-inventory congruence, per-artifact recorded hashes), stages the current source through the canonical add pipeline, applies the destination-set difference, and updates the profile in the same transaction. It does not apply the uninstall digest guard, because it expects the source to have changed. That guard and 'add --force' semantics are unchanged.
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.
Four commits on
backlog-lifecycle-sync, basemain.Feature 030 — backlog lifecycle sync (
fd62232)Committed
.dude/backlog.md/.htmlwent stale after task-state mutations, so the backlog showed no active work while features were in flight. Adds mechanical refresh at the three task-state mutation chokepoints plus a terminalresolvedidea lifecycle. Acceptance caught and fixed a live defect whererecovery.mjsaccepted onlydraft/defined, breakinginspect()once a resolved ledger existed.Idea resolution (
43415f7)Resolves
good-enough-delivery; its intent was already covered by existing YAGNI, materially-ready, and escalation rules.Agent YAGNI directives (
4097ec8)The Tester and Reviewer were the only two agents missing the YAGNI governing rule their siblings carry — the two roles that decide how many edge cases get written and how much rework gets demanded. Tester gets a tailored clause: cover reachable inputs and real caller paths, not contrived permutations.
Feature 031 — transactional pack refresh (
3aaa6b7)Editing a pack source made that pack unrefreshable.
removerefuses on the Feature 028 recorded-source digest guard;add --forceearly-returns as already installed. The documented remove-then-add path was unusable in exactly the case it documented.Adds
compose refresh <pack>: proves installed-side authority (exact profile bytes, whole-profile currency, files-to-inventory congruence, per-artifact recorded hashes), stages the current source through the canonical add pipeline, applies the destination-set difference, and updates the profile in one all-or-restored transaction. It deliberately does not apply the uninstall digest guard, because it expects the source to have changed. That guard andadd --forcesemantics are unchanged.Verified end-to-end in a throwaway real bundle: the edited-source case that fails
removereturns[OK] refreshed pack "coding" (5 replaced, 0 added, 0 removed).Verification
2308 tests, 2304 pass, 0 fail, 4 pre-existing skips · lint 0/0 · compose verify 16 packs clean · build-dev idempotent with source/generated parity · release build 64 files, single lint warning identical at HEAD ·
git diff --checkclean.Each task received independent review before close.