Skip to content

chore(deps): combined safe/tested dependency updates (8 of 11 open Renovate PRs) - #155

Merged
Decipher merged 8 commits into
developfrom
chore/deps-major-updates
Aug 12, 2026
Merged

chore(deps): combined safe/tested dependency updates (8 of 11 open Renovate PRs)#155
Decipher merged 8 commits into
developfrom
chore/deps-major-updates

Conversation

@Decipher

Copy link
Copy Markdown
Owner

Summary

Combines the remaining open Renovate PRs on GitHub into one branch against develop (all target main directly, bypassing GitFlow). Supersedes the earlier safe-only batch — every one of these is individually bumped, empirically tested (test suite, lint, typecheck, nuxt generate build — not just Renovate's own stale CI), and committed as its own commit within this branch.

Included (8)

Excluded (3) — real, empirically-found blockers, not a rubber-stamp

  • npm:pnpm v11 (chore(deps): update dependency npm:pnpm to v11 #133): looked like the safest change in the batch (single-line version pin), but pnpm v11 cannot install the project at all — it fails to apply nuxt/patches/druxt.patch (ERR_PNPM_PATCH_FAILED) against the git-sourced druxt dependency. Confirmed with a clean A/B: the exact same lockfile installs cleanly under pnpm v10, and fails consistently under v11 (tested by setting packageManager to a real corepack-resolved pnpm@11.21.0 and doing a fully clean reinstall, twice). Needs its own investigation into pnpm v11's patch-application changes, or regenerating the patch, before it's safe.
  • nuxt 4.5.1 + @nuxt/test-utils v4 (chore(deps): update dependency nuxt to v4.5.1 [security] #144, chore(deps): update devdependency @nuxt/test-utils to v4 #137): these two are coupled (bumping nuxt alone breaks ESLint with 458 new errors and makes the entire Vitest suite fail to collect), but bumping both together still fails — @nuxt/test-utils v4 itself requires vitest@^4.0.2, which is a third major bump not present in any open Renovate PR. This needs its own dedicated pass (nuxt + test-utils + vitest + re-validating the coverage/reporter config together), not a drive-by dependency bump.
  • @nuxt/ui v4 (chore(deps): update dependency @nuxt/ui to v4 [security] #143): bumps cleanly, all tests/lint/typecheck pass, but pnpm run generate shows [Icon] failed to load icon for ~7 distinct icons (lucide:rss, lucide:heart, lucide:github, lucide:zoom-in, lucide:external-link, simple-icons:drupal, simple-icons:githubsponsors) across multiple pages. Confirmed real: the generated HTML has an empty <span class="iconify ..."> with no inlined SVG (previously these were inlined at build time), meaning the icon only appears after client-side JS hydration instead of being in the static HTML — a real regression on /writing's repository cards (GitHub Sponsors icon) and elsewhere. Needs its own icon-collection/config investigation before it's safe.

Test plan

  • Each of the 8 included changes individually verified (test/lint/build) before being combined
  • Full combined-state verification after all 8: pnpm test (582/582), pnpm lint — both on a fully clean node_modules reinstall
  • cweagans/composer-patches v2 verified against a live .devtools/ Drupal instance: patches apply, 29/29 PHPUnit tests pass
  • pnpm v11's install-breaking regression caught by an explicit clean A/B test (same lockfile, pnpm v10 vs v11) — not left in the branch

Mirror of gitlab.local MR !40 (primary repo) — already green there.

@coderabbitai

coderabbitai Bot commented Aug 12, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: ad68762e-4f6c-4319-bdf9-14ee7d96d2b0

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Aug 12, 2026

Copy link
Copy Markdown

Preview site

pr-155: https://pr-155--stuartclark.netlify.app

Updated 2026-08-12T08:46:42Z

@codecov

codecov Bot commented Aug 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.76%. Comparing base (edd81d2) to head (4466382).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #155   +/-   ##
========================================
  Coverage    99.76%   99.76%           
========================================
  Files           75       75           
  Lines         1268     1268           
  Branches       324      324           
========================================
  Hits          1265     1265           
  Misses           3        3           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions

github-actions Bot commented Aug 12, 2026

Copy link
Copy Markdown

✅ Lighthouse Audit · unknown

All pages within budget — 12 pages scanned.

Route FCP LCP CLS TBT Perf
/ 2.9s 3.0s 0.000 20.500 0.89
/about 2.7s 3.1s 0.000 62.000 0.89
/community 2.6s 2.7s 0.016 10.500 0.92
/open-source 2.7s 4.8s 0.000 0.000 0.78
/writing 2.8s 2.9s 0.000 30.500 0.90
/writing/custom-formatters-410-20260731 2.9s 3.4s 0.063 0.000 0.86
/writing/decoupling-configuration-config-pages-20220412 2.6s 3.0s 0.015 0.000 0.90
/writing/field-tokens-200-20260722 2.6s 2.7s 0.019 0.000 0.92
/writing/hello-world-20211126 2.9s 3.0s 0.142 0.000 0.84
/writing/jsonapi-views-120-20260812 2.7s 2.8s 0.005 0.000 0.91
/writing/layout-paragraphs-module-20220301 2.6s 2.7s 0.055 0.000 0.92
/writing/what-no-images-20220315 2.6s 2.8s 0.039 0.000 0.92

Thresholds: FCP ≤ 3.5s, LCP ≤ 5.0s, CLS ≤ 0.25, TBT ≤ 600.000, Perf ≥ 0.75

…izer core patch

composer-patches v2 defaults drupal/core patches to depth 2 (matching
Drupal.org's conventional a/core/... diff header), but this custom patch
uses a/modules/... (already relative to web/core/) and needs depth 1.
Without an explicit override, v2 silently fails to apply it
(ERR: 'No available patcher was able to apply patch'), which was masked
locally by a stale vendor/ directory retaining the already-patched file
from before the v1->v2 bump — only surfaced on a genuinely clean install.

Also commits patches.lock.json, the new per-patch lockfile v2 introduces
for reproducibility (analogous to composer.lock).
@Decipher
Decipher merged commit 3f608d9 into develop Aug 12, 2026
10 of 11 checks passed
@Decipher
Decipher deleted the chore/deps-major-updates branch August 12, 2026 08:49
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