Skip to content

fix(audit): match meta description names case-insensitively - #297

Open
shashank-sn wants to merge 1 commit into
every-app:mainfrom
shashank-sn:codex/fix-audit-meta-description-case
Open

fix(audit): match meta description names case-insensitively#297
shashank-sn wants to merge 1 commit into
every-app:mainfrom
shashank-sn:codex/fix-audit-meta-description-case

Conversation

@shashank-sn

Copy link
Copy Markdown

Fixes #293.

Site audits treated the name value on meta description tags as case-sensitive, so valid tags such as <meta name="Description"> were reported as missing. The parser now recognizes description names regardless of capitalization while preserving content trimming and first-description-wins behavior.

Regression coverage verifies lowercase, title-case, uppercase, and mixed-case names; exact extracted content; duplicate handling; and absent or empty descriptions.

Validation:

  • pnpm run ci:check
  • pnpm run test:ci — 1,180 tests passed
  • pnpm vite build
  • pnpm --dir web run types:check
  • pnpm --dir web run build

Docker image verification was not run because the local Docker daemon is unavailable.

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.

Site audit reports a meta description as missing when it isn't

1 participant