feat(skills): vendor the outsource skill (explore/review/write)#14
Open
OriNachum wants to merge 1 commit into
Open
feat(skills): vendor the outsource skill (explore/review/write)#14OriNachum wants to merge 1 commit into
OriNachum wants to merge 1 commit into
Conversation
Cite-don't-import copy of the canonical outsource skill from guildmaster: hand a scoped repo task to convertible (a *different* engine/mind) for a diverse second opinion — outsource review|explore|write. Portable wrapper; resolves the convertible CLI from PATH. Patch bump per version-check rule. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
5 tasks
Review Summary by QodoVendor outsource skill for diverse second opinions via convertible
WalkthroughsDescription• Vendors the canonical outsource skill from guildmaster with three verbs • explore and review run read-only in isolated git worktrees at HEAD • write implements changes on a drive branch or opens a PR with --pr • Portable wrapper resolves convertible CLI from PATH or local dev fallback • Patch version bump (0.2.0 → 0.2.1) per version-check CI rule Diagramflowchart LR
User["User invokes outsource verb"] -->|explore/review| ReadOnly["Read-only in throwaway worktree"]
User -->|write| InPlace["In-place drive branch or PR"]
ReadOnly --> Convertible["convertible drive CLI"]
InPlace --> Convertible
Convertible --> Result["TaskResult summary printed"]
File Changes1. .claude/skills/outsource/scripts/outsource.sh
|
Code Review by Qodo
Context used✅ Compliance rules (platform):
22 rules 1. Missing outsource entry in docs
|
|
![qodo-code-review[bot]](https://avatars.githubusercontent.com/in/484649?s=60&v=4){kind=small}
Comment on lines
+98
to
+103
| ## Provenance | ||
|
|
||
| This is a **first-party convertible** skill — `agentculture/convertible` is its | ||
| origin. guildmaster **re-broadcasts** it to the mesh (the same inbound pattern as | ||
| the devague-origin workflow skills), tracking it in `docs/skill-sources.md`. The | ||
| `cite, don't import` policy holds: downstream repos copy it, they don't symlink |
There was a problem hiding this comment.
1. Missing outsource entry in docs 📘 Rule violation § Compliance
A new vendored skill outsource was added under .claude/skills/outsource/, but docs/skill-sources.md has no corresponding provenance/divergence entry even though the skill’s SKILL.md claims it is tracked there. This breaks the required audit trail/provenance map for vendored skills and will cause drift, confusion, and harder future re-vendoring and license/provenance reviews.
Agent Prompt
## Issue description
The PR vendors a new skill at `.claude/skills/outsource/` but does not add/update the required provenance/divergence documentation in `docs/skill-sources.md`, despite `SKILL.md` stating it is tracked there.
## Issue Context
`docs/skill-sources.md` is the required upstream/downstream map (vendoring provenance registry) for vendored skills and is used to guide future re-syncs/re-vendoring. The newly added `outsource` skill must be recorded with its source (guildmaster re-vendor / true origin), acquisition date/commit/tag if known, and divergence notes (even if “verbatim”); alternatively, if the intent is not to track it there, the claim in `SKILL.md` should be removed/adjusted to avoid misinformation.
## Fix Focus Areas
- .claude/skills/outsource/SKILL.md[98-104]
- docs/skill-sources.md[20-44]
ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools
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.
Summary
outsourceskill from guildmaster (cite-don't-import pattern)outsource review | explore | write— hand a scoped task to convertible (a different engine/mind) and fold its answer back; diversity catches what the author's mind glides pastexplore,review) run in an isolated throwaway git worktree and cannot touch the working treeTest plan
.claude/skills/outsource/SKILL.mdfrontmattername: outsourcematches directory namescripts/outsource.shis executable and resolvesconvertiblefrom PATHoutsource exploreagainst a small scope to verify the worktree isolation🤖 Generated with Claude Code