Merge branch 'reuse-room-resource' into return-of-the-skill-commands

Author: lukemelia

packages/host/app/components/ai-assistant/panel.gts

@@ -188,11 +188,14 @@ export default class AiAssistantPanel extends Component<Signature> {
           </div>
         {{else if this.isReady}}
           {{! below if statement is covered in 'isReady' check above but added due to glint not realizing it }}
-          {{#if this.matrixService.currentRoomId}}
-            <Room
-              @roomId={{this.matrixService.currentRoomId}}
-              @monacoSDK={{this.monacoSDK}}
-            />
+          {{#if this.roomResource}}
+            {{#if this.matrixService.currentRoomId}}
+              <Room
+                @roomId={{this.matrixService.currentRoomId}}
+                @roomResource={{this.roomResource}}
+                @monacoSDK={{this.monacoSDK}}
+              />
+            {{/if}}
           {{/if}}
         {{else}}
           <LoadingIndicator

packages/host/app/components/matrix/room.gts

@@ -39,6 +39,7 @@ import UpdateSkillActivationCommand from '@cardstack/host/commands/update-skill-
 import { Message } from '@cardstack/host/lib/matrix-classes/message';
 import type { StackItem } from '@cardstack/host/lib/stack-item';
 import { getAutoAttachment } from '@cardstack/host/resources/auto-attached-card';
+import { RoomResource } from '@cardstack/host/resources/room';
 
 import type CardService from '@cardstack/host/services/card-service';
 import type CommandService from '@cardstack/host/services/command-service';
@@ -66,6 +67,7 @@ import type { Skill } from '../ai-assistant/skill-menu';
 interface Signature {
   Args: {
     roomId: string;
+    roomResource: RoomResource;
     monacoSDK: MonacoSDK;
   };
 }
@@ -77,7 +79,7 @@ export default class Room extends Component<Signature> {
         class='room'
         data-room-settled={{this.doWhenRoomChanges.isIdle}}
         data-test-room-settled={{this.doWhenRoomChanges.isIdle}}
-        data-test-room-name={{this.roomResource.name}}
+        data-test-room-name={{@roomResource.name}}
         data-test-room={{@roomId}}
       >
         <AiAssistantConversation
@@ -147,15 +149,12 @@ export default class Room extends Component<Signature> {
                 @autoAttachedFile={{this.autoAttachedFile}}
                 @filesToAttach={{this.filesToAttach}}
               />
-
-              {{#if this.roomResource}}
-                <LLMSelect
-                  @selected={{this.roomResource.activeLLM}}
-                  @onChange={{this.roomResource.activateLLM}}
-                  @options={{this.supportedLLMs}}
-                  @disabled={{this.roomResource.isActivatingLLM}}
-                />
-              {{/if}}
+              <LLMSelect
+                @selected={{@roomResource.activeLLM}}
+                @onChange={{@roomResource.activateLLM}}
+                @options={{this.supportedLLMs}}
+                @disabled={{@roomResource.isActivatingLLM}}
+              />
             </div>
           </div>
         </footer>
@@ -217,10 +216,6 @@ export default class Room extends Component<Signature> {
   @service private declare operatorModeStateService: OperatorModeStateService;
   @service private declare loaderService: LoaderService;
 
-  private get roomResource() {
-    return this.matrixService.roomResources.get(this.args.roomId);
-  }
-
   private autoAttachmentResource = getAutoAttachment(
     this,
     () => this.topMostStackItems,
@@ -519,25 +514,25 @@ export default class Room extends Component<Signature> {
   };
 
   private isDisplayingCode = (message: Message) => {
-    return !!this.roomResource?.isDisplayingCode(message);
+    return this.args.roomResource.isDisplayingCode(message);
   };
 
   private toggleViewCode = (message: Message) => {
-    this.roomResource?.toggleViewCode(message);
+    this.args.roomResource.toggleViewCode(message);
   };
 
   private doMatrixEventFlush = restartableTask(async () => {
     await this.matrixService.flushMembership;
     await this.matrixService.flushTimeline;
-    await this.roomResource?.loading;
+    await this.args.roomResource.loading;
   });
 
   private get messages() {
-    return this.roomResource?.messages ?? [];
+    return this.args.roomResource.messages;
   }
 
-  private get skills() {
-    return this.roomResource?.skills ?? [];
+  private get skills(): Skill[] {
+    return this.args.roomResource.skills;
   }
 
   private get supportedLLMs(): string[] {
@@ -555,7 +550,7 @@ export default class Room extends Component<Signature> {
   }
 
   private get room() {
-    let room = this.roomResource?.matrixRoom;
+    let room = this.args.roomResource.matrixRoom;
     return room;
   }