Skip to content

feat(style_tool): allow update_style by style_id#129

Open
dakotayounger wants to merge 1 commit into
webflow:mainfrom
dakotayounger:feat/update-style-style-id
Open

feat(style_tool): allow update_style by style_id#129
dakotayounger wants to merge 1 commit into
webflow:mainfrom
dakotayounger:feat/update-style-style-id

Conversation

@dakotayounger
Copy link
Copy Markdown

Problem

query_styles returns base utility style_id values (isComboClass: false). update_style requires style_name and resolves combo shells when names collide — agents cannot clear text-wrap on six goboon base utility IDs.

Evidence site: 67357e5b3470f7b3ed6b8017 (goboon)

Base id (query) Name-based update_style id isComboClass
da317553-de32-cbab-b54f-45d9e7ce308e d46038f7-… true
8a893afb-5285-9a83-26aa-19bc009eee60 59e46359-… true

Re-audit after six name-based clears: 6/6 base IDs still have text-wrap.

Changes

  • Add optional style_id to update_style zod object in src/tools/deStyle.ts
  • Make style_name optional when style_id is set
  • .refine: require style_id or style_name

Patch verified with git apply on main @ c4fa339.

Verify before merge

  • Designer bridge RPC accepts style_id on update_style and updates the base style, not a combo
  • Regression: existing name-based updates unchanged
  • Mirror optional style_id in published style_tool JSON schema if maintained separately

Related

Boon internal patch kit: tools/webflow-mcp-style-id-patch/ (not in this PR — upstream only).

Made with Cursor

Add optional style_id to update_style; make style_name optional when
style_id is set. Require at least one via zod refine so agents can
target base utilities without combo name resolution.

Evidence: goboon site 67357e5b3470f7b3ed6b8017 — name-based update hits
isComboClass: true shells for base utility IDs from query_styles.
Co-authored-by: Cursor <cursoragent@cursor.com>
@dakotayounger dakotayounger requested a review from a team as a code owner June 4, 2026 15:17
@dakotayounger dakotayounger requested review from bertenator and removed request for a team June 4, 2026 15:17
@dakotayounger
Copy link
Copy Markdown
Author

Autonomous repro (Boon internal runbook — no secrets)

20-minute autonomous watcher (duration=1200000ms) shows marketplace MCP cannot clear goboon base utilities by style_id: bridge falls back to update_style by style_name only (plugin schema has no style_id on update_style until this PR lands).

Evidence on Boon-Company/boon-brain branch agent/cursor/goboon-textwrap-fix:

After merge: wire patched or upstream server in Cursor MCP (webflow-patched / updated marketplace) and re-run clear + publish per runbook.

@dakotayounger
Copy link
Copy Markdown
Author

Production blocker for Boon (goboon.co): we need update_style to honor style_id so six base utility classes can clear text-wrap via MCP before publish. Hosted MCP resolves combo shells only; audit stays at 6 until this merges. Expedited review appreciated — happy to provide repro or Designer bridge logs.

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.

1 participant