ci: streamline checks and release artifacts - #76
Merged
Conversation
Centralize project commands in package.json and remove the duplicate justfile.\n\nBuild tagged npm and OCI artifacts once, then publish them in isolated jobs.
There was a problem hiding this comment.
Pull request overview
Streamlines project checks and release automation while centralizing commands in package scripts.
Changes:
- Replaces Just tasks with unified package scripts.
- Simplifies PR checks and separates npm/GHCR publishing.
- Adds OCI metadata, CODEOWNERS, and aligned documentation.
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
package.json |
Centralizes build and validation scripts. |
justfile |
Removes the secondary task runner. |
.github/workflows/pr.yml |
Runs the unified PR check. |
.github/workflows/release.yml |
Builds and publishes reusable release artifacts. |
.github/CODEOWNERS |
Protects release-sensitive files. |
Containerfile |
Uses the explicit Codex package name. |
src/runtimes/openai/app-server-client.ts |
Uses explicit Codex runtime constants. |
tests/driver-artifact-contract.test.ts |
Updates the image script contract. |
README.md |
Documents the new commands. |
.env.example |
Updates live-test instructions. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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
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.
Summary
Why
The project no longer needs a second task runner.
Checks, builds, and publishing should have one clear path without repeating work across CI jobs.
Impact
Pull requests no longer build container images.
Version tags produce one npm package and one OCI image, with npm publishing gated by the npm environment.
The image is published with the package version and latest tags.
Validation
vp run checkCloses #75