commonmarker 1.0.0.pre11 → 1.0.0.pre12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8625202aca416574d42bd5ca2dea12d42c3d91708bda01a9bbcfbbfc21a18cae
-  data.tar.gz: 51b1a7df2e9249f52e6ca7298035e64e3e758ecf0c71b872709c566752e7941f
+  metadata.gz: 66ae092bc1da22502edf2dccb8709f14c87128503eedf76e1c17aa348686dfe9
+  data.tar.gz: 20cebb526aa41fb1f1f98ad4958e943e5c5e6dcefcea0620732ac01553a0fd2e
 SHA512:
-  metadata.gz: baed4aa07a0f802b1ddf7ed20732983d29f1703a63d6896fa48d9352ee73aa3476387085b78b02ff69a6d4be2ae8d5cda7bf22d25b64e69885306a4a56d22c40
-  data.tar.gz: ad2af728905d0e1a80332f34142b2ab59b451dae278021e444c501f345a2d65485f2ee13063b7211cdc019a54e6107f74b04948cc61205d1e94f13855282b1dc
+  metadata.gz: 31dad33f45ec8106299d7089ff1e72971b202ab853b83f3fcb884ac640f760941575c6f7494dd13e3d1acf6a449cfc48a1d2cb0b3123b149c996824ce771142b
+  data.tar.gz: 28e6ae936f089c690fc1b18bc03ec8a1ba4e247ef5e55ecd46e37eebf2223bd8a298e096a81e46a9968ab467196ae9d5d952b5586254c2dbcbabcbd745834e84

data/Cargo.lock

