fix: OPTIC-1287: Project config validation should ignore xml comments

[bmartel]

PR fulfills these requirements

• Commit message(s) and PR title follows the format [fix|feat|ci|chore|doc]: TICKET-ID: Short description of change made ex. fix: DEV-XXXX: Removed inconsistent code usage causing intermittent errors
• Tests for the changes have been added/updated (for bug fixes/features)
• Docs have been added/updated (for bug fixes/features)
• Best efforts were made to ensure docs/code are concise and coherent (checked for spelling/grammatical errors, commented out code, debug logs etc.)
• Self-reviewed and ran all changes on a local instance (for bug fixes/features)

Change has impacts in these area(s)

(check all that apply)

• Product design
• Backend (Database)
• Backend (API)
• Frontend

Describe the reason for change

When using xml comments which may contain code samples that have a name property like<!--document.querySelector('[name="field"]')--> that also occurs in the label config xml <Taxonomy name="field"> it would fail to validate stating that Config cannot contain non-unique names. This is not correct as the values do not occur within the Label config portion of the xml.

Does this PR introduce a breaking change?

(check only one)

• Yes, and covered entirely by feature flag(s)
• Yes, and covered partially by feature flag(s)
• No
• Not sure (briefly explain the situation below)

What level of testing was included in the change?

(check all that apply)

• e2e
• integration
• unit

Which logical domain(s) does this change affect?

Project Config Validation

[netlify]

✅ Deploy Preview for heartex-docs canceled.

Name	Link
🔨 Latest commit	62903f5
🔍 Latest deploy log	https://app.netlify.com/sites/heartex-docs/deploys/672e0da879a3d30008a3116d

[netlify]

✅ Deploy Preview for label-studio-docs-new-theme canceled.

Name	Link
🔨 Latest commit	62903f5
🔍 Latest deploy log	https://app.netlify.com/sites/label-studio-docs-new-theme/deploys/672e0da8f480570008f48f64

[codecov]

Codecov Report

Attention: Patch coverage is 73.91304% with 6 lines in your changes missing coverage. Please review.

Project coverage is 76.66%. Comparing base (de77967) to head (62903f5).
Report is 8 commits behind head on develop.

Files with missing lines	Patch %	Lines
label_studio/core/label_config.py	73.91%	6 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #6613      +/-   ##
===========================================
+ Coverage    76.63%   76.66%   +0.02%     
===========================================
  Files          169      169              
  Lines        13838    13844       +6     
===========================================
+ Hits         10605    10613       +8     
+ Misses        3233     3231       -2     

Flag	Coverage Δ
pytests	76.66% <73.91%> (+0.02%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.