Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open API GitHub Plugin Skill + Frontend Auth (MSAL + Basic / PAT) #641

Merged
merged 47 commits into from May 1, 2023

Conversation

teresaqhoang
Copy link
Contributor

@teresaqhoang teresaqhoang commented Apr 25, 2023

Motivation and Context

This PR adds:

  • Swagger files detailing List and Get Pull Request(s) functions.
  • Example 22 c on how to leverage imported OpenAPI GitHub Skill
  • Basic (+ User PAT) and Msal Authentication flows in Copilot chat for connector plug-ins, sending auth data if user enables plug-in
  • Parsing headers in SemanticKernelController to import respective OpenAPI Skill if auth information is provided

Description

image
image
image
image
image

Contribution Checklist

@github-actions github-actions bot added .NET Issue or Pull requests regarding .NET code kernel.core labels Apr 25, 2023
@adrianwyatt adrianwyatt self-assigned this Apr 25, 2023
Copy link
Member

@Vivihung Vivihung left a comment

Choose a reason for hiding this comment

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

I took a pass on the changes in Copilot Chats. LGTM and have a couple curious questions. I might not be helpful in changes outside of Copilot Chats.

@teresaqhoang teresaqhoang requested a review from amsacha May 1, 2023 18:24
@dluc dluc dismissed their stale review May 1, 2023 21:50

dismissed

@teresaqhoang teresaqhoang added the PR: ready to merge PR has been approved by all reviewers, and is ready to merge. label May 1, 2023
@adrianwyatt adrianwyatt merged commit 309840b into microsoft:main May 1, 2023
11 checks passed
awharrison-28 pushed a commit to awharrison-28/semantic-kernel that referenced this pull request May 1, 2023
…crosoft#641)

### Motivation and Context
This PR adds:

- Swagger files detailing List and Get Pull Request(s) functions. 
- Example 22 c on how to leverage imported OpenAPI GitHub Skill
- Basic (+ User PAT) and Msal Authentication flows in Copilot chat for
connector plug-ins, sending auth data if user enables plug-in
- Parsing headers in SemanticKernelController to import respective
OpenAPI Skill if auth information is provided

### Description
![image](https://user-images.githubusercontent.com/125500434/234473631-cb01eac2-12b7-4f78-a939-bc99f70d2184.png)

![image](https://user-images.githubusercontent.com/125500434/234473648-976fa0a5-f857-4693-b72b-b2f85a4d1eee.png)

![image](https://user-images.githubusercontent.com/125500434/234473826-9d0264a8-1e4c-464e-a59f-84affa1ae99a.png)

![image](https://user-images.githubusercontent.com/125500434/234473842-1efcbf7d-4bd4-4015-b903-c13bdd173191.png)

![image](https://user-images.githubusercontent.com/125500434/235235863-1353180f-f679-4445-afaf-4b4c2154d922.png)
@teresaqhoang teresaqhoang deleted the openapi-github-pr-skill branch May 15, 2023 16:59
codebrain pushed a commit to searchpioneer/semantic-kernel that referenced this pull request May 16, 2023
…crosoft#641)

### Motivation and Context
This PR adds:

- Swagger files detailing List and Get Pull Request(s) functions. 
- Example 22 c on how to leverage imported OpenAPI GitHub Skill
- Basic (+ User PAT) and Msal Authentication flows in Copilot chat for
connector plug-ins, sending auth data if user enables plug-in
- Parsing headers in SemanticKernelController to import respective
OpenAPI Skill if auth information is provided

### Description
![image](https://user-images.githubusercontent.com/125500434/234473631-cb01eac2-12b7-4f78-a939-bc99f70d2184.png)

![image](https://user-images.githubusercontent.com/125500434/234473648-976fa0a5-f857-4693-b72b-b2f85a4d1eee.png)

![image](https://user-images.githubusercontent.com/125500434/234473826-9d0264a8-1e4c-464e-a59f-84affa1ae99a.png)

![image](https://user-images.githubusercontent.com/125500434/234473842-1efcbf7d-4bd4-4015-b903-c13bdd173191.png)

![image](https://user-images.githubusercontent.com/125500434/235235863-1353180f-f679-4445-afaf-4b4c2154d922.png)
dehoward pushed a commit to lemillermicrosoft/semantic-kernel that referenced this pull request Jun 1, 2023
…crosoft#641)

### Motivation and Context
This PR adds:

- Swagger files detailing List and Get Pull Request(s) functions. 
- Example 22 c on how to leverage imported OpenAPI GitHub Skill
- Basic (+ User PAT) and Msal Authentication flows in Copilot chat for
connector plug-ins, sending auth data if user enables plug-in
- Parsing headers in SemanticKernelController to import respective
OpenAPI Skill if auth information is provided

### Description
![image](https://user-images.githubusercontent.com/125500434/234473631-cb01eac2-12b7-4f78-a939-bc99f70d2184.png)

![image](https://user-images.githubusercontent.com/125500434/234473648-976fa0a5-f857-4693-b72b-b2f85a4d1eee.png)

![image](https://user-images.githubusercontent.com/125500434/234473826-9d0264a8-1e4c-464e-a59f-84affa1ae99a.png)

![image](https://user-images.githubusercontent.com/125500434/234473842-1efcbf7d-4bd4-4015-b903-c13bdd173191.png)

![image](https://user-images.githubusercontent.com/125500434/235235863-1353180f-f679-4445-afaf-4b4c2154d922.png)
golden-aries pushed a commit to golden-aries/semantic-kernel that referenced this pull request Oct 10, 2023
…crosoft#641)

### Motivation and Context
This PR adds:

- Swagger files detailing List and Get Pull Request(s) functions. 
- Example 22 c on how to leverage imported OpenAPI GitHub Skill
- Basic (+ User PAT) and Msal Authentication flows in Copilot chat for
connector plug-ins, sending auth data if user enables plug-in
- Parsing headers in SemanticKernelController to import respective
OpenAPI Skill if auth information is provided

### Description
![image](https://user-images.githubusercontent.com/125500434/234473631-cb01eac2-12b7-4f78-a939-bc99f70d2184.png)

![image](https://user-images.githubusercontent.com/125500434/234473648-976fa0a5-f857-4693-b72b-b2f85a4d1eee.png)

![image](https://user-images.githubusercontent.com/125500434/234473826-9d0264a8-1e4c-464e-a59f-84affa1ae99a.png)

![image](https://user-images.githubusercontent.com/125500434/234473842-1efcbf7d-4bd4-4015-b903-c13bdd173191.png)

![image](https://user-images.githubusercontent.com/125500434/235235863-1353180f-f679-4445-afaf-4b4c2154d922.png)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kernel Issues or pull requests impacting the core kernel .NET Issue or Pull requests regarding .NET code PR: ready for review All feedback addressed, ready for reviews PR: ready to merge PR has been approved by all reviewers, and is ready to merge.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants