Skip to content

fix: generate PaperViz container build inputs - #3

Merged
GoatInAHat merged 1 commit into
mainfrom
fix/container-release-inputs
Sep 7, 2026
Merged

GoatInAHat merged 1 commit into
mainfrom
fix/container-release-inputs

Conversation

@GoatInAHat

Copy link
Copy Markdown
Owner

The first container release failed because its fresh checkout lacked web/src/ops.json, a ToolFactory build product that is intentionally untracked. Generate the container inputs before invoking Docker, matching the successful package and Pages jobs. Exclude local dependencies and environment files from the Docker context, document the local build prerequisite, and smoke-test the built image's CLI and web assets.

Validation: reproduced a clean checkout, ran the published ToolFactory 0.1.2 build, confirmed the operations schema exists, and completed the locked web install and production build. ToolFactory drift check and whitespace checks pass. Runtime tests, release packaging and Pages passed in the previous run; the next dispatch verifies the actual Linux container build.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 7, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-07T22:38:05.589279Z 4f68f61 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@GoatInAHat
GoatInAHat merged commit 8277d92 into main Sep 7, 2026
4 checks passed
@GoatInAHat
GoatInAHat deleted the fix/container-release-inputs branch September 7, 2026 22:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant