-
Notifications
You must be signed in to change notification settings - Fork 48
Deploy sample application in container app - Python #89
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
Closed
Closed
Changes from 1 commit
Commits
Show all changes
131 commits
Select commit
Hold shift + click to select a range
7c1cbc9
Update README title from '.NET SDK' to 'SDK Samples' (#1)
pontemonti d81af2c
Added CODEOWNERS and .gitignore files (#2)
pontemonti c3c1e74
Add .NET Semantic Kernel Sample Agent (#3)
pontemonti 1d90369
Update License to MIT License (#4)
rahuldevikar761 39a239c
Added Python OpenAI sample (#5)
pontemonti 6f236de
Add LangChain Sample (#7)
JesuTerraz 4cbd949
Add Node.js Claude Sample Agent (#6)
pontemonti 1c0b548
Fix Agent Notifs and update env (#8)
JesuTerraz ea11209
[Python][AgentFramework] Adding Python Agent Framework Sample (#9)
JesuTerraz 9b124ba
Add TM (#11)
rahuldevikar761 e99aa17
Revise README for Agent 365 Sample Agent (#13)
pontemonti d221266
Add TM (#14)
rahuldevikar761 e733e9c
Update Agents SDK for LangChain Sample (#15)
JesuTerraz 45fffa8
Adding Notifications on AF (#16)
mrunalhirve128 0b9adea
Add Copilot instructions for code review and validation (#21)
rahuldevikar761 db60215
Update Sample with API Change (#19)
JesuTerraz 1b01b67
Adding Vercel AI SDK sample with claude model (#23)
dbezic 7716ce5
Add n8n sample agent (#24)
rbrighenti 75e3a7d
Correct example agent name to use correct casing in line with n8n (#26)
rbrighenti cadc62f
Introducing perplexity Sample Agent (#18)
aubreyquinn d521cf1
Add OpenAI Nodejs Sample (#28)
JesuTerraz fea98e1
Rename "Microsoft Agents A365" to "Microsoft Agent 365" in documentat…
Copilot 51edfde
Add Devin Agent Sample (#22)
walterluna 45854e9
Fix OpenAI JS ToolingManifest (#33)
JesuTerraz 35b4a33
add updated nodejs claude implementation (#32)
msftairaamane 224b2dc
Quickstart LangChain Agent (#25)
JesuTerraz 2398b60
Introducing Playground notification handling in Perplexity agent (#34)
aubreyquinn 000ff20
Add Cursor IDE Prompt Guide for Agent365 (#29)
shinsi-fathima-rahman 348b4a9
Update samples to remove environment id from calls and settings (#10)
joratz 6c78dbe
Fix formatting of 'Microsoft Agent365' to 'Microsoft Agent 365' (#39)
pontemonti 292c85c
add quickstart claude agent before A365 extension (#42)
msftairaamane a5b3168
Update package.json description for clarity (#45)
pontemonti e8dccd5
Update devin agent sample (#48)
walterluna 42e7d80
updating samples read me docs (#47)
abdulanu0 df4e892
Update Node.js OpenAI sample (#50)
pontemonti 03f59e9
Introducing thinking indicator in Perplexity sample agent (#37)
aubreyquinn 4112c7a
updating auth handler (#53)
abdulanu0 2430fd0
Update Python Agent Framework sample (#58)
pontemonti 684a02c
Update Python OpenAI sample agent (#57)
pontemonti 251c405
Introducing observability in Perplexity agent (#44)
aubreyquinn acc29c9
Add prompt injection defense to agent system prompt (#40)
efpiva 8d931c9
Update Package References to Use npm Registry + Remove local preinsta…
JesuTerraz 0971787
Add temporariy thumbnails to the related samples (#63)
Alive-Fish 7bbfda7
Update .NET Semantic Kernel Sample Agent (#61)
pontemonti df82639
Update tooling manifest (#60)
pontemonti f473a8a
Introducing Microsoft Teams manifest file for Perplexity (#51)
aubreyquinn 004f8a2
Add manifest template for n8n Sample (#49)
rbrighenti 6dc7d20
Potential fix for code scanning alert no. 1: Workflow does not contai…
pontemonti 9e8d76d
Add CI workflow for Node.js OpenAI sample agent (#65)
pontemonti ffaa2ea
Remove local reference in python samples (#62)
JesuTerraz 132d1c3
Google ADK Sample with Tooling (#70)
JesuTerraz 9794391
Updated formatting on readme file (#71)
aubreyquinn 1611d2e
Perplexity: introducing the published agents-a365 packages (#72)
aubreyquinn c086ffc
Add Devin Agent's manifest sample (#52)
walterluna c44f444
reference public package dependencies (#64)
walterluna 805f9ae
Perplexity: added telemetry markers to all paths in the code (#73)
aubreyquinn aa4c676
Updated the scope for messaging. (#78)
Reza-sh 29c31bf
post public documentation changes (#80)
shinsi-fathima-rahman 764ec3b
Adding points for the users who contributed towards code review & qua…
LavanyaK235 4083f53
Add Agent Framework and Semantic Kernel enhancements (#85)
MattB-msft 7b293c3
Fix agentic user token retrieval for n8n Sample (#75)
rbrighenti 0af6f5c
Add package-lock.json to .gitignore (#76)
pontemonti 462149b
Delete nodejs/claude/quickstart-before/package-lock.json (#88)
pontemonti 889e04f
Update activity for sample application
tirthdoshi009 1e660f6
Update activity for sample application
tirthdoshi009 fb91b81
Update python/agent-framework/sample-agent/host_agent_server.py
tirthdoshi009 d66f523
Update python/agent-framework/sample-agent/host_agent_server.py
tirthdoshi009 447ad69
Adding docker file to run the container
tirthdoshi009 a7ddfd7
Adding docker file to run the container
tirthdoshi009 739305f
Merge branch 'users/tirthdoshi/local-playground' of https://github.co…
tirthdoshi009 6c7b64f
Merge branch 'users/tirthdoshi/local-playground' of https://github.co…
tirthdoshi009 b424420
Add the generated workflow file
tirthdoshi009 7aedf72
Add the generated workflow file
tirthdoshi009 11e4b91
Dummy push for running
tirthdoshi009 147dac1
Dummy push for running
tirthdoshi009 5a25573
Automate RG as well
tirthdoshi009 6769aa0
Automate RG as well
tirthdoshi009 acb5b3d
Create a docker image
tirthdoshi009 99cba82
Create a docker image
tirthdoshi009 16a6f7a
Update directory structure
tirthdoshi009 910fa5a
Update directory structure
tirthdoshi009 ca7662d
Remove Docker Buildx setup to fix docker-compose context error
tirthdoshi009 e90a8c0
Remove Docker Buildx setup to fix docker-compose context error
tirthdoshi009 d8207dc
Remove Docker Buildx setup to fix docker-compose context error
tirthdoshi009 8f9364a
Remove Docker Buildx setup to fix docker-compose context error
tirthdoshi009 12b29e6
Check directory structure
tirthdoshi009 75fafb2
Check directory structure
tirthdoshi009 c970c26
Fix docker environment vars
tirthdoshi009 3e42674
Fix docker environment vars
tirthdoshi009 5e16ae4
Docker image build
tirthdoshi009 730d539
Docker image build
tirthdoshi009 714b7c6
Remove fixing environment vars and debug
tirthdoshi009 71f385d
Remove fixing environment vars and debug
tirthdoshi009 8aab1a5
ACR login simplified
tirthdoshi009 7af4c80
ACR login simplified
tirthdoshi009 1a333e4
Get tocken
tirthdoshi009 2263d77
Get tocken
tirthdoshi009 f2a1256
simplify workflow
tirthdoshi009 5de1417
simplify workflow
tirthdoshi009 80f6f1f
Fix docker context
tirthdoshi009 a0decdb
Fix docker context
tirthdoshi009 56ccfca
Add permissions
tirthdoshi009 8b343c5
Add permissions
tirthdoshi009 3d69c71
Add a debug step
tirthdoshi009 fb7b831
Add a debug step
tirthdoshi009 7009477
Build docker image, file path corrected
tirthdoshi009 e700d23
Build docker image, file path corrected
tirthdoshi009 dcb80b5
Update indent and remove reserved variable
tirthdoshi009 327f3c4
Update indent and remove reserved variable
tirthdoshi009 df325b9
Update syntax error
tirthdoshi009 e890c04
Update syntax error
tirthdoshi009 b8cde9b
Add optional Application Insights integration
tirthdoshi009 75084ac
Add optional Application Insights integration
tirthdoshi009 9b3dab9
Add container app logs viewing step to deployment workflow
tirthdoshi009 16c348c
Add container app logs viewing step to deployment workflow
tirthdoshi009 7f16967
Add complete environment configuration to workflow
tirthdoshi009 b261a8d
Add complete environment configuration to workflow
tirthdoshi009 70d4f03
Trigger workflow run
tirthdoshi009 d3ca65d
Trigger workflow run
tirthdoshi009 808d12b
Trigger workflow run
tirthdoshi009 003451b
Trigger workflow run
tirthdoshi009 a55f300
Trigger workflow run
tirthdoshi009 a18f5a9
Trigger workflow run
tirthdoshi009 272e2dd
Add a comment once deployed
tirthdoshi009 4109e8c
Resolve merge conflicts
tirthdoshi009 4c73ead
Update comment in PR
tirthdoshi009 daef7fa
Update previous comment instead of ccreating a new one
tirthdoshi009 f087043
Remove local branch within PR
tirthdoshi009 d4ddc70
Delete python/agent-framework/sample-agent/APPLICATION_INSIGHTS.md
tirthdoshi009 bbcd192
Remove workflow
tirthdoshi009 3e9688e
Merge branch 'users/tirthdoshi/local-playground' of https://github.co…
tirthdoshi009 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Critical: The
auth_handlersparameter was removed from the message activity handler. This breaks authentication for the agents playground.The
on_messagehandler at line 170 should includeauth_handlers=handlerto ensure proper authentication:@self.agent_app.activity("message", auth_handlers=handler)Comparing with line 154 (
installationUpdate) and the similar implementation inpython/openai/sample-agent/host_agent_server.py:114, all activity handlers should include the auth_handlers parameter. The removal of this parameter is likely causing the playground connectivity issue mentioned in the PR description.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tirthdoshi009 can you address this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rahuldevikar761 Hey Rahul,

The auth handler does not really do anything in this code. I think it's being misplaced. In order for the playground to send the message to the agent, the correct way is this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
https://learn.microsoft.com/en-us/microsoft-365/agents-sdk/quickstart?pivots=python
Documentation ref