Fix widget routing infinite loop and room switching

Three critical fixes for widget routing:

1. Infinite loop prevention (MainWidget.ts)
   - Add guard to switchContentView() tracking currentViewType/entityId
   - Skip re-render when already showing the same content
   - Prevents cascading events from triggering infinite widget recreation

2. Remove duplicate event emission (RoomListWidget.ts)
   - Remove direct content:opened emit from selectRoom()
   - The collaboration/content/open command already emits with proper contentItemId
   - Reduces event noise and potential race conditions

3. Critical pageState fix (MainWidget.ts)
   - ROOM_SELECTED handler now calls pageState.setContent() BEFORE switchContentView()
   - ChatWidget reads from pageState as priority 1, was loading stale room
   - Now room switching correctly updates both tab AND chat content

Result: Clicking rooms in sidebar now properly switches tabs and content
without infinite loops or stale data.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: Joel

src/debug/jtag/generated-command-schemas.json

@@ -1,5 +1,5 @@
 {
-  "generated": "2026-01-06T21:38:05.260Z",
+  "generated": "2026-01-06T22:05:05.706Z",
   "version": "1.0.0",
   "commands": [
     {

src/debug/jtag/package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@continuum/jtag",
-  "version": "1.0.6888",
+  "version": "1.0.6890",
   "description": "Global CLI debugging system for any Node.js project. Install once globally, use anywhere: npm install -g @continuum/jtag",
   "config": {
     "active_example": "widget-ui",

src/debug/jtag/package.json

@@ -3,5 +3,5 @@
  * DO NOT EDIT MANUALLY
  */
 
-export const VERSION = '1.0.6888';
+export const VERSION = '1.0.6890';
 export const PACKAGE_NAME = '@continuum/jtag';

src/debug/jtag/shared/version.ts

@@ -208,21 +208,16 @@ export class RoomListWidget extends EntityScrollerWidget<RoomEntity> {
 
     // Emit room selection IMMEDIATELY so ChatWidget switches fast
     // Include uniqueId for human-readable URL building
+    // NOTE: MainWidget handles view switching via ROOM_SELECTED
+    // NOTE: Tab creation happens via collaboration/content/open command below
+    // DO NOT emit content:opened here - the command will emit it with proper contentItemId
     Events.emit(UI_EVENTS.ROOM_SELECTED, {
       roomId,
       roomName,
       uniqueId: roomEntity.uniqueId || roomEntity.name || roomId  // Prefer uniqueId for URLs
     });
 
-    // Emit content:opened for MainWidget tab update (optimistic)
-    // Use uniqueId for human-readable URLs
-    Events.emit('content:opened', {
-      contentType: 'chat',
-      entityId: roomEntity.uniqueId || roomId,  // Use uniqueId for content state
-      title: roomName
-    });
-
-    // Persist to server in BACKGROUND (don't block UI)
+    // Persist to server in BACKGROUND - command emits content:opened with proper data
     const userId = this.userState?.userId;
     if (userId) {
       Commands.execute<ContentOpenParams, ContentOpenResult>('collaboration/content/open', {

src/debug/jtag/widgets/chat/room-list/RoomListWidget.ts

@@ -31,6 +31,10 @@ export class MainWidget extends BaseWidget {
   private currentContent: ContentInfo | null = null;
   private contentStateAdapter: PositronContentStateAdapter;
 
+  // Guard against infinite re-render loops
+  private currentViewType: string | null = null;
+  private currentViewEntityId: string | undefined = undefined;
+
   constructor() {
     super({
       widgetName: 'MainWidget',
@@ -327,9 +331,19 @@ export class MainWidget extends BaseWidget {
    * Emits RIGHT_PANEL_CONFIGURE to update right panel based on content type's layout
    */
   private switchContentView(contentType: string, entityId?: string): void {
+    // GUARD: Prevent infinite re-render loops by checking if already showing this content
+    if (this.currentViewType === contentType && this.currentViewEntityId === entityId) {
+      console.log(`🔄 MainPanel: Already showing ${contentType}/${entityId || 'default'}, skipping re-render`);
+      return;
+    }
+
     const contentView = this.shadowRoot?.querySelector('.content-view');
     if (!contentView) return;
 
+    // Track what we're rendering to prevent loops
+    this.currentViewType = contentType;
+    this.currentViewEntityId = entityId;
+
     const widgetTag = getWidgetForType(contentType);
 
     // Create widget element with entity context
@@ -639,7 +653,7 @@ export class MainWidget extends BaseWidget {
 
     // IMPORTANT: Also listen for ROOM_SELECTED as reliable backup
     // RoomListWidget emits this and it definitely works (sidebar highlights change)
-    Events.subscribe(UI_EVENTS.ROOM_SELECTED, (data: { roomId: string; roomName: string; uniqueId?: string }) => {
+    Events.subscribe(UI_EVENTS.ROOM_SELECTED, async (data: { roomId: string; roomName: string; uniqueId?: string }) => {
       console.log('📋 MainPanel: Received ROOM_SELECTED event:', data.roomName);
 
       // Only switch to chat if we're currently viewing chat content
@@ -656,6 +670,11 @@ export class MainWidget extends BaseWidget {
       const newPath = buildContentPath('chat', urlIdentifier);
       this.updateUrl(newPath);
 
+      // CRITICAL: Set pageState BEFORE creating widget - ChatWidget reads from pageState
+      // Resolve entity for proper display name
+      const resolved = await RoutingService.resolve('chat', urlIdentifier);
+      pageState.setContent('chat', urlIdentifier, resolved || { id: data.roomId, displayName: data.roomName, uniqueId: urlIdentifier });
+
       // Switch to the selected chat room (use uniqueId for content view)
       this.switchContentView('chat', urlIdentifier);