NullVoxPopuli

Reproduce renderComponent error: 'Cannot read property of undefined: reading syscall'

1

Similar to: https://github.com/emberjs/ember.js/pull/20995 but maybe more representative of real-world usage (renderComponent called from a modifier, or external library, etc). When the owner is passed to renderComponent, and renderComponent is called...

Reproduce renderComponent error: 'attempted to close a tracking frame, but one was not open'

1

This happens when: - renderComponent is called twice, sequentially - the renderedComponent calls another component - and the rendered component calls a helper - the owner is passed to renderComponent...

### 🐞 Describe the Bug [Repro](https://limber.glimdown.com/edit?c=JYWwDg9gTgLgBAYQuCA7Apq%2BAzKy4DkAAgOYA2oI6UA9AMbKQZYEDcAUKJLHAN5wwoAQzoBrdABM4AXzi58xcpWo1BI0cFQk2nFD35oZcvCEJF0IAEYqQECcGzBqO9ugAe3eBPTYhAVzJ4OjIhAGdQuAAJdDIyCAB1aDIpdxhMCQikFGZ4XnY4OCI1MUk4Bj8sOABeOAAGDny4AGJNOigLTHgagAoASmqAPgEAC2BQgDpyyoBqGoBGBoKAHjTwELSBxuWwAYBNCD84YaEAN3QyihKpGGHzyz8YGENeXhuxyYOsaVkYUHQJpY0HbsLZwJb3R6GGAATzA6CqACIIU9UAi%2BLxDAjgsAxGi3hMWqg2h0vtIBghLqJAci0JtlqoLGB1uhNtJ2EA&format=gjs) I expect this to work: ```gjs import Component from '@glimmer/component'; import { tracked } from '@glimmer/tracking'; import { on } from '@ember/modifier'; export default...

I don't want to write an RFC for this right now, as I think it'd be better to prototype the code first during render, we always know the owner. so...

Tests for Issue #20984

1

seems like it may have very subtle timing -- so far... unable to reproduce the issue

I think this can be non-breaking, keeping the same api. Hopefully reducing the amount of code we need to maintain

Prototype what it would be like to remove the classic / pre-octane things from @ember/component

1

Spiking out an idea I had: - https://github.com/NullVoxPopuli/legacy-ember-component - Generally, I want to figure out if all the legacy stuff _can_ be extracted to a library and used independently --...

Move devDeps to devDeps

1

Just exploring this: https://node-modules.dev/chart/sunburst#[email protected]&install=ember-source&[email protected] Relevant: https://discord.com/channels/480462759797063690/485447409296736276/1413906895323922462

Upgrade glimmer-vm ( pre-merge verification for VM PR#1690 )

1

Testing out https://github.com/glimmerjs/glimmer-vm/pull/1690 before merge over there in the VM I don't expect types to pass with the tarballs included here because the dep graph isn't _correct_ when using tarballs...

### 🐞 Describe the Bug missing types ### 🔬 Minimal Reproduction just import it in a ts/gts file ### 😕 Actual Behavior no types ### 🤔 Expected Behavior types ###...