Skip to content

Fix review feedback: duplicate H1, back-to-top.ts ref, ****** stripping, singular/plural#1084

Merged
pethers merged 2 commits intonews/deep-inspection-2026-03-12-betald-polisutbildning-1686e6c8ca78187bfrom
copilot/sub-pr-1080
Mar 12, 2026
Merged

Fix review feedback: duplicate H1, back-to-top.ts ref, ****** stripping, singular/plural#1084
pethers merged 2 commits intonews/deep-inspection-2026-03-12-betald-polisutbildning-1686e6c8ca78187bfrom
copilot/sub-pr-1080

Conversation

Copy link
Contributor

Copilot AI commented Mar 12, 2026

Addresses review comments from PR #1080 review thread #3936161887.

HTML fixes (EN + SV deep-inspection articles)

  • Duplicate <h1>: Removed second <h1> after site tagline; added aria-hidden="true" to duplicate tagline div — matches pattern in other generated pages
  • back-to-top.ts.js: GitHub Pages won't serve .ts with correct MIME type; switched to compiled ../js/back-to-top.js consistent with other articles
  • Singular/plural: "1 of 1 documents""1 of 1 document"

Auth token fix (scripts/mcp-client/client.ts)

getDefaultAuthToken() stripped legacy ****** prefix from env vars and Authorizationheader but not fromgateway.apiKey`. Added the same normalization:

// Before
if (apiKey) return apiKey;

// After
if (apiKey) return apiKey.replace(/^Bearer\s+/i, '');

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

…ip Bearer from gateway.apiKey, fix singular/plural

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>
Copilot AI changed the title [WIP] Update deep-inspection for paid police education proposal Fix review feedback: duplicate H1, back-to-top.ts ref, ****** stripping, singular/plural Mar 12, 2026
@pethers pethers marked this pull request as ready for review March 12, 2026 12:51
@pethers pethers merged commit 182aa0b into news/deep-inspection-2026-03-12-betald-polisutbildning-1686e6c8ca78187b Mar 12, 2026
@pethers pethers deleted the copilot/sub-pr-1080 branch March 12, 2026 12:52
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