Fix/vistab close

BUG: Wrong vis being closed when multiple vis are open at the same time

Changes:

1. removeVisualization:

• Changed || to ?? for index handling
• Added validation check before operations

2. reorderVisState:

• Added bounds validation check for indices
• Changed the active index updating logic to handle three cases:
  1. Moving the active tab
  2. Moving a tab before current active tab
  3. Moving a tab after current active tab

Some more validation checks added to other reducers