@@ -226,9 +226,9 @@ dependencies = [
 
 [[package]]
 name = "comrak"
-version = "0.19.0"
+version = "0.20.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "82c995deda3bfdebd07d0e2af79e9da13e4b1be652b21a746f3f5b24bf0a49ef"
+checksum = "9f18e72341e6cdc7489cffb76f993812a14a906db54dedb020044ccc211dcaae"
 dependencies = [
  "clap",
  "derive_builder",

data/README.md

@@ -105,14 +105,26 @@ providing further niceties.
 
 #### Syntax Highlighter Plugin
 
+The library comes with [a set of pre-existing themes](https://docs.rs/syntect/5.0.0/syntect/highlighting/struct.ThemeSet.html#implementations) for highlighting code:
+
+- `"base16-ocean.dark"`
+- `"base16-eighties.dark"`
+- `"base16-mocha.dark"`
+- `"base16-ocean.light"`
+- `"InspiredGitHub"`
+- `"Solarized (dark)"`
+- `"Solarized (light)"`
+
 ````ruby
 code = <<~CODE
   ```ruby
   def hello
-  puts "hello"
+    puts "hello"
   end
+  ```
 CODE
 
+# pass in a theme name from a pre-existing set
 puts Commonmarker.to_html(code, plugins: { syntax_highlighter: { theme: "InspiredGitHub" } })
 
 # <pre style="background-color:#ffffff;" lang="ruby"><code>
@@ -123,32 +135,49 @@ puts Commonmarker.to_html(code, plugins: { syntax_highlighter: { theme: "Inspire
 # </code></pre>
 ````
 
-To disable this plugin, pass `nil`:
+By default, the plugin uses the `"base16-ocean.dark"` theme to syntax highlight code.
+
+To disable this plugin, set the value to `nil`:
+
+````ruby
+code = <<~CODE
+  ```ruby
+  def hello
+    puts "hello"
+  end
+  ```
+CODE
 
-```ruby
 Commonmarker.to_html(code, plugins: { syntax_highlighter: nil })
-# or
-Commonmarker.to_html(code, plugins: { syntax_highlighter: { theme: nil } })
-```
 
-You can also provide a `path` to a directory containing `.tmtheme` files to load:
+# <pre lang="ruby"><code>def hello
+#   puts &quot;hello&quot;
+# end
+# </code></pre>
+````
+
+To output CSS classes instead of `style` attributes, set the `theme` key to `""`:
 
-```ruby
+````ruby
+code = <<~CODE
+  ```ruby
+  def hello
+    puts "hello"
+  end
+CODE
 
-Commonmarker.to_html(code, plugins: { syntax_highlighter: { theme: "Monokai", path: "./themes" } })
-```
+Commonmarker.to_html(code, plugins: { syntax_highlighter: { theme: "" } })
 
-##### Available themes
+# <pre class="syntax-highlighting"><code><span class="source ruby"><span class="meta function ruby"><span class="keyword control def ruby">def</span></span><span class="meta function ruby"> # <span class="entity name function ruby">hello</span></span>
+#   <span class="support function builtin ruby">puts</span> <span class="string quoted double ruby"><span class="punctuation definition string begin ruby">&quot;</span>hello<span class="punctuation definition string end ruby">&quot;</span></span>
+# <span class="keyword control ruby">end</span>\n</span></code></pre>
+````
 
-Here's [a list of themes available by default](https://docs.rs/syntect/5.0.0/syntect/highlighting/struct.ThemeSet.html#implementations):
+To use a custom theme, you can provide a `path` to a directory containing `.tmtheme` files to load:
 
-- `"base16-ocean.dark"`
-- `"base16-eighties.dark"`
-- `"base16-mocha.dark"`
-- `"base16-ocean.light"`
-- `"InspiredGitHub"`
-- `"Solarized (dark)"`
-- `"Solarized (light)"`
+```ruby
+Commonmarker.to_html(code, plugins: { syntax_highlighter: { theme: "Monokai", path: "./themes" } })
+```
 
 ## Output formats
 

data/ext/commonmarker/Cargo.toml

@@ -5,7 +5,7 @@ edition = "2021"
 
 [dependencies]
 magnus =  "0.6"
-comrak = { version = "0.19", features = ["shortcodes"] }
+comrak = { version = "0.20", features = ["shortcodes"] }
 syntect = { version = "5.1", features = ["plist-load"] }
 
 [lib]

data/ext/commonmarker/src/lib.rs

@@ -18,10 +18,7 @@ use options::iterate_options_hash;
 
 mod plugins;
 use plugins::{
-    syntax_highlighting::{
-        fetch_syntax_highlighter_path, fetch_syntax_highlighter_theme,
-        SYNTAX_HIGHLIGHTER_PLUGIN_DEFAULT_THEME,
-    },
+    syntax_highlighting::{fetch_syntax_highlighter_path, fetch_syntax_highlighter_theme},
     SYNTAX_HIGHLIGHTER_PLUGIN,
 };
 
@@ -57,78 +54,83 @@ fn commonmark_to_html(args: &[Value]) -> Result<String, magnus::Error> {
 
         let theme = match rb_plugins.get(Symbol::new(SYNTAX_HIGHLIGHTER_PLUGIN)) {
             Some(syntax_highlighter_options) => {
-                fetch_syntax_highlighter_theme(syntax_highlighter_options)?
-            }
-            None => SYNTAX_HIGHLIGHTER_PLUGIN_DEFAULT_THEME.to_string(), // no `syntax_highlighter:` defined
-        };
-
-        let path = match rb_plugins.get(Symbol::new(SYNTAX_HIGHLIGHTER_PLUGIN)) {
-            Some(syntax_highlighter_options) => {
-                fetch_syntax_highlighter_path(syntax_highlighter_options)?
+                match fetch_syntax_highlighter_theme(syntax_highlighter_options) {
+                    Ok(theme) => theme,
+                    Err(e) => {
+                        return Err(e);
+                    }
+                }
             }
-            None => PathBuf::from("".to_string()), // no `syntax_highlighter:` defined
+            None => None, // no `syntax_highlighter:` defined
         };
 
-        if !path.eq(&PathBuf::from("".to_string())) && !path.exists() {
-            return Err(Error::new(
-                exception::arg_error(),
-                "path does not exist".to_string(),
-            ));
-        }
-
-        if theme.is_empty() && path.exists() {
-            return Err(Error::new(
-                exception::arg_error(),
-                "`path` also needs `theme` passed into the `syntax_highlighter`",
-            ));
-        }
-        if path.exists() && !path.is_dir() {
-            return Err(Error::new(
-                exception::arg_error(),
-                "`path` needs to be a directory",
-            ));
-        }
-
-        if path.exists() {
-            let builder = SyntectAdapterBuilder::new();
-            let mut ts = ThemeSet::load_defaults();
-
-            match ts.add_from_folder(&path) {
-                Ok(_) => {}
-                Err(e) => {
-                    return Err(Error::new(
-                        exception::arg_error(),
-                        format!("failed to load theme set from path: {e}"),
-                    ));
+        match theme {
+            None => syntax_highlighter = None,
+            Some(theme) => {
+                if theme.is_empty() {
+                    // no theme? uss css classes
+                    adapter = SyntectAdapter::new(None);
+                    syntax_highlighter = Some(&adapter);
+                } else {
+                    let path = match rb_plugins.get(Symbol::new(SYNTAX_HIGHLIGHTER_PLUGIN)) {
+                        Some(syntax_highlighter_options) => {
+                            fetch_syntax_highlighter_path(syntax_highlighter_options)?
+                        }
+                        None => PathBuf::from("".to_string()), // no `syntax_highlighter:` defined
+                    };
+
+                    if path.exists() {
+                        if !path.is_dir() {
+                            return Err(Error::new(
+                                exception::arg_error(),
+                                "`path` needs to be a directory",
+                            ));
+                        }
+
+                        let builder = SyntectAdapterBuilder::new();
+                        let mut ts = ThemeSet::load_defaults();
+
+                        match ts.add_from_folder(&path) {
+                            Ok(_) => {}
+                            Err(e) => {
+                                return Err(Error::new(
+                                    exception::arg_error(),
+                                    format!("failed to load theme set from path: {e}"),
+                                ));
+                            }
+                        }
+
+                        // check if the theme exists in the dir
+                        match ts.themes.get(&theme) {
+                            Some(theme) => theme,
+                            None => {
+                                return Err(Error::new(
+                                    exception::arg_error(),
+                                    format!("theme `{}` does not exist", theme),
+                                ));
+                            }
+                        };
+
+                        adapter = builder.theme_set(ts).theme(&theme).build();
+
+                        syntax_highlighter = Some(&adapter);
+                    } else {
+                        // no path? default theme lookup
+                        ThemeSet::load_defaults()
+                            .themes
+                            .get(&theme)
+                            .ok_or_else(|| {
+                                Error::new(
+                                    exception::arg_error(),
+                                    format!("theme `{}` does not exist", theme),
+                                )
+                            })?;
+                        adapter = SyntectAdapter::new(Some(&theme));
+                        syntax_highlighter = Some(&adapter);
+                    }
                 }
             }
-
-            ts.themes.get(&theme).ok_or_else(|| {
-                Error::new(
-                    exception::arg_error(),
-                    format!("theme `{}` does not exist", theme),
-                )
-            })?;
-
-            adapter = builder.theme_set(ts).theme(&theme).build();
-
-            syntax_highlighter = Some(&adapter);
-        } else if theme.is_empty() || theme == "none" {
-            syntax_highlighter = None;
-        } else {
-            ThemeSet::load_defaults()
-                .themes
-                .get(&theme)
-                .ok_or_else(|| {
-                    Error::new(
-                        exception::arg_error(),
-                        format!("theme `{}` does not exist", theme),
-                    )
-                })?;
-            adapter = SyntectAdapter::new(&theme);
-            syntax_highlighter = Some(&adapter);
         }
-
         comrak_plugins.render.codefence_syntax_highlighter = syntax_highlighter;
 
         Ok(markdown_to_html_with_plugins(

data/ext/commonmarker/src/plugins/syntax_highlighting.rs

@@ -7,28 +7,43 @@ use crate::EMPTY_STR;
 
 pub const SYNTAX_HIGHLIGHTER_PLUGIN_THEME_KEY: &str = "theme";
 pub const SYNTAX_HIGHLIGHTER_PLUGIN_PATH_KEY: &str = "path";
-pub const SYNTAX_HIGHLIGHTER_PLUGIN_DEFAULT_THEME: &str = "base16-ocean.dark";
 
-pub fn fetch_syntax_highlighter_theme(value: Value) -> Result<String, magnus::Error> {
+pub fn fetch_syntax_highlighter_theme(value: Value) -> Result<Option<String>, magnus::Error> {
     if value.is_nil() {
         // `syntax_highlighter: nil`
-        return Ok(EMPTY_STR.to_string());
+        return Ok(None);
+    }
+
+    let syntax_highlighter_plugin: RHash = match TryConvert::try_convert(value) {
+        Ok(plugin) => plugin, // `syntax_highlighter: { theme: "<something>" }`
+        Err(e) => {
+            // not a hash!
+            return Err(e);
+        }
+    };
+
+    if syntax_highlighter_plugin.is_nil() || syntax_highlighter_plugin.is_empty() {
+        return Err(magnus::Error::new(
+            magnus::exception::type_error(),
+            "theme cannot be blank hash",
+        ));
     }
 
-    let syntax_highlighter_plugin: RHash = TryConvert::try_convert(value)?;
     let theme_key = Symbol::new(SYNTAX_HIGHLIGHTER_PLUGIN_THEME_KEY);
 
     match syntax_highlighter_plugin.get(theme_key) {
         Some(theme) => {
             if theme.is_nil() {
-                // `syntax_highlighter: { theme: nil }`
-                return Ok(EMPTY_STR.to_string());
+                return Err(magnus::Error::new(
+                    magnus::exception::type_error(),
+                    "theme cannot be nil",
+                ));
             }
-            Ok(theme.try_convert::<String>()?)
+            Ok(TryConvert::try_convert(theme)?)
         }
         None => {
-            // `syntax_highlighter: {  }`
-            Ok(EMPTY_STR.to_string())
+            // `syntax_highlighter: { theme: nil }`
+            Ok(None)
         }
     }
 }

data/lib/commonmarker/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Commonmarker
-  VERSION = "1.0.0.pre11"
+  VERSION = "1.0.0.pre12"
 end

metadata

@@ -1,15 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: commonmarker
 version: !ruby/object:Gem::Version
-  version: 1.0.0.pre11
+  version: 1.0.0.pre12
 platform: ruby
 authors:
 - Garen Torikian
 - Ashe Connor
-autorequire:
+autorequire: 
 bindir: exe
 cert_chain: []
-date: 2023-11-13 00:00:00.000000000 Z
+date: 2023-12-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rb_sys
@@ -56,7 +56,7 @@ dependencies:
         version: '1.2'
 description: A fast, safe, extensible parser for CommonMark. This wraps the comrak
   Rust crate.
-email:
+email: 
 executables: []
 extensions:
 - ext/commonmarker/extconf.rb
@@ -88,7 +88,7 @@ metadata:
   funding_uri: https://github.com/sponsors/gjtorikian/
   source_code_uri: https://github.com/gjtorikian/commonmarker
   rubygems_mfa_required: 'true'
-post_install_message:
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -103,8 +103,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: 3.3.22
 requirements: []
-rubygems_version: 3.4.21
-signing_key:
+rubygems_version: 3.4.22
+signing_key: 
 specification_version: 4
 summary: CommonMark parser and renderer. Written in Rust, wrapped in Ruby.
 test_files: []