local-llm-acp-record-is-put-right - #813
Merged
Merged
Conversation
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
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.
local-llm-acplanded on 2026-10-01 in #803 with every box ticked, andthe sweep has refused to archive it ever since. Waiting longer would not
have helped.
owesNothing(describeTaskDebts(...))— the one answer the sweep and themerge gate both take — is
falsefor it. Item 4.4 is**Delegated to local-llm-acp**, and its record was written on thecheckbox line with nothing indented beneath it.
isUnrecordedTaskreadsa delegated item as recording nothing when the lines under it are empty,
whatever is on the line, because nothing can mechanically tell evidence
appended to an item's sentence from the item's own instructions.
Measured at
513a5c96:describeTaskDebtsreports 4.4 underunrecorded, andvalidate --change local-llm-acpreturns"ok": false.After this change, both are clean.
Why it landed green. The gate resolves the change from
--change "$GITHUB_HEAD_REF", and #803's branch wasfix-local-llm-acp,not
local-llm-acp. A name no active change has is skipped rather thanguessed at, so the debt check never ran. The title was the change id; the
branch was not, and the gate reads the branch.
What this does
git diff --word-diffshows no word added or removed, only line breaksand indentation. Nothing re-verified, nothing reworded, no other item
touched.
openspec/README.md, next to "one changeis one pull request", naming this as the measured case.
openspec-workbenchrequirement so the rule is writtendown: a delegated or human-only item's evidence goes under the item.
No source changes, no behaviour changes, no changeset. After this lands,
the sweep archives
local-llm-acpon its next pass.Deliberately not done
Making the gate check every change whose
tasks.mda pull requesttouches would have caught this — and would also fail a pull request
carrying another change's freshly opened proposal. That rule has its own
blast radius and belongs in its own change, not smuggled into a record
correction.
Checks
typecheck0 errors ·lint0 errors (3 pre-existing warnings already onmain) ·lint:englishpassed · tests 287 files, all passed (cli 192,core 2011 + 62, extension 499, server 118, webui 687) ·
openspec validate --changes --strict2 passed, 0 failed · merge gate"ok": true.