Feedback & Idea: Improving Edge-Case Safety and Completing Hotkey-Map Placeholders

[AkshatRaj00]

Hi everyone,

I’ve been diving into the ProjectVisBug codebase recently and I really love how modular and clean the architecture is. While exploring the repository, I noticed a couple of areas where we could either enhance code reliability or help move some long-standing placeholders forward.

I wanted to open a discussion here to see if the maintainers are open to contributions regarding these two points:

1. Adding Defensive Guard Clauses for Dynamic Elements

In app/components/selection/distance.element.js, the position setter destructures and passes line_model downstream without an explicit check:

set position ({ line_model, node_label_id }) {
  this.styleProps = line_model
  this.$shadow.innerHTML = this.render(line_model, node_label_id)
}