Skip to content

fix: add missing whitespace before import keyword in code previews #432

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
danielroe merged 5 commits into from
Jan 30, 2026
Merged

fix: add missing whitespace before import keyword in code previews #432

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
05afb94
fix: add missing whitespace before `import` keyword in code previews
mikouaji Jan 30, 2026
3c492b3
[autofix.ci] apply automated fixes
autofix-ci[bot] Jan 30, 2026
6a9d7a2
fix: bump cache version
mikouaji Jan 30, 2026
bf311ce
Merge branch 'main' into fix/missing-whitespace-before-import
mikouaji Jan 30, 2026
227e1ba
Merge branch 'main' into fix/missing-whitespace-before-import
mikouaji Jan 30, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion server/api/registry/file/[...pkg].get.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ import {
ERROR_PACKAGE_VERSION_AND_FILE_FAILED,
} from '#shared/utils/constants'

const CACHE_VERSION = 2
const CACHE_VERSION = 3

// Maximum file size to fetch and highlight (500KB)
const MAX_FILE_SIZE = 500 * 1024
Expand Down
15 changes: 12 additions & 3 deletions server/utils/code-highlight.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -220,11 +220,20 @@ function linkifyImports(html: string, options?: LinkifyOptions): string {
// or: <span>import</span><span>(</span><span>'module'</span>
// Note: require often has a leading space in the span from Shiki
result = result.replace(
/(<span[^>]*>)\s*(require|import)(<\/span>)(<span[^>]*>\(<\/span>)(<span[^>]*>)(['"][^'"]+['"])<\/span>/g,
(match, spanOpen, keyword, spanClose, parenSpan, stringSpanOpen, moduleSpecifier) => {
/(<span[^>]*>)(\s*)(require|import)(<\/span>)(<span[^>]*>\(<\/span>)(<span[^>]*>)(['"][^'"]+['"])<\/span>/g,
(
match,
spanOpen,
whitespace,
keyword,
spanClose,
parenSpan,
stringSpanOpen,
moduleSpecifier,
) => {
const href = getHref(moduleSpecifier)
if (!href) return match
return `${spanOpen}${keyword}${spanClose}${parenSpan}${stringSpanOpen}<a href="${href}" class="import-link">${moduleSpecifier}</a></span>`
return `${spanOpen}${whitespace}${keyword}${spanClose}${parenSpan}${stringSpanOpen}<a href="${href}" class="import-link">${moduleSpecifier}</a></span>`
},
)

Expand Down
Loading