docs: update changelog, readme, cmdlet reference, and permissions for…#6
Merged
Conversation
… v1.1.0 - Add v1.1.0 changelog with features, security fixes, and maintenance - Add Get-CustomSecurityAttributesReport to README cmdlet table and quick start - Add CSA cmdlet section to CMDLET-REFERENCE.md - Add CustomSecAttributeDefinition.Read.All to PERMISSIONS.md - Update module/README.md file tree with new and renamed files
Renamed a function for clarity and removed unused parameters.
There was a problem hiding this comment.
Pull request overview
Updates the project documentation to reflect the v1.1.0 release, including the new Custom Security Attributes report cmdlet and its associated docs/permissions.
Changes:
- Add v1.1.0 release notes to
CHANGELOG.md. - Document
Get-CustomSecurityAttributesReportin the root README anddocs/CMDLET-REFERENCE.md. - Extend
docs/PERMISSIONS.mdwith the additional Custom Security Attribute definition scope and update the module file tree inmodule/README.md.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| README.md | Updates module description and adds Get-CustomSecurityAttributesReport to quick start + cmdlet list. |
| module/README.md | Updates module overview and file tree to include the CSA cmdlet and new/renamed private scripts. |
| docs/PERMISSIONS.md | Adds CustomSecAttributeDefinition.Read.All to summary/union/per-cmdlet scope lists. |
| docs/CMDLET-REFERENCE.md | Adds a new section documenting Get-CustomSecurityAttributesReport parameters and examples. |
| CHANGELOG.md | Adds a new [1.1.0] section detailing features, fixes, security, and maintenance items. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| [License: MIT](LICENSE) | ||
|
|
||
| PowerShell module for Microsoft Graph–backed reports: Intune Enrollment Flows, Intune Anomalies, Entra Admin Roles, and M365 License Assignment. Connects to Microsoft Graph and generates HTML/CSV reports. | ||
| PowerShell module for Microsoft Graph–backed reports: Intune Enrollment Flows, Intune Anomalies, Entra Admin Roles, M365 License Assignment, and Custom Security Attributes. Connects to Microsoft Graph and generates interactive HTML reports. |
Comment on lines
119
to
123
| - CustomSecAttributeAssignment.Read.All | ||
| - CustomSecAttributeDefinition.Read.All | ||
| - User.Read.All | ||
| - Organization.Read.All | ||
| - Mail.Send |
royklo
pushed a commit
that referenced
this pull request
Jun 17, 2026
…al devices
Resolve-IntuneLapsAnomalies read \$d.OwnerType but Get-AllDeviceData emits
DeviceOwnership. Every real device hit the LAPS tab with an empty Ownership
cell. The initial smoke test passed because it synthesized OwnerType directly
on the fake device - the test was masking the bug instead of catching it.
Fix:
- Read \$d.DeviceOwnership in Resolve-IntuneLapsAnomalies.
- Switch all three resolver output literals to [PSCustomObject][ordered]@{}
so property iteration order is deterministic. While here, align literal
order with the HTML <td> column order (Severity before Status) - the
report rendered correctly via property-name access but Export-Csv on the
same objects would have emitted them in the wrong order.
- Update test fixture to use the real Get-AllDeviceData field name
(DeviceOwnership) AND assert the materialised value, not just the column
presence, so a future field-name drift can't sneak past the test the way
it did this time.
Fixes copilot findings #1-#5 on the PR. Finding #6 (-UseBasicParsing breaks
PS 7+) is rejected as empirically wrong: the parameter is a documented no-op
on PS 6+, verified working on PS 7.5.4, and the settings catalog has been
downloading successfully all session.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
… v1.1.0
Summary of changes
Brief description of what this PR does.
Related issue
Fixes #(issue number) — or "None".
Checklist
Invoke-Pester ./Tests/Consistency.Tests.ps1)