Skip to content

feat(ai-report): added detailed pass section for report package#7637

Merged
v-sharmachir merged 10 commits intomainfrom
users/v-sharmachir/agentreport
Aug 12, 2025
Merged

feat(ai-report): added detailed pass section for report package#7637
v-sharmachir merged 10 commits intomainfrom
users/v-sharmachir/agentreport

Conversation

@v-sharmachir
Copy link
Copy Markdown
Contributor

@v-sharmachir v-sharmachir commented Aug 1, 2025

Details

The passed section does not show the components that passed the check. To improve clarity, added logic so that passed section include the components that successfully passed the check.

Added optional parameter expandPassSectionDetails in axe-results-report.ts, based on this parameter expanded pass section will be displayed, else the existing format is displayed.

expandPassSectionDetails contains expandPassSection boolean to expand the pass section and expandByTags array of string to expand more details like html for only those rules which consist of that tag.

Report screenshot when expandPassSectionDetails is passed:

image

When expandPassSection true is passed

Screenshot 2025-08-07 142223

When expandByTags with some specific tag is passed. In this example, Experimental is passed in expandByTags parameter so all the rules having that tag shows passing components. This will be used to pass 'ai-scan' tag so that we show passing components for new rules and not axe-core rules.

Motivation

Addresses issue - https://github.com/1ES-microsoft/accessibility-insights-agent/issues/912

Context

Pull request checklist

  • Addresses an existing issue: https://github.com/1ES-microsoft/accessibility-insights-agent/issues/912
  • Ran yarn fastpass
  • Added/updated relevant unit test(s) (and ran yarn test)
  • Verified code coverage for the changes made. Check coverage report at: <rootDir>/test-results/unit/coverage
  • PR title AND final merge commit title both start with a semantic tag (fix:, chore:, feat(feature-name):, refactor:). See CONTRIBUTING.md.
  • (UI changes only) Added screenshots/GIFs to description above
  • (UI changes only) Verified usability with NVDA/JAWS

@v-viyada v-viyada marked this pull request as ready for review August 7, 2025 16:36
@v-viyada v-viyada requested a review from a team as a code owner August 7, 2025 16:36
@v-viyada v-viyada changed the title feat(ai-report): added detailed pass section for agent report feat(ai-report): added detailed pass section for report package Aug 7, 2025
@v-sharmachir v-sharmachir merged commit 2251709 into main Aug 12, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants