Release v1.46.0

[pattonwebz]

Changelog items from merges on develop since main (from scripts/prep_release.sh):

Steve Jones (6):

• fix: show success icon when no issues found in Accessibility Analysis panel
• Fix inaccurate accessible name on settings navigation landmark
• fix: convey summary status in Reading Level card accessible name

William Patton (11):

• [Backport] Release v1.45.0
• fix: strip leading v from release tag before using as SVN version
• PRO-1168: fix: don't flag .ogg audio as video content in video_present rule
• Exclude core cover blocks from aria-hidden rule and move spacer/separator to selector
• Fix: FK reading grade of 0 < x < 1 collapses to 0 (normalize to 1)
• Add @SInCE update tool and release prep integration
• Tests: add Jest coverage for 6 scanner rules without tests
• chore: update plugin URI to equalizedigital.com
• Update POT translation file
• chore: regenerate hooks docs
• PTC accessibility-checker (1985): Automatic Translations July 08, 2026 18:43:12797590

Verified: the new tools/update-since-tags.php step rewrote all @since x.x.x placeholders in files changed since v1.45.0 to 1.46.0 (commit 0f660e2); no placeholders remain in changed PHP files.

🤖 Generated with Claude Code

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 1c6f10f6-bfbf-41b0-b96b-4cd1d4209e71

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch release/1.46.0

---

_{Comment @coderabbitai help to get the list of available commands.}

[gemini-code-assist]

Code Review

This pull request bumps the plugin version to 1.46.0 and introduces Flesch-Kincaid grade level normalization to prevent simple content from being incorrectly flagged. It also updates several accessibility rules (such as excluding decorative cover blocks from aria-hidden checks and preventing Ogg audio from being flagged as video), adds comprehensive Jest and PHPUnit tests, and automates @SInCE tag updates during release preparation. The review feedback highlights a compatibility issue with mapfile -d in older Bash versions on macOS, a potential TypeError in image-input-has-alt.js if alt is undefined, and a test isolation issue in colorContrastFailure.test.js where the mocked getContext method is deleted rather than restored.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

[gemini-code-assist]

The mapfile -d option was introduced in Bash 4.4. However, macOS still ships with Bash 3.2 by default, where this option is not supported and will cause the script to fail with an invalid option error.

To ensure compatibility across development environments (especially macOS), consider using a while read loop with read -d to safely parse the NUL-delimited output.

Suggested change

	mapfile -d '' SINCE_CHANGES < <(git diff --name-only -z -- '*.php')
	SINCE_CHANGES=()
	while IFS= read -r -d '' line; do
	SINCE_CHANGES+=("$line")
	done < <(git diff --name-only -z -- '*.php')

[pattonwebz]

Good catch — applied in ab7e2ff with a while read -r -d '' loop. Worth noting it's slightly worse than stated: macOS system bash (3.2) lacks mapfile entirely, not just the -d option, so the script would have died at this step on any Mac using system bash. Verified the replacement handles paths with spaces and empty input identically.

[gemini-code-assist]

If alt is undefined (which can occur in some non-standard DOM environments or custom test mocks), alt !== null will evaluate to true, but calling alt.trim() will throw a TypeError.

To make this check more robust and defensive, verify that alt is explicitly of type string before calling .trim() on it.

Suggested change

	const alt = node.getAttribute( 'alt' );
	return alt !== null && alt.trim() !== '';
	const alt = node.getAttribute( 'alt' );
	return typeof alt === 'string' && alt.trim() !== '';

[pattonwebz]

Declining — per the DOM spec, Element.getAttribute() returns a string or null, never undefined, and that holds in every environment this check runs in: real browsers via the axe-core page scanner, and jsdom in the Jest suite. The premise ("non-standard DOM environments or custom test mocks") describes environments this code doesn't execute in, so alt !== null && alt.trim() !== '' is safe as written and the typeof guard would be defensive noise.

[gemini-code-assist]

Using delete HTMLCanvasElement.prototype.getContext completely removes the getContext method from the prototype. This can pollute the global environment and potentially break other tests running after this suite if they rely on the default JSDOM canvas implementation (which returns null or uses the canvas package).

A safer approach is to store the original getContext method in beforeAll and restore it in afterAll to ensure proper test isolation.

	let originalGetContext;
	beforeAll( async () => {
		originalGetContext = HTMLCanvasElement.prototype.getContext;
		// The color-contrast-matches built-in matcher calls _isIconLigature, which
		// needs HTMLCanvasElement.getContext. Without it, axe throws and excludes
		// every element before the check runs. Provide a minimal mock so the matcher
		// completes (equal measureText widths -> not an icon ligature -> element passes
		// through to the contrast check).
		HTMLCanvasElement.prototype.getContext = function() {
			return {
				font: '',
				measureText: ( text ) => ( { width: text.length * 8 } ),
				fillText: () => {},
				clearRect: () => {},
				fillRect: () => {},
				drawImage: () => {},
				getImageData: () => ( { data: new Uint8ClampedArray( 4 ) } ),
			};
		};

		axe.configure( { rules: [ colorContrastRule ] } );
	} );

	afterAll( () => {
		HTMLCanvasElement.prototype.getContext = originalGetContext;
		axe.reset();
	} );

[pattonwebz]

Declining — Jest gives every test file its own fresh jsdom environment, so prototype mutations cannot leak across suites, and within this file the delete runs in afterAll with nothing executing after it. There's no pollution path here. Save-and-restore is marginally tidier style and worth adopting if these suites ever share an environment, but there's no defect as things stand.

[github-actions]

✅ Accessibility Checker build (primary only)

• Artifact: Download accessibility-checker-1.46.0-1830-c811a634.zip
• Workflow run: View logs
• Playground (primary): Open with plugin preinstalled