chore(deps): combined web dependency update (react 19.3.0, AG Grid 36.2.0, jsdom 30.1.1, @types/node 26.6.2) - #309
Merged
Merged
Conversation
The "Start MinIO" step exits 125 on every CI run, including main, and reds both Test Python and ARQ worker integration: docker: Error response from daemon: unauthorized: access to the requested resource is not authorized (quay.io/minio/minio:RELEASE.2025-09-07T16-13-09Z) Root cause is upstream distribution, not this repo: - quay.io now grants anonymous pulls an empty action list for minio/minio (401), - minio/minio and minio/mc were deleted from Docker Hub on 2026-09-11, - the upstream repo was archived 2026-04-25 (Community Edition is source-only) and dl.min.io/server/ returns 410. github.com/minio/minio GitHub Releases still serves the official binary for this exact tag, together with a published .sha256sum. Download it, verify a checksum pinned inline, and run it directly. The test contract is unchanged and verified locally: localhost:9000, localhost:9001, MINIO_ROOT_USER/MINIO_ROOT_PASSWORD, and /minio/health/live, with the identical step in both jobs. The readiness loop now fails the step (and dumps the server log) instead of silently passing when MinIO never starts. Signed-off-by: Roland Salardon <r.salardon@gmail.com>
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
1 participant
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.
Supersedes #303, #305, #306, #307, #308.
Why a single PR
Each Dependabot PR is safe in isolation except where they interact:
reactto 119.3.0 but leavesreact-domat 19.2.7 →Incompatible React versionshard-fails 36/54 vitest files on its own.ag-grid-community(36.1.0 + 36.2.0) →TS2769×6 inFightsGrid,PlayersGrid,SquadRollupsGrid,TargetRollupsGrid,WebhookDlqGrid,WebhookSubscriptionsGrid. Neither is green alone; together they still need a regenerated lockfile (web/pnpm-lock.yamlconflicts in both merge orders).web/pnpm-lock.yaml.Landing them one by one turns
mainred in between. This PR carries all five plus the two fixes that make them coherent.Changes vs
mainweb/package.json:react19.2.7 → 19.3.0react-dom19.2.7 → 19.3.0 (required: React enforces an exact react/react-dom version match)@types/node^26.5→^26.6web/pnpm-lock.yamlregenerated so that:ag-grid-community/ag-grid-react/ag-stackall resolve to 36.2.0 (single copy each),ag-charts-types14.1.0 → 14.2.0jsdom30.1.0 → 30.1.1 (plus its own transitive subtree)react/react-dom19.3.0,@types/react19.3.0@types/node26.6.2No unrelated packages were changed;
pnpm-workspace.yamloverridesis byte-identical tomain.Verification (local, isolated worktree)
pnpm install --frozen-lockfile✅pnpm exec tsc --noEmit✅ 0 errorspnpm exec eslint .✅ 0 errors (16 pre-existing warnings)pnpm exec vitest run --coverage✅ 384 pass / 0 fail / 3 skip (lines 79.5, branches 68.3, functions 73.6)API_BASE_URL=http://api:8000 pnpm run build✅pnpm audit --audit-level=high --ignore GHSA-mh99-v99m-4gvg✅Review checklist
Lint WebBuild Web imagePlaywright (chromium)— this is the first branch to exercise AG Grid 36.2.0 in a browser; it wasskippedon chore(deps-dev): bump ag-grid-community from 36.1.0 to 36.2.0 in /web #305/chore(deps-dev): bump ag-grid-react from 36.1.0 to 36.2.0 in /web #308 because itneeds: lint-webvisual-regressioniscontinue-on-errorwith stale baselines — advisory onlyCI infrastructure fix (included)
pull for
minio/minioand the repository was deleted from Docker Hub on 2026-09-11,which had been reding
Test PythonandARQ worker integrationon every runincluding
main.f8f3171replaces the unavailable registry image with the pinned officialGitHub Release binary for the same tag (
RELEASE.2025-09-07T16-13-09Z), verifiedagainst a SHA-256 pinned inline before execution.
Test PythonandARQ worker integrationare now green.mergeState=CLEAN.Closes #303
Closes #305
Closes #306
Closes #307
Closes #308