fix(mirror): resolve PC-to-Discord message detection & channel routing (#148)

[dgomez407]

This PR fixes Issue #148 where messages typed directly in the Antigravity IDE on a PC were not being mirrored back to Discord/Telegram despite /mirror being turned ON.

Root Cause Analysis

1. DOM Selector Scope: UserMessageDetector (src/services/userMessageDetector.ts) relied on a strict .bg-input.p-2 selector. In modern Antigravity / Cascade builds, user message container classes vary (e.g. bg-gray-500/15 p-2, div[class*="bg-input"], [data-message-author-role="user"]), causing DOM detection to return null.
2. Channel Resolution Fallback: In JoinCommandHandler.routeMirroredMessage (src/commands/joinCommandHandler.ts), detected messages were routed strictly via chatSessionRepo.findByDisplayName(projectName, chatTitle). If a session title was unmapped or a user enabled /mirror in a project channel without creating a dedicated session sub-channel via /join, findByDisplayName returned null and the message was silently dropped.

---

Fixes Implemented

1. Robust User Message DOM Detection

• Updated DETECT_USER_MESSAGE_SCRIPT in src/services/userMessageDetector.ts to query all user bubble selector candidates:

  const bubbles = Array.from(scope.querySelectorAll(
      '.bg-input.p-2, div[class*="bg-gray-"][class*="p-2"], div[class*="bg-input"], [data-message-author-role="user"]'
  ));

2. Workspace Channel Fallback Routing

• Updated JoinCommandHandler.routeMirroredMessage in src/commands/joinCommandHandler.ts to fall back to the project's workspace channel (bridge.approvalChannelByWorkspace.get(projectName)) when findByDisplayName yields null:

  const session = this.chatSessionRepo.findByDisplayName(projectName, chatTitle);
  let targetChannelId = session?.channelId;
  
  if (!targetChannelId && bridge) {
      const workspaceChannel = bridge.approvalChannelByWorkspace.get(projectName);
      if (workspaceChannel) {
          targetChannelId = workspaceChannel.id;
      }
  }

---

Verification & Test Results

• Unit Tests:
  • tests/commands/joinCommandHandler.test.ts: Added test verifying workspace channel fallback routing (12/12 tests passing).
  • tests/services/userMessageDetector.test.ts: Verified DOM detector and deduplication (11/11 tests passing).
• Full Test Suite: npm test executed with 115 test suites passed (1,558 tests passed, 0 failures).

[coderabbitai]

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 545f77b6-bb41-4b84-9449-1dc6f7cfab09

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[dgomez407]

Will pause till prior stacked PRs get merged.