feat(wrap): distill half off by default behind wrap.distill - #665
Merged
Merged
Conversation
A plain /kit:wrap now runs the landing half only: board rows, commit, merge own PRs, deploy check, tidy, pull, activity line, retro, report. The distill half (the pre-step-0 candidate scan, the before/after seams, all of step 7 with its drain) runs when wrap.distill is true in the operator or kit-root kit.toml, or when the invocation carries the word distill (/kit:wrap distill [repo...]), or when the ask says to distill. With the switch off the report carries Built: SKIPPED: distill off and Seam: SKIPPED: distill off, which the report lint already accepts as a real skip. The knob resolves root-only like the other [wrap] autonomy knobs because the distill half writes to home repos. Surfaces: kit.toml key, commands/wrap.md switch paragraph and per-step gates, module-registry row, test-wrap fence and lint cases, changelog COMPAT entry naming the behavior change at the default.
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.
What
/kit:wrapnow has two halves and a switch. A plain/kit:wrapruns the landing half only: board rows, commit, merge own PRs, deploy check, tidy, pull, activity line, retro, report. The distill half (the pre-step-0 candidate scan, thebefore/afterseams, all of step 7 with its drain) runs when[wrap] distill = truein the operator or kit-rootkit.toml, or when the invocation carriesdistill(/kit:wrap distill [repo...]), or when the ask says to distill.Why
The distill half is the expensive half (precedent pass, lane classifier, builds in home repos, a note-store grep) and "wrap up" means land the session. The operator who wants the rest says so. The knob resolves root-only like the other
[wrap]autonomy knobs because the distill half writes to home repos.Report shape with the switch off
lib/wrap/report-lint.shalready accepts both as a real skip (the reject list isnothing,no candidate,none).Surfaces
kit.toml: the key and its comment.commands/wrap.md: the switch paragraph, the argument, per-step gates on the scan, step -1, step 7, the report template.lib/config/module-registry.md: the row plus the index list, and which five knobs idle while the half is off.tests/test-wrap.sh: root-only fence (default false, operator wins, project ignored), presence checks, the lint shape.docs/CHANGELOG.md: COMPAT entry naming the behavior change at the default.docs/verification/wrap-distill-knob.md: proof of done.Proof
bash tests/test-wrap.sh: all 515 passed (7 new).lib/gate/negctl.sh: default flipped totrue, suite RED (wrap.distill ships as false), restored GREEN.tests/run-all.sh: 150 of 152 ok;test-orchestrate-wavefrontandtest-pane-viewerhit the runner's 300s ceiling, untouched by this branch.Not proven
A live wrap with the switch off, and a live
/kit:wrap distill. The first real wrap after merge is that run.