signal.Set "has" observes too much

[seb-odoo]

• Set created with data = signal.Set(new Set());
• Using data().has(...) on a template.
• Adding another element in the set data().add(...) (not the one observed)
• The templare re-renders anyway.

import { Component, mount, xml, signal, onPatched } from "@odoo/owl";

class Root extends Component {
    static template = xml`<div>Hello Owl! <t t-out="this.data().has(1)"/></div>`;
    data = signal.Set(new Set());

    setup() {
        onPatched(() => console.log("patched"));
        setTimeout(() => this.data().add(2), 100);
    }
}

mount(Root, document.body, { templates: TEMPLATES, dev: true });

The bug is not present if using data = proxy(new Set()); instead (and removing () from callers)

import { Component, mount, xml, signal, onPatched, proxy } from "@odoo/owl";

class Root extends Component {
    static template = xml`<div>Hello Owl! <t t-out="this.data.has(1)"/></div>`;
    data = proxy(new Set());

    setup() {
        onPatched(() => console.log("patched"));
        setTimeout(() => this.data.add(2), 100);
    }
}

mount(Root, document.body, { templates: TEMPLATES, dev: true });

signal.Set is supposed to be more efficient as it is shallow (?), but in practice proxy performs better.

Is it a bug or do I miss something?

[seb-odoo]

Note 1: other collections are probably impacted too.

Note 2: it would be nice if the constructor could create the collection when no param is provided.