Teams

[cooper-grc]

Summary by cubic

Add Microsoft Teams as a first-class source and tool. You can search Teams messages via smart queries and post messages to channels.

• New Features

  • Register Teams source and tool client in the gateway.
  • New Teams provider (Microsoft Graph) to list teams/channels and read channel/chat messages.
  • Smart query inference: InferTeamsQuery with filename templating and limits.
  • View filters: team, channel, from, after, before.
  • Default filename format: {date}_{team}_{channel}_{user}_{id}.txt.
  • New teams tool with post-message (team, channel, text).
  • Microsoft OAuth for Teams: prompt=consent, clearer user-facing errors for admin consent and other failures.
  • Capability mapping now recognizes full Graph scope URLs for Teams write access.
  • Added tests for provider behavior, OAuth scopes/consent, and capability detection.

• Migration

  • Microsoft OAuth now requests Teams scopes; reauthorize to grant: https://graph.microsoft.com/Team.ReadBasic.All, https://graph.microsoft.com/Channel.ReadBasic.All, https://graph.microsoft.com/ChannelMessage.Read.All, https://graph.microsoft.com/Chat.Read, https://graph.microsoft.com/ChannelMessage.Send, https://graph.microsoft.com/Chat.ReadWrite, https://graph.microsoft.com/User.Read, offline_access.
  • Tenant admin consent may be required for some scopes; we now surface a clear message with next steps.

^{Written for commit 5004a45. Summary will update on new commits.}

[cubic-dev-ai]

6 issues found across 22 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="pkg/tools/clients/teams.go">

<violation number="1" location="pkg/tools/clients/teams.go:86">
P2: `resolveIDs` only searches the first page of Graph results and ignores `@odata.nextLink`, so lookups can fail for users with many teams/channels.</violation>
</file>

<file name="pkg/sources/providers/teams.go">

<violation number="1" location="pkg/sources/providers/teams.go:292">
P1: `ReadResult` passes message/reply IDs to `fetchReply` in the wrong order, so reply reads call the wrong Graph endpoint.</violation>

<violation number="2" location="pkg/sources/providers/teams.go:349">
P1: Result IDs are serialized with `:` delimiters without escaping, so parsing is ambiguous when any component contains `:`.</violation>

<violation number="3" location="pkg/sources/providers/teams.go:695">
P2: Channel-query parsing splits on whitespace, so `in:Team Name/Channel Name` cannot be parsed and incorrectly falls back to broad search.</violation>
</file>

<file name="pkg/sources/queries/baml_client/functions.go">

<violation number="1" location="pkg/sources/queries/baml_client/functions.go:728">
P1: Do not panic on encode failure; return the error so callers can handle it.</violation>
</file>

<file name="pkg/gateway/services/view_filters.go">

<violation number="1" location="pkg/gateway/services/view_filters.go:316">
P2: Team/channel values are injected into `in:` without escaping, so names containing spaces break Teams channel-query parsing and silently degrade to broader search.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review, or fix all with cubic.}