Skip to content

Add Permission Handling documentation to all language READMEs#879

Open
jamesmontemagno wants to merge 3 commits intogithub:mainfrom
jamesmontemagno:update-readmes-with-perms
Open

Add Permission Handling documentation to all language READMEs#879
jamesmontemagno wants to merge 3 commits intogithub:mainfrom
jamesmontemagno:update-readmes-with-perms

Conversation

@jamesmontemagno
Copy link

This pull request updates the documentation for all SDKs (Node.js, Python, Go, .NET) to clarify that a permission handler is now required when creating or resuming a Copilot session. It adds detailed sections and examples for "Permission Handling", including how to use built-in helpers or provide custom logic to approve or deny tool executions. The changes also update all code samples, quick starts, and configuration references to include the required permission handler.

Copilot AI and others added 2 commits March 17, 2026 21:49
Co-authored-by: jamesmontemagno <1676321+jamesmontemagno@users.noreply.github.com>
…-permissions

docs: Add Permission Handling section to all language READMEs
@jamesmontemagno jamesmontemagno requested a review from a team as a code owner March 17, 2026 22:02
Copilot AI review requested due to automatic review settings March 17, 2026 22:02
Copy link
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

This PR updates the SDK documentation across Node.js/TypeScript, Python, Go, and .NET to reflect that an explicit permission handler is now required when creating or resuming a Copilot session, and adds new “Permission Handling” sections with examples and result-kind reference tables.

Changes:

  • Update quick starts and session creation/resume examples to include the required permission handler.
  • Add per-language “Permission Handling” documentation with approve-all and custom handler examples.
  • Document permission decision result kinds and per-tool “skip permission” configuration.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 8 comments.

File Description
python/README.md Adds required on_permission_request guidance and Python permission handler examples/tables.
nodejs/README.md Adds required onPermissionRequest guidance and TS permission handler examples/tables.
go/README.md Adds required OnPermissionRequest guidance and Go permission handler examples/tables.
dotnet/README.md Adds required OnPermissionRequest guidance and .NET permission handler examples/tables.

You can also share your feedback on Copilot code review. Take the survey.

Co-authored-by: Copilot Autofix powered by AI <175728472+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.

3 participants