[#6399] Added validation to fileset dialog

[Pranaykarvi]

What changes were proposed in this pull request?

The validation schema for the key field within the propItems array was modified to allow hyphens in file names. The regular expression for the key was updated to:

/^[a-zA-Z_][a-zA-Z0-9_-]*$/

Why are the changes needed?

Fix: issue #6399
The UI was incorrectly rejecting file names containing hyphens when creating a fileset, even though hyphens were allowed in the name specification. The changes ensure that hyphens are properly validated as part of the file name.

Does this PR introduce any user-facing change?

Yes, this PR allows users to use hyphens in file names when creating a files

How was this patch tested?

The changes were tested by creating filesets with hyphens in the names via the UI, ensuring they were accepted correctly.

[mchades]

Thanks for your PR!

Could you plz follow the format like other PR titles?

[Pranaykarvi]

Thanks for your PR!

Could you plz follow the format like other PR titles?

done, please could you check now?

[mchades]

plz make the CI pass. see: https://github.com/apache/gravitino/actions/runs/13173847662/job/36768887540?pr=6400

[justinmclean]

There's some formatting issue with your PR, to make the tests pass you'll need to run prettier with --write to format the code correctly.

[Pranaykarvi]

There's some formatting issue with your PR, to make the tests pass you'll need to run prettier with --write to format the code correctly.

@justinmclean Thanks for the heads-up! I ran Prettier with --write, and the formatting should be correct now. Let me know if anything else needs fixing

[Pranaykarvi]

plz make the CI pass. see: https://github.com/apache/gravitino/actions/runs/13173847662/job/36768887540?pr=6400

@mchades CI has been passed

[jerryshao]

@LauraXia123 , can you please help to review this PR?

[LauraXia123]

@Pranaykarvi Hi，you can update the keyRegex which from @/lib/utils/regex is ok.

[Pranaykarvi]

@Pranaykarvi Hi，you can update the keyRegex which from @/lib/utils/regex is ok.

@LauraXia123 I made the changes in the regex file

[LauraXia123]

I read the keyRegex carefully and made sure it already supports hyphens. The issue #6399 is about the fileset name.
And you need to update the nameRegex and nameRegexDesc replace keyRegex.
Note the addition of three connectors on the back end：/=-

[LauraXia123]

@Pranaykarvi Hi, can you run prettier to fix formatting?

[Pranaykarvi]

@Pranaykarvi Hi, can you run prettier to fix formatting?

@LauraXia123 done please could you check now?

[jerryshao]

@Pranaykarvi @LauraXia123 can you please check build error here?

[LauraXia123]

@Pranaykarvi can you run pnpm lint to check the lint issue?

[Pranaykarvi]

@jerryshao @LauraXia123
please could you check now?

[Pranaykarvi]

@Pranaykarvi @LauraXia123 can you please check build error here?

@jerryshao passed can u check now?

[LauraXia123]

@Pranaykarvi Hi, the nameRegex rules should be consistent with the back end nameRegex = /^\w[\w/=-]{0,63}$/, and the description should be changed as well. Thanks

[tengqm]

lgtm

[jerryshao]

@Pranaykarvi Hi, the nameRegex rules should be consistent with the back end nameRegex = /^\w[\w/=-]{0,63}$/, and the description should be changed as well. Thanks

@Pranaykarvi can you please address the comments mentioned by @LauraXia123 ? If you don't have time, we can help to continue on this PR, thanks.

[Pranaykarvi]

@jerryshao @LauraXia123 i have made the changes now can you check?

[LauraXia123]

LGTM, thanks

[Pranaykarvi]

@jerryshao @mchades I’ve made the requested changes, and @LauraXia123 has approved them. Could you please review the PR and, if everything looks good, proceed with merging? Thanks!