fix(release): correct package identifier and restore GitHub Release step
- Revert workflow to use internal identifier 'llmx' instead of scoped name - Revert build_npm_package.py to only accept internal identifiers - Restore "Create GitHub Release" step that was incorrectly removed - Package name '@valknarthing/llmx' is read from package.json by the script This matches OpenAI's approach where they use 'codex' not '@openai/codex' in workflow. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
10
.github/workflows/rust-release.yml
vendored
10
.github/workflows/rust-release.yml
vendored
@@ -447,6 +447,16 @@ jobs:
|
|||||||
--release-version "${{ steps.release_name.outputs.name }}" \
|
--release-version "${{ steps.release_name.outputs.name }}" \
|
||||||
--package llmx
|
--package llmx
|
||||||
|
|
||||||
|
- name: Create GitHub Release
|
||||||
|
uses: softprops/action-gh-release@v2
|
||||||
|
with:
|
||||||
|
name: ${{ steps.release_name.outputs.name }}
|
||||||
|
tag_name: ${{ github.ref_name }}
|
||||||
|
files: dist/**
|
||||||
|
# Mark as prerelease only when the version has a suffix after x.y.z
|
||||||
|
# (e.g. -alpha, -beta). Otherwise publish a normal release.
|
||||||
|
prerelease: ${{ contains(steps.release_name.outputs.name, '-') }}
|
||||||
|
|
||||||
- uses: facebook/dotslash-publish-release@v2
|
- uses: facebook/dotslash-publish-release@v2
|
||||||
env:
|
env:
|
||||||
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
|
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
|
||||||
|
|||||||
@@ -31,7 +31,7 @@ def parse_args() -> argparse.Namespace:
|
|||||||
parser = argparse.ArgumentParser(description="Build or stage the LLMX CLI npm package.")
|
parser = argparse.ArgumentParser(description="Build or stage the LLMX CLI npm package.")
|
||||||
parser.add_argument(
|
parser.add_argument(
|
||||||
"--package",
|
"--package",
|
||||||
choices=("llmx", "llmx-responses-api-proxy", "llmx-sdk", "@valknarthing/llmx"),
|
choices=("llmx", "llmx-responses-api-proxy", "llmx-sdk"),
|
||||||
default="llmx",
|
default="llmx",
|
||||||
help="Which npm package to stage (default: llmx).",
|
help="Which npm package to stage (default: llmx).",
|
||||||
)
|
)
|
||||||
@@ -76,9 +76,6 @@ def main() -> int:
|
|||||||
args = parse_args()
|
args = parse_args()
|
||||||
|
|
||||||
package = args.package
|
package = args.package
|
||||||
# Normalize scoped package name to internal identifier
|
|
||||||
if package == "@valknarthing/llmx":
|
|
||||||
package = "llmx"
|
|
||||||
version = args.version
|
version = args.version
|
||||||
release_version = args.release_version
|
release_version = args.release_version
|
||||||
if release_version:
|
if release_version:
|
||||||
|
|||||||
Reference in New Issue
Block a user