Skip to content

Conversation

@bskinny
Copy link
Contributor

@bskinny bskinny commented Jan 5, 2026

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)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
  • If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

@github-actions
Copy link

github-actions bot commented Jan 5, 2026

⚠️ Recommendation: Add qa/skip-qa label

This PR does not modify any files shipped with the agent.

To help streamline the release process, please consider adding the qa/skip-qa label if these changes do not require QA testing.

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a 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".

@bskinny bskinny force-pushed the external-pr/microsoft-copilot-v2-assets branch from adae6b5 to e7f7c5c Compare January 5, 2026 21:04
@bskinny bskinny force-pushed the external-pr/microsoft-copilot-v2-assets branch from e7f7c5c to b87e4b6 Compare January 5, 2026 21:24
Copy link
Contributor

@domalessi domalessi left a 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!

Comment on lines 5 to 8
[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.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
[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.

Comment on lines 10 to 11
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.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
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.

Comment on lines 19 to 20
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:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
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:

Comment on lines 32 to 33
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.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
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>
@bskinny
Copy link
Contributor Author

bskinny commented Jan 6, 2026

@domalessi Great feedback, applied all of it.

@bskinny bskinny marked this pull request as draft January 6, 2026 13:56
@bskinny bskinny changed the title Microsoft Copilot v2: Assets for webhook-based AI Interactions (Do no… [SIMI-88] Microsoft Copilot v2: Assets for webhook-based AI Interactions (Do no… Jan 6, 2026
domalessi
domalessi previously approved these changes Jan 6, 2026
Copy link
Contributor

@domalessi domalessi left a 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 :)

Comment on lines 10 to 11
- 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
Copy link
Contributor

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.

Suggested change
- 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

Copy link
Contributor Author

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.

@bskinny bskinny force-pushed the external-pr/microsoft-copilot-v2-assets branch from df4df50 to 780a007 Compare January 6, 2026 21:15
@temporal-github-worker-1 temporal-github-worker-1 bot dismissed domalessi’s stale review January 6, 2026 21:15

Review from domalessi is dismissed. Related teams and files:

  • documentation
    • microsoft_copilot/README.md
Copy link
Contributor

@domalessi domalessi left a comment

Choose a reason for hiding this comment

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

@bskinny 💪 💪

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants