Skip to content

fix(skills): rewrite custom-code-management with full workflow and missing sections#20

Merged
yan-xie-webflow merged 2 commits intomainfrom
fix/custom-code-management-rewrite
Apr 7, 2026
Merged

fix(skills): rewrite custom-code-management with full workflow and missing sections#20
yan-xie-webflow merged 2 commits intomainfrom
fix/custom-code-management-rewrite

Conversation

@yan-xie-webflow
Copy link
Copy Markdown
Contributor

@yan-xie-webflow yan-xie-webflow commented Apr 6, 2026

Summary

Rewrite of the custom-code-management skill — the most underdeveloped skill in the repo (~50 lines, missing required sections). Now ~90 lines with full coverage.

What was wrong

  • Missing webflow_guide_tool (only MCP skill without it)
  • No ## Examples section (required by CONTRIBUTING.md)
  • No ## Guidelines section (required by CONTRIBUTING.md)
  • No phased workflow (flat numbered steps)
  • Missing page-level script operations (get_page_script, upsert_page_script, delete_all_page_scripts)
  • Original incorrectly stated "Site-level only (no page-specific via MCP)" — the MCP does support page-level scripts
  • Inconsistent header casing (## Important note## Important Note)

All original guidance preserved

Original guidance Status in rewrite
register (store) → apply (attach) concept Covered in Phase 2 (registration vs application status)
list_registered_scripts / list_applied_scripts Important Note + Phase 2
add_inline_site_script with displayName, sourceCode, version, location, canCopy Phase 4 step 6
delete_all_site_scripts — no selective delete Multiple warnings in Planning + Guidelines
Max 10,000 chars per script Important Note + Guidelines
No <script> tags (Webflow adds them) Important Note + Guidelines
displayName + version must be unique Guidelines
Hosted scripts not available via MCP Guidelines
Require "add" to confirm adding Phase 3
Require "delete all" to confirm removal Phase 3
Remind user to publish / suggest safe-publish Phase 5 + Guidelines

What's new

  • webflow_guide_tool as first step
  • 5-phase workflow: Discovery → Analysis → Planning → Execution → Verification
  • 4 examples: view scripts, add GTM, remove all, page-specific tracking
  • Page-level operations: get_page_script, upsert_page_script, delete_all_page_scripts
  • Explicit confirmation gates with specific keywords
  • Header vs footer placement recommendations

🤖 Generated with Claude Code

yan-xie-nk and others added 2 commits April 5, 2026 23:30
…ssing sections

The custom-code-management skill was severely underdeveloped compared
to all other MCP-based skills (~50 lines vs 150-600+). This rewrite:

- Adds webflow_guide_tool as first step (was completely missing)
- Adds phased workflow: Discovery → Analysis → Planning → Execution → Verification
- Adds Examples section with 5 concrete examples (view, add GTM, remove all, chat widget, page-level)
- Adds Guidelines section (script rules, confirmation requirements, scope, safety, errors)
- Adds page-level script operations (get_page_script, upsert_page_script, delete_all_page_scripts)
- Fixes header casing: "Important note" → "Important Note"
- Adds explicit confirmation gates before all mutations

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Same coverage, tighter format: compressed examples into shorthand
flows, merged guidelines into a single flat list.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Copy Markdown
Collaborator

@viratatwebflow viratatwebflow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks much more clean now!

@yan-xie-webflow yan-xie-webflow merged commit 3b403ea into main Apr 7, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants