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

create a sample agent #3

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

create a sample agent #3

wants to merge 14 commits into from

Conversation

Fr4nc3
Copy link
Collaborator

@Fr4nc3 Fr4nc3 commented Jan 22, 2025

Purpose

  • ...

Does this introduce a breaking change?

  • Yes
  • No

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

@Copilot Copilot bot review requested due to automatic review settings January 22, 2025 21:39
Copy link

@Copilot 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.

Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.

Files not reviewed (1)
  • src/frontend/wwwroot/home/home.html: Language not supported
Comments suppressed due to low confidence (2)

src/backend/agents/baker_agent.py:10

  • The function bake_cookies should have a type hint for its return value.
async def bake_cookies(cookie_type: str, quantity: int) -> str:

src/backend/agents/baker_agent.py:13

  • The function prepare_dough should have a type hint for its return value.
async def prepare_dough(dough_type: str) -> str:

@@ -90,6 +90,9 @@
case "GenericAgent":
agentIcon = "manager";
break;
case "BakerAgent":
agentIcon = "manager";
Copy link
Preview

Copilot AI Jan 22, 2025

Choose a reason for hiding this comment

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

The agentIcon for BakerAgent is incorrectly set to 'manager'. It should be set to an icon related to baking.

Suggested change
agentIcon = "manager";
agentIcon = 'baker';

Copilot is powered by AI, so mistakes are possible. Review output carefully before use.

Positive Feedback
Negative Feedback

Provide additional feedback

Please help us improve GitHub Copilot by sharing more details about this comment.

Please select one or more of the options
marktayl1 and others added 13 commits January 22, 2025 22:57
…g fixes (microsoft#52)

* fix: ui changes (#1)

* fix: in progress status color after fetch task details (microsoft#5)

* cancel notification message updated (microsoft#7)

* Update task.js (microsoft#9)

* Stages overflow issue fix (microsoft#10)

* fix: added space to the agent (microsoft#13)

* Approve reject buttons titles disabling buttons and (microsoft#15)

* Fix: UX becomes damaged when chat outputs sample code for a task (microsoft#14)

* task page UI updates

* UI updated code for task

* Task page UI updated code

* status section UI update in task page

* Added custom event (microsoft#24)

* feat: added custom event

* Logs updated

* modify code

* added exception logs

* added exception logs for cosmos memory

---------

Co-authored-by: Roopan P M <v-roopanpm@microsoft.com>

* fix: task with zero stages cannot show the page, spins forever and rai test prompt (microsoft#41)

* Updated the workflow for build and push docker

* updated the repo name

* Update requirements.txt

* Update requirements.txt

* Update requirements.txt

* feat: Integrated application insights instrumentation key into the bicep file (microsoft#42)

* feat: Integrated application insights instrumentation key into bicep files

* added application insights instrumentation key into env and readme file

* updated json file

* upgraded json file

* Update docker-build-and-push.yml

* Update docker-build-and-push.yml

* Update docker-build-and-push.yml

* Update docker-build-and-push.yml

* Update docker-build-and-push.yml

* Update docker-build-and-push.yml

* pyLint issues fixed

* lint issues fixed

---------

Co-authored-by: Roopan-Microsoft <168007406+Roopan-Microsoft@users.noreply.github.com>
Co-authored-by: Roopan P M <v-roopanpm@microsoft.com>

* Update test.yml (microsoft#43)

* Disabling Text Area functionality (microsoft#47)

* fix: Usability and Alignments changes (microsoft#48)

* Name update and padding removed

* fix home page padding and cards height

* remove gap in tasks section

* Update docker-build-and-push.yml to debug

* Update docker-build-and-push.yml

* Update docker-build-and-push.yml to take the correct event name

* Update docker-build-and-push.yml

* fix: text area background color (microsoft#50)

* fix: text area background color

* Cursor hover style and Light dark color updated

---------

Co-authored-by: Prashant-Microsoft <v-pmalusare@microsoft.com>
Co-authored-by: Kiran-Siluveru-Microsoft <v-ksiluveru@microsoft.com>
Co-authored-by: Mohan-Microsoft <v-mvenudass@microsoft.com>
Co-authored-by: Harmanpreet-Microsoft <v-harmanprka@microsoft.com>
…icrosoft#53)

* error handled and removed console logs

* extra spaces removed
…across various files in the backend (microsoft#58)

* fix: ui changes (#1)

* fix: in progress status color after fetch task details (microsoft#5)

* cancel notification message updated (microsoft#7)

* Update task.js (microsoft#9)

* Stages overflow issue fix (microsoft#10)

* fix: added space to the agent (microsoft#13)

* Approve reject buttons titles disabling buttons and (microsoft#15)

* Fix: UX becomes damaged when chat outputs sample code for a task (microsoft#14)

* task page UI updates

* UI updated code for task

* Task page UI updated code

* status section UI update in task page

* Added custom event (microsoft#24)

* feat: added custom event

* Logs updated

* modify code

* added exception logs

* added exception logs for cosmos memory

---------

Co-authored-by: Roopan P M <v-roopanpm@microsoft.com>

* fix: task with zero stages cannot show the page, spins forever and rai test prompt (microsoft#41)

* Updated the workflow for build and push docker

* updated the repo name

* Update requirements.txt

* Update requirements.txt

* Update requirements.txt

* feat: Integrated application insights instrumentation key into the bicep file (microsoft#42)

* feat: Integrated application insights instrumentation key into bicep files

* added application insights instrumentation key into env and readme file

* updated json file

* upgraded json file

* Update docker-build-and-push.yml

* Update docker-build-and-push.yml

* Update docker-build-and-push.yml

* Update docker-build-and-push.yml

* Update docker-build-and-push.yml

* Update docker-build-and-push.yml

* pyLint issues fixed

* lint issues fixed

---------

Co-authored-by: Roopan-Microsoft <168007406+Roopan-Microsoft@users.noreply.github.com>
Co-authored-by: Roopan P M <v-roopanpm@microsoft.com>

* Update test.yml (microsoft#43)

* Disabling Text Area functionality (microsoft#47)

* fix: Usability and Alignments changes (microsoft#48)

* Name update and padding removed

* fix home page padding and cards height

* remove gap in tasks section

* Update docker-build-and-push.yml to debug

* Update docker-build-and-push.yml

* Update docker-build-and-push.yml to take the correct event name

* Update docker-build-and-push.yml

* fix: text area background color (microsoft#50)

* fix: text area background color

* Cursor hover style and Light dark color updated

* feat: customize track events (microsoft#54)

* feat: customized track event

* pylint fix

* pylint fix

---------

Co-authored-by: Prashant-Microsoft <v-pmalusare@microsoft.com>
Co-authored-by: Kiran-Siluveru-Microsoft <v-ksiluveru@microsoft.com>
Co-authored-by: Mohan-Microsoft <v-mvenudass@microsoft.com>
Co-authored-by: Harmanpreet-Microsoft <v-harmanprka@microsoft.com>
* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update LocalDeployment.md

* Update LocalDeployment.md

* Update azure_app_service_auth_setup.md

* Update LocalDeployment.md

* Update README.md
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