fix(bash): arguments highlight

Author: tuyen-at-work

CHANGES.md

@@ -26,6 +26,7 @@ Core Grammars:
 - enh(json) add json5 support [Kerry Shetline][]
 - fix(css) `unicode-range` parsing, issue #4253 [Kerry Shetline][]
 - fix(csharp) Support digit separators [te-ing][]
+- fix(bash) arguments highlight for Bash/Shell, issue #4288 [tuyen-at-work][]
 
 Documentation:
 
@@ -55,6 +56,7 @@ CONTRIBUTORS
 [te-ing]: https://github.com/te-ing
 [Anthony Martin]: https://github.com/anthony-c-martin
 [NriotHrreion]: https://github.com/NriotHrreion
+[tuyen-at-work]: https://github.com/tuyen-at-work
 
 
 ## Version 11.11.1

src/languages/bash.js

@@ -117,6 +117,16 @@ export default function(hljs) {
     relevance: 0
   };
 
+  const BASH_ARGUMENTS = {
+    variants: [
+      {
+        className: "attr",
+        begin: /[\t\u0020](-){1,2}[\w\d-]+/,
+        relevance: 0,
+      },
+    ],
+  };
+
   const KEYWORDS = [
     "if",
     "then",
@@ -392,6 +402,7 @@ export default function(hljs) {
     contains: [
       KNOWN_SHEBANG, // to catch known shells and boost relevancy
       hljs.SHEBANG(), // to catch unknown shells but still highlight the shebang
+      BASH_ARGUMENTS,
       FUNCTION,
       ARITHMETIC,
       COMMENT,

test/markup/bash/escaped-quote.expect.txt

@@ -1,2 +1,2 @@
 <span class="hljs-comment"># Escaped double-quote is not a string</span>
-<span class="hljs-built_in">echo</span> <span class="hljs-string">&#x27;&quot;quoted&quot;&#x27;</span> | <span class="hljs-built_in">tr</span> -d \&quot; &gt; text.txt
+<span class="hljs-built_in">echo</span> <span class="hljs-string">&#x27;&quot;quoted&quot;&#x27;</span> | <span class="hljs-built_in">tr</span><span class="hljs-attr"> -d</span> \&quot; &gt; text.txt

test/markup/bash/no-numbers.expect.txt

@@ -1,3 +1,3 @@
 <span class="hljs-comment"># numbers aren&#x27;t highlighted in bash as their semantics is</span>
 <span class="hljs-comment"># not strictly defined for command line parameters</span>
-$ <span class="hljs-built_in">tail</span> -10 access.log
+$ <span class="hljs-built_in">tail</span><span class="hljs-attr"> -10</span> access.log

test/markup/bash/strings.expect.txt

@@ -2,11 +2,11 @@ SCRIPT_DIR=<span class="hljs-string">&quot;<span class="hljs-subst">$( cd <span
 TLS_DIR=<span class="hljs-string">&quot;<span class="hljs-variable">$SCRIPT_DIR</span>/../src/main/resources/tls&quot;</span>
 ROOT_DIR=<span class="hljs-string">&quot;<span class="hljs-variable">$SCRIPT_DIR</span>/..&quot;</span>
 
-jshell -s - &lt;&lt; <span class="hljs-string">EOF
+jshell<span class="hljs-attr"> -s</span> - &lt;&lt; <span class="hljs-string">EOF
 System.out.printf(&quot;Procs: %s%n&quot;, getdata())
 EOF</span>
 
-jshell -s - &lt;&lt;&lt;<span class="hljs-string">&#x27;System.out.printf(&quot;Procs: %s%n&quot;, getdata())&#x27;</span>
+jshell<span class="hljs-attr"> -s</span> - &lt;&lt;&lt;<span class="hljs-string">&#x27;System.out.printf(&quot;Procs: %s%n&quot;, getdata())&#x27;</span>
 
 <span class="hljs-built_in">cat</span> &lt;&lt;&lt; <span class="hljs-string">&#x27;$VARIABLE&#x27;</span>
 <span class="hljs-built_in">cat</span> &lt;&lt;&lt; <span class="hljs-string">&quot;<span class="hljs-variable">$VARIABLE</span>&quot;</span>

test/markup/bash/token-containing-keyword.expect.txt

@@ -1,5 +1,5 @@
 <span class="hljs-comment"># a keyword as part of an option</span>
-mycmd --disable-foo
+mycmd<span class="hljs-attr"> --disable-foo</span>
 
 <span class="hljs-comment"># a keyword as part of a parameter</span>
 some-cmd set-some-setting

test/markup/dockerfile/default.expect.txt

@@ -7,7 +7,7 @@
 <span class="hljs-keyword">COPY</span><span class="language-bash"> www.conf /etc/php5/fpm/pool.d/</span>
 
 <span class="hljs-keyword">RUN</span><span class="language-bash"> apt-get update \
-    &amp;&amp; apt-get install -y php5-fpm php-apc php5-curl php5-gd php5-intl php5-mysql</span>
+    &amp;&amp; apt-get install<span class="hljs-attr"> -y</span> php5-fpm php-apc php5-curl php5-gd php5-intl php5-mysql</span>
 <span class="hljs-keyword">RUN</span><span class="language-bash"> <span class="hljs-built_in">mkdir</span> /tmp/sessions</span>
 
 <span class="hljs-keyword">ENV</span> APPLICATION_ENV dev

test/markup/shell/command-continuation.expect.txt

@@ -1,14 +1,14 @@
 <span class="hljs-meta prompt_">$ </span><span class="language-bash">docker run \
-      --publish=7474:7474 --publish=7687:7687 \
-      --volume=/neo4j/data:/data \
-      --volume=/neo4j/plugins:/plugins \
-      --volume=/neo4j/conf:/conf \
-      --volume=/logs/neo4j:/logs \
-      --user=<span class="hljs-string">&quot;<span class="hljs-subst">$(id -u neo4j)</span>:<span class="hljs-subst">$(id -g neo4j)</span>&quot;</span> \
-      --group-add=<span class="hljs-variable">$groups</span> \
+     <span class="hljs-attr"> --publish</span>=7474:7474<span class="hljs-attr"> --publish</span>=7687:7687 \
+     <span class="hljs-attr"> --volume</span>=/neo4j/data:/data \
+     <span class="hljs-attr"> --volume</span>=/neo4j/plugins:/plugins \
+     <span class="hljs-attr"> --volume</span>=/neo4j/conf:/conf \
+     <span class="hljs-attr"> --volume</span>=/logs/neo4j:/logs \
+     <span class="hljs-attr"> --user</span>=<span class="hljs-string">&quot;<span class="hljs-subst">$(id -u neo4j)</span>:<span class="hljs-subst">$(id -g neo4j)</span>&quot;</span> \
+     <span class="hljs-attr"> --group-add</span>=<span class="hljs-variable">$groups</span> \
       neo4j:3.4</span>
 <span class="hljs-meta prompt_">&gt; </span><span class="language-bash">/bin/cat \.travis.yml\
- -b | <span class="hljs-built_in">head</span> -n1</span>
+<span class="hljs-attr"> -b</span> | <span class="hljs-built_in">head</span><span class="hljs-attr"> -n1</span></span>
      1	language: node_js
 <span class="hljs-meta prompt_">&gt; </span><span class="language-bash"><span class="hljs-built_in">echo</span> <span class="hljs-string">&#x27;hello&#x27;</span></span>
 hello

test/markup/shell/single.expect.txt

@@ -1 +1 @@
-<span class="hljs-meta prompt_">$ </span><span class="language-bash"><span class="hljs-built_in">echo</span> <span class="hljs-string">&quot;<span class="hljs-variable">$HOME</span>&quot;</span> -n</span>
+<span class="hljs-meta prompt_">$ </span><span class="language-bash"><span class="hljs-built_in">echo</span> <span class="hljs-string">&quot;<span class="hljs-variable">$HOME</span>&quot;</span><span class="hljs-attr"> -n</span></span>