fix(runtime): always call component lifecycle hooks

[johnjenkins]

What is the current behavior?

As per the linked issues, Stencil makes class components less easy to reason and not behave like the dynamic JS classes they are. Adding and extending class behaviour dynamically makes DX much better.

GitHub Issue Number:

#6165

What is the new behavior?

Removed the static flags / checks around lifecycle hook use - they are always called now.

Any argument around whether to always run lifecycle hooks hinges on any potential size savings won by Stencil's current optimisations.

The following is the result from a single, simple component without any lifecycle hooks (remember if any single component uses a lifecycle hook, any difference is diminished).

Before latest stencil:

dist-custom-elements core:
+--------------------------------------------------------------+
| Size | 12.71 KiB |
|--------------------------------------------------------------|
| Gzipped | 5.27 KiB |
|--------------------------------------------------------------|

dist browser core:
+--------------------------------------------------------------+
| Size | 14.97 KiB |
|--------------------------------------------------------------|
| Gzipped | 6.2 KiB |
|--------------------------------------------------------------|

dist esm core (no-minification):
+--------------------------------------------------------------+
| Size | 39.54 KiB |
|--------------------------------------------------------------|
| Gzipped | 9.99 KiB |
|--------------------------------------------------------------|

After (this branch / consistent lifecycle hooks):

dist-custom-elements core:
+--------------------------------------------------------------+
| Size | 12.92 KiB |
|--------------------------------------------------------------|
| Gzipped | 5.34 KiB |
|--------------------------------------------------------------|

dist browser core:
+--------------------------------------------------------------+
| Size | 15.38 KiB |
|--------------------------------------------------------------|
| Gzipped | 6.34 KiB |
|--------------------------------------------------------------|

dist esm core (no minification):
+--------------------------------------------------------------+
| Size | 40.68 KiB |
|--------------------------------------------------------------|
| Gzipped | 10.19 KiB |
|--------------------------------------------------------------|

^ in these, 'worst case' scenarios, stripping away the ability to dynamically use lifecycle hooks saves 0.07-.2 kb

• Fixes feat: always enable all render functions #6165
• Fixes Lifecycle methods cannot all be declared as field holding arrow function #2217
• Fixes Keep component lifecycle after build #2195

Documentation

Does this introduce a breaking change?

• Yes
• No

Testing

New wdio tests

Other information