Skip to content

fix(errors) - nested fields are parsed now#930

Merged
gabrielseco merged 2 commits intomainfrom
pybr-3710-fix-snake-case-issue
Apr 16, 2026
Merged

fix(errors) - nested fields are parsed now#930
gabrielseco merged 2 commits intomainfrom
pybr-3710-fix-snake-case-issue

Conversation

@gabrielseco
Copy link
Copy Markdown
Collaborator

@gabrielseco gabrielseco commented Apr 16, 2026

Before

image

Now

image

Note

Medium Risk
Touches shared error-normalization logic used across multiple flows; a mistake could regress how field-level validation errors are labeled or displayed, though the change is localized and covered by new tests.

Overview
Improves field-error normalization to resolve labels from nested meta structures, including slash-delimited field paths (e.g. service_duration/expiration_date), instead of only supporting flat Meta lookups.

Introduces and exports NestedMeta/MetaValues (marking Meta as deprecated), updates consumers (handleStepError and the example ReviewMeta renderer) to use the nested type, and adds unit tests covering nested-path label resolution.

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

@gabrielseco gabrielseco self-assigned this Apr 16, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 16, 2026

📦 Bundle Size Report

Metric Current Previous Change Status
Total (gzip) 208.21 kB 208.05 kB +167 B (+0.1%) 🔴
Total (raw) 559.04 kB 558.87 kB +167 B (+0.0%) 🔴
CSS (gzip) 20.11 kB 20.11 kB 0 B (0%) 🟢
CSS (raw) 103.71 kB 103.71 kB 0 B (0%) 🟢

Size Limits

  • ✅ Total gzipped: 208.21 kB / 250 kB (83.3%)
  • ✅ Total raw: 559.04 kB / 600 kB (93.2%)
  • ✅ CSS gzipped: 20.11 kB / 25 kB (80.4%)

Largest Files (Top 5)

  1. chunk-72RFYT5Y.js - 13.83 kB (new)
  2. index.css - 10.05 kB (0 B (0%))
  3. styles.css - 10.05 kB (0 B (0%))
  4. index.js - 6.05 kB (0 B (0%))
  5. chunk-GV72N5CU.js - 5.93 kB (new)
View All Files (310 total)
File Size (gzip) Change
chunk-72RFYT5Y.js 13.83 kB new
index.css 10.05 kB 0 B (0%)
styles.css 10.05 kB 0 B (0%)
index.js 6.05 kB 0 B (0%)
chunk-GV72N5CU.js 5.93 kB new
chunk-R47Q7R6C.js 5.82 kB 0 B (0%)
chunk-PR5FZBPF.js 4.4 kB new
chunk-KLN2FG5Y.js 4.23 kB new
chunk-XXXLKG43.js 4.06 kB new
chunk-TIJRONPS.js 3.45 kB new

✅ Bundle size check passed

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 16, 2026

📊 Coverage Report

⚠️ Coverage decreased

Metric Current Previous Change Status
Lines 89.91% 89.94% -0.02% 🔴
Statements 89.58% 89.61% -0.02% 🔴
Functions 87.47% 87.47% 0%
Branches 79.81% 79.78% +0.03% 🟢

Detailed Breakdown

Lines Coverage
  • Covered: 3584 / 3986
  • Coverage: 89.91%
  • Change: -0.02% (9 lines)
Statements Coverage
  • Covered: 3647 / 4071
  • Coverage: 89.58%
  • Change: -0.02% (9 statements)
Functions Coverage
  • Covered: 984 / 1125
  • Coverage: 87.47%
  • Change: 0% (0 functions)
Branches Coverage
  • Covered: 2225 / 2788
  • Coverage: 79.81%
  • Change: +0.03% (8 branches)

✅ Coverage check passed

@github-actions
Copy link
Copy Markdown
Contributor

Deploy preview for remote-flows-example-app ready!

✅ Preview
https://remote-flows-example-czxdmho10-remotecom.vercel.app

Built with commit 67a4918.
This pull request is being automatically deployed with vercel-action

@github-actions
Copy link
Copy Markdown
Contributor

Deploy preview for remote-flows ready!

✅ Preview
https://remote-flows-a9r29m16z-remotecom.vercel.app

Built with commit 67a4918.
This pull request is being automatically deployed with vercel-action

@gabrielseco gabrielseco merged commit 6ae3a28 into main Apr 16, 2026
11 checks passed
@gabrielseco gabrielseco deleted the pybr-3710-fix-snake-case-issue branch April 16, 2026 07:20
@gabrielseco gabrielseco mentioned this pull request Apr 17, 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