[PR #32] [MERGED] chore: update CI container toolchain and release flow #31

Closed
opened 2026-10-26 22:23:07 +00:00 by bsakaguchi · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/Comfy-Org/comfyui-ci-container/pull/32
Author: @DrJKL
Created: 9/28/2026
Status: ✅ Merged
Merged: 9/28/2026
Merged by: @DrJKL

Base: amp/orb-setup ← Head: amp/playwright-1.63.0


📝 Commits (4)

  • a9591fb chore: update Playwright to 1.63.0
  • 2c9419b chore: align Node default with frontend
  • 3233bee chore: streamline container release flow
  • ef960a8 docs: use frontend setup action

📊 Changes

4 files changed (+51 additions, -64 deletions)

View changed files

➕ .agents/skills/updating-ci-container/SKILL.md (+18 -0)
📝 .github/workflows/build-and-push.yml (+11 -17)
📝 Dockerfile (+4 -4)
📝 README.md (+18 -43)

📄 Description

Publishes the updated test toolchain and makes main the only automatic release path.

Changes

  • updates Playwright from v1.61.1-noble to v1.63.0-noble
  • updates the preinstalled Node.js default from 25 to 26
  • removes pull request builds and duplicate builds triggered by generated version tags
  • trims stale README sections and uses the frontend's shared setup action
  • adds the updating-ci-container repository skill

Release order

This container must be published before ComfyUI_frontend can consume it. After this PR merges, the release workflow publishes the image and opens the frontend update PR. That frontend PR must update @playwright/test to 1.63.0 before it merges.

Stacked on #31.

Verification

  • docker buildx build --check .
  • actionlint .github/workflows/*.yml
  • Playwright image manifest checked for linux/amd64 and linux/arm64
  • confirmed ComfyUI_frontend's shared setup action reads .nvmrc through actions/setup-node@v6
  • workspace skill reload detected updating-ci-container

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/Comfy-Org/comfyui-ci-container/pull/32 **Author:** [@DrJKL](https://github.com/DrJKL) **Created:** 9/28/2026 **Status:** ✅ Merged **Merged:** 9/28/2026 **Merged by:** [@DrJKL](https://github.com/DrJKL) **Base:** `amp/orb-setup` ← **Head:** `amp/playwright-1.63.0` --- ### 📝 Commits (4) - [`a9591fb`](https://github.com/Comfy-Org/comfyui-ci-container/commit/a9591fbe848c7bde6bf49793e53062e8ecdfe8ac) chore: update Playwright to 1.63.0 - [`2c9419b`](https://github.com/Comfy-Org/comfyui-ci-container/commit/2c9419b103e3ba1eddd8641fc94c278af1602851) chore: align Node default with frontend - [`3233bee`](https://github.com/Comfy-Org/comfyui-ci-container/commit/3233beed81b725c0cfbe8f604210f7cbc793ae5f) chore: streamline container release flow - [`ef960a8`](https://github.com/Comfy-Org/comfyui-ci-container/commit/ef960a83be72389f82c31461fea52ddd4c4a04c5) docs: use frontend setup action ### 📊 Changes **4 files changed** (+51 additions, -64 deletions) <details> <summary>View changed files</summary> ➕ `.agents/skills/updating-ci-container/SKILL.md` (+18 -0) 📝 `.github/workflows/build-and-push.yml` (+11 -17) 📝 `Dockerfile` (+4 -4) 📝 `README.md` (+18 -43) </details> ### 📄 Description Publishes the updated test toolchain and makes `main` the only automatic release path. ## Changes - updates Playwright from `v1.61.1-noble` to `v1.63.0-noble` - updates the preinstalled Node.js default from 25 to 26 - removes pull request builds and duplicate builds triggered by generated version tags - trims stale README sections and uses the frontend's shared setup action - adds the `updating-ci-container` repository skill ## Release order This container must be published before ComfyUI_frontend can consume it. After this PR merges, the release workflow publishes the image and opens the frontend update PR. That frontend PR must update `@playwright/test` to 1.63.0 before it merges. Stacked on #31. ## Verification - `docker buildx build --check .` - `actionlint .github/workflows/*.yml` - Playwright image manifest checked for linux/amd64 and linux/arm64 - confirmed ComfyUI_frontend's shared setup action reads `.nvmrc` through `actions/setup-node@v6` - workspace skill reload detected `updating-ci-container` --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
bsakaguchi 2026-10-26 22:23:07 +00:00
Sign in to join this conversation.
No labels
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Comfy-Org/comfyui-ci-container#31
No description provided.