-
Notifications
You must be signed in to change notification settings - Fork 1.5k
[SIMI-88] Microsoft Copilot v2: Assets for webhook-based AI Interactions (Do no… #22263
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
base: master
Are you sure you want to change the base?
[SIMI-88] Microsoft Copilot v2: Assets for webhook-based AI Interactions (Do no… #22263
Conversation
|
This PR does not modify any files shipped with the agent. To help streamline the release process, please consider adding the |
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.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: adae6b580e
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
microsoft_copilot/assets/monitors/spike_in_microsoft_copilot_guardrail_responses.json
Show resolved
Hide resolved
adae6b5 to
e7f7c5c
Compare
…t merge until backend confirms)
e7f7c5c to
b87e4b6
Compare
domalessi
left a comment
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.
Thanks for the changes, @bskinny ! I made some suggestions to tighten up the language and ensure adherence with Datadog style guidelines. Give that a look-through and let me know if you have any questions. Flag me down when this is ready for re-review!
microsoft_copilot/README.md
Outdated
| [Microsoft Copilot][1] is an AI assistant integrated into Microsoft 365, the cloud-based suite of productivity and | ||
| communication tools that includes Microsoft Office applications. Once a user within the Microsoft 365 organization | ||
| is assigned a Copilot license, they can access Copilot directly through Copilot Chat or through Microsoft 365 | ||
| applications, including Outlook, Word, Excel, PowerPoint, OneNote, and Teams. |
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.
| [Microsoft Copilot][1] is an AI assistant integrated into Microsoft 365, the cloud-based suite of productivity and | |
| communication tools that includes Microsoft Office applications. Once a user within the Microsoft 365 organization | |
| is assigned a Copilot license, they can access Copilot directly through Copilot Chat or through Microsoft 365 | |
| applications, including Outlook, Word, Excel, PowerPoint, OneNote, and Teams. | |
| [Microsoft Copilot][1] is an AI assistant integrated into Microsoft 365 applications such as Outlook, Word, Excel, PowerPoint, OneNote, and Teams. After a user in a Microsoft 365 organization is assigned a Copilot license, they can access Copilot through Copilot Chat or directly within supported Microsoft 365 applications. |
microsoft_copilot/README.md
Outdated
| By integrating Microsoft Copilot usage tracking with Datadog, you can ingest and analyze user activity to gain insight | ||
| into the value Microsoft Copilot delivers within your Microsoft 365 environment. |
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.
| By integrating Microsoft Copilot usage tracking with Datadog, you can ingest and analyze user activity to gain insight | |
| into the value Microsoft Copilot delivers within your Microsoft 365 environment. | |
| By integrating Microsoft Copilot usage tracking with Datadog, you can ingest logs and metrics that reflect user activity and assess the value Copilot delivers within your Microsoft 365 environment. |
microsoft_copilot/README.md
Outdated
| Microsoft computes Copilot usage reports on a time-delayed basis of [up to 72 hours][3]; real-time AI interactions | ||
| are not affected by this. Upon detection of a new report, this integration: |
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.
| Microsoft computes Copilot usage reports on a time-delayed basis of [up to 72 hours][3]; real-time AI interactions | |
| are not affected by this. Upon detection of a new report, this integration: | |
| Microsoft computes Copilot usage reports with a delay of [up to 72 hours][3]. Real-time AI interaction logs are not affected by this delay. Upon detection of a new report, this integration: |
microsoft_copilot/README.md
Outdated
| The Microsoft Copilot integration collects a log message for each user's Copilot activity once per day as well as | ||
| a real-time log message for each AI interaction. |
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.
| The Microsoft Copilot integration collects a log message for each user's Copilot activity once per day as well as | |
| a real-time log message for each AI interaction. | |
| The Microsoft Copilot integration collects daily summary log messages for user Copilot activity and real-time log messages for individual AI interactions. |
Co-authored-by: domalessi <111786334+domalessi@users.noreply.github.com>
|
@domalessi Great feedback, applied all of it. |
domalessi
left a comment
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.
Looks great! Thanks for the changes, @bskinny. Left one nit to review but certainly not blocking :)
microsoft_copilot/README.md
Outdated
| - Track user activity through logs that capture each user's "last activity" per Microsoft 365 application | ||
| - Identify the number of Microsoft Copilot licenses that have not been used for extended periods |
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.
Is "last activity" an exact field name exposed to users? If not, we can remove the quotes.
| - Track user activity through logs that capture each user's "last activity" per Microsoft 365 application | |
| - Identify the number of Microsoft Copilot licenses that have not been used for extended periods | |
| - Track user activity through logs that capture each user's last activity per Microsoft 365 application | |
| - Identify Microsoft Copilot licenses that have not been used for extended periods |
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.
No nit too small. Just pushed your suggestions, the quotes around "last activity" were not needed.
df4df50 to
780a007
Compare
Review from domalessi is dismissed. Related teams and files:
- documentation
- microsoft_copilot/README.md
domalessi
left a comment
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.
@bskinny 💪 💪
This version of the ms copilot integration is a superset of the v1.0.0, with AI Interactions present as log messages.
Do not merge until backend capability is present, which could be later in 2026.
Review checklist (to be filled by reviewers)
qa/skip-qalabel if the PR doesn't need to be tested during QA.backport/<branch-name>label to the PR and it will automatically open a backport PR once this one is merged