Skip to content

powershell-lsp: rule-rationale override layer (N1.1 slice 1) + ROADMAP re-scope + v1.24.2 (000125)#85

Merged
manderse21 merged 3 commits into
mainfrom
dispatch/000125-rationale-override-v1242
Jul 10, 2026
Merged

powershell-lsp: rule-rationale override layer (N1.1 slice 1) + ROADMAP re-scope + v1.24.2 (000125)#85
manderse21 merged 3 commits into
mainfrom
dispatch/000125-rationale-override-v1242

Conversation

@manderse21

Copy link
Copy Markdown
Owner

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 $OwnedRationaleOverrides to scripts/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. PSShouldProcess is inside the PSES-15 default surface (reflection-confirmed), so its improved line reaches every user; the other three are base-only. The Write-Host fix leads with Write-Information/Write-Verbose, never Write-Output.

All seven inherited design constraints (000124 leg-2 survey) hold, each proven RED->GREEN:

  • overrides hand-authored in the generator, artifact regenerated (never hand-edited);
  • -Check drift 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 replaced derived text);
  • the generator THROWS on a vacuous override (equals the derived text), on a non-base key, and on an owned-code collision;
  • artifact records overrides + override_count = 4; 180-char cap, printable-ASCII fail-closed, LF, no BOM;
  • integration proofs: a Write-Host finding under ruleset=base carries the override text (not the Write-Output-led derived text); the PSShouldProcess override 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 ThrowTerminatingError deferred-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 the PATCH: 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, no CONTRACT.md change, 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

@manderse21 manderse21 merged commit cf1c5f1 into main Jul 10, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants