Skip to content

docs: audit and refresh Teams MCP integration docs#390

Closed
diberry wants to merge 1 commit intobradygaster:devfrom
diberry:squad/258-teams-mcp-docs-audit
Closed

docs: audit and refresh Teams MCP integration docs#390
diberry wants to merge 1 commit intobradygaster:devfrom
diberry:squad/258-teams-mcp-docs-audit

Conversation

@diberry
Copy link
Copy Markdown
Collaborator

@diberry diberry commented Mar 14, 2026

Summary

End-to-end audit of Teams MCP integration documentation per #258.

Changes Made

Fixed broken references:

  • Removed incorrect reference to microsoft/IF-MCP-Server-for-Microsoft-Teams (that repo is actually a Court Listener demo, not a general Teams MCP server)
  • Added link to working community reference implementation: benleane83's teams-webhook-mcp.js gist

Clarified MCP server setup:

  • Updated all placeholder paths from path/to/.js to /absolute/path/to/.js for clarity
  • Added explicit instructions that users need to download or create MCP server scripts
  • Pointed users to mcpmarket.com for finding compatible MCP servers
  • Simplified Teams setup to focus on webhook approach (fastest, no Azure AD setup required)

Files updated:

  • docs/src/content/docs/features/notifications.md — main Teams integration docs
  • docs/src/content/docs/features/mcp.md — MCP setup guide
  • docs/src/content/docs/concepts/portability.md — MCP config example

What Works Now

  • Docs accurately reflect that MCP servers are external implementations, not bundled with Squad
  • Community members can follow benleane83's working reference implementation
  • Path placeholders clearly indicate they require actual file paths
  • Setup instructions focus on practical, working approaches

Closes #258

- Removed broken reference to microsoft/IF-MCP-Server-for-Microsoft-Teams (it's actually a Court Listener demo, not a general Teams MCP server)
- Clarified that path/to/*.js placeholders require actual MCP server implementations
- Added link to community reference implementation (benleane83's teams-webhook-mcp.js gist)
- Updated all MCP config examples to use /absolute/path/to/ format for clarity
- Simplified Teams setup to focus on webhook approach (fastest, most practical)
- Updated iMessage MCP server link to point to marketplace search instead of specific (potentially dead) link

Closes bradygaster#258

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@diberry
Copy link
Copy Markdown
Collaborator Author

diberry commented Mar 14, 2026

@bradygaster Ready for review. This addresses issue #258 (Teams MCP docs audit). Labels needed: squad:pao, type:docs, status:in-review.

@diberry
Copy link
Copy Markdown
Collaborator Author

diberry commented Mar 14, 2026

Consolidated into PR #389 — all changes from this PR have been merged into the consolidated docs quality PR.

@diberry diberry closed this Mar 14, 2026
bradygaster pushed a commit that referenced this pull request Mar 15, 2026
Merges diberry's consolidated docs PR #389.
Closes #258, Fixes #351, Addresses #197 item 6, Addresses #398.
19 files, -1,471 net lines. Bloat reduction across guide.md, tips-and-tricks.md, sample-prompts.md, and more.
Supersedes PRs #390, #391, #392, #395, #397, #399.
tamirdresher pushed a commit to tamirdresher/squad that referenced this pull request Mar 16, 2026
…radygaster#413)

* chore(squad): quality review findings — 7 issues filed

Quality audit complete: 5 agents assessed CLI across testing, coverage, stability, accessibility, UX.
Results: 4 P0 blockers (bradygaster#365bradygaster#368), 3 P1 items (bradygaster#369bradygaster#371).
Blocking: Waingro dead sessions, ErrorBoundary, dropped input; Marquez help text consistency.

Changes:
- Logged session summary to .squad/log/2026-02-24T0205-quality-review-complete.md
- Updated .squad/identity/now.md with quality review findings and new issue numbers

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* chore(squad): merge decision — Marquez UX audit findings

Quality assessment merged from inbox (Grade B): 11 improvements (3 P0, 4 P1, 4 P2). help text, stub commands, vocabulary, separators, roster.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: visual polish — separators, help, status, agents, thinking indicator

- Replace ASCII hyphens with box-drawing chars (U+2500) in AgentPanel
  and MessageStream separators for visual consistency
- Restructure /help with section headers, column alignment, shortcuts
- Redesign /status with brand mark and clean key-value layout
- Replace /agents bracket notation ([IDLE]) with emoji + status words
- Fix ThinkingIndicator double-dim making text nearly invisible
- Add 'quit' to bare-word exit (matches /quit behavior)
- Update tests to match new output format

Closes bradygaster#388, bradygaster#389, bradygaster#390, bradygaster#391, bradygaster#392, bradygaster#393

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* docs: update Marquez history with visual polish session

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
tamirdresher pushed a commit to tamirdresher/squad that referenced this pull request Mar 16, 2026
…radygaster#466)

- Bold header, dash separator, aligned key-value pairs
- Dimmed root path for visual hierarchy
- Working section uses [WORK]/[STREAM] vocabulary with activity hints
- Imports ANSI codes from existing output module

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants