Skip to content

[codex] reuse release artifacts for npm staging - #27312

Merged
tamird merged 1 commit into
mainfrom
codex/reuse-release-artifacts-for-npm
Jun 10, 2026
Merged

[codex] reuse release artifacts for npm staging#27312
tamird merged 1 commit into
mainfrom
codex/reuse-release-artifacts-for-npm

Conversation

@tamird

@tamird tamird commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

The release job already downloads every workflow artifact into dist,
but npm staging creates a new cache and downloads the six target
artifacts again.

Reuse dist as the staging script's artifact cache while preserving the
existing download fallback for missing artifacts and standalone callers.
The script retains ownership of temporary caches but does not delete a
caller-provided directory.

In https://github.com/openai/codex/actions/runs/27242495616, the duplicate
download transferred 3.3 GiB and took 4 minutes 13 seconds. This should
reduce total release time by about 4 minutes.

The release job downloads all workflow artifacts into dist before it
stages npm packages. stage_npm_packages.py then creates a fresh cache
and downloads the six final target artifacts again.

Allow callers to provide an artifact directory and point the release
job at dist. Keep the existing temporary cache and download behavior
for standalone callers and for any missing artifacts.

Run 27242495616 spent 4 minutes 13 seconds redownloading 3.3 GiB during
npm staging. Reusing dist is expected to reduce total workflow time by
about 4 minutes.
@tamird
tamird marked this pull request as ready for review June 10, 2026 05:01
@tamird
tamird requested a review from a team as a code owner June 10, 2026 05:01
@tamird
tamird merged commit 387adc6 into main Jun 10, 2026
31 checks passed
@tamird
tamird deleted the codex/reuse-release-artifacts-for-npm branch June 10, 2026 20:15
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 10, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants