fix: elementInternals.setValidity() clear stale props

package.json

@@ -1,6 +1,6 @@
 {
   "name": "element-internals-polyfill",
-  "version": "3.0.1",
+  "version": "3.0.2",
   "description": "A polyfill for the element internals specification",
   "main": "dist/index.js",
   "module": "dist/index.js",

src/element-internals.ts

@@ -243,14 +243,14 @@ export class ElementInternals implements globalThis.ElementInternals {
     const validity = validityMap.get(this);
     const validityChangesObj: Partial<ValidityState> = {};
     for (const key in validityChanges) {
+      if (key === "valid") {
+        continue;
+      }
       validityChangesObj[key] = validityChanges[key];
     }
-    if (Object.keys(validityChangesObj).length === 0) {
-      setValid(validity);
-    }
-    const check = { ...validity, ...validityChangesObj };
-    delete check.valid;
-    const { valid } = reconcileValidity(validity, check, this.form);
+
+    setValid(validity);
+    const { valid } = reconcileValidity(validity, validityChangesObj, this.form);
 
     if (!valid && !validationMessage) {
       throw new DOMException(

test/ElementInternals.test.js

@@ -299,6 +299,23 @@ describe('The ElementInternals polyfill', () => {
       expect(internals.validity.valid).to.be.true;
     });
 
+    it('will clear omitted validity flags on subsequent setValidity calls', () => {
+      internals.setValidity({
+        valueMissing: true
+      }, 'This field is required');
+
+      expect(internals.validity.valueMissing).to.be.true;
+      expect(internals.validity.valid).to.be.false;
+
+      internals.setValidity({
+        customError: false
+      });
+
+      expect(internals.validity.valueMissing).to.be.false;
+      expect(internals.validity.customError).to.be.false;
+      expect(internals.validity.valid).to.be.true;
+    });
+
     it('will throw if setValidity is called with a flag and no validation message', () => {
       expect(() => {
         internals.setValidity({