chore(xpkg, tinyhttps): xpkg 0.0.60 and tinyhttps 0.3.2 - #490
Merged
Merged
Conversation
xpkg 0.0.60 (openxlings/libxpkg#45): ExecutionContext::hook_log -- a hook's child processes write to a log file instead of the caller's terminal -- and pkginfo.build_dep reads the variable xlings exports for a namespaced name. tinyhttps 0.3.2 (mcpplibs/tinyhttps#19): download_to_file reports a write the disk refused (writeFailed, bytesWritten counts what reached the file, bytesReceived what the network delivered). The GitHub tag tarballs and the GitCode mirror copies (mcpp-res/xpkg, mcpp-res/tinyhttps) were downloaded in full and are byte-identical: xpkg 348d78e8...ce5a, tinyhttps 6c9af965...fe5b. Added to all three platform blocks of each recipe; checked by loading both files with lua.
Sunrisepeak
pushed a commit
to openxlings/xlings
that referenced
this pull request
Sep 29, 2026
…rite to a log, one error per failed download (#636) Plan, review rulings (R1–R10) and readings: `.agents/docs/2026-09-29-download-attribution-and-install-noise-plan.md`. Consumes libxpkg 0.0.60 (openxlings/libxpkg#45) through mcpplibs/mcpp-index#490. Companion index change: openxlings/xim-pkgindex (7-Zip quiet + build dep, chatgpt without the Qt shims). ## What a user saw on 2026.9.29.1 (a real home) - `install chatgpt@26.917.71314` → `sha256 mismatch … got 68e23f…`. The recipe was right; the disk was at 100 %. `68e23f…` is the sha256 of the correct file's first 220,979,200 bytes: tinyhttps never checks `ofs.write`, the truncated file was hashed, the host was demoted and its retries skipped. - The same failure printed three errors: a reasonless `[pkg] failed:` in the middle of the progress frames, the reason, and `download artifact missing`. - `install chatgpt` reinstalling qt-base printed 7-Zip's banners, listings and a repaired `ERROR` between `[74/88]` and `[75/88]`. - `self update` on a current home printed `xlings -> 2026.9.29.1` twice. - `info chatgpt` printed the same 35 deps as `runtime deps` and again as `build deps`. ## Changes | Area | Change | |---|---| | Download attribution (A2) | `xlings::tinyhttps::FailureKind` {Source, Transfer, Local, Cancelled}, decided once in the xhttp wrapper. What landed on disk is compared with what the transfer reported **before** `onVerify`: a file shorter than received is Local; a short transfer is Transfer (retried); only a complete file with the wrong content is Source (next candidate, host demoted). A destination that cannot be opened is Local | | Local failures (A3) | Stop every candidate (another mirror writes to the same disk), demote no host, `E_DISK_FULL` + hint, the free space in the message. The duplicate "incomplete transfer" check in the downloader is gone (one answerer). indexfetch stops on Local too | | Space pre-check (A4) | At the first Content-Length, a disk that cannot hold the rest stops the transfer before it is spent. An unreadable free-space figure never refuses. `XLINGS_DOWNLOAD_SPACE_CHECK=off` | | One error per failed download (B) | The progress callback's -1 only marks the panel row; the result loop emits one Failed status with the reason, code, hint and plan key; the per-node loop no longer reports the consequence | | Hook output (C2) | Every `run_hook` gets `ctx.hook_log = <home>/logs/hooks/<ns>-<name>@<ver>.<hook>.log` (libxpkg 0.0.60 redirects the hook's children there). A failure prints the last 20 lines and `full log: <path>`; a hook past 15 s prints `… <pkg> install hook running 2m: <last log line>` (interface: `progress` phase `hook`, protocol 1.5). `XLINGS_HOOK_OUTPUT=inherit` / `-v` restore the terminal (never in the interface or TUI); `XLINGS_HOOK_HEARTBEAT=<first>:<every>` | | `use` / `self update` (D) | `use` announces a switch only when something moved (`alreadyActive`: the target AND every release member already at the selected versions). `self update` on a current home ends with `xlings is already at <v> (latest)` | | `info` (E1) | A recipe written `deps = { ... }` shows one `deps` row | | Build | xpkg 0.0.60; version 2026.9.30.1 | ## Tests - unit: `DownloadAttribution.*` (5), `XimDownloaderTest.ALocalWriteFailureIsReportedAsDiskFullNotAsTheSource`, `XvmSwitchPlan.AlreadyActiveOnlyWhenTheWholeReleaseIsInPlace`; protocol version 1.5. - E2E-127 `hook_output_log_test.sh` H1..H5 — H1 fails on 2026.9.29.1 (the hook's commands in the install output). - E2E-128 `download_failure_report_test.sh` F1..F3 — F1 fails on 2026.9.29.1 (reasonless `failed:` and `download artifact missing`). - libc++: `mcpp build --toolchain llvm@20.1.7` on Linux. ## Corrected during implementation (plan §8.4) - The interface never leaked children's stdout into NDJSON: `StdoutCapture` already forwarded it to stderr as `[stray stdout]`. C2's gain there is the log and the tail on failure. - "The target is active" is not "nothing moved": a release member can still move.
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.
chore(xpkg, tinyhttps): xpkg 0.0.60 and tinyhttps 0.3.2
xpkg 0.0.60 (openxlings/libxpkg#45): ExecutionContext::hook_log -- a hook's
child processes write to a log file instead of the caller's terminal -- and
pkginfo.build_dep reads the variable xlings exports for a namespaced name.
tinyhttps 0.3.2 (mcpplibs/tinyhttps#19): download_to_file reports a write the
disk refused (writeFailed, bytesWritten counts what reached the file,
bytesReceived what the network delivered).
The GitHub tag tarballs and the GitCode mirror copies (mcpp-res/xpkg,
mcpp-res/tinyhttps) were downloaded in full and are byte-identical:
xpkg 348d78e8...ce5a, tinyhttps 6c9af965...fe5b. Added to all three platform
blocks of each recipe; checked by loading both files with lua.