Skip to content

Data 35039 Copilot Communication Compliance Monitoring Configured#776

Merged
merill merged 5 commits intomainfrom
Data-35039
Feb 4, 2026
Merged

Data 35039 Copilot Communication Compliance Monitoring Configured#776
merill merged 5 commits intomainfrom
Data-35039

Conversation

@komalp2025
Copy link
Collaborator

@komalp2025 komalp2025 commented Jan 16, 2026

@komalp2025 komalp2025 marked this pull request as draft January 16, 2026 06:34
@komalp2025 komalp2025 force-pushed the Data-35039 branch 2 times, most recently from 4089c11 to fd71a24 Compare January 28, 2026 06:46
@komalp2025 komalp2025 changed the title draft - 35039 Data 35039 Copilot Communication Compliance Monitoring Configured Jan 29, 2026
@komalp2025 komalp2025 marked this pull request as ready for review January 29, 2026 08:09
@komalp2025 komalp2025 requested a review from alexandair January 29, 2026 08:12
@alexandair alexandair requested a review from Copilot January 29, 2026 08:31
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 pull request adds a new assessment test and associated remediation guidance to verify that Communication Compliance is configured to monitor Copilot interactions.

Changes:

  • Introduces Test-Assessment-35039 PowerShell test to detect Communication Compliance supervisory review rules targeting Copilot, ensure associated policies are enabled with review mailboxes, and optionally check Unified Audit Log evidence.
  • Adds Test-Assessment.35039.md documentation describing the rationale and remediation steps for configuring Copilot-related Communication Compliance policies.
  • (Note: TestMeta.json currently has no entry for TestId 35039; if the harness depends on it, a follow-up change may be needed to register this test.)

Reviewed changes

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

File Description
src/powershell/tests/Test-Assessment.35039.ps1 New test implementation that queries supervisory review rules/policies, evaluates Copilot-targeting configuration and enabled state, and generates detailed markdown output including audit-log evidence.
src/powershell/tests/Test-Assessment.35039.md New remediation guidance and verification steps for configuring and validating Communication Compliance policies targeting Copilot interactions.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@komalp2025 komalp2025 added the ready for review PR is ready for review and merging label Jan 30, 2026
Copy link
Collaborator

@alexandair alexandair left a comment

Choose a reason for hiding this comment

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

Why are you wrapping code in @()?
The @() around command output/pipelines can be useful in some cases, but the codebase generally doesn't use this pattern constantly.

Please, remove all the unnecessary @() wrappers.

@komalp2025
Copy link
Collaborator Author

Why are you wrapping code in @()? The @() around command output/pipelines can be useful in some cases, but the codebase generally doesn't use this pattern constantly.

Please, remove all the unnecessary @() wrappers.

Hi @alexandair I have removed it. Changes are ready for review, except for broken link.

@alexandair alexandair self-requested a review February 3, 2026 10:30
Copy link
Collaborator

@alexandair alexandair left a comment

Choose a reason for hiding this comment

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

@komalp2025 A broken link in .md still needs to be fixed.
LGTM

Updated links in Test-Assessment.35039.md for accuracy.
@komalp2025
Copy link
Collaborator Author

@komalp2025 A broken link in .md still needs to be fixed. LGTM

Hi @alexandair I have updated the broken link.

@komalp2025 komalp2025 requested a review from alexandair February 4, 2026 06:07
@merill merill merged commit 24d65e5 into main Feb 4, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready for review PR is ready for review and merging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants