From 5d5a4ba153b8bfc41bd87c730331ac24eb285faa Mon Sep 17 00:00:00 2001 From: nicolethoen Date: Fri, 20 Mar 2026 10:49:19 -0400 Subject: [PATCH 1/3] chore: fix a11y tests --- .github/workflows/build-lint-test.yml | 2 +- package.json | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/build-lint-test.yml b/.github/workflows/build-lint-test.yml index c35cc7e..84463fb 100644 --- a/.github/workflows/build-lint-test.yml +++ b/.github/workflows/build-lint-test.yml @@ -147,7 +147,7 @@ jobs: - run: yarn install --immutable if: steps.yarn-cache.outputs.cache-hit != 'true' - name: Install Chrome for Puppeteer - run: npx puppeteer browsers install chrome + run: npx @puppeteer/browsers install chrome@stable - uses: actions/cache@v4 id: dist name: Cache dist diff --git a/package.json b/package.json index 3d25af3..0d21590 100644 --- a/package.json +++ b/package.json @@ -8,7 +8,7 @@ "react", "log", "viewer", - "logviewer", + "logviewer" ], "workspaces": [ "packages/*" From 75c8f32589e47394cc896ab9d67631c7120f5566 Mon Sep 17 00:00:00 2001 From: nicolethoen Date: Fri, 20 Mar 2026 10:52:29 -0400 Subject: [PATCH 2/3] fix: use @puppeteer/browsers@latest for Chrome install in CI The project's transitive @puppeteer/browsers dependency is too old and uses stale download URLs, causing 404 errors. Co-Authored-By: Claude Opus 4.6 --- .github/workflows/build-lint-test.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/build-lint-test.yml b/.github/workflows/build-lint-test.yml index 84463fb..da63271 100644 --- a/.github/workflows/build-lint-test.yml +++ b/.github/workflows/build-lint-test.yml @@ -147,7 +147,7 @@ jobs: - run: yarn install --immutable if: steps.yarn-cache.outputs.cache-hit != 'true' - name: Install Chrome for Puppeteer - run: npx @puppeteer/browsers install chrome@stable + run: npx @puppeteer/browsers@latest install chrome@stable - uses: actions/cache@v4 id: dist name: Cache dist From 6bf6c4db5c2cfbd19e55582dbe64a79ec74866a8 Mon Sep 17 00:00:00 2001 From: nicolethoen Date: Fri, 20 Mar 2026 10:58:04 -0400 Subject: [PATCH 3/3] fix: add puppeteer as direct devDependency for a11y CI puppeteer was only a transitive dependency via @patternfly/patternfly-a11y, so npx couldn't find it in Yarn 4. Adding it as a direct devDependency matches the setup in other patternfly repos. Co-Authored-By: Claude Opus 4.6 --- .github/workflows/build-lint-test.yml | 2 +- packages/module/package.json | 1 + yarn.lock | 1 + 3 files changed, 3 insertions(+), 1 deletion(-) diff --git a/.github/workflows/build-lint-test.yml b/.github/workflows/build-lint-test.yml index da63271..c35cc7e 100644 --- a/.github/workflows/build-lint-test.yml +++ b/.github/workflows/build-lint-test.yml @@ -147,7 +147,7 @@ jobs: - run: yarn install --immutable if: steps.yarn-cache.outputs.cache-hit != 'true' - name: Install Chrome for Puppeteer - run: npx @puppeteer/browsers@latest install chrome@stable + run: npx puppeteer browsers install chrome - uses: actions/cache@v4 id: dist name: Cache dist diff --git a/packages/module/package.json b/packages/module/package.json index 3a7d88b..337cce0 100644 --- a/packages/module/package.json +++ b/packages/module/package.json @@ -47,6 +47,7 @@ "@patternfly/react-table": "^6.0.0", "camel-case": "^3.0.0", "monaco-editor": "^0.34.1", + "puppeteer": "^24.15.0", "react-monaco-editor": "^0.51.0", "resize-observer-polyfill": "^1.5.1", "tslib": "^2.0.0" diff --git a/yarn.lock b/yarn.lock index db8e9ce..0e31237 100644 --- a/yarn.lock +++ b/yarn.lock @@ -4064,6 +4064,7 @@ __metadata: camel-case: "npm:^3.0.0" memoize-one: "npm:^5.1.0" monaco-editor: "npm:^0.34.1" + puppeteer: "npm:^24.15.0" react-monaco-editor: "npm:^0.51.0" resize-observer-polyfill: "npm:^1.5.1" tslib: "npm:^2.0.0"