fix: resolve four local skills onboarding friction points#43
Merged
scotthavird merged 1 commit intomainfrom Apr 11, 2026
Merged
fix: resolve four local skills onboarding friction points#43scotthavird merged 1 commit intomainfrom
scotthavird merged 1 commit intomainfrom
Conversation
- Add --dry-run flag to `skills generate` (was documented but unimplemented; silently ignored on every invocation) - Add diagnostic error when repo filter yields zero transcripts: tells user how many global transcripts exist and how to opt out with --repo='' - Improve 5-conversation minimum messaging: distinguishes "not enough new transcripts" from "not enough total" and explains --force clearly - Fix outputLocalSkillsGenerated to accurately report write failures: marks unwritten skills inline, shows "N of M skills written" when partial, and prints an actionable error when 0 were written (was silently reporting success) - Fix URL: promptconduit.io/skills → promptconduit.dev/skills Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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
Four friction points in
promptconduit skills generate --localthat would cause silent failures or confusing errors for first-time users (e.g., from a Show HN post).Changes
cmd/skills.go--dry-runflag onskillsGenerateCmd(was documented in README and DocsPage but never wired up — silently ignored)promptconduit.io/skills→promptconduit.dev/skillscmd/skills_local.go--repoauto-detection filters all transcripts to zero, now returns an actionable error explaining how many global transcripts exist and how to use--repo=''to opt out. Previously silently fell through to the 5-conv guard with a confusing message.skillsDryRunand call newoutputLocalSkillsDryRuninstead of writing files. Shows what would be written with predicted paths.outputLocalSkillsGeneratedpreviously printed "N skills written" even when all writes failed. Now marks unwritten skills inline with "Not written (see errors below)", reports "N of M skills written" for partial success, and prints an actionable stderr error when 0 were written.Testing
make build— compiles cleanlymake test— all tests pass