Compliance Dashboard implementation#328
Merged
gusfcarvalho merged 4 commits intomainfrom Feb 25, 2026
Merged
Conversation
…ntrols Aggregate evidence status across resolved profile controls with group and summary rollups, optional SSP implementation coverage, integration tests, and Swagger updates.
…and add edge case tests
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request implements a compliance dashboard feature that tracks compliance progress for OSCAL profiles. It includes a new API endpoint, bug fixes for handling controls/groups across multiple catalogs, and comprehensive test coverage.
Changes:
- Added new
/api/oscal/profiles/{id}/compliance-progressendpoint that aggregates compliance status across controls in a profile - Fixed cross-catalog handling in control and group merging functions by including catalog_id in merge keys
- Added nil parent ID validation checks to prevent potential runtime errors in rollup functions
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| internal/api/handler/oscal/profile_compliance.go | New handler implementing compliance progress endpoint with status aggregation, group rollups, and SSP implementation tracking |
| internal/api/handler/oscal/profiles.go | Bug fixes for cross-catalog control/group merging and added nil pointer safety checks in parent lookups |
| internal/api/handler/oscal/profiles_test.go | Unit tests for cross-catalog control and group merging behavior |
| internal/api/handler/oscal/profiles_integration_test.go | Comprehensive integration tests covering compliance progress endpoint, edge cases, and SSP implementation tracking |
| docs/swagger.yaml | OpenAPI documentation for new compliance progress endpoint |
| docs/swagger.json | OpenAPI documentation in JSON format |
| docs/docs.go | Generated Swagger documentation code |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…ent details and update integration tests
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.
No description provided.