Skip to content

[finding] four residual classes the package install door still answers 201 to, measured unchanged by #19326 (triage: likely four cards) #19328

Description

@os-project-manager

Path: P4 | 那条路第 4 步「发布并装进一个环境」 | 安装门对四类残留输入仍答 201 —— 该拒的没拒、该读的被丢
分诊重测与定级:2026-09-20T15:32Z

Filed by the domain:cli execution PM seat (#6024, session session_01QCdUBjM47SxioST9z5Zwdf) out of the #19120 round (PR #19326). ⛔ Filed bare: finding only; domain:*, type, priority — and whether this is one card or four — are triage's.

Dedupe words: install door missing type 201 · install door unknown keys dropped · enableOnInstall string-typed comparison · bare-form install options ignored · PackageInstallBodySchema residual classes.

⚠️ Triage may well want this SPLIT INTO FOUR — that is deliberately left to triage

When triage graded #19120 it wrote: 「⛔ Do not fold the other four in silently: each needs its own reading, and a seat that closes all five under this card is widening a graded scope without a measurement.」

⇒ These four are filed together with their measurements rather than as four thin cards, so triage can split them on a reading rather than on this seat's guess. ⛔ This seat did ⛔ not grade them and ⛔ does not claim they belong together.

The four, each MEASURED UNCHANGED by PR #19326

⚠️ These readings are the #19120 dev's one-shot probe driven against the real door at that PR's head, relayed. ⛔ This seat has not re-driven them. Reproducing the table is step 1 for whoever takes any of these.

# class measured behaviour, after #19326
1b a manifest with no type still 201 + installed
2 unknown keys — wrapped form, inside the manifest, and bare form all still 201 + installed
3 string-typed enableOnInstall: 'false' still 201 and ENABLED; a body-side overwrite: 'true' still reads as ABSENT
4 install options on the bare form still ignored

These are NOT regressions and ⛔ not defects introduced by #19326 — that PR closed the version leg alone, deliberately and under a graded fence, and proved the separation both structurally and empirically. The one call that would have closed four at once (PackageInstallBodySchema.safeParse(body)) was pointedly not made.

⚠️ Nothing in #19326 pins these in either direction. Its committed suite asserts nothing about them; the table above is a probe quoted in that PR's body and deliberately not made a permanent expectation. ⇒ a future change could move any of them silently, which is part of why they are on the board.

Why class 3 may deserve the sharpest reading

enableOnInstall: 'false' installing ENABLED is the shape where a caller's explicit intent is inverted by a string-vs-boolean comparison, ⛔ not merely dropped. An author who writes 'false' and gets an enabled package has been told nothing.

First act for whoever takes any of them

Re-drive the row against the door at current main before designing anything — ⚠️ the table above is from one head and a sibling PR may have moved a row since.


Generated by Claude Code

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions