feat(api): standardize frontmatter, update validation, register 24 api skills#5
Open
gaurav0107 wants to merge 2 commits intoLambdaTest:mainfrom
Open
feat(api): standardize frontmatter, update validation, register 24 api skills#5gaurav0107 wants to merge 2 commits intoLambdaTest:mainfrom
gaurav0107 wants to merge 2 commits intoLambdaTest:mainfrom
Conversation
Contributor
|
@gaurav0107 I see similar changes here as in #4 about the smallcasing and normalizing of names. Please verify. @SparshKesari , do we need the languages and other things in SKILL.md as provided by him in commit 2b87157? |
…-case Addresses the review comment on PR LambdaTest#4 about rebasing onto api-skill/ after upstream's api/ -> api-skill/ rename (PR LambdaTest#6). - Renames all 18 api-skill/ subdirectories from mixed-case with spaces to lowercase hyphen-case - Renames references/ (plural) to reference/ (singular) for the 2 api skills that had playbook reference directories, matching repo convention - Updates api-skill/README.md directory tree to reflect new structure and fixes relative links to CONTRIBUTING.md and LICENSE - Fixes typo 'dcoumentation' -> 'documentation' in postman-to-newman Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
… in index Rebased onto upstream's api/ -> api-skill/ rename (PR LambdaTest#6), applied on top of PR LambdaTest#4's subdirectory normalization. - Add languages, category (api-testing), license (MIT), and metadata to all 24 api-skill/ SKILL.md files to match the test-skill frontmatter standard already enforced by scripts/validate_skills.py for 46 non-api skills - Update validate_skills.py to scan api-skill/ (was api/) and add api-design to VALID_CATEGORIES - Add 'api-skill' to SKIP_DIRS so nested skills aren't double-counted - Register all 24 api-skill entries in skills_index.json with full descriptions and trigger keywords - Sort all 70 skills alphabetically, update total_skills to 70 Validation: Skills found: 70 Errors: 0 Warnings: 47 (expected — api-skills are single-file, no playbooks) Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
gaurav0107
force-pushed
the
feat/api-skill-standardization
branch
from
2b87157 to
f86b2ff
Compare
Author
|
@SparshKesari — on whether Also rebased #4 and #5 onto main after the |
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
2 participants
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
languages,category(api-testing),license(MIT), andmetadatato all 24 api SKILL.md files to match the test skill frontmatter standardvalidate_skills.pyto scanapi/subdirectories and addsapi-testingandapi-designto valid categoriesskills_index.jsonwith full descriptions including trigger keywordstotal_skillsfrom 46 to 70Changes by area
scripts/validate_skills.pyos.walk(api/)scanning, new categories,apiin SKIP_DIRSskills_index.jsonValidation
Test plan
python3 scripts/validate_skills.py— expect 70 skills, 0 errorspython3 -c "import json; d=json.load(open('skills_index.json')); print(len(d['skills']))"— expect 70name,description,languages,category,license,metadata)🤖 Generated with Claude Code