Skip to content

build(deps): вдигане на undici override до ^7.29.0 - #282

Merged
todorkolev merged 1 commit into
mainfrom
fix/bump-undici
Aug 4, 2026
Merged

build(deps): вдигане на undici override до ^7.29.0#282
todorkolev merged 1 commit into
mainfrom
fix/bump-undici

Conversation

@todorkolev

Copy link
Copy Markdown
Collaborator

Отпушва CI. main е червен от 3e76949 насам и нищо не може да се мърджне, включително #281.

Одитът на зависимости в задачата check намира пет уязвимости срещу undici@7.28.0 - GHSA-4cwx-7wf7-3272 (CVSS 7.4), GHSA-8xcm-r25x-g524, GHSA-jr45-8vmc-qm54, GHSA-m8rv-5g2x-5cg5, GHSA-v3r7-h72x-cjcm - всичките поправени в 7.29.0.

Съществуващият диапазон ^7.28.0 вече допускаше 7.29.0; заключващият файл просто не беше помръдвал. Вдигам долната граница на ^7.29.0, за да не може да се върне тихо назад, и опреснявам lock-а.

Дифът в pnpm-lock.yaml не съдържа нищо освен undici. Пакетът се стига през wranglerminiflare (локалният fetch) и никога не влиза в runtime-а на Worker-а, тоест има същия статут като останалите пинове в този блок.

Five advisories were disclosed against undici 7.28.0 — GHSA-4cwx-7wf7-3272 (CVSS 7.4),
GHSA-8xcm-r25x-g524, GHSA-jr45-8vmc-qm54, GHSA-m8rv-5g2x-5cg5 and GHSA-v3r7-h72x-cjcm —
all fixed in 7.29.0. The `check` job's dependency audit fails on them, so main has been red
since 3e76949 and nothing can merge.

The existing `^7.28.0` range already admitted 7.29.0; the lockfile had simply never moved.
Raise the floor to ^7.29.0 so it cannot silently fall back, and refresh the lock. The diff
touches nothing but undici. Reached through wrangler→miniflare (local fetch), so it never
ships to the Worker runtime — same standing as the other pins in this block.
@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown

Test coverage

Workspace Lines Δ Branches Δ Functions Statements
apps/etl 74.05% +0.05pp 58.22% +0.02pp 70.21% 71.42%
apps/web 91.02% +1.32pp 82.50% +0.70pp 91.19% 89.73%
packages/config 92.85% +0.05pp 72.22% +0.02pp 92.85% 89.18%
packages/db 94.54% +0.34pp 79.38% +0.38pp 87.19% 91.55%
packages/ingest 86.37% +0.57pp 80.46% +0.46pp 79.27% 84.40%
packages/shared 95.50% +0.10pp 80.83% +0.83pp 92.30% 89.56%
Total (informational) 91.03% 80.34% 86.94% 88.78%

✅ No workspace dropped below its baseline (tolerance 0.5pp).

📈 Coverage rose by more than 1pp — run node scripts/check-coverage.mjs --update locally and commit coverage-baseline.json to ratchet the threshold up.

@todorkolev
todorkolev merged commit 0dc5d80 into main Aug 4, 2026
5 checks passed
lyubomir-bozhinov added a commit to lyubomir-bozhinov/sigma that referenced this pull request Aug 4, 2026
…acbg gate midt-bg#281)

Two commits landed on main while the midt-bg#226 sync was in flight — an undici
override bump to ^7.29.0 and a cacbg completeness-gate fix (phantom rows /
--limit). Clean merge; the surface is pnpm-workspace.yaml + scripts/cacbg,
none of it inside the six measured workspaces, so the coverage baseline is
untouched and the gate stays green.

Verified against the merged tree: scripts tests 51 pass, cacbg pipeline tests
84 pass (run via the register-ts loader, as the CI job does), coverage ratchet
green for all six workspaces.
LyuboslavLyubenov added a commit to LyuboslavLyubenov/sigma that referenced this pull request Aug 5, 2026
Clean automatic merge: no conflicts. Upstream advanced 3 commits since the
PR's previous rebase (related-persons midt-bg#226, undici bump midt-bg#282, cacbg fix midt-bg#281);
none of those touch the PR's test-only surface (apps/web/test/integration/*,
docs/spec/integration-testing.md, apps/web/vitest.integration.config.ts).
The single auto-merged file is docs/README.md, which gained a new ADR entry
in upstream (0032); the merge preserves the alphabetical/numerical ordering
without re-flowing the PR's content.

Verification (local):
- pnpm typecheck → 7/7 packages clean
- pnpm --filter @sigma/web test → 530 passing (52 files, 8 integration files,
  41 integration tests, 0 .skip)
- pnpm lint → (run separately)
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