René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Bug in beta tokenizer","articleBody":"Just looking at the code I think there is a bug here:\r\n\r\nhttps://github.com/scriptcoded/sql-highlight/blob/909c361424255f5af2a5e87b96dcd9e93e61123d/lib/index.js#L42C21-L42C21\r\n\r\nThis only matches litteral periods `.`. I assume that you meant to match any character `/(?\u003cunknown\u003e.+?)/`.\r\n\r\nWithout this change (now that it's using `.matchAll()`) anything in the input SQL string that does not match the regex will be removed from the results and not visible to the user. I assume that is undesired.\r\n\r\n(Also of note that this approach will at most match a single character in every `unknown` group match.)\r\n\r\nAnd a bit off topic, but I'm using this highlighter with a non-standard SQL flavor, which uses several additional keywords. These keywords will now be highlighted as identifiers. I hope the highlighting for those is not too disturbing. 😅","author":{"url":"https://github.com/Qtax","@type":"Person","name":"Qtax"},"datePublished":"2024-01-08T16:44:28.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":8},"url":"https://github.com/178/sql-highlight/issues/178"}