Add Execution Procedures, split oversized refs, improve structure#1
Open
whiletrue0x wants to merge 4 commits intofeatbit:mainfrom
Open
Add Execution Procedures, split oversized refs, improve structure#1whiletrue0x wants to merge 4 commits intofeatbit:mainfrom
whiletrue0x wants to merge 4 commits intofeatbit:mainfrom
Conversation
Comprehensive quality audit of all 17 skills using skill-forge methodology. Covers security, structure, quality, and publishing readiness validation. Result: PASS with 2 minor warnings (nested reference cross-link, collection size at threshold). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Regenerated REVIEW-REPORT.md and REVIEW-REPORT.pdf with fresh skill-forge audit - Added missing metadata.author: FeatBit to deployment-docker and deployment-aws skills - All 17 skills now pass full validation (security, structure, quality, publishing) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Full skill-forge review covering all 17 skills and 15 reference files. Corrects previous report's terminology assessment and adds 3 systemic patterns + 7 per-skill findings. Also gitignore conversation exports. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Add Execution Procedure with pseudocode to all 17 SKILL.md files - Add YAML frontmatter to all 18 reference files - Split troubleshooting.md (919 lines) into 3 files by category - Split environment-variables.md: extract OTEL config into separate file - Remove redundant sections (When to Use, Source, Read Next) from all skills - Normalize terminology: "feature flag" across all SDK skills - Fix AGENTS.md stale naming example (featbit-dotnet-sdk → featbit-sdks-dotnet) - Improve README: value proposition, Quick Start, reference-style badges, logo - Move review report to docs/, remove old PDF report - Fix graceful-skip conditions and one-sided conditionals in SDK skills - Consolidate duplicate code blocks in featbit-sdks-java Co-Authored-By: Claude Opus 4.6 (1M context) <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
## Execution Procedurewith pseudocode to all 17 SKILL.md files — agents now follow structured workflows instead of interpreting prosename,description) to all 18 reference filestroubleshooting.md(919 lines) → 3 files,environment-variables.md(754 lines) → 2 filesWhen to Use,Source,Read Next) that duplicated frontmatter descriptions across all skillsfeatbit-dotnet-sdk→featbit-sdks-dotnet)featbit-sdks-javaWhy
Ran a comprehensive skill-forge v3 review and found 5 systemic patterns across all 17 skills. The core issue: every skill was missing an Execution Procedure, causing AI agents to treat skills as passive knowledge documents rather than structured workflows. Full review report included in
docs/REVIEW-REPORT.md.Changes by category
Structure (all 17 skills)
## Execution Procedurewith a pseudocode blockOversized files (deployment-docker)
troubleshooting.md(919 lines) →troubleshooting-setup.md+troubleshooting-infrastructure.md+troubleshooting-recovery.mdenvironment-variables.md(754 lines) → core vars +environment-variables-otel.mdQuality (SDK skills)
README
.github/logo-light.svg,.github/logo-dark.svg)Test plan
npx skills add featbit/featbit-skillsstill installs correctly🤖 Generated with Claude Code