:fix: Fix auto update of max regex length when the current value is m…

[wryonik]

…ore than previous one

[rutefig]

Hi! Thanks for working on this. I tested the changes and wanted to share some feedback.

What I'm seeing:

When I test the updated code, the maxLength field doesn't seem to be auto-updating at all anymore. It appears to only accept manual user input and doesn't calculate based on the regex output when patterns change.

Specifically:

• The field keeps the default length value
• Changes to the regex pattern don't trigger any automatic maxLength updates
• Error handling doesn't seem to be displaying as expected

I recorded a quick demo showing the behavior if that would help debug this. It looks like the auto-update functionality might not be triggering in the first place.

Would it be worth investigating why the auto-update isn't working before adding the conditional logic? I'm wondering if there might be something else going on that's preventing the feature from working as intended. Happy to help test further or provide more details!

Screen.Recording.2025-10-02.at.12.46.29.mov

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch shubham/reg-564-fe-ux-fix-max-length-auto-calculated-even-after-the-user

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[rutefig]

@wryonik what is the status of this PR?