fix: a failed write no longer follows you to the next row

ts/lock-codes-card.integration.test.ts

@@ -693,6 +693,21 @@ describe('LockCodesCard integration', () => {
             expect((card as any)._editingSlot).toBe(1);
             expect((card as any)._editValue).toBe('1234');
         });
+
+        it('a failed write does not follow you to the next row', async () => {
+            // The banner renders above the whole slot list, so a failure left
+            // over from slot 1 reads as if it belongs to whatever row you
+            // moved on to.
+            (card as any)._editingSlot = 1;
+            (card as any)._editValue = '1234';
+            sendMessagePromiseMock.mockRejectedValueOnce(new Error('Code already in use'));
+            await (card as any)._saveCode(1);
+            expect((card as any)._writeError).toBe('Code already in use');
+
+            (card as any)._startEditing({ stopPropagation: () => {} }, { code: 'empty', slot: 2 });
+
+            expect((card as any)._writeError).toBeUndefined();
+        });
         /* eslint-enable @typescript-eslint/no-explicit-any */
     });
 

ts/lock-codes-card.ts

@@ -57,8 +57,6 @@ class LockCodesCard extends LockCodesCardBase {
     _data?: LockCoordinatorData;
     _error?: string;
 
-    /** Whether codes were revealed before editing started */
-
     set hass(hass: HomeAssistant) {
         this._hass = hass;
         void this._subscribe();
@@ -194,6 +192,10 @@ class LockCodesCard extends LockCodesCardBase {
     // Editing methods for unmanaged slots
     private _startEditing(e: Event, slot: LockCoordinatorSlotData): void {
         e.stopPropagation();
+        // Belongs to the row it happened on. Moving to another row without
+        // cancelling first would otherwise leave the previous row's failure
+        // on screen above the one now being typed into.
+        this._writeError = undefined;
         // Deliberately does NOT reveal. `_revealed` is card-global and
         // `_formatCode` reads it for every row, so revealing to edit one
         // unmanaged slot put every managed user's PIN on screen in cleartext