fix(ci): regenerate uv.lock for ogx-client 1.2.0 on release-1.2.x#6300
Merged
Conversation
pyproject.toml pins ogx-client==1.2.0 but the committed uv.lock still resolved ogx-client 1.1.3 (and carried a stale >=1.1.3 specifier), so the pre-commit uv-lock check failed on the branch. Regenerate the lock to match: ogx-client 1.1.3 -> 1.2.0 and drop the no-longer-required pyaml transitive dependency. Co-Authored-By: Claude Opus 4.8 <[email protected]> Signed-off-by: Charlie Doern <[email protected]>
cdoern
requested review from
bbrowning,
franciscojavierarceo,
leseb,
mattf,
raghotham and
skamenan7
as code owners
July 14, 2026 17:58
The ogx-client-typescript entry in the pypi.yml build matrix was
commented out on release-1.2.x while its external repo lacked a matching
release branch. That left it as the only entry defining `repo`, so
`repository: ${{ matrix.repo }}` referenced an undefined matrix
property and the actionlint pre-commit hook failed.
The matching release-1.2.x branch now exists in ogx-client-typescript,
so restore the matrix entry to match main. The runtime should-build
guard still skips the TypeScript build, so publish behavior is unchanged;
this only restores the matrix type so actionlint passes.
Co-Authored-By: Claude Opus 4.8 <[email protected]>
Signed-off-by: Charlie Doern <[email protected]>
This was referenced Jul 14, 2026
cdoern
added a commit
that referenced
this pull request
Jul 14, 2026
…) (#6302) ## Summary Backport of #6301 to `release-1.2.x`, enabling the TypeScript client in the release publish pipeline now that `ogx-ai/ogx-client-typescript` has a matching `release-1.2.x` branch. `release-1.2.x` needed **three** edits (one more than #6301 on main, whose publish-matrix entry was already live): 1. Remove the build `should-build` TEMPORARY skip for `ogx-client-typescript`. 2. **Restore the `ogx-client-typescript` entry in the publish matrix** — it was still commented out here; #6300 only restored the *build* matrix entry (to fix actionlint). 3. Remove the publish `should-publish` TEMPORARY skip. ## Effect With this on `release-1.2.x`, a `packages: all` (default) or `clients-only` release run builds **and** publishes `ogx-client-typescript` to npm, alongside the Python packages. A 1.2.1 cut will then include the TS client. ## Test plan - `actionlint` pre-commit hook passes on the edited `pypi.yml`. - The npm build/publish path is exercised on the next release run. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Signed-off-by: Charlie Doern <[email protected]> Co-authored-by: Claude Opus 4.8 <[email protected]>
pull Bot
pushed a commit
to Ucg2c3/llama-stack
that referenced
this pull request
Jul 14, 2026
…exist (ogx-ai#6301) ## Summary Both the build and publish matrices in `pypi.yml` special-cased `ogx-client-typescript` with an unconditional `skip=true`, added while the external `ogx-client-typescript` repo lacked matching release branches: ```bash # TEMPORARY: skip ogx-client-typescript until external repo has release branch if [ "$PACKAGE" == "ogx-client-typescript" ]; then echo "skip=true" ... ``` Those branches now exist (`main` and `release-1.2.x` in `ogx-ai/ogx-client-typescript`), so the temporary guard is no longer needed. Remove both guards (build `should-build` and publish `should-publish`) so the TypeScript client flows through the normal `packages` selection (`all` / `clients-only`) like every other package. No matrix changes needed on `main` — both TS entries were already live (they're what keeps actionlint happy on `matrix.repo`). This only removes the runtime skip. ## Effect - `packages: all` (default) and `packages: clients-only` now build **and** publish `ogx-client-typescript` to npm. - Python packages unchanged. ## Backport note The `release-1.2.x` backport needs one extra change beyond this diff: its **publish** matrix entry for `ogx-client-typescript` is still commented out (only the build-matrix entry was restored in ogx-ai#6300), so it must be uncommented there too. ## Test plan - `actionlint` pre-commit hook passes on the edited `pypi.yml`. - The npm build/publish path is exercised on the next `clients-only`/`all` release run. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Signed-off-by: Charlie Doern <[email protected]> Co-authored-by: Claude Opus 4.8 <[email protected]>
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
1 participant
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
pyproject.tomlonrelease-1.2.xpinsogx-client==1.2.0, but the committeduv.lockstill resolvedogx-client1.1.3 (and carried a stale>=1.1.3specifier). The pre-commit uv-lock check regenerates the lock and diffs it, so it failed on the branch — this is the pre-commit failure seen on the--insecurebackport (#6293).Change
Regenerate
uv.lockto matchpyproject.toml:ogx-client1.1.3 → 1.2.0pyamltransitive dependencyNo source changes; lockfile only.
Test plan
Separate from the build-job failures on #6293, which are addressed by the
install-ogx-clientin-repo generation fix (#6297, being backported torelease-1.2.x).🤖 Generated with Claude Code