feat: import owned public GitHub skills#2444
Conversation
|
@vyctorbrzezowski is attempting to deploy a commit to the Amantus Machina Team on Vercel. A member of the Team first needs to authorize it. |
|
Codex review: needs real behavior proof before merge. Reviewed June 1, 2026, 4:04 PM ET / 20:04 UTC. Summary Reproducibility: unclear. The review failed before ClawSweeper could establish a reproduction path. Review metrics: none identified. Merge readiness Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch. Risk before merge
Maintainer options:
Next step before merge
Review detailsBest possible solution: Retry the Codex review after fixing the execution failure. Do we have a high-confidence way to reproduce the issue? Unclear. The review failed before ClawSweeper could establish a reproduction path. Is this the best way to solve the issue? Unclear. Retry the review first so ClawSweeper can evaluate the actual issue and fix direction. AGENTS.md: unclear because the file could not be read completely. Codex review notes: model gpt-5.5, reasoning high; reviewed against cb6ced7906f7. Label changesLabel changes:
Label justifications:
Evidence reviewedWhat I checked:
Likely related people:
What the crustacean ranks mean
Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics. How this review workflow works
|
|
🦞🧹 I asked ClawSweeper to review this item again. Re-review progress:
|
|
@clawsweeper re-review Addressed the remaining compatibility point in
Validation after the fix:
|
|
🦞🧹 I asked ClawSweeper to review this item again. Re-review progress:
|
|
@clawsweeper re-review |
|
🦞🧹 I asked ClawSweeper to review this item again. Re-review progress:
|
…wned-public-import # Conflicts: # src/__tests__/import.route.test.tsx # src/routes/dashboard.tsx # src/routes/import.tsx
Summary
Adds the owned-public GitHub import flow for ClawHub skills:
/importnow scans the signed-in GitHub account for realSKILL.md/ legacyskills.mdcandidates, then lets users select, review, and publish multiple skills in one flow.GITHUB_TOKENis configured by using GitHub Code Search; the fallback is bounded and handles truncated trees via archive detection.Demo
export-1780093667365.mp4
Video demo: https://sleek-vigil-s2a3.here.now/github-import-demo.mp4
Visuals
Review Findings Addressed
Codex review accepted/fixed these P2 findings:
defaultSelectedso only intended files start selected.gpt-4and de-dupes togpt-4-2.One P2 recommendation was intentionally not applied: re-adding version/tag controls. The import UX intentionally exposes display name, slug, icon, selected files, and MIT-0 confirmation only; version/tags stay defaulted by the import pipeline.
Latest ClawSweeper follow-up addressed:
skills.mdcompatibility for URL parsing, archive/tree candidate detection, GitHub Code Search discovery, docs, and specs.bunx tsc --noEmit --pretty false, andbun run ci:staticafter these fixes.Validation
bunx vitest run src/__tests__/import.route.test.tsx convex/githubImport.test.ts convex/lib/githubImport.test.ts— 35 tests passed.bunx vitest run src/__tests__/import.route.test.tsx src/__tests__/skills-publish-route.test.tsx src/routes/-dashboard.test.tsx convex/githubImport.test.ts convex/lib/githubImport.test.ts— 58 tests passed during the review/fix pass.bun run ci:static— passed.bunx tsc --noEmit --pretty false— passed.bun run ci:unit— 231 files, 2564 tests passed; coverage summary: 86.52% statements, 75.09% branches, 87.73% functions, 90.17% lines.bun run ci:types-build— passed, including app/package typechecks and production build. Build emitted the existing non-fatal Shiki wasm fallback warning.