fix(client): prevent infinite recursion in add access key dialog

[sentry]

This PR addresses a RangeError: Maximum call stack size exceeded occurring in the AddAccessKeyDialog component.

Root Cause:
The attributeChangedCallback in add-access-key-dialog was unconditionally calling runAccessKeyChecks(newValue) for every attribute change. This created an infinite recursion cycle due to:

1. The accessKey property having reflect: true, causing LitElement to reflect property changes back to attributes.
2. MWC's md-filled-text-field.setValidity (triggered by error state changes) synchronously calling setAttribute, which in turn re-triggered attributeChangedCallback.

Solution:
Introduced a guard in attributeChangedCallback to ensure runAccessKeyChecks is only invoked when the accesskey attribute specifically changes. This breaks the recursive cycle. The attribute name accesskey (lowercase) is used because Lit 3.x automatically lowercases property names when reflecting to attributes.

Tests:
Added a new spec file (index.spec.ts) to cover the behavior of the attributeChangedCallback guard, ensuring that:

• runAccessKeyChecks is called when accesskey changes.
• runAccessKeyChecks is not called for unrelated attribute changes.
• No stack overflow occurs when accesskey is changed.

Fixes OUTLINE-CLIENTS-NM61S

_{@sentry <feedback>: Autofix iterates on these changes
@sentry stop iterating: Autofix stops iterating on this run}

_{This PR was automatically generated by Sentry at no cost. You can adjust this setting at any time.}

[greptile-apps]

Confidence Score: 5/5

[Medium risk] Fixes infinite recursion in access key dialog.

The PR appears safe to merge; no new actionable issues remain.

What we checked:

• Unrelated attributes rerun key checks: The callback runs key checks only when the changed attribute is accesskey. Changes to open skip that call.

Summary

Limits AddAccessKeyDialog validation to changes to accesskey, matching the attribute for its reflected accessKey property.

• Unrelated attribute changes no longer restart access-key checks.

_{Reviews (2) · Last reviewed commit: "fix(client/web): prevent infinite recurs..." · Reviewed by Greptile}