chore(ci): call the shared claude-review workflow - #74
Merged
Merged
Conversation
Reduces claude.yml to its triggers and permissions grant; the author gate, runner selection and the claude-code-action and checkout pins now come from RoboFinSystems/robosystems/.github/workflows/claude-review.yml, where they are maintained once rather than in each repo. Also picks up the SHA-pinned checkout from the shared workflow - this repo was on the mutable actions/checkout@v7 tag. No behaviour change: the gate and permissions are identical, and select-runner returns ["ubuntu-latest"] without an API call while RUNNER_LABELS is github-hosted. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0113UQ7evQKHNf1L1tFiZzLD
jfrench9
added a commit
that referenced
this pull request
Sep 2, 2026
Deletes this repo's copy of `tag-release.yml` (~400 lines) and calls the shared one in `robosystems`. **This is how the release-notes fix reaches this repo.** The local copy extracted the changelog with `.content[0].text`, which assumes the first content block is text. Once `CLAUDE_MODEL` moved to a model that thinks by default, responses lead with a thinking block, so that returned `null` and tripped the empty-content fallback — every release here has been emitting bare commit stats since early August. ## It also fixes the wrong product identity The copy here was inherited from the web app repo and never re-pointed. It told Claude: - `- App: RoboSystems (Comprehensive financial platform hub web application)` - "a **Next.js SDK library** release" - `- **Live App:** [robosystems.ai](https://robosystems.ai)` None of which describe an MCP client. Even with the parser fixed, this repo would have generated notes about the wrong product and linked the wrong site. It now passes `product_name: 'RoboSystems MCP Client'` and `project_kind: 'MCP client SDK'`, with no live link. ##⚠️ Merge order **RoboFinSystems/robosystems#1344 must merge first** — the shared workflow needs the new inputs, and merging this first would fail the next release rather than just degrade it. Independent of #74 (the `claude.yml` change), which is on a separate branch.
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.
Reduces
claude.ymlto its triggers plus the permissions grant ( 54 → 28 lines). The author gate, runner selection, and theclaude-code-action/checkoutpins now come fromRoboFinSystems/robosystems/.github/workflows/claude-review.yml.Also picks up the SHA-pinned checkout — this repo was on the mutable
actions/checkout@v7tag.No behaviour change: the gate and permissions are identical, and
select-runnerreturns["ubuntu-latest"]without an API call whileRUNNER_LABELSisgithub-hosted.Verified end to end in
robosystems(run 33592717239:check-runner-availability→claude, both green) and piloted in robosystems-python-client#206.