coder / coder/chat

feat(adapters/msteams): land Microsoft Teams adapter as experimental (needs live-tenant validation)

Open
#6 0 comments 0 reactions 0 assignees View on GitHub
enhancement ready-for-human
Dominant language
Go
Stars
2
Forks
0
Avg merge
50m
Merged PRs (30d)
23

Description

Track landing the Microsoft Teams adapter (`adapters/msteams`) at the **experimental** tier, per ADR 0007.

## Status

A high-quality spike exists as draft PR #4: self-contained (zero core-runtime changes, zero new dependencies), stdlib-only Bot Framework JWT/JWKS verification, versioned opaque thread IDs carrying `conversationReference`, reuse of `internal/ratelimit` and the `Observer` hooks, ~78% statement coverage against a mocked Bot Framework server.

The spike **resolved ADR 0007 open question 9** (no `msbotbuilder-go`; hand-rolled stdlib JWT validation works, no new deps). The remaining **9 of 10 open questions require a live Azure Bot registration + Teams tenant** to validate (ack/turn semantics, endorsement behavior on production keys, markdown fidelity, `serviceUrl`/conversation-id stability, proactive posting prerequisites, `aadObjectId` stability, etc.). See the 10-point checklist in PR #4.

## Path to land

1. A human with Azure Bot + Teams tenant access runs the live validation checklist from PR #4.
2. Update ADR 0007 → `Accepted`, recording confirmed findings.
3. Mark the package experimental in `adapters/msteams/doc.go` (honest tier labeling).
4. Undraft PR #4, pass CI + `@codex review` + `@codex security review`, merge.

Blocked on: live-tenant access (human).

---
_Generated with [`mux`](https://github.com/coder/mux) • Model: `anthropic:claude-fable-5` • Thinking: `xhigh`_

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.