Address comments

Author: laghee

injected/src/content-feature.js

@@ -240,7 +240,7 @@ export default class ContentFeature extends ConfigFeature {
      * @param {Obj} object - object whose property we are wrapping (most commonly a prototype, e.g. globalThis.BatteryManager.prototype)
      * @param {Key} propertyName
      * @param {import('./wrapper-utils.js').StrictPropertyDescriptorGeneric<Obj, Key>} descriptor - requires all descriptor options to be defined because we can't validate correctness based on TS types
-    */
+     */
     defineProperty(object, propertyName, descriptor) {
         // make sure to send a debug flag when the property is used
         // NOTE: properties passing data in `value` would not be caught by this
@@ -258,7 +258,36 @@ export default class ContentFeature extends ConfigFeature {
             }
         });
 
-        return defineProperty(object, String(propertyName), /** @type {any} */ (descriptor));
+        // Build complete strict descriptor all at once so TS doesn't complain about missing properties
+        let strictDescriptor;
+
+        if (descriptor.value !== undefined || descriptor.writable !== undefined) {
+            // Data descriptor
+            strictDescriptor = {
+                configurable: descriptor.configurable ?? false,
+                enumerable: descriptor.enumerable ?? false,
+                value: descriptor.value,
+                writable: descriptor.writable ?? false,
+            };
+        } else if (descriptor.get !== undefined || descriptor.set !== undefined) {
+            // Accessor descriptor
+            strictDescriptor = {
+                configurable: descriptor.configurable ?? false,
+                enumerable: descriptor.enumerable ?? false,
+                get: descriptor.get ?? (() => undefined),
+                set: descriptor.set ?? (() => {}),
+            };
+        } else {
+            // Default data descriptor
+            strictDescriptor = {
+                configurable: descriptor.configurable ?? false,
+                enumerable: descriptor.enumerable ?? false,
+                value: undefined,
+                writable: false,
+            };
+        }
+
+        return defineProperty(object, String(propertyName), strictDescriptor);
     }
 
     /**

injected/src/features/web-compat.js

@@ -1,4 +1,3 @@
-// TypeScript is disabled for this file due to intentional DOM polyfills (e.g., Notification) that are incompatible with the DOM lib types.
 import ContentFeature from '../content-feature.js';
 // eslint-disable-next-line no-redeclare
 import { URL } from '../captured-globals.js';