Skip to content

Document Copilot Spaces PAT requirements - #2479

Merged
SamMorrowDrums merged 1 commit into
mainfrom
bestra/copilot-spaces-fgp-readme
May 14, 2026
Merged

SamMorrowDrums merged 1 commit into
mainfrom
bestra/copilot-spaces-fgp-readme

Conversation

@Bestra

@Bestra Bestra commented May 14, 2026

Copy link
Copy Markdown
Contributor

Summary

Document the authentication constraints for the remote-only Copilot Spaces tools in the README. The new note explains why fine-grained PAT users can still see the tools, but still need the right org installation, organization_copilot_spaces: read, and repo access for repo-backed resources.

Why

The README currently lists get_copilot_space and list_copilot_spaces without explaining why a fine-grained PAT might still see those tools but fail when it tries to use them. This is a small docs followup so the org-installed token requirement and the repo-backed-resource visibility behavior are easier to discover.

What changed

  • Added an authentication note to the remote-only Copilot Spaces section in README.md.
  • Called out that fine-grained PATs are not hidden by classic PAT scope filtering.
  • Documented the org installation + organization_copilot_spaces: read requirement and the repo-backed-resource access requirement.

MCP impact

  • No tool or API changes
  • Tool schema or behavior changed
  • New tool added

Prompts tested (tool changes only)

  • Not applicable; docs-only change.

Security / limits

  • No security or limits impact
  • Auth / permissions considered
    • This PR only documents the existing Copilot Spaces auth requirements for fine-grained PATs.
  • Data exposure, filtering, or token/size limits considered
    • The README now calls out that org-owned spaces with repo-backed resources may be hidden if the token lacks repo access.

Tool renaming

  • I am renaming tools as part of this PR (e.g. a part of a consolidation effort)
    • I have added the new tool aliases in deprecated_tool_aliases.go
  • I am not renaming tools as part of this PR

Lint & tests

  • Linted locally with ./script/lint
  • Tested locally with ./script/test

Docs

  • Not needed
  • Updated (README / docs / examples)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@Bestra Bestra closed this May 14, 2026
@Bestra Bestra reopened this May 14, 2026
@Bestra
Bestra marked this pull request as ready for review May 14, 2026 19:32
@Bestra
Bestra requested a review from a team as a code owner May 14, 2026 19:32
Copilot AI review requested due to automatic review settings May 14, 2026 19:32

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the documentation for the remote-only Copilot Spaces toolset to clarify why fine-grained PAT users may still see the tools listed but encounter authentication/authorization failures when invoking them.

Changes:

  • Added an “Authentication note” to the Copilot Spaces section describing fine-grained PAT visibility vs usability.
  • Documented the org installation + organization_copilot_spaces: read requirement for org-owned spaces.
  • Documented that repo-backed resources may require additional repository access or the space can appear “not found”.
Show a summary per file
File Description
README.md Adds an authentication/permissions note for remote-only Copilot Spaces tools to reduce confusion for fine-grained PAT users.

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 0

@SamMorrowDrums
SamMorrowDrums merged commit b8be4c1 into main May 14, 2026
31 checks passed
@SamMorrowDrums
SamMorrowDrums deleted the bestra/copilot-spaces-fgp-readme branch May 14, 2026 21:23
social4hyq pushed a commit to social4hyq/homebrew-core that referenced this pull request Sep 20, 2026
github-mcp-server 1.0.5

Created-by: HarmonybrewBot
Commit-by: HarmonybrewBot
Merged-by: HarmonybrewBot
Description: Created by `brew bump`

---

Created with `brew bump-formula-pr`.<details>
  <summary>release notes</summary>
  <pre>## What's Changed
* Add ifc label for list_issues tool by @gokhanarkan in github/github-mcp-server#2453
* Add ifc label for get_file_contents tool by @gokhanarkan in github/github-mcp-server#2454
* Add missing pagination on get_reviews by @RossTarrant in github/github-mcp-server#2367
* Add optional `rationale` parameter to `update_issue_type` tool by @alondahari in github/github-mcp-server#2458
* Add ifc label for search_issues tool by @gokhanarkan in github/github-mcp-server#2456
* Add ifc label for issue_read tool by @gokhanarkan in github/github-mcp-server#2457
* Add ifc label for search_repositories tool by @gokhanarkan in github/github-mcp-server#2459
* Return minimal code search results with text match snippets by @SamMorrowDrums in github/github-mcp-server#2476
* Replace ingress IFC reader list with private marker by @gokhanarkan in github/github-mcp-server#2478
* Document Copilot Spaces PAT requirements by @Bestra in github/github-mcp-server#2479
* Add tool to list repo collaborators by @JoannaaKL in github/github-mcp-server#2477
* Add tool for discussion comment write operations by @RossTarrant in github/github-mcp-server#2427
* Upgrade go-github to v 0.87 by @iulia-b in github/github-mcp-server#2452

## New Contributors
* @alondahari made their first contribution in github/github-mcp-server#2458
* @Bestra made their first contribution in github/github-mcp-server#2479

**Full Changelog**: https://github.com/github/github-mcp-server/compare/v1.0.4...v1.0.5</pre>
  <p>View the full release notes at <a href="https://github.com/github/github-mcp-server/releases/tag/v1.0.5">https://github.com/github/github-mcp-server/releases/tag/v1.0.5</a>.</p>
</details>
<hr>

See merge request: Harmonybrew/homebrew-core!3094
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.

3 participants