Skip to content

Conversation

@briandelmsft
Copy link
Owner

No description provided.

@briandelmsft briandelmsft requested a review from Copilot June 4, 2025 16:28
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 enhances error handling, adds mail message enrichment, and updates version/configuration details.

  • Introduces STATServerError and retry logic for 5xx responses in shared/rest.py.
  • Implements mail message enrichment pipeline: counts, comments, KQL tables, and list getters across modules.
  • Bumps function version, extends host bundle range, tweaks README URL, and updates VS Code debug config.

Reviewed Changes

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

Show a summary per file
File Description
shared/rest.py Added STATServerError import, catch block, and 5xx handler.
modules/version.json Updated "FunctionVersion" to "2.2.3".
modules/kql.py Included mail_entities in KQL query concatenation.
modules/base.py Imported datetime, added enrich_mail_message, mail counts, comments, and updated entity counts and append logic.
classes/init.py Defined STATServerError, added CreatedTime, MailMessages, and updated getters for lists and KQL.
host.json Raised extensionBundle version range to [4.0.0, 5.0.0).
README.md Prefixed AZURE_AUTHORITY_HOST with https://.
.vscode/launch.json Changed debug "type" to "debugpy" and added connect settings.
Comments suppressed due to low confidence (2)

modules/base.py:92

  • The comment variable is only defined inside the preceding if block, but add_incident_comment is called unconditionally. Guard this call or initialize comment to avoid an UnboundLocalError when no comments are generated.
        rest.add_incident_comment(base_object, comment)

host.json:13

  • Bumping the extensionBundle to major version 4 may require Azure Functions v4 runtime. Verify compatibility with your target Function runtime before upgrading.
    "version": "[4.0.0, 5.0.0)"

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.

2 participants