Skip to content

Add advanced alt-text demo fixture - #63

Draft
taarikashenafi wants to merge 1 commit into
mainfrom
taarikashenafi-prepare-scanner-demo
Draft

Add advanced alt-text demo fixture#63
taarikashenafi wants to merge 1 commit into
mainfrom
taarikashenafi-prepare-scanner-demo

Conversation

@taarikashenafi

Copy link
Copy Markdown
Contributor

Summary

  • add a synthetic advanced fixture for keyword stuffing, contextually inaccurate alt text, and an accurate control
  • add a credential-free demo:verify command that emits machine-readable scanner-shaped findings with mocked model verdicts clearly labeled
  • add a credential-gated demo:live command that scans only the synthetic fixture with GitHub Models and optional Azure Vision enrichment
  • document exact credentials, modes, and demo commands

Technical boundaries

  • the five deterministic rules remain enabled and credential-free by default
  • keyword stuffing and generated remediation remain part of the opt-in alt-text-quality model rule
  • mocked output replaces only the external model response; extraction, rule mapping, and Finding emission use production code
  • no GitHub Models or Azure calls were made while preparing this PR

Validation

  • npm test (164 tests)
  • npm run typecheck
  • npm run build
  • npm run lint
  • npm run format:check
  • npm run demo:verify
  • verified npm run demo:live refuses to start without a model token

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 40132dd1-60d2-4441-ab8b-e8c5dc8e049e
Comment thread scripts/demo-support.ts
response.end(contents)
} catch (error) {
response.writeHead(500, {'Content-Type': 'text/plain; charset=utf-8'})
response.end(error instanceof Error ? error.message : String(error))
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