René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Attrs initialization without `@attr` decorator","articleBody":"Hi there 😄 \r\n\r\nAfter upgrading from v1.1.4 to v1.3.2, I noticed that my attributes were no longer shadowed by `initializeAttrs()`. I started digging and stumbled upon this piece of logic introduced in #191:\r\n\r\nhttps://github.com/github/catalyst/blob/a8fb3ba59a4ac258d0fe64a50ecd3b6997f5f016/src/attr.ts#L39\r\n\r\nIt prevents `initializeAttrs()` from doing anything the second time around, thus skipping the manual initialization of attributes. Also, it seems like this change was one of the main things addressed by the above-mentioned PR.\r\n\r\nSo, what would be the correct way now to initialize the attributes without the `@attr` decorator?\r\n\r\nThanks!\r\n\r\n---\r\n\r\n**Example**\r\n\r\n```html\r\n\u003cscript\u003e\r\n  class InfoMessage extends HTMLElement {\r\n    open = true\r\n\r\n    connectedCallback() {\r\n      initializeAttrs(this, ['open'])\r\n    }\r\n  }\r\n\r\n  controller(InfoMessage)\r\n\u003c/script\u003e\r\n\r\n\u003cinfo-message data-open=\"false\"\u003e\u003c/info-message\u003e\r\n```\r\n\r\n1. `controller(InfoMessage)` wraps original `connectedCallback()` and calls:\r\n\r\nhttps://github.com/github/catalyst/blob/a8fb3ba59a4ac258d0fe64a50ecd3b6997f5f016/src/core.ts#L14\r\n\r\n2. `InfoMessage` is marked as `initialized` (`attrs.ts`):\r\n\r\nhttps://github.com/github/catalyst/blob/a8fb3ba59a4ac258d0fe64a50ecd3b6997f5f016/src/attr.ts#L40\r\n\r\n3. The original `connectedCallback()` is executed, but `initializeAttrs()` would hit that early-return condition.\r\n4. `this.open` is stuck in its default state (`true`) 😔","author":{"url":"https://github.com/kristoforsalmin","@type":"Person","name":"kristoforsalmin"},"datePublished":"2022-04-29T11:50:12.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":4},"url":"https://github.com/207/catalyst/issues/207"}