provide debug utilities for inspector via event communication#20580
provide debug utilities for inspector via event communication#20580patricklx wants to merge 5 commits intoemberjs:mainfrom
Conversation
|
maybe @chancancode ? |
c7e2bab to
592e9cf
Compare
bb68625 to
208881f
Compare
208881f to
9307366
Compare
|
We now need this to proceed with emberjs/rfcs#1015 For the VM side of this, glimmerjs/glimmer-vm#1489, benchmarks are running, so we'll know if this is safe. |
There was a problem hiding this comment.
I think this implementation is good for now, but we'll need to change things later, because I think we want ember-source to push panels into the inspector somehow.
needs some planning there, but it'd be great if ember-source's part of the inspector could be tested within ember-source, and than the inspector slurps those up (automatic updates depending on ember-source version!)
There was a problem hiding this comment.
I think pushing panels is not needed. more important would be to push a ember_debug directly from ember.js instead of having inspectors ember_debug try to support multiple versions.
There was a problem hiding this comment.
well the thinking here is that because ember.js is has more people working on it, it could be updated more easily than the inspector -- and then the inspector also doesn't need to maintain compatibility with everything going forward -- as it would be provided by ember-source
we should probs move to talking about this on the discord though -- lots of details to work through
|
Are these things not already accessible via their existing private import paths? |
|
Not with ember static source enabled. Then it's not accessible. |
|
Biggest issue is the boot load event. And the whole ember source needs to be loaded before inspector can access it. (With AMD) There have been multiple issues around that in the past. |
|
E.g. emberjs/ember-inspector#2051 As an example |
Development AssetsDiff --- main/out.txt 2025-04-10 17:47:56.000000000 +0000
+++ pr/./pr-14704709762/out.txt 2025-04-28 09:37:40.000000000 +0000
@@ -1,6 +1,6 @@
- 2.2M └─┬ .
-1015K ├─┬ @ember
- 205K │ ├─┬ -internals
+ 2.3M └─┬ .
+1020K ├─┬ @ember
+ 206K │ ├─┬ -internals
69K │ │ ├─┬ views
64K │ │ │ └─┬ lib
23K │ │ │ ├── mixins
@@ -27,7 +27,7 @@
4.1K │ │ └── environment
183K │ ├─┬ routing
28K │ │ └── lib
- 149K │ ├─┬ object
+ 150K │ ├─┬ object
66K │ │ └─┬ lib
62K │ │ └── computed
114K │ ├─┬ template-compiler
@@ -37,7 +37,7 @@
4.1K │ │ └── -internal
66K │ ├─┬ application
5.6K │ │ └── lib
- 52K │ ├─┬ debug
+ 57K │ ├─┬ debug
21K │ │ └── lib
38K │ ├─┬ array
4.9K │ │ └── lib
@@ -63,23 +63,23 @@
4.2K │ ├── deprecated-features
4.1K │ ├── template-factory
4.1K │ └── version
- 709K ├── shared-chunks
- 384K ├─┬ @glimmer
- 166K │ ├── runtime
+ 898K ├── shared-chunks
+ 203K ├─┬ @glimmer
60K │ ├── opcode-compiler
30K │ ├── manager
- 22K │ ├── validator
14K │ ├── program
12K │ ├── reference
11K │ ├── destroyable
- 10K │ ├─┬ tracking
- 4.4K │ │ └── primitives
10K │ ├── util
+ 8.9K │ ├─┬ tracking
+ 4.5K │ │ └── primitives
8.1K │ ├── node
7.3K │ ├── global-context
6.4K │ ├── wire-format
+ 5.5K │ ├── runtime
5.0K │ ├── vm
4.9K │ ├── encoder
+ 4.7K │ ├── validator
4.6K │ ├── owner
4.1K │ └── env
60K ├─┬ ember-testingDetails
|
this would allow inspector to communicate with emberjs without relying on any globals like the define, require functions and without the barrel file.
it would be required for inspector to provide more functionality when using vite or also when building with embroider webpack and staticEmberSources=true
would also like to glimmerjs/glimmer-vm#1489
as part of this
emberjs/ember-inspector#2554