Skip to content

improvement(flint): set json language on the pages code input#5643

Merged
waleedlatif1 merged 1 commit into
stagingfrom
flint-json-language
Jul 13, 2026
Merged

improvement(flint): set json language on the pages code input#5643
waleedlatif1 merged 1 commit into
stagingfrom
flint-json-language

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

Type of Change

  • Improvement

Testing

Typecheck, biome, and block registry tests (83) pass

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Jul 13, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Skipped Skipped Jul 13, 2026 8:20pm

Request Review

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cursor review

@cursor

cursor Bot commented Jul 13, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Single declarative block metadata change with no runtime or API behavior impact.

Overview
Adds language: 'json' on the Flint block’s Pages (JSON) code sub-block for Generate Pages, so the workflow editor treats that field as JSON (syntax highlighting and inline validation) like other integration JSON code inputs (e.g. Apify, A2A).

No changes to Flint API params, tool wiring, or block inputs/outputs—only block UI configuration.

Reviewed by Cursor Bugbot for commit 1c69c8b. Configure here.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 1c69c8b. Configure here.

@greptile-apps

greptile-apps Bot commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR marks the Flint Pages code input as JSON. The main change is:

  • Adds JSON language metadata to the Flint Pages (JSON) code field.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.

Important Files Changed

Filename Overview
apps/sim/blocks/blocks/flint.ts Adds JSON language metadata to the Flint Pages code subblock.

Reviews (1): Last reviewed commit: "improvement(flint): set json language on..." | Re-trigger Greptile

@greptile-apps

greptile-apps Bot commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR updates the Flint Pages code input metadata.

  • Adds language: 'json' to the Pages (JSON) code field.
  • Aligns the Flint block config with other JSON code inputs.

Confidence Score: 5/5

This looks safe to merge after a small cleanup to the editor behavior.

  • The changed config is type-valid.
  • Runtime parsing and workflow execution are unchanged.
  • The visible gap is limited to JSON-specific highlighting in the editor.

apps/sim/blocks/blocks/flint.ts

Important Files Changed

Filename Overview
apps/sim/blocks/blocks/flint.ts Adds JSON language metadata to the Flint Pages code input; the config is valid, but JSON-specific highlighting is not applied by the current editor path.

Reviews (2): Last reviewed commit: "improvement(flint): set json language on..." | Re-trigger Greptile

Comment thread apps/sim/blocks/blocks/flint.ts
@waleedlatif1 waleedlatif1 merged commit f80162d into staging Jul 13, 2026
18 checks passed
@waleedlatif1 waleedlatif1 deleted the flint-json-language branch July 13, 2026 21:48
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