TODOs in the codebase

[luoling8192]

TODO:

• show more info with selected node
  

  devtools/packages/devtools/src/app/components/flowmap/ModuleFlow.vue

  Line 400 in e0838a0

  <!-- TODO: show more info with selected node -->

• should we add filters for node_modules?
  

  devtools/packages/devtools/src/app/pages/session/[session]/graph/index.vue

  Line 192 in 43fc5c5

  <!-- TODO: should we add filters for node_modules? -->

• install @types/splitpanes once updated
  

  devtools/packages/devtools/src/app/splitpanes.d.ts

  Line 1 in 0c5bdaa

  // TODO install @types/splitpanes once updated

devtools/packages/devtools/src/app/components/data/ChunkDetails.vue

Line 27 in 7dea7f5

<!-- TODO -->

[webfansplz]

@luoling8192 Thanks for the help. I've updated the todo (removed the WIP and hold-off todo).

I think it would be better for us to merge it into #9 for management. @antfu WDYT? (The TODO mentioned here needs your confirmation on whether to keep it or provide more information.)

[antfu]

The reason I put TODOs in the code instead of #9 is to avoid having the same thing duplicated in two places, which would be difficult to manage and track. I'd say that if people want to take on a task, they should leave a comment in #9 with a permanent link to the code to reference.

[GeoffreyParrier]

Here more todos to add and one that has been updated (ModuleFlow.vue#400 to ModuleFlowDetails.vue#90)

Also maybe add link for "todos" search at the top ?
https://github.com/search?q=repo%3Avitejs%2Fdevtools%20todo&type=code

Hope it helps 🙌

TODOS

Update the first one

Update from /ModuleFlow.vue to:

devtools/packages/devtools-vite/src/app/components/flowmap/ModuleFlowDetails.vue

Line 90 in 4542bdb

<!-- TODO: show more info with selected node -->

New todos

• get metadata from api endpoint

devtools/packages/devtools-webext/src/window.ts

Line 6 in 4542bdb

// TODO (hold-off): get metadata from api endpoint

---

• redesign how the manager is passed

devtools/packages/devtools-vite/src/node/cli.ts

Line 40 in 4542bdb

// TODO: redesign how the manager is passed

devtools/packages/devtools-vite/src/node/cli.ts

Line 99 in 4542bdb

// TODO: redesign how the manager is passed

devtools/packages/devtools-vite/src/modules/rpc/runtime/server/metadata.ts

Line 14 in 4542bdb

// TODO: redesign how the manager is passed

---

• detect vite env

devtools/packages/devtools-webext/src/devtools-background.ts

Line 16 in 4542bdb

// TODO (hold-off): detect vite env

---

• graph view

devtools/packages/devtools-vite/src/app/pages/session/[session]/chunks.vue

Line 25 in 4542bdb

<!-- TODO: graph view -->

---

• static server

devtools/packages/devtools-vite/src/modules/rpc/runtime/plugin.ts

Line 39 in 4542bdb

// TODO (hold-off): static server