remove-markdown 0.3.0 → 0.5.2

package/.github/workflows/default.yaml

@@ -0,0 +1,27 @@
+name: Run tests
+
+on:
+  push:
+    branches: [ main ]
+  pull_request:
+    branches: [ main ]
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+
+    strategy:
+      matrix:
+        node-version: [14.x, 16.x, 18.x]
+        # See supported Node.js release schedule at https://nodejs.org/en/about/releases/
+
+    steps:
+    - uses: actions/checkout@v3
+    - name: Use Node.js ${{ matrix.node-version }}
+      uses: actions/setup-node@v3
+      with:
+        node-version: ${{ matrix.node-version }}
+        cache: 'npm'
+    - run: npm ci
+    - run: npm run build --if-present
+    - run: npm test

package/README.md

@@ -1,4 +1,4 @@
-[![CircleCI](https://circleci.com/gh/stiang/remove-markdown.svg?style=svg&circle-token=cac2feef7dc90e6b8578aec361be369412be1c6a)](https://circleci.com/gh/stiang/remove-markdown)
+![default workflow](https://github.com/stiang/remove-markdown/actions/workflows/default.yaml/badge.svg)
 
 ## What is it?
 **remove-markdown** is a node.js module that will remove (strip) Markdown formatting from text.
@@ -43,5 +43,6 @@ PRs are very much welcome. Here are some ideas for future enhancements:
 ## Credits
 The code is based on [Markdown Service Tools - Strip Markdown](http://brettterpstra.com/2013/10/18/a-markdown-service-to-strip-markdown/) by Brett Terpstra.
 
-## Author
-Stian Grytøyr
+## Authors
+Stian Grytøyr (original creator)
+[zuchka](https://github.com/zuchka) (maintainer since 2023)

package/index.js

@@ -4,11 +4,14 @@ module.exports = function(md, options) {
   options.stripListLeaders = options.hasOwnProperty('stripListLeaders') ? options.stripListLeaders : true;
   options.gfm = options.hasOwnProperty('gfm') ? options.gfm : true;
   options.useImgAltText = options.hasOwnProperty('useImgAltText') ? options.useImgAltText : true;
+  options.abbr = options.hasOwnProperty('abbr') ? options.abbr : false;
+  options.replaceLinksWithURL = options.hasOwnProperty('replaceLinksWithURL') ? options.replaceLinksWithURL : false;
+  options.htmlTagsToSkip = options.hasOwnProperty('htmlTagsToSkip') ? options.htmlTagsToSkip : [];
 
   var output = md || '';
 
   // Remove horizontal rules (stripListHeaders conflict with this rule, which is why it has been moved to the top)
-  output = output.replace(/^(-\s*?|\*\s*?|_\s*?){3,}\s*$/gm, '');
+  output = output.replace(/^(-\s*?|\*\s*?|_\s*?){3,}\s*/gm, '');
 
   try {
     if (options.stripListLeaders) {
@@ -19,7 +22,7 @@ module.exports = function(md, options) {
     }
     if (options.gfm) {
       output = output
-        // Header
+      // Header
         .replace(/\n={2,}/g, '\n')
         // Fenced codeblocks
         .replace(/~{3}.*\n/g, '')
@@ -28,9 +31,31 @@ module.exports = function(md, options) {
         // Fenced codeblocks
         .replace(/`{3}.*\n/g, '');
     }
+    if (options.abbr) {
+      // Remove abbreviations
+      output = output.replace(/\*\[.*\]:.*\n/, '');
+    }
     output = output
-      // Remove HTML tags
+    // Remove HTML tags
       .replace(/<[^>]*>/g, '')
+
+    var htmlReplaceRegex = new RegExp('<[^>]*>', 'g');
+    if (options.htmlTagsToSkip.length > 0) {
+      // Using negative lookahead. Eg. (?!sup|sub) will not match 'sup' and 'sub' tags.
+      var joinedHtmlTagsToSkip = '(?!' + options.htmlTagsToSkip.join("|") + ')';
+
+      // Adding the lookahead literal with the default regex for html. Eg./<(?!sup|sub)[^>]*>/ig
+      htmlReplaceRegex = new RegExp(
+          '<' +
+          joinedHtmlTagsToSkip +
+          '[^>]*>', 
+          'ig'
+      );
+    }
+
+    output = output
+      // Remove HTML tags
+      .replace(htmlReplaceRegex, '')
       // Remove setext-style headers
       .replace(/^[=\-]{2,}\s*$/g, '')
       // Remove footnotes?
@@ -39,22 +64,30 @@ module.exports = function(md, options) {
       // Remove images
       .replace(/\!\[(.*?)\][\[\(].*?[\]\)]/g, options.useImgAltText ? '$1' : '')
       // Remove inline links
-      .replace(/\[(.*?)\][\[\(].*?[\]\)]/g, '$1')
+      .replace(/\[([^\]]*?)\][\[\(].*?[\]\)]/g, options.replaceLinksWithURL ? '$2' : '$1')
       // Remove blockquotes
-      .replace(/^\s{0,3}>\s?/g, '')
+      .replace(/^(\n)?\s{0,3}>\s?/gm, '$1')
+      // .replace(/(^|\n)\s{0,3}>\s?/g, '\n\n')
       // Remove reference-style links?
       .replace(/^\s{1,2}\[(.*?)\]: (\S+)( ".*?")?\s*$/g, '')
       // Remove atx-style headers
-      .replace(/^(\n)?\s{0,}#{1,6}\s+| {0,}(\n)?\s{0,}#{0,} {0,}(\n)?\s{0,}$/gm, '$1$2$3')
-      // Remove emphasis (repeat the line to remove double emphasis)
-      .replace(/([\*_]{1,3})(\S.*?\S{0,1})\1/g, '$2')
-      .replace(/([\*_]{1,3})(\S.*?\S{0,1})\1/g, '$2')
+      .replace(/^(\n)?\s{0,}#{1,6}\s*( (.+))? +#+$|^(\n)?\s{0,}#{1,6}\s*( (.+))?$/gm, '$1$3$4$6')
+      // Remove * emphasis
+      .replace(/([\*]+)(\S)(.*?\S)??\1/g, '$2$3')
+      // Remove _ emphasis. Unlike *, _ emphasis gets rendered only if 
+      //   1. Either there is a whitespace character before opening _ and after closing _.
+      //   2. Or _ is at the start/end of the string.
+      .replace(/(^|\W)([_]+)(\S)(.*?\S)??\2($|\W)/g, '$1$3$4$5')
       // Remove code blocks
       .replace(/(`{3,})(.*?)\1/gm, '$2')
       // Remove inline code
       .replace(/`(.+?)`/g, '$1')
-      // Replace two or more newlines with exactly two? Not entirely sure this belongs here...
-      .replace(/\n{2,}/g, '\n\n');
+      // // Replace two or more newlines with exactly two? Not entirely sure this belongs here...
+      // .replace(/\n{2,}/g, '\n\n')
+      // // Remove newlines in a paragraph
+      // .replace(/(\S+)\n\s*(\S+)/g, '$1 $2')
+      // Replace strike through
+      .replace(/~(.*?)~/g, '$1');
   } catch(e) {
     console.error(e);
     return md;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "remove-markdown",
-  "version": "0.3.0",
+  "version": "0.5.2",
   "description": "Remove Markdown formatting from text",
   "main": "index.js",
   "scripts": {
@@ -8,7 +8,7 @@
   },
   "repository": {
     "type": "git",
-    "url": "https://github.com/stiang/remove-markdown.git"
+    "url": "https://github.com/zuchka/remove-markdown.git"
   },
   "keywords": [
     "markdown"
@@ -16,12 +16,12 @@
   "author": "Stian Grytøyr",
   "license": "MIT",
   "bugs": {
-    "url": "https://github.com/stiang/remove-markdown/issues"
+    "url": "https://github.com/zuchka/remove-markdown/issues"
   },
-  "homepage": "https://github.com/stiang/remove-markdown",
+  "homepage": "https://github.com/zuchka/remove-markdown",
   "devDependencies": {
     "chai": "^4.0.2",
-    "mocha": "^2.1.0",
+    "mocha": "^10.2.0",
     "should": "^5.0.0"
   }
 }

package/test/remove-markdown.js

@@ -69,6 +69,18 @@ describe('remove Markdown', function () {
       expect(removeMd(string)).to.equal(expected);
     });
 
+    it('should remove emphasis only if there is no space between word and emphasis characters.', function () {
+      const string = 'There should be no _space_, *before* *closing * _ephasis character _.';
+      const expected = 'There should be no space, before *closing * _ephasis character _.';
+      expect(removeMd(string)).to.equal(expected);
+    });
+
+    it('should remove "_" emphasis only if there is space before opening and after closing emphasis characters.', function () {
+      const string = '._Spaces_ _ before_ and _after _ emphasised character results in no emphasis.';
+      const expected = '.Spaces _ before_ and _after _ emphasised character results in no emphasis.';
+      expect(removeMd(string)).to.equal(expected);
+    });
+
     it('should remove double emphasis', function () {
       const string = '**this sentence has __double styling__**';
       const expected = 'this sentence has double styling';
@@ -109,18 +121,31 @@ describe('remove Markdown', function () {
             expect(removeMd(test.string)).to.equal(test.expected);
         });
     });
+    
+    it('should remove blockquotes over multiple lines', function () {
+      const string = '> I am a blockquote firstline  \n>I am a blockquote secondline';
+      const expected = 'I am a blockquote firstline  \nI am a blockquote secondline';
+      expect(removeMd(string)).to.equal(expected);
+    });
+
+    it('should remove blockquotes following other content', function () {
+      const string = '## A headline\n\nA paragraph of text\n\n> I am a blockquote';
+      const expected = 'A headline\n\nA paragraph of text\n\nI am a blockquote';
+
+      expect(removeMd(string)).to.equal(expected);
+    });
 
     it('should not remove greater than signs', function () {
       var tests = [
-          { string: '100 > 0', expected: '100 > 0' },
-          { string: '100 >= 0', expected: '100 >= 0' },
-          { string: '100>0', expected: '100>0' },
-          { string: '> 100 > 0', expected: '100 > 0' },
-          { string: '1 < 100', expected: '1 < 100' },
-          { string: '1 <= 100', expected: '1 <= 100' },
+        { string: '100 > 0', expected: '100 > 0' },
+        { string: '100 >= 0', expected: '100 >= 0' },
+        { string: '100>0', expected: '100>0' },
+        { string: '> 100 > 0', expected: '100 > 0' },
+        { string: '1 < 100', expected: '1 < 100' },
+        { string: '1 <= 100', expected: '1 <= 100' },
       ];
       tests.forEach(function (test) {
-          expect(removeMd(test.string)).to.equal(test.expected);
+        expect(removeMd(test.string)).to.equal(test.expected);
       });
     });
 
@@ -141,5 +166,34 @@ describe('remove Markdown', function () {
       const expected = '\nThis is a heading\n\nThis is a paragraph with a link.\n\nThis is another heading\n\nIn Getting Started we set up something foo.\n\n  Some list\n  With items\n    Even indented';
       expect(removeMd(paragraph)).to.equal(expected);
     });
+
+    it('should not strip paragraphs without content', function() {
+      const paragraph = '\n#This paragraph\n##This paragraph#';
+      const expected = paragraph;
+      expect(removeMd(paragraph)).to.equal(expected);
+    });
+
+    it('should not trigger ReDoS with atx-headers', function () {
+      const start = Date.now();
+
+      const paragraph = '\n## This is a long "'+' '.repeat(200)+'" heading ##\n';
+      const expected = /\nThis is a long " {200}" heading\n/;
+      expect(removeMd(paragraph)).to.match(expected);
+
+      const duration = Date.now()-start;
+      expect(duration).to.be.lt(1000);
+    });
+
+    it('should work fast even with lots of whitespace', function () {
+      const string = 'Some text with lots of                                                                                                                                                                                                       whitespace';
+      const expected = 'Some text with lots of                                                                                                                                                                                                       whitespace';
+      expect(removeMd(string)).to.equal(expected);
+    });
+
+    it('should still remove escaped markdown syntax', function () {
+      const string = '\# Heading in _italic_';
+      const expected = 'Heading in italic';
+      expect(removeMd(string)).to.equal(expected);
+    });
   });
 });

package/.circleci/config.yml

@@ -1,84 +0,0 @@
-# This configuration was automatically generated from a CircleCI 1.0 config.
-# It should include any build commands you had along with commands that CircleCI
-# inferred from your project structure. We strongly recommend you read all the
-# comments in this file to understand the structure of CircleCI 2.0, as the idiom
-# for configuration has changed substantially in 2.0 to allow arbitrary jobs rather
-# than the prescribed lifecycle of 1.0. In general, we recommend using this generated
-# configuration as a reference rather than using it in production, though in most
-# cases it should duplicate the execution of your original 1.0 config.
-version: 2
-jobs:
-  build:
-    working_directory: ~/stiang/remove-markdown
-    parallelism: 1
-    shell: /bin/bash --login
-    # CircleCI 2.0 does not support environment variables that refer to each other the same way as 1.0 did.
-    # If any of these refer to each other, rewrite them so that they don't or see https://circleci.com/docs/2.0/env-vars/#interpolating-environment-variables-to-set-other-environment-variables .
-    environment:
-      CIRCLE_ARTIFACTS: /tmp/circleci-artifacts
-      CIRCLE_TEST_REPORTS: /tmp/circleci-test-results
-    # In CircleCI 1.0 we used a pre-configured image with a large number of languages and other packages.
-    # In CircleCI 2.0 you can now specify your own image, or use one of our pre-configured images.
-    # The following configuration line tells CircleCI to use the specified docker image as the runtime environment for you job.
-    # We have selected a pre-built image that mirrors the build environment we use on
-    # the 1.0 platform, but we recommend you choose an image more tailored to the needs
-    # of each job. For more information on choosing an image (or alternatively using a
-    # VM instead of a container) see https://circleci.com/docs/2.0/executor-types/
-    # To see the list of pre-built images that CircleCI provides for most common languages see
-    # https://circleci.com/docs/2.0/circleci-images/
-    docker:
-    - image: circleci/build-image:ubuntu-14.04-XXL-upstart-1189-5614f37
-      command: /sbin/init
-    steps:
-    # Machine Setup
-    #   If you break your build into multiple jobs with workflows, you will probably want to do the parts of this that are relevant in each
-    # The following `checkout` command checks out your code to your working directory. In 1.0 we did this implicitly. In 2.0 you can choose where in the course of a job your code should be checked out.
-    - checkout
-    # Prepare for artifact and test results  collection equivalent to how it was done on 1.0.
-    # In many cases you can simplify this from what is generated here.
-    # 'See docs on artifact collection here https://circleci.com/docs/2.0/artifacts/'
-    - run: mkdir -p $CIRCLE_ARTIFACTS $CIRCLE_TEST_REPORTS
-    # Dependencies
-    #   This would typically go in either a build or a build-and-test job when using workflows
-    # Restore the dependency cache
-    - restore_cache:
-        keys:
-        # This branch if available
-        - v1-dep-{{ .Branch }}-
-        # Default branch if not
-        - v1-dep-master-
-        # Any branch if there are none on the default branch - this should be unnecessary if you have your default branch configured correctly
-        - v1-dep-
-    # The following line was run implicitly in your 1.0 builds based on what CircleCI inferred about the structure of your project. In 2.0 you need to be explicit about which commands should be run. In some cases you can discard inferred commands if they are not relevant to your project.
-    - run: if [ -z "${NODE_ENV:-}" ]; then export NODE_ENV=test; fi
-    - run: export PATH="~/stiang/remove-markdown/node_modules/.bin:$PATH"
-    - run: npm install
-    # Save dependency cache
-    - save_cache:
-        key: v1-dep-{{ .Branch }}-{{ epoch }}
-        paths:
-        # This is a broad list of cache paths to include many possible development environments
-        # You can probably delete some of these entries
-        - vendor/bundle
-        - ~/virtualenvs
-        - ~/.m2
-        - ~/.ivy2
-        - ~/.bundle
-        - ~/.go_workspace
-        - ~/.gradle
-        - ~/.cache/bower
-        - ./node_modules
-    # Test
-    #   This would typically be a build job when using workflows, possibly combined with build
-    # The following line was run implicitly in your 1.0 builds based on what CircleCI inferred about the structure of your project. In 2.0 you need to be explicit about which commands should be run. In some cases you can discard inferred commands if they are not relevant to your project.
-    - run: npm test
-    # Teardown
-    #   If you break your build into multiple jobs with workflows, you will probably want to do the parts of this that are relevant in each
-    # Save test results
-    - store_test_results:
-        path: /tmp/circleci-test-results
-    # Save artifacts
-    - store_artifacts:
-        path: /tmp/circleci-artifacts
-    - store_artifacts:
-        path: /tmp/circleci-test-results

package/.npmignore

@@ -1 +0,0 @@
-node_modules/