diff --git a/src/utils/url.test.ts b/src/utils/url.test.ts index b8f9f2ac..44f34a2d 100644 --- a/src/utils/url.test.ts +++ b/src/utils/url.test.ts @@ -3,6 +3,7 @@ import { invoke } from '@tauri-apps/api/core' import { revealItemInDir } from '@tauri-apps/plugin-opener' import { copyLocalPath, + isUrlValue, normalizeExternalUrl, openExternalUrl, openLocalFile, @@ -24,10 +25,17 @@ function setClipboard(writeText: (value: string) => Promise) { describe('normalizeExternalUrl', () => { it('keeps valid http URLs and normalizes bare domains', () => { + expect(normalizeExternalUrl('https://example.com')).toBe('https://example.com') expect(normalizeExternalUrl('https://example.com/docs')).toBe('https://example.com/docs') expect(normalizeExternalUrl('example.com/docs')).toBe('https://example.com/docs') }) + it('rejects pure numeric values instead of treating them as bare domains', () => { + expect(normalizeExternalUrl('2026')).toBeNull() + expect(normalizeExternalUrl('0')).toBeNull() + expect(isUrlValue('2026')).toBe(false) + }) + it('rejects malformed or unsupported URLs', () => { expect(normalizeExternalUrl('https://exa mple.com')).toBeNull() expect(normalizeExternalUrl('javascript:alert(1)')).toBeNull() diff --git a/src/utils/url.ts b/src/utils/url.ts index 83f63cc6..8c667276 100644 --- a/src/utils/url.ts +++ b/src/utils/url.ts @@ -28,6 +28,7 @@ export function normalizeExternalUrl(value: string): string | null { } if (parseHttpUrl(trimmed)) return trimmed + if (!trimmed.includes('.')) return null const bareDomainCandidate = `https://${trimmed}` const parsedBareDomain = parseHttpUrl(bareDomainCandidate)