tiennm99/miti99
1
0
Fork 0
mirror of https://github.com/tiennm99/miti99.git synced 2026-08-03 22:24:18 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(mt-add-post): preserve YouTube video id in duplicate check

Browse Source 
bareUrl() dropped the query string, collapsing every youtube.com/watch
URL to the same bare key and flagging all YouTube links as duplicates.
Keep host identity params (v for youtube) so distinct videos are not
treated as duplicates.
This commit is contained in:
tiennm99
2026-05-30 15:48:52 +07:00
parent 9e8c56af26
commit 9bf8dbbf77
1 changed files with 17 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.claude/skills/mt-add-post/scripts/prepare-url.js
+17 -2
View File
@@ -38,11 +38,26 @@ function cleanUrl(rawUrl) {
}
}
// Extract the bare URL (scheme + host + path) — used for stricter duplicate checks
// Some sites carry the resource identity in a query param, not the path
// (e.g. YouTube /watch?v=ID). Stripping the query for these collapses every
// item to the same bare URL, causing false-positive duplicates. Preserve the
// identity param for those hosts.
const IDENTITY_PARAMS = {
"youtube.com": "v",
"www.youtube.com": "v",
"m.youtube.com": "v",
};
// Extract the bare URL (scheme + host + path) — used for stricter duplicate checks.
// Keeps the host's identity query param when one is defined above.
function bareUrl(targetUrl) {
try {
const p = new URL(targetUrl);
return `${p.protocol}//${p.host}${p.pathname}`.replace(/\/$/, "");
let bare = `${p.protocol}//${p.host}${p.pathname}`.replace(/\/$/, "");
const idParam = IDENTITY_PARAMS[p.host.toLowerCase()];
const idValue = idParam ? p.searchParams.get(idParam) : null;
if (idValue) bare += `?${idParam}=${idValue}`;
return bare;
} catch {
return targetUrl.split("?")[0].replace(/\/$/, "");
}