Skip to content

fix: classify lock reads and dispose rejected OpenAPI bodies - #182

Merged
smiggleworth merged 2 commits into
developfrom
release/050-cli-read-failures
Oct 10, 2026
Merged

smiggleworth merged 2 commits into
developfrom
release/050-cli-read-failures

Conversation

@smiggleworth

@smiggleworth smiggleworth commented Oct 10, 2026 •

Copy link
Copy Markdown
Contributor

A one-shot lock preflight EACCES could be treated as rename contention and admit an update, while rejected OpenAPI downloads could retain streaming response bodies. Keep read and rename error classification separate, and dispose rejected or redirected bodies through the existing cleanup path. Preserve load limits, complete successful responses and the bounded Windows observation retry.

Validation: ten new/strengthened assertions fail against immutable pre-fix source; all 455 unit cases and the main release checks pass locally. A real local HTTPS probe changes five rejected-response cases from one or two live streaming sockets to no live sockets or further body writes. Thirty-five added cases include nineteen malformed-input cases, five real local installer process outcomes, two preflight denials and nine transport cases; one existing transport assertion is strengthened. Packed checks retain two names, reject nine old imports and fifteen private paths under actual TypeScript 6.0.3/7.0.2. Full matrix, source assertions, process-recovery and timeout boundaries are recorded in docs/0.5.0-hardening.md.

All five packed templates, the peer floor, production audit, formatting and 22 unchanged performance gates are qualified separately. Refs #159; closure waits for exact PR and merged checks. No package version, dependency, public surface, workflow or budget changes. The complete candidate graph/site and maintainer full review still gate publication.

The initial Windows run passed the publication assertions but fixture removal hit EBUSY after CLI termination. The test now waits for inherited installer/shell handles to close after stopping its owned installer. The 22-case publication suite, typecheck and formatting pass locally; hosted checks run on the corrected commit. Production source is unchanged by this test correction.

@smiggleworth
smiggleworth merged commit 1ec1965 into develop Oct 10, 2026
10 checks passed
@smiggleworth
smiggleworth deleted the release/050-cli-read-failures branch October 10, 2026 04:58
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.

1 participant