Skip to content

fix(prettifyFormValues) - strip html tags#1044

Merged
gabrielseco merged 5 commits into
mainfrom
strip-html-tags-prettifyFormValues
May 22, 2026
Merged

fix(prettifyFormValues) - strip html tags#1044
gabrielseco merged 5 commits into
mainfrom
strip-html-tags-prettifyFormValues

Conversation

@gabrielseco
Copy link
Copy Markdown
Collaborator

@gabrielseco gabrielseco commented May 21, 2026

We strip html tags that can appear from the schema

Before

image

Now

image

Note

Low Risk
Low risk: this only changes display formatting of schema-provided labels/options by stripping markup/entities, with tests added to lock in behavior.

Overview
prettifyFormValues now strips HTML tags and decodes HTML entities from schema-provided field labels and radio/select option labels before returning them, preventing markup from leaking into review/summary UI.

Adds a new stripHtml helper (DOMParser-based) in utils.ts and extends utils.test.ts with coverage for HTML/entity stripping in both labels and option-derived prettyValues.

Reviewed by Cursor Bugbot for commit 3de9b86. Bugbot is set up for automated code reviews on this repo. Configure here.

@gabrielseco gabrielseco self-assigned this May 21, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 21, 2026

📦 Bundle Size Report

Metric Current Previous Change Status
Total (gzip) 211.85 kB 211.94 kB -84 B (-0.0%) 🟢
Total (raw) 574.53 kB 574.41 kB +128 B (+0.0%) 🔴
CSS (gzip) 20.86 kB 20.86 kB 0 B (0%) 🟢
CSS (raw) 108.83 kB 108.83 kB 0 B (0%) 🟢

Size Limits

  • ✅ Total gzipped: 211.85 kB / 250 kB (84.7%)
  • ✅ Total raw: 574.53 kB / 600 kB (95.8%)
  • ✅ CSS gzipped: 20.86 kB / 25 kB (83.4%)

Largest Files (Top 5)

  1. chunk-UGPYPGVV.js - 14 kB (new)
  2. styles.css - 10.43 kB (0 B (0%))
  3. index.css - 10.43 kB (0 B (0%))
  4. index.js - 6.11 kB (+1 B (+0.0%))
  5. chunk-ER46UFIU.js - 5.89 kB (0 B (0%))
View All Files (312 total)
File Size (gzip) Change
chunk-UGPYPGVV.js 14 kB new
styles.css 10.43 kB 0 B (0%)
index.css 10.43 kB 0 B (0%)
index.js 6.11 kB +1 B (+0.0%)
chunk-ER46UFIU.js 5.89 kB 0 B (0%)
chunk-CCI2RP3T.js 5.81 kB new
chunk-EFL7QFFN.js 5.29 kB new
chunk-EDSB7P64.js 4.62 kB new
chunk-SOCG425C.js 4.45 kB new
chunk-C5PA6MLW.js 4.06 kB new

✅ Bundle size check passed

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 21, 2026

📊 Coverage Report

✅ Coverage increased! 🎉

Metric Current Previous Change Status
Lines 90.07% 90.06% +0.01% 🟢
Statements 89.74% 89.73% +0.01% 🟢
Functions 87.74% 87.73% +0.01% 🟢
Branches 80.15% 80.11% +0.04% 🟢

Detailed Breakdown

Lines Coverage
  • Covered: 3702 / 4110
  • Coverage: 90.07%
  • Change: +0.01% (6 lines)
Statements Coverage
  • Covered: 3762 / 4192
  • Coverage: 89.74%
  • Change: +0.01% (6 statements)
Functions Coverage
  • Covered: 1002 / 1142
  • Coverage: 87.74%
  • Change: +0.01% (1 functions)
Branches Coverage
  • Covered: 2310 / 2882
  • Coverage: 80.15%
  • Change: +0.04% (6 branches)

✅ Coverage check passed

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 21, 2026

Deploy preview for remote-flows ready!

Project:remote-flows
Status: ✅  Deploy successful!
Preview URL:https://remote-flows-eqbg7lx5g-remotecom.vercel.app
Latest Commit:3de9b86

Deployed with vercel-action

Comment thread src/lib/utils.ts Outdated
Comment thread src/lib/utils.ts
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, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 4d7796d. Configure here.

Comment thread src/lib/utils.ts Outdated
@gabrielseco gabrielseco requested a review from jordividaller May 22, 2026 06:07
Comment thread src/lib/utils.ts Outdated
@gabrielseco gabrielseco merged commit 87af0bf into main May 22, 2026
10 checks passed
@gabrielseco gabrielseco deleted the strip-html-tags-prettifyFormValues branch May 22, 2026 09:59
@gabrielseco gabrielseco mentioned this pull request May 22, 2026
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.

2 participants