feat(autoskills): detect Xquik SDK projects - #182
Open
kriptoburak wants to merge 19 commits into
Open
Conversation
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.
What changed
x-twitter-scraperdependency as Xquik.e1ccbd1709a6cc290307b7815ddf26d929ab31a1.Why
Projects using the published Xquik package had no AutoSkills match. Exact package detection avoids unrelated Twitter packages.
The explicit audit marker separates OpenAI-reviewed records from skipped and ambiguous historical records. Audited syncs now re-review old approvals that lack this marker. No-review syncs retain their cache until an audit can run.
The whole-registry validator also replaces the existing Elysia placeholder bundle hash with its computed SHA-256 value. Without this correction, the validator rejects the upstream registry before any Skill can install.
Fixes #181.
Review changes
Retry-Afterbehavior.Verification
pnpm --dir packages/autoskills test: 378 passedpnpm --dir packages/autoskills buildpnpm fmt:checkpnpm lint: passed with 11 existing warnings outside this changepnpm buildpnpm --dir packages/autoskills validate:registryreports 1 defect already present onmain: the FastMCP map entry has no registry entry. This pull request does not change FastMCP.SkillSpector scanned all 71 public components. It reports 1 MIT-license heuristic, 0 confirmed issues, and a safe 3/100 risk score.
Checklist
Summary by CodeRabbit
Nuevas funciones
Seguridad
Documentación
Mejoras