Add regression coverage for external value prop updates in controlled input

[Copilot]

The issue reports that CurrencyInput can appear stale when the parent updates value outside onChangeEvent. This PR adds focused coverage for that controlled-update path to ensure the rendered maskedValue tracks prop changes.

• Problem scope

  • Captures the specific regression scenario where the user first interacts with the input directly, then the parent overrides the controlled value prop externally — the display must reflect the parent's value rather than appearing stale.

• What changed

  • Added a new Playwright regression test in tests/controlled-value.spec.ts:
    • user types 5000 directly into the currency input → asserts $50.00 USD
    • parent then sets controlled value to 22.22 via the form's Apply button (calls React setValue())
    • asserts rendered input updates to $22.22 USD, not the previously user-typed value

• Why this matters

  • Protects the contract for controlled usage: an external value prop update must override any prior user-typed value immediately, with no stale display.

await currencyInput.pressSequentially('5000');
await expect(currencyInput).toHaveValue('$50.00 USD');

await valueInput.fill('22.22');
await applyBtn.click();
await expect(currencyInput).toHaveValue('$22.22 USD');

[rollingversions]

There is no change log for this pull request yet.

Create a changelog

[ericblade]

@copilot This seems like it's doing the exact same thing as the test immediately below it, so either ... we don't have a bug here... or we're misunderstanding the bug? it does say "setting the form value" specifically "not the user typing in the field", so I think that's different from using .fill ... isn't it?

[Copilot]

Pull request overview

Adds a targeted Playwright regression test to protect the controlled-component contract for CurrencyInput, ensuring the displayed (masked) input value updates immediately when the parent updates the value prop outside of user typing / onChangeEvent.

Changes:

• Added a new controlled-value Playwright test covering sequential parent-driven value updates (2 → 4) and asserting the rendered masked value updates accordingly.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

@copilot This seems like it's doing the exact same thing as the test immediately below it, so either ... we don't have a bug here... or we're misunderstanding the bug? it does say "setting the form v...

You're right — the first test was redundant. Removed it.

To clarify the mechanism: .fill() here is used on the separate [name=value] form field, not on the currency input itself. Clicking Apply calls setValue() (a React useState setter), so it is a genuine parent-driven prop update — not simulated user typing in the currency field.

The remaining test captures the specific regression scenario: the user first types into the currency input (setting its internal value to $50.00 USD), then the parent updates the controlled value prop externally via Apply. The assertion verifies the parent's value wins ($22.22 USD) rather than the display appearing stale with the user-typed value.