Skip to content

chore(deps): bump undici from 8.7.0 to 8.9.0 in /.repos/effect-smol - #48

Closed
dependabot[bot] wants to merge 1 commit into
mainfrom
dependabot/npm_and_yarn/dot-repos/effect-smol/undici-8.9.0
Closed

chore(deps): bump undici from 8.7.0 to 8.9.0 in /.repos/effect-smol#48
dependabot[bot] wants to merge 1 commit into
mainfrom
dependabot/npm_and_yarn/dot-repos/effect-smol/undici-8.9.0

Conversation

@dependabot

@dependabot dependabot Bot commented on behalf of github Aug 3, 2026

Copy link
Copy Markdown

Bumps undici from 8.7.0 to 8.9.0.

Release notes

Sourced from undici's releases.

v8.9.0

⚠️ Security fixes

High severity

  • GHSA-4cwx-7wf7-3272: malformed qualified private Cache-Control directives could cause cross-user information disclosure in shared caches or a parse-time crash. The cache parser now treats empty qualified directives conservatively and safely handles mixed qualified and unqualified directives. Fixed by 4fe5bc5f with regression coverage in 9f09b49a.

Medium severity

  • GHSA-m8rv-5g2x-5cg5: a malicious type property on a duck-typed blob-like HTTP/1.1 request body could inject CRLF sequences into the generated content-type header. Undici now coerces and validates the value before adding it to the request. Fixed by 7d3cf924.
  • GHSA-jr45-8vmc-qm54: optional whitespace around = in qualified no-cache and private directives could bypass shared-cache restrictions and disclose authenticated data across users. Cache-Control parsing now normalizes these forms and applies conservative cache decisions. Fixed by c601fff1.
  • GHSA-8xcm-r25x-g524: the retry interceptor could expose a stale Content-Length after resuming a partial response, potentially causing downstream response desynchronization, hangs, or corruption. Undici now rejects partial responses whose Content-Length is inconsistent with Content-Range. Fixed by e11a68ed, with corrected fixtures in 2b3f7493.
  • GHSA-v3r7-h72x-cjcm: unsanitized domain and unparsed values passed to setCookie() could inject cookie attributes. Undici now validates cookie domains, paths, and unparsed attributes more strictly. Fixed by 10d93fc3.

Additional hardening

Undici now validates non-string header values after coercion, including array elements, preventing crafted toString() or Symbol.toPrimitive implementations from introducing CRLF sequences. This defense-in-depth change was made in 354a151f.

What's Changed

New Contributors

Full Changelog: nodejs/undici@v8.8.0...v8.9.0

v8.8.0

What's Changed

... (truncated)

Commits
  • 21a8e1e Bumped v8.9.0 (#5589)
  • 7d3cf92 fix: validate blob body content type
  • c601fff fix(cache): harden cache directive parsing
  • 2b3f749 test(retry): correct broken content-range fixtures in retry-handler.js
  • e11a68e fix(retry): reject partial content length mismatch
  • 9f09b49 test: cover crash on mixed unqualified and qualified private cache directives
  • 4fe5bc5 fix: handle empty qualified private cache directive
  • 10d93fc fix: harden cookie domain, path, and unparsed attribute validation
  • a17e301 Ignore auto-generated .npmrc on Windows (#5583)
  • a0922b0 fix: handle frozen globalThis in setGlobalDispatcher (#5574)
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
    You can disable automated security fix PRs for this repo from the Security Alerts page.

Bumps [undici](https://github.com/nodejs/undici) from 8.7.0 to 8.9.0.
- [Release notes](https://github.com/nodejs/undici/releases)
- [Commits](nodejs/undici@v8.7.0...v8.9.0)

---
updated-dependencies:
- dependency-name: undici
  dependency-version: 8.9.0
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot Bot added dependencies Pull requests that update a dependency file javascript Pull requests that update javascript code labels Aug 3, 2026
@radroid

radroid commented Aug 8, 2026

Copy link
Copy Markdown
Owner

Closing — this is noise, not a fix, and merging it would cost us on every sync.

.repos/effect-smol is a vendored reference subtree, not a workspace package:

  • It is not matched by any glob in pnpm-workspace.yaml (apps/*, infra/*, oxlint-plugin-t3code, packages/*, scripts), so nothing here is ever installed, bundled or shipped. The advisory is not reachable from any artifact we build.
  • ci.yml explicitly sparse-checks-out !/.repos/ — the code in this PR is not even present when CI runs.
  • It is maintained by scripts/sync-reference-repos.ts as a git subtree pulled from the real upstream (added in chore: add vendored reference repo subtree sync tooling pingdotgg/t3code#2902 / chore: add Alchemy reference repo subtree pingdotgg/t3code#2918). A fork-local edit to a file inside the subtree diverges it from its source, so the next subtree pull conflicts — and the conflict lands in a ~30k-line lockfile.
  • It sits inside the daily rebase-onto-upstream path, so the same hunk would be re-conflicted on every upstream sync.

The right place to fix this advisory is effect-smol upstream; it reaches us for free on the next subtree pull.

.github/dependabot.yml now ignores the vendored directories so this stops recurring.

@radroid radroid closed this Aug 8, 2026
@radroid
radroid deleted the dependabot/npm_and_yarn/dot-repos/effect-smol/undici-8.9.0 branch August 8, 2026 05:59
@dependabot @github

dependabot Bot commented on behalf of github Aug 8, 2026

Copy link
Copy Markdown
Author

OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let me know by commenting @dependabot ignore this major version or @dependabot ignore this minor version.

If you change your mind, just re-open this PR and I'll resolve any conflicts on it.

radroid added a commit that referenced this pull request Aug 8, 2026
…tron pin

Two things the electron bump in this PR pulls in with it.

`.github/dependabot.yml` (new, fork-owned — upstream has none, so no ledger
row) ignores `.repos/**`. Those subtrees are reference source: no
`pnpm-workspace.yaml` glob matches them, nothing there is installed or shipped,
and `ci.yml` sparse-checkout-excludes `!/.repos/`. They were still 41 of the
fork's 107 open alerts and had already produced #48 and #50, both closed
unmerged — merging either would have diverged a subtree from its source and
re-conflicted on every rebase. The wildcard `ignore` is the lever that reaches
security updates; `open-pull-requests-limit: 0` is documented as
version-updates-only and would not have worked alone.

SEAMS.md, per its own self-reference rule. `apps/desktop/package.json` becomes
row 36 (+1/-1, churn 15, risk 30) with a note against the tripwire: a version
pin has no fork-owned home, and it retires itself once upstream passes 41.10.3.
`pnpm-lock.yaml` grows +93 -> +125/-69, which moves it past ChatView.tsx to the
top of the table at risk 12416.

Header totals go 35 files / +1700 -187 -> 36 / +1733 -257. Both figures were
regenerated with the documented script against merge-base 30c9622; the script
reproduces the old header exactly on origin/main, so the delta is the change.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file javascript Pull requests that update javascript code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant