Skip to content
This repository was archived by the owner on Sep 1, 2026. It is now read-only.

Devices - 51030 - Device antivirus protection is measured across the existing tenant device population - #1536

Merged
Anton Staykov (astaykov) merged 8 commits into
devfrom
DEVICE-51030
Sep 1, 2026
Merged

Devices - 51030 - Device antivirus protection is measured across the existing tenant device population#1536
Anton Staykov (astaykov) merged 8 commits into
devfrom
DEVICE-51030

Conversation

@Manoj-Kesana

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI 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 introduces a new “Device antivirus protection” metric sourced from Microsoft Defender Advanced Hunting and surfaces it in the report UI, enabling antivirus coverage to be measured across the tenant’s existing device population.

Changes:

  • Add an Advanced Hunting query in the PowerShell tenant info collector to compute protectedDeviceCount and emit it under TenantInfo.DeviceOverview.DeviceAntivirusProtection.
  • Add a reusable DeviceAntivirusProtectionCard and place it into the dashboard and devices insights UI.
  • Extend SwgDefenseLayers with an optional stacked layout mode and enable it on the dashboard.

Reviewed changes

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

Show a summary per file
File Description
src/report/src/pages/Dashboard.tsx Adds the new antivirus protection card to the Devices section and adjusts section ordering/layout.
src/report/src/config/report-data.ts Extends the report data model with DeviceAntivirusProtection and protectedDeviceCount.
src/report/src/components/overview/swg-defense-layers.tsx Adds a stacked prop to control layout responsiveness for the SWG visualization.
src/report/src/components/overview/device-antivirus-protection.tsx New UI card rendering protected vs unprotected device counts and percentages.
src/report/src-curent/pages/Dashboard.tsx Adds the new antivirus protection card to the “current” dashboard variant and adjusts grid layout.
src/report/src-curent/config/report-data.ts Mirrors the report data model extension for the “current” report variant.
src/report/src-curent/components/overview/device-antivirus-protection.tsx Mirrors the new antivirus protection card for the “current” report variant.
src/report/src-curent/components/DevicesInsightsAccordion.tsx Replaces the previous placeholder antivirus card with the reusable DeviceAntivirusProtectionCard.
src/powershell/private/tenantinfo/devices/Add-ZtDeviceOverview.ps1 Collects protectedDeviceCount via Advanced Hunting and attaches it to DeviceOverview.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread src/powershell/private/tenantinfo/devices/Add-ZtDeviceOverview.ps1 Outdated
Comment thread src/report/src/components/overview/device-antivirus-protection.tsx
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@Manoj-Kesana Manoj Kesana (Manoj-Kesana) added the ready for review PR is ready for review and merging label Aug 28, 2026
@Manoj-Kesana
Manoj Kesana (Manoj-Kesana) marked this pull request as ready for review August 28, 2026 07:40
@astaykov

Copy link
Copy Markdown
Collaborator

Hey Manoj Kesana (@Manoj-Kesana),

Please update your feature branch from dev then regenerate the report templates. Because I merged #1537 which also changes the report template this branch must be first updated from dev, then report templates regenerated.

@astaykov Anton Staykov (astaykov) left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please address my comments - the hunting query execution must be fixed before further review can be performed.

@astaykov

Copy link
Copy Markdown
Collaborator

The tile for Anti-Virus protection has different height when rendered on overview page vs. on devices page:
Overview:
image

On devices it is unneccessarily tall:

image image

resp.

image

It should not be taller then Device Ownership on the devices page.

@astaykov
Anton Staykov (astaykov) merged commit 17c6ce3 into dev Sep 1, 2026
2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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