fix: rm terminalLink.isSupported checks

They're not needed - the terminal-link package checks this internally and formats the URL using parentheses and zero-width spaces. As it is the logic is dropping the URL in terminals like VSCode's/Cursor's.

Author: mmkal

source/util.js

@@ -34,7 +34,7 @@ const _npRootDirectory = fileURLToPath(new URL('..', import.meta.url));
 export const {package_: npPackage, rootDirectory: npRootDirectory} = await readPackage(_npRootDirectory);
 
 export const linkifyIssues = (url, message) => {
-	if (!(url && terminalLink.isSupported)) {
+	if (!url) {
 		return message;
 	}
 
@@ -50,15 +50,15 @@ export const linkifyIssues = (url, message) => {
 };
 
 export const linkifyCommit = (url, commit) => {
-	if (!(url && terminalLink.isSupported)) {
+	if (!url) {
 		return commit;
 	}
 
 	return terminalLink(commit, `${url}/commit/${commit}`);
 };
 
 export const linkifyCommitRange = (url, commitRange) => {
-	if (!(url && terminalLink.isSupported)) {
+	if (!url) {
 		return commitRange;
 	}
 

test/util/hyperlinks.js

@@ -36,7 +36,7 @@ test('linkifyIssues returns raw message if terminalLink is not supported', verif
 	linksSupported: false,
 }, ({t, util: {linkifyIssues}}) => {
 	const message = 'Commit message - fixes #6';
-	t.is(linkifyIssues(MOCK_REPO_URL, message), message);
+	t.is(linkifyIssues(MOCK_REPO_URL, message), `${message} (\u200B${MOCK_REPO_URL}/issues/6\u200B)`);
 });
 
 test('linkifyCommit correctly links commits', verifyLinks, {
@@ -54,7 +54,7 @@ test('linkifyCommit returns raw commit hash if url is not provided', verifyLinks
 test('linkifyCommit returns raw commit hash if terminalLink is not supported', verifyLinks, {
 	linksSupported: false,
 }, ({t, util: {linkifyCommit}}) => {
-	t.is(linkifyCommit(MOCK_REPO_URL, MOCK_COMMIT_HASH), MOCK_COMMIT_HASH);
+	t.is(linkifyCommit(MOCK_REPO_URL, MOCK_COMMIT_HASH), `${MOCK_COMMIT_HASH} (\u200B${MOCK_REPO_URL}/commit/${MOCK_COMMIT_HASH}\u200B)`);
 });
 
 test('linkifyCommitRange returns raw commitRange if url is not provided', verifyLinks, {
@@ -66,7 +66,7 @@ test('linkifyCommitRange returns raw commitRange if url is not provided', verify
 test('linkifyCommitRange returns raw commitRange if terminalLink is not supported', verifyLinks, {
 	linksSupported: false,
 }, ({t, util: {linkifyCommitRange}}) => {
-	t.is(linkifyCommitRange(MOCK_REPO_URL, MOCK_COMMIT_RANGE), MOCK_COMMIT_RANGE);
+	t.is(linkifyCommitRange(MOCK_REPO_URL, MOCK_COMMIT_RANGE), `${MOCK_COMMIT_RANGE} (\u200B${MOCK_REPO_URL}/compare/${MOCK_COMMIT_RANGE}\u200B)`);
 });
 
 test('linkifyCommitRange correctly links commit range', verifyLinks, {