fix(form): honor Codex review feedback on Field

[soulevilx]

Addresses the three inline review comments from PR #22 (Codex bot) on components/form/src/components/Field.tsx:

1. Preserve child's own disabled state — Field only forces disabled when the host explicitly sets Field.disabled or the form is busy; an idle form no longer overrides a control's own disabled={true}.
2. Accessible label for composite controls — the label gets an id and Field injects aria-labelledby into the control (RadioGroup fieldset, groups) so it is named; existing aria-label/aria-labelledby on the child is respected.
3. aria-required propagation — Field.required now sets aria-required on the control, not just the visible asterisk.

Tests added for all three behaviors (31 form tests passing).

[qodo-code-review]

ⓘ Qodo reviews are paused because the subscription is no longer active. Ask your workspace admin to reactivate the subscription to resume reviews. Manage billing

[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.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 13fa4d5e-dba0-499e-b3ab-383f836dba9b

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

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

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.}

[codacy-production]

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Coverage 100.00% diff coverage · +0.18% coverage variation

Metric	Results
Coverage variation	✅ +0.18% coverage variation (-1.00%)
Diff coverage	✅ 100.00% diff coverage

View coverage diff in Codacy

Coverage variation details

	Coverable lines	Covered lines	Coverage
Common ancestor commit (a6ac2fc)	1304	959	73.54%
Head commit (d04e094)	1313 (+9)	968 (+9)	73.72% (+0.18%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details

	Coverable lines	Covered lines	Diff coverage
Pull request (#23)	10	10	100.00%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

_{NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.}

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d04e094f06

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

Forward the generated label into RadioGroup

When Field wraps the repository's RadioGroup, this injected prop is discarded because components/ui/src/components/RadioGroup.tsx only destructures id, aria-invalid, and aria-describedby; its role="radiogroup" therefore still has neither aria-labelledby nor an aria-label when the group-level label prop is omitted. The new test passes only because it uses a raw <div> rather than the supported composite control, so the advertised Field label="Plan tier" composition remains unnamed to assistive technology. Forward aria-labelledby through RadioGroup to its accessible group node and cover that actual composition.

AGENTS.md reference: components/form/AGENTS.md:L37-L37

Useful? React with 👍 / 👎.

[chatgpt-codex-connector]

Preserve a child's existing aria-required state

When a child declares its own aria-required={true} and Field.required is omitted or false, cloneElement receives aria-required: undefined here and removes the child's attribute. This recreates the disabled-state regression for required semantics: an idle Field should only force aria-required when its own required prop is true and otherwise leave the child's value untouched.

AGENTS.md reference: components/form/AGENTS.md:L37-L37

Useful? React with 👍 / 👎.