move static define() block to end of class body

[bemky]

Summary

Follow-up to #37. Even with the cache-invalidation fix in scanPrototypesFor, the first instance constructed by an upgrade-during-class-init still ends up with empty static metadata — the cache no longer poisons subsequent instances, but that one early instance is broken. This change eliminates the timing window entirely.

customElements.define synchronously upgrades any pre-existing matching elements in the DOM, which runs the constructor. The constructor reads assignableAttributes, events, bindMethods, etc., and several components also touch class-level singletons like static observer = new ResizeObserver(...). Calling define() mid-class-body — as we did everywhere, right after static tagName = ... — let that upgrade fire before the rest of the static field initializers had run.

Move every static { this.define() } block to be the last static in its class so registration always happens after the class is fully initialized. Document the requirement on KompElement.define's JSDoc and update the FormField example.

Verified with a smoke test: parsing <komp-dropzone enabled> into the DOM and then importing the module now correctly resolves all three keys of assignableAttributes (enabled, dragHereOverlay, dragOverOverlay) on the upgraded instance.

Drops a leftover empty if (!window.customElements.get('komp-floater')) {} block at the bottom of floater.js (looks like a leftover from a previous registration refactor).

Test plan

• npm test — 33 passing
• npm run docs builds without warnings
• Smoke test: HTML-first <komp-dropzone> resolves full assignableAttributes after import
• Existing demos render unchanged

🤖 Generated with Claude Code