Turns out that ~the cast to a number~ the check isItemPath in parseInternalLinks failed because the parentheses in the raw markdown were included in the match.
This PR fixes it by only matching characters that make part sense as part of the URL. Only matching /items/\d+ is not enough since we also need to match any potential commentId query parameter.
I noticed that I received no item mention notification for this: https://stacker.news/items/582653
Turns out that ~the cast to a number~ the check
isItemPath
inparseInternalLinks
failed because the parentheses in the raw markdown were included in the match.This PR fixes it by only matching characters that make part sense as part of the URL. Only matching
/items/\d+
is not enough since we also need to match any potentialcommentId
query parameter.