fix: hyperlink truncation edge case

Author: pchiwan

package.json

@@ -1,6 +1,6 @@
 {
   "name": "markdown-truncate",
-  "version": "1.0.2",
+  "version": "1.0.3",
   "description": "A zero-dependency, vanilla JavaScript utility to truncate markdown text.",
   "main": "lib/index.js",
   "scripts": {

src/index.js

@@ -118,7 +118,7 @@ const truncate = (text, limit, ellipsis) => {
 
   let outputText = truncateString(text);
 
-  if (ellipsis) {
+  if (ellipsis && outputText.length < text.length) {
     outputText += '...';
   }
 
@@ -128,7 +128,7 @@ const truncate = (text, limit, ellipsis) => {
 module.exports = function (text = '', options = {}) {
   const { limit, ellipsis } = options || {}
 
- if (isNaN(parseInt(limit, 10)) || text.length < limit) {
+ if (isNaN(parseInt(limit, 10)) || text.length <= limit) {
     return text;
   }
 

src/index.test.js

@@ -54,6 +54,11 @@ describe('truncateMarkdown test suite', () => {
       'a [hyperlink is](https://google.com)...',
       { limit: 15, ellipsis: true },
     ],
+    [
+      'in hyperlinks [only the label counts towards truncate limit](https://google.com)',
+      'in hyperlinks [only the label counts towards truncate limit](https://google.com)',
+      { limit: 58, ellipsis: true },
+    ],
     ['this www.google.com is also properly truncated', 'this www.goo...', { limit: 12, ellipsis: true }],
     [
       '"quotes" and other special <characters> remain unchanged',