powershell-lsp: rule-rationale override layer (N1.1 slice 1) + ROADMAP re-scope + v1.24.2 (000125)#85
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.
Dispatch 000125 -- combined sweep train. Three code-bearing legs land here; the survey/poll/PK legs are records-only in the hub outbox. HELD for Mike's merge gate -- nothing tagged, verified, or published.
Leg 1 -- owned rationale-override layer (N1.1 slice 1)
Adds
$OwnedRationaleOverridestoscripts/regen-rule-rationales.ps1: hand-authored why+fix rationales that REPLACE the weak auto-derived PSSA text for four idiom-family codes --PSShouldProcess,PSUseSupportsShouldProcess,PSAvoidUsingWriteHost,PSAvoidShouldContinueWithoutForce.PSShouldProcessis inside the PSES-15 default surface (reflection-confirmed), so its improved line reaches every user; the other three are base-only. TheWrite-Hostfix leads withWrite-Information/Write-Verbose, neverWrite-Output.All seven inherited design constraints (000124 leg-2 survey) hold, each proven RED->GREEN:
-Checkdrift goes RED on an override's presence/key/text, on an artifact-only edit, and on a pin bump changing the derived text under an override (the artifact records the replacedderivedtext);overrides+override_count = 4; 180-char cap, printable-ASCII fail-closed, LF, no BOM;Write-Hostfinding underruleset=basecarries the override text (not the Write-Output-led derived text); thePSShouldProcessoverride renders on the DEFAULT surface.Surgical: exactly the 4 override entries changed; the other 50 PSSA + 5 owned entries are byte-identical to HEAD.
Leg 2 -- ROADMAP N1.1 re-scope (docs)
Re-scopes the Horizon 1 N1.1 entry from new detection to guidance quality per the 000124 evidence (no idiom candidate clears a 0%-FP bar as new detection). Names slice 1 = 000125; keeps
ThrowTerminatingErrordeferred-unmeasurable with the corpus gap named. Also fixes Section 3 row 8 (an approved deviation -- a feedback-table cell that would otherwise contradict the re-scoped entry). Exactly two hunks.Leg 5 -- release prep (PATCH -> 1.24.2)
Lockstep bump of both manifests + dated
[1.24.2]CHANGELOG entry with thePATCH:lead-in. Classified PATCH from the on-disk policy: a wording-quality improvement to rationale lines that already render, not a new capability. Same content-fix class as [1.23.1]/[1.24.1]; explicitly NOT leaned on [1.21.1] (whose PATCH rested on the default surface being byte-unchanged, which does not hold here).Scope
No
base.psd1/ ruleset edit, noCONTRACT.mdchange, no knob, no new detection rule. Full local suite green on both hosts modulo a pre-existing, intermittent, host-independent ServeShim-lifecycle flake (documented in the outbox; unrelated subsystem).https://claude.ai/code/session_01Jq8dQqSvpmcTQRFK4jjEob