chore: merge develop, address PR review comments

- Resolve merge conflict in prompt-box-files.html (merge rendering + a11y changes)
- Remove tracked JSON report files (aria-validation-report.json, wcag-compliance-report.json)
- Fix .gitignore: only ignore JSON reports in tests/_output/, not the whole directory
- Simplify CI: remove unused affected-only input from _test-a11y-specs.yml
- Simplify CI: remove infoResults from set-commit-status (ci-a11y-specs reports its own status)

Author: dtopalov

.github/workflows/_test-a11y-specs.yml

@@ -2,11 +2,6 @@ name: Test A11y Specs
 
 on:
   workflow_call:
-    inputs:
-      affected-only:
-        required: false
-        type: boolean
-        default: true
 
 concurrency:
   group: test-a11y-specs-${{ github.ref }}
@@ -56,37 +51,25 @@ jobs:
       - name: Build HTML package
         run: npm run build --prefix packages/html
 
-      - name: Run unified A11y tests (affected only)
+      - name: Run unified A11y tests (affected components)
         id: a11y-tests
-        if: ${{ inputs.affected-only }}
         continue-on-error: true
         run: |
           xvfb-run --auto-servernum --server-args="-screen 0, 1366x768x24" npm run test:a11y:unified -- --affected --verbose 2>&1 | tee a11y-test-output.txt
 
-      - name: Run unified A11y tests (all components)
-        id: a11y-tests-all
-        if: ${{ !inputs.affected-only }}
-        continue-on-error: true
-        run: |
-          xvfb-run --auto-servernum --server-args="-screen 0, 1366x768x24" npm run test:a11y:unified -- --verbose 2>&1 | tee a11y-test-output.txt
-
       - name: Report test results
         run: |
           echo "## A11y Spec Test Results" >> $GITHUB_STEP_SUMMARY
           echo "" >> $GITHUB_STEP_SUMMARY
 
-          OUTCOME="${{ steps.a11y-tests.outcome || steps.a11y-tests-all.outcome }}"
-
-          if [ "$OUTCOME" == "success" ]; then
+          if [ "${{ steps.a11y-tests.outcome }}" == "success" ]; then
             echo ":white_check_mark: Unified A11y tests passed (ARIA + WCAG)" >> $GITHUB_STEP_SUMMARY
           else
             echo ":x: A11y tests failed" >> $GITHUB_STEP_SUMMARY
           fi
 
-          if [ "${{ inputs.affected-only }}" == "true" ]; then
-            echo "" >> $GITHUB_STEP_SUMMARY
-            echo "_Tested affected components only for faster CI._" >> $GITHUB_STEP_SUMMARY
-          fi
+          echo "" >> $GITHUB_STEP_SUMMARY
+          echo "_Tested affected components only for faster CI._" >> $GITHUB_STEP_SUMMARY
 
           echo "" >> $GITHUB_STEP_SUMMARY
           echo "### Test Output" >> $GITHUB_STEP_SUMMARY
@@ -100,8 +83,7 @@ jobs:
 
       - name: Check for failures
         run: |
-          OUTCOME="${{ steps.a11y-tests.outcome || steps.a11y-tests-all.outcome }}"
-          if [ "$OUTCOME" == "failure" ]; then
+          if [ "${{ steps.a11y-tests.outcome }}" == "failure" ]; then
             echo "A11y spec tests failed. See summary above for details."
             exit 1
           fi

.github/workflows/ci.yml

@@ -97,7 +97,7 @@ jobs:
   set-commit-status:
     name: Set commit status
     runs-on: ubuntu-latest
-    needs: [ detect-changes, ci-lint-scripts, ci-lint-styles, ci-docs, ci-test-units, ci-test-integrations, ci-a11y, ci-a11y-specs, ci-test-html-spec ]
+    needs: [ detect-changes, ci-lint-scripts, ci-lint-styles, ci-docs, ci-test-units, ci-test-integrations, ci-a11y, ci-test-html-spec ]
     timeout-minutes: 30
     if: ${{ always() && needs.detect-changes.outputs.snapshot_sha }}
 
@@ -121,12 +121,6 @@ jobs:
               ['success', 'Status check > Visual']
             ];
 
-            // Non-blocking checks - reported with actual status but don't prevent merging
-            // (configure as non-required in GitHub repo settings)
-            const infoResults = [
-              ['${{ needs.ci-a11y-specs.result }}', 'Status check > A11y Specs (non-blocking)']
-            ];
-
             const mapState = (r) => {
               if (r === 'success') return 'success';
               if (r === 'cancelled' || r === 'skipped') return 'failure';
@@ -136,30 +130,17 @@ jobs:
             const sha = '${{ needs.detect-changes.outputs.snapshot_sha }}';
             const targetUrl = `${process.env.GITHUB_SERVER_URL}/${process.env.GITHUB_REPOSITORY}/actions/runs/${process.env.GITHUB_RUN_ID}`;
 
-            await Promise.all([
-              ...results.map(([result, contextName]) =>
-                github.rest.repos.createCommitStatus({
-                  owner: context.repo.owner,
-                  repo: context.repo.repo,
-                  sha,
-                  context: contextName,
-                  state: mapState(result),
-                  target_url: targetUrl,
-                  description: 'Automated visual snapshot commit'
-                })
-              ),
-              ...infoResults.map(([result, contextName]) =>
-                github.rest.repos.createCommitStatus({
-                  owner: context.repo.owner,
-                  repo: context.repo.repo,
-                  sha,
-                  context: contextName,
-                  state: mapState(result),
-                  target_url: targetUrl,
-                  description: result === 'success' ? 'All a11y spec tests passed' : 'A11y spec tests failed (non-blocking)'
-                })
-              )
-            ]);
+            await Promise.all(results.map(([result, contextName]) =>
+              github.rest.repos.createCommitStatus({
+                owner: context.repo.owner,
+                repo: context.repo.repo,
+                sha,
+                context: contextName,
+                state: mapState(result),
+                target_url: targetUrl,
+                description: 'Automated visual snapshot commit'
+              })
+            ));
 
   ci-visual:
     name: Status check > Visual

.gitignore

@@ -21,5 +21,5 @@ debug.log
 .nx/cache
 .nx/workspace-data
 
-# Generated test output
-tests/_output/
+# Generated a11y test reports (not tracked; screenshots in tests/_output/ ARE tracked)
+tests/_output/*.json

packages/html/src/prompt-box/tests/prompt-box-files.tsx

@@ -19,7 +19,7 @@ export default () =>(
                 <PromptBoxLineModeMulti
                     placeholder="Type a message"
                     endAffix={<>
-                        <IconButton icon="paperclip-outline-alt-right" size="small" fillMode="flat" />
+                        <IconButton icon="paperclip-outline-alt-right" size="small" fillMode="flat" rounded="full" />
                         <div className="k-spacer"></div>
                     </>}
                 />