Fix bugs, harden validator, add upload .zip builder#3
Open
mustafa-coursera wants to merge 1 commit into
Open
Conversation
On top of the flat sibling layout established in #4: - Fill in SEARCH-DELIVER.md (was a 7-line stub) with full Step 2 procedure: per-modality routing table, four branches with tool-call shapes, roleplay scenario-prep flow, result-shaping rules, and failure modes. - Fix get_course_materials → get_course_material (singular) in SKILL.md and README.md to match the real Coursera MCP tool name. - Tighten the SKILL.md description from 1013 → 692 chars, leaving headroom under the 1024-char spec ceiling. - Harden scripts/validate-skills.mjs with spec-derived checks: name ≤ 64 chars, description ≤ 1024 chars, warn when SKILL.md > 500 lines. Add a top-of-file comment pointing at the spec URL. - Add scripts/package-skill.mjs to build dist/<skill>.zip for uploaders that take a .zip / .skill bundle (the single-.md upload path can't carry sibling step files). - Document the bundle workflow in README. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
mustafa-coursera
force-pushed
the
mfurniturewala/spec-align-references
branch
from
ebd2954 to
dccaeb0
Compare
mustafa-coursera
changed the title
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.
Builds on the flat sibling layout from #4. Brings across the real bug fixes that #4 didn't include.
Summary
SEARCH-DELIVER.md. Was a 7-line stub. Now a full Step 2 procedure: routing table, four per-modality branches with tool-call shapes, roleplay scenario-prep flow, result-shaping rules, failure modes.get_course_materials(plural — doesn't exist) →get_course_material(singular — the real Coursera MCP tool). Affects SKILL.md and README.description. 1013 → 692 chars. Was one edit away from breaking the 1024-char spec ceiling.scripts/validate-skills.mjs. Spec-derived checks: name ≤ 64 chars, description ≤ 1024 chars, warn when SKILL.md exceeds 500 lines (progressive-disclosure recommendation). Spec URL comment at top.scripts/package-skill.mjs. Buildsdist/<skill-name>.zipfor the Coursera upload modal's.zip/.skillpath. The single-.mdupload path can't carry sibling step files; this script bundles everything correctly.Test plan
node scripts/validate-skills.mjs→Skill validation passed.get_course_materials(plural) references remain (grep clean)node scripts/package-skill.mjs learn-with-courseraproducesdist/learn-with-coursera.zipcontainingSKILL.md,DIAGNOSE.md,SEARCH-DELIVER.md,PATH-FINDER.mddist/confirmed ignored by.gitignoreget_course_materialwithout error🤖 Generated with Claude Code