Repository navigation
chore(deps): weekly dependency update (2026-09-19) - #10
Closed
robertying wants to merge 1 commit into
Closed
robertying wants to merge 1 commit into
robertying wants to merge 1 commit into
Conversation
Update pnpm to 12.4.2 and bump all workspace dependencies to latest via `pnpm update --latest -r`. Notable changes: - eslint bumped to 10.11.0 by `pnpm update --latest`, then capped back to 9.39.5 (latest 9.x): eslint-config-next pulls in eslint-plugin-react 7.37.5 (its latest release), whose `getFilename()` peer API usage is incompatible with ESLint 10's flat-config context, breaking `pnpm lint` with a hard crash. No newer eslint-plugin-react/eslint-config-next release supports ESLint 10 yet. - @graphql-codegen/client-preset, @types/node, daisyui, dotenv (17->18 major), prettier bumped to latest patch/minor/major with no breakage. Local verification (sandboxed, no access to the private GraphQL API): - lint: pass - codegen/build: could not be exercised — no .env.local and the sandbox's egress policy blocks api.tsinghua.app, so gql/ was never generated and the build fails on the resulting missing `gql` module import. CI has the required secrets/network access and should be checked. - dev smoke test: `pnpm dev`, GET / returned 200 with the expected page title and no errors/warnings in the startup log (routes depending on generated gql/ code were not reachable, consistent with the above). See PR description for full details, the next.config.ts TypeScript-CLI workaround assessment, and items needing human review. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01C454fCeZafg1A5YR4UU7Un
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
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.
Summary
Weekly automated dependency update. Ran
corepack use pnpm@latest && corepack install, thenpnpm update --latest -r.@graphql-codegen/client-preset: 6.1.3 -> 6.2.0@types/node: 26.4.1 -> 26.6.2daisyui: 5.7.37 -> 5.7.42dotenv: 17.4.2 -> 18.0.1 (major) — no breakage observed; only used incodegen.tsfordotenv.config(), API unchanged.prettier: 3.9.6 -> 3.9.8eslint: bumped to 10.11.0 (major) bypnpm update --latest, then capped back down to 9.39.5 (see below).All other dependencies were already at their latest resolvable versions (no change).
Capped package:
eslint->9.39.5pnpm update --latestbumpedeslintto 10.11.0. This brokepnpm lintwith a hard crash:Root cause:
eslint-config-next(already at its latest release, 16.3.5) pulls ineslint-plugin-react@7.37.5(also its latest release), which still calls the pre-flat-configcontext.getFilename()API removed by ESLint 10.eslint-plugin-react's own published peerDependencies cap at^9.7, confirming ESLint 10 isn't supported yet. This is an unsupported-version incompatibility in the toolchain, not a real code issue, soeslintwas capped back to9.39.5(latest 9.x) to keep lint green.pnpm lintpasses cleanly at this pin. This should be revisited onceeslint-plugin-react/eslint-config-nextpublish ESLint 10 support.Local check results (this sandbox)
pnpm lint: pass (clean, no warnings)pnpm codegen: could not run — requiresGRAPHQL_API_URL/GRAPHQL_ADMIN_SECRET(normally from.env.local, which does not exist in this checkout) to hit the live schema atapi.tsinghua.app. This sandbox's network egress policy also blocks that host outright. This is an environment limitation, not a dependency-update regression —.env.localwas never touched per policy. CI has the required secrets and network access and should be used to verify codegen/build for this PR.pnpm build: failed, but only as a downstream consequence of the above —gql/(gitignored, generated by codegen) was never produced, so webpack fails withModule not found: Can't resolve 'gql'in the two pages that import generated types (app/(main)/courses/page.tsx,app/(main)/courses/[id]/page.tsx). No other build errors were observed.pnpm dev): pass. Server started,GET /returned HTTP 200 with the expected page title (星期四 Thursday) and no errors in the startup log. (Routes under/coursesdepend on the ungeneratedgql/module and were not exercised, consistent with the codegen limitation above.)next.config.tsworkaround assessmentExisting comment/workaround:
Checked after this week's bumps:
@typescript/typescript6is still at6.0.2(unchanged; already latest on the registry) and its published package still only shipsbin/tsc6, nobin/tsc. The workaround is still necessary — not safe to remove yet. (Did not touch this file.) No other unusual workaround/pin comments were found elsewhere in the repo besides this one.GitHub Actions
Checked all
uses:pins in.github/workflows/CI.yml(actions/checkout@v7,actions/setup-node@v7,pnpm/action-setup@v6,actions/cache@v6) against upstream tags. All four are already floating major-version tags pointing at the latest available major (checkout v7.0.1, setup-node v7.0.0, action-setup v6.1.0, cache v6.1.0 are the newest releases in their respective majors — no v8/v7 exists beyond what's already referenced). No workflow changes needed.Dockerfile
FROM node:lts-alpineis a floating tag — left alone per policy. No other hardcoded/pinned versions found worth flagging.Needs review
pnpm codegen/pnpm buildcould not be verified locally in this sandbox (no.env.local, no network access toapi.tsinghua.app). Please confirm CI'scheckjob (which runspnpm codegenthenpnpm buildwith real secrets) passes on this branch before merging.pnpm peers checkreports an unmet peer —graphql-request@7.4.0wantsgraphql@"14 - 16"butgraphql@17.0.2is installed. Both versions were already present before this week's update (introduced in a prior week's update) and neitherpnpm update --latestchanged them (each is already at its own latest release). Not capped/touched this run since it's outside this week's diff and its real-world impact couldn't be verified without the live schema. Worth a follow-up.eslintcapped at9.39.5(see above) — re-check for an ESLint-10-compatibleeslint-config-next/eslint-plugin-reactrelease in a future run and un-cap when available.🤖 Generated with Claude Code
https://claude.ai/code/session_01C454fCeZafg1A5YR4UU7Un
Generated by Claude Code