Skip to content

update wrapper - #741

Merged
Paul Lizer (paullizer) merged 6 commits into
Developmentfrom
pluginloggerasync
Mar 2, 2026
Merged

update wrapper#741
Paul Lizer (paullizer) merged 6 commits into
Developmentfrom
pluginloggerasync

Conversation

@Bionic711

Copy link
Copy Markdown
Collaborator

Updates the wrapper to function with async kernel_functions.

CopilotAI review requested due to automatic review settings March 2, 2026 22:46

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

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 Semantic Kernel plugin invocation logging wrapper to better support async kernel functions, alongside a small schema-validation tweak and dependency bumps to align with the newer SK behavior.

Changes:

  • Refactors plugin_function_logger to create async vs sync wrappers based on coroutine detection.
  • Updates JSON schema validation to validate full schemas with $ref resolution.
  • Adjusts a few Python dependencies (notably semantic-kernel, plus loosening gunicorn/pyjwt constraints).

Reviewed changes

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

FileDescription
application/single_app/semantic_kernel_plugins/plugin_invocation_logger.pyAdds async-aware wrapper logic for plugin function logging.
application/single_app/requirements.txtUpdates dependency constraints to support the async wrapper/SK version.
application/single_app/json_schema_validation.pySwitches plugin schema validation to use $ref resolution at the root schema level.

Comment threadapplication/single_app/requirements.txt
Comment threadapplication/single_app/semantic_kernel_plugins/plugin_invocation_logger.py Outdated
Comment threadapplication/single_app/requirements.txt
CopilotAI review requested due to automatic review settings March 2, 2026 23:14

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

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

Comment threadapplication/single_app/config.py
@paullizer
Paul Lizer (paullizer) merged commit 433a7c3 into DevelopmentMar 2, 2026
7 of 8 checks passed
@paullizer
Paul Lizer (paullizer) deleted the pluginloggerasync branch March 2, 2026 23:33
Sign up for freeto 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.

4 participants

@Bionic711@paullizer@nadoylemsft