-
Notifications
You must be signed in to change notification settings - Fork 0
Support/openclaw #30
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support/openclaw #30
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,106 @@ | ||
| name: Release CLI | ||
|
|
||
| on: | ||
| pull_request: | ||
| branches: | ||
| - master | ||
| types: | ||
| - closed | ||
| paths: | ||
| - "packages/cli/**" | ||
| - ".github/workflows/release-cli.yml" | ||
| push: | ||
| branches: | ||
| - master | ||
| paths: | ||
| - "packages/cli/**" | ||
|
|
||
| permissions: | ||
| contents: read | ||
|
|
||
| concurrency: | ||
| group: release-cli-${{ github.workflow }}-${{ github.ref }} | ||
| cancel-in-progress: ${{ github.event_name == 'pull_request' }} | ||
|
|
||
| jobs: | ||
| validate: | ||
| name: Validate CLI release | ||
| if: github.event_name == 'push' || github.event.pull_request.merged == true | ||
| runs-on: ubuntu-latest | ||
| timeout-minutes: 15 | ||
|
|
||
| steps: | ||
| - name: Checkout | ||
| uses: actions/checkout@v6 | ||
|
|
||
| - name: Setup pnpm | ||
| uses: pnpm/action-setup@v6 | ||
|
|
||
| - name: Setup Node.js | ||
| uses: actions/setup-node@v6 | ||
| with: | ||
| node-version: 24 | ||
| cache: pnpm | ||
| cache-dependency-path: pnpm-lock.yaml | ||
|
|
||
| - name: Install dependencies | ||
| run: pnpm install --frozen-lockfile | ||
|
|
||
| - name: Test CLI | ||
| run: pnpm --filter @clankeroverflow/cli test | ||
|
|
||
| - name: Check CLI types | ||
| run: pnpm --filter @clankeroverflow/cli check-types | ||
|
|
||
| - name: Build CLI and plugin manifests | ||
| run: pnpm --filter @clankeroverflow/cli build | ||
|
|
||
| - name: Check package contents | ||
| run: pnpm --filter @clankeroverflow/cli pack --pack-destination /tmp | ||
|
|
||
| - name: Preview ClawHub bundle | ||
| run: pnpm dlx clawhub@0.18.0 package publish ./packages/cli --family bundle-plugin --dry-run | ||
|
|
||
| publish: | ||
| name: Publish CLI and plugins | ||
| if: github.event_name == 'push' || github.event.pull_request.merged == true | ||
| needs: validate | ||
| runs-on: ubuntu-latest | ||
| timeout-minutes: 15 | ||
| environment: production | ||
| permissions: | ||
| contents: read | ||
| id-token: write | ||
| env: | ||
| CLAWHUB_TOKEN: ${{ secrets.CLAWHUB_TOKEN }} | ||
|
|
||
| steps: | ||
| - name: Checkout | ||
| uses: actions/checkout@v6 | ||
|
|
||
| - name: Setup pnpm | ||
| uses: pnpm/action-setup@v6 | ||
|
|
||
| - name: Setup Node.js | ||
| uses: actions/setup-node@v6 | ||
| with: | ||
| node-version: 24 | ||
| cache: pnpm | ||
| cache-dependency-path: pnpm-lock.yaml | ||
| registry-url: https://registry.npmjs.org | ||
|
|
||
| - name: Install dependencies | ||
| run: pnpm install --frozen-lockfile | ||
|
|
||
| - name: Publish CLI package | ||
| working-directory: packages/cli | ||
| env: | ||
| NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }} | ||
| run: pnpm publish --access public --provenance --no-git-checks | ||
|
|
||
| - name: Configure ClawHub token fallback | ||
| if: env.CLAWHUB_TOKEN != '' | ||
| run: pnpm dlx clawhub@0.18.0 login --token "$CLAWHUB_TOKEN" --no-browser | ||
|
Comment on lines
+101
to
+103
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
When Useful? React with 👍 / 👎. |
||
|
|
||
| - name: Publish ClawHub bundle | ||
| run: pnpm dlx clawhub@0.18.0 package publish ./packages/cli --family bundle-plugin --owner clankeroverflow | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,15 @@ | ||
| { | ||
| "name": "clankeroverflow", | ||
| "version": "1.0.12", | ||
| "description": "Search-first memory for AI coding agents — log and reuse verified fixes across sessions", | ||
| "author": { | ||
| "name": "ClankerOverflow", | ||
| "url": "https://clankeroverflow.com" | ||
| }, | ||
| "homepage": "https://clankeroverflow.com", | ||
| "repository": "https://github.com/oussama/clankeroverflow", | ||
| "license": "MIT", | ||
| "keywords": ["debugging", "solutions", "knowledge-base", "mcp"], | ||
| "skills": "./skills/", | ||
| "mcpServers": "./.mcp.json" | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,11 @@ | ||
| { | ||
| "id": "clankeroverflow", | ||
| "name": "ClankerOverflow", | ||
| "description": "Search-first memory for AI coding agents — log and reuse verified fixes across sessions", | ||
| "version": "1.0.12", | ||
| "configSchema": { | ||
| "type": "object", | ||
| "additionalProperties": false | ||
| }, | ||
| "skills": ["./skills"] | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When a PR that changes
packages/cli/**is merged intomaster, GitHub emits both thepull_request: closedevent configured above and thispushevent. GitHub documents that a merged PR'sGITHUB_REFis the target branch ref, so both runs also share the concurrency key from lines 21-23 (GitHub Docs). Depending on event ordering, the PR run can cancel a push run while it is publishing, or the push run can execute afterward and attempt to publish the same immutable npm and ClawHub versions again. Keep only one publish trigger, or make one event validation-only.Useful? React with 👍 / 👎.