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

Document Copilot Spaces PAT requirements#2479
SamMorrowDrums merged 1 commit into
mainfrom
bestra/copilot-spaces-fgp-readme

Conversation

@Bestra
Copy link
Copy Markdown
Contributor

@Bestra Bestra commented May 14, 2026

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
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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
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