Land ESLint v10 modernization (#284) + open dep/feature PRs + undici CVE fix#307
Land ESLint v10 modernization (#284) + open dep/feature PRs + undici CVE fix#307chad-fossa wants to merge 8 commits into
Conversation
The report JSON is interpolated into the run script as raw text, so a single quote in dependency copyright text (e.g. "Georg Reinke (<...>)") broke the echo '...' quoting and failed with a bash syntax error / exit 2. Pass the report via an env var instead. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
The env-var approach from the previous commit fixed the quoting bug but hit MAX_ARG_STRLEN: the attribution report is ~350KB on a single line, so placing it in an env var made the kernel reject starting bash with 'Argument list too long'. A quoted heredoc keeps the value in the script file (no argv/env size limit) and treats it literally (no quoting issues). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Bumps [actions/checkout](https://github.com/actions/checkout) from 6 to 7. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@v6...v7) --- updated-dependencies: - dependency-name: actions/checkout dependency-version: '7' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
Merges open PRs for combined end-to-end testing: - #303 add include-unused-deps input (feature) - #306 bump actions/checkout 6->7 - #284 ESLint v10 flat-config modernization (typescript-eslint + import-x) - #302 eslint ->10.3.0, #300 globals ->17.6.0 (pinned into #284's ranges) Obsoleted by #284 (packages removed), so not included: - #279 @typescript-eslint/parser, #301 eslint-plugin, #292 @eslint/compat Preserved from main (NOT reverted despite #284 being based pre-them): - #278 fossa test --diff push-event fix - #291 dependabot @types/node major guard Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015cyhpr2n4CDui7j9AVWukq
e2e result (run 28587298720)All functional steps passed under Node 24 with every merged PR's changes:
The single failure is This is not introduced by the merge: none of those packages are in the lockfile changes, |
@actions/http-client pulled undici 6.24.1, flagged by fossa test for CVE-2026-6733, CVE-2026-11525, CVE-2026-9679 (all fixed in 6.27.0). Add a resolutions entry and rebuild dist so the fix ships in the bundle. lodash@4.17.5 and pydantic-ai-slim@0.0.48 were also flagged but are not in this repo's manifests (phantom FOSSA-project issues); not fixable here. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015cyhpr2n4CDui7j9AVWukq
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Organization UI (inherited) Review profile: ASSERTIVE Plan: Pro Run ID: ⛔ Files ignored due to path filters (3)
📒 Files selected for processing (8)
👮 Files not reviewed due to content moderation or server errors (8)
Warning Walkthrough skippedFile diffs could not be summarized. Comment |
Add a dogfood job that runs the published @v2 action against this repo, complementing the working-copy (./) jobs which test the PR's own code. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015cyhpr2n4CDui7j9AVWukq
Purpose
Integration / test branch — merges the open PRs together so their combined behavior gets exercised end-to-end by
test.yml(realfossa analyze/test/container/report/--diffagainst the FOSSA key). Not intended to merge as-is; it's a green-light check that the open PRs are mutually compatible.What's included
include-unused-depsinput (feature)actions/checkout6 → 7typescript-eslint+eslint-plugin-import-x, drops airbnb/standard stack)Deliberately NOT included
Obsoleted by #284 — it removes the packages these bump:
@typescript-eslint/parser— package removed by Upgrade to ESLint v10 and modernize lint config #284@typescript-eslint/eslint-plugin— package removed by Upgrade to ESLint v10 and modernize lint config #284@eslint/compat— package removed by Upgrade to ESLint v10 and modernize lint config #284Preserved from
main, NOT reverted (#284 was branched before these landed; a naive merge would have reverted them):fossa test --diffpush-event fix@types/nodemajor-version guardLocal verification (Node 24)
yarn install— clean lockfile regenyarn lint— passes on the new flat configyarn build—dist/rebuilt and in sync;include-unused-depspresent in bundleTesting
Pushing this branch triggers
test.yml, which is the real e2e suite. Watch the checks tab.