Skip to content

[comp] Production Deploy#2396

Merged
tofikwest merged 8 commits intoreleasefrom
main
Mar 30, 2026
Merged

[comp] Production Deploy#2396
tofikwest merged 8 commits intoreleasefrom
main

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

This is an automated pull request to release the candidate branch into production, which will trigger a deployment.
It was created by the [Production PR] action.

github-actions bot and others added 3 commits March 30, 2026 19:29
Add `bodyEncoding: "json" | "form"` field to the fetch step schema.
Default is "json" (current behavior, nothing breaks).

When "form":
- Serializes body as URL-encoded key=value pairs (URLSearchParams)
- Sets Content-Type to application/x-www-form-urlencoded
- Template interpolation ({{variables}}) still resolves before encoding

Also updated httpPost/httpPut/httpPatch in check-context to:
- Respect Content-Type from passed headers (don't override with JSON)
- Send string bodies as-is (don't double-stringify form-encoded data)

This unblocks OAuth2 client_credentials flows (Sophos, ADP,
ServiceNow, etc.) and any legacy API requiring form-encoded POST.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
feat(integration-platform): add bodyEncoding option to fetch step
@cursor
Copy link
Copy Markdown

cursor bot commented Mar 30, 2026

PR Summary

Medium Risk
Changes how editor content is normalized and how integration DSL HTTP bodies/headers are encoded, which can alter rendering and request behavior for existing saved content and integrations. Risk is moderate due to broad impact across editors and runtime HTTP calls, but changes are additive with test coverage for validation.

Overview
Editor content handling is consolidated and made more robust. Policy editors now rely on the shared validateAndFixTipTapContent without ad-hoc sanitization, and the UI editor explicitly syncs readOnly via editor.setEditable to avoid stale editable state.

TipTap validation is expanded to handle real-world malformed content. The validator now parses stringified JSON nodes, fixes malformed lists by wrapping/merging orphaned listItems, and strips problematic textStyle marks; extensive new tests cover these scenarios.

Integration DSL fetch steps gain form body support. fetch steps accept optional bodyEncoding (json/form), interpreter encodes form bodies and sets Content-Type, and the runtime HTTP helpers now allow string bodies and only default Content-Type: application/json when not provided (also ensuring refreshed tokens are picked up on retries).

Written by Cursor Bugbot for commit e4fb01a. This will update automatically on new commits. Configure here.

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 30, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
app (staging) Ready Ready Preview, Comment Mar 30, 2026 9:27pm
comp-framework-editor Ready Ready Preview, Comment Mar 30, 2026 9:27pm
portal (staging) Ready Ready Preview, Comment Mar 30, 2026 9:27pm

Request Review

[dev] [Marfuen] mariano/eng-197-when-going-in-a-policy-getting-a-node-with-invalid-content
tofikwest and others added 2 commits March 30, 2026 16:21
…r token refresh

buildHeaders was called once outside the request lambda, capturing
the current auth token. On 401 retry after token refresh,
executeRequest re-invokes the lambda but the stale token was reused.

Moved buildHeaders inside the lambda for httpPost, httpPut, and
httpPatch so each retry picks up the refreshed currentAccessToken.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
fix(integration-platform): move buildHeaders inside request lambda for token refresh
Updated PolicyEditorWrapper to include isPending state from usePermissions, allowing the editor to remain editable while permissions are loading. This prevents premature locking of the editor when checking for update permissions.
Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

…er (#2400)

Removed isPending state from usePermissions in PolicyEditorWrapper to streamline the logic for determining the editor's read-only state. This change ensures that the editor's lock state is based solely on the user's update permissions, enhancing clarity and maintainability.

Co-authored-by: Mariano Fuentes <marfuen98@gmail.com>
@claudfuen
Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 3.13.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants