Repository navigation
fix(ci): repair ecosystem-sync false positives after websearch goNative - #103
Merged
Merged
Conversation
Re-apply the websearch integration on top of the latest main. - Switch websearch skill from archived Python SIN-Code-Websearch-Skill to Go-native OpenSIN-Code/web_search_bundle. - mcpclient registry now uses the built binary at SIN_SKILLS_DIR/web_search_bundle/sin-websearch and falls back to PATH. - Registry also uses the default skills dir (~/.local/share/sin-code/skills) when SIN_SKILLS_DIR env is unset, matching skillmgr behavior. - skillmgr verifies the Go entrypoint by building sin-websearch into the repo root with -o sin-websearch. - Add registry tests for local-binary, PATH-fallback, and default-skills-dir. - Add E2E test (build tag e2e) that installs websearch from GitHub and verifies the binary runs --help. - Add skill-e2e.yml GitHub Actions workflow to run the E2E test on main/push, on PRs touching skillmgr/mcpclient, and manually. - Update ECOSYSTEM.md to list web_search_bundle instead of the archived SIN-Code-Websearch-Skill.
The ecosystem-sync check grepped every quoted lowercase string in registry.go, which now produced false positives like 'mcp', 'os', and 'serve' after the goNative helper for the websearch skill was added. - Rewrite the server-name extraction to pull the actual short names from the shortName map and any explicit Name: entries in DefaultServers. - Replace SIN-Code-Websearch-Skill with web_search_bundle==v0.4.2 in requirements-ecosystem.txt to match ECOSYSTEM.md.
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
🏆 CEO Audit — A+ (100.0/100)
📥 Download full report (Markdown)
|
🏆 CEO Audit — A+ (100.0/100)
📥 Download full report (Markdown) Run ID:
|
This branch was successfully deployed
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.
The ecosystem-sync check grepped every quoted lowercase string in registry.go, which now produced false positives like 'mcp', 'os', and 'serve' after the goNative helper for the websearch skill was added.