-
Notifications
You must be signed in to change notification settings - Fork 171
fix: MCP proxy uses the repo space instead of activeSpace #124
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
Merged
Merged
Changes from all commits
Commits
Show all changes
6 commits
Select commit
Hold shift + click to select a range
4282a2a
fix: MCP proxy uses the repo space instead of activeSpace
MaheshtheDev 5d6c501
fix: advertise that search_memory defaults to the repo space
MaheshtheDev 490b27a
chore: bump plugin to 0.1.7
MaheshtheDev d595260
chore: trim mcp-proxy comments to one function-level line
MaheshtheDev f55b6ae
Revert "chore: bump plugin to 0.1.7"
MaheshtheDev ae2bac0
ci: auto-bump plugin version on merge to main
MaheshtheDev File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,44 @@ | ||
| name: Release | ||
|
|
||
| on: | ||
| push: | ||
| branches: [main] | ||
|
|
||
| permissions: | ||
| contents: write | ||
|
|
||
| jobs: | ||
| bump: | ||
| if: ${{ !startsWith(github.event.head_commit.message, 'chore(release):') }} | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v6 | ||
| with: | ||
| fetch-depth: 0 | ||
|
|
||
| - uses: actions/setup-node@v6 | ||
| with: | ||
| node-version: "20" | ||
|
|
||
| - name: Bump patch version | ||
| run: | | ||
| git config user.name "github-actions[bot]" | ||
| git config user.email "41898282+github-actions[bot]@users.noreply.github.com" | ||
| npm version patch --no-git-tag-version | ||
| VERSION="$(node -p "require('./package.json').version")" | ||
| node -e ' | ||
| const fs = require("fs"); | ||
| const version = require("./package.json").version; | ||
| for (const file of ["plugin/.claude-plugin/plugin.json", "latest.json"]) { | ||
| const next = fs.readFileSync(file, "utf8").replace( | ||
| /"version":\s*"[^"]+"/, | ||
| `"version": "${version}"`, | ||
| ); | ||
| fs.writeFileSync(file, next); | ||
| } | ||
| ' | ||
| git add package.json package-lock.json plugin/.claude-plugin/plugin.json latest.json | ||
| git commit -m "chore(release): ${VERSION}" | ||
|
|
||
| - name: Push version bump | ||
| run: git push origin HEAD:main | ||
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
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
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
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
Oops, something went wrong.
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.
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 multiple commits reach
mainbefore their release runs finish, each run checks out its own event SHA and creates a different bump commit; after one run or a later merge advancesmain, this unconditional push can be rejected as non-fast-forward. Git permits ordinary branch updates only when the new tip descends from the remote tip (git-push documentation), so the workflow can fail and omit the promised per-merge version bump. Cancel stale runs or fetch/rebase and regenerate the bump before retrying the push.Useful? React with 👍 / 👎.