Updates siteglide-cli gui to latest improvements from pos-cli source, plus initial support for AI skills and MCP. - #81
Open
MattJonesSiteglide wants to merge 38 commits into
Open
Updates siteglide-cli gui to latest improvements from pos-cli source, plus initial support for AI skills and MCP. #81MattJonesSiteglide wants to merge 38 commits into
MattJonesSiteglide wants to merge 38 commits into
Conversation
…hen it's safe to start work again or commit the pull to git.
… gives the server more time before checking again. This can mean fast processes get done much more quickly, whereas overall the polling will not over-do it.
…at local Siteglide MCP repo (private)
Adding gui interface.
Add pos cli check command
Skip IDE MCP registration unless alpha npm credentials exist; check installed version and offer alpha upgrades when MCP is already configured. Point local MCP dependency at Siteglide-MCP. Co-authored-by: Cursor <cursoragent@cursor.com>
…sets same as app assets when deploying.
…s starting with "."; add default module skip list; adding quicker script file for testing internally;
MattJonesSiteglide
marked this pull request as ready for review
August 17, 2026 16:30
MattJonesSiteglide
marked this pull request as draft
August 18, 2026 09:58
Author
|
Sorry last minute changes to where to expect the live published MCP version to be were buggy + realised I need to remove the app vs. marketplace_builder change from this branch as I'd only removed from my other branch. Hold off on the testing for a moment. |
…es to marketplace_builder/ app to now use a function so it can be replaced later if agreed.
…nt, but should upgrade when MCP available. Should priortise install from npm, but will use github as backup if Siteglide (for pre-alpha testing - temporary really).
Co-authored-by: Cursor <cursoragent@cursor.com>
MattJonesSiteglide
marked this pull request as ready for review
August 18, 2026 13:40
Member
|
Contains this too I think? -> https://feedback.siteglide.com/p/siteglide-cli-when-pulling-skip-files-with-a-physical-file Mentioned here -> #78 (comment) Confirmed as not relevant any more -> https://discord.com/channels/@me/1026792778384015370/1539929662342299739 |
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
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.
Contains all of:
https://feedback.siteglide.com/p/siteglide-cli-pull-should-pull-all-public-module-files
https://feedback.siteglide.com/p/siteglide-cli-add-command-to-download-latest-ai-skills-into
https://feedback.siteglide.com/p/siteglide-cli-better-logs-ui
Does not contain:
https://feedback.siteglide.com/p/siteglide-mcp (but contains code which references and installs it)
Testing:
Siteglide CLI can work independently of Siteglide MCP for now. It is not an npm dependency. (Maybe it can be- just not while you're figuring out how to release and still want to test).
However, when pulling, it checks
1st - is it published on npm?
2nd do you have github access to private repo
If either are true, it will npm install the package and set up the files so its discoverable by your AI.
This is not really the right way of doing it necessarily, it's just easiest compromise prior to a decision being made!
Let me know if I can change anything to make it easier to test!