jekyll-kroki 0.5.0 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 39c2c3f9463c5788864d00c4ed80e9aa826a1aa1d1380dbff2be92407958f0e5
-  data.tar.gz: 38340a5a1deaa705a0b7cde596e5eb007da3d84526b2641e9cad6e9f38e86809
+  metadata.gz: 4f2efa9a6bac231d6efdac15ccf493e387cdb1a9890f1b774ddf839b8fb0b599
+  data.tar.gz: f51d26aa98904cd284f8bea24113f6225fdc5511651c0b7a1d432898f47397fa
 SHA512:
-  metadata.gz: dbedea8bbf380c047c91cee38bc7fade0c9a6a0e3aa8735b7a18e3361219cb796fb3b89b17583db402f0b03bf083f13892d53a7677cc904d12f6be6990bfee98
-  data.tar.gz: 6c3cf11ca513c6b16331547bcd69003b77dd62c3a09b46c52e440684a5001f8c47318962037faa3a0bce5cf1e99c2dfe4a2f5f4e00f4bc0eb893ef0fb174efa6
+  metadata.gz: 1b4685d93410a2d8ea9ce9a3c97049f28ea2d4fdfcea144357ae388e0c59af9d93fdadb8c872ea80d7b48510358bd9cfedd980ae72cdfef7f586ce6ac49d915b
+  data.tar.gz: 3158b7e07f81b144d249161984b0874c48a383187a3c82c564a818864cc3401b9a5938d2b5e3632a1ed3993acb3f356d42861f6b8ed1d014ce109405537b2ef7

data/.devcontainer/devcontainer.json

@@ -1,6 +1,6 @@
 {
 	"name": "Ruby",
-	"image": "mcr.microsoft.com/devcontainers/ruby:3.4-bullseye",
+	"image": "mcr.microsoft.com/devcontainers/ruby:4.0-bullseye",
 	"customizations": {
 		"vscode": {
 			"extensions": [

data/.rubocop.yml

@@ -5,7 +5,7 @@ plugins:
 
 AllCops:
   NewCops: enable
-  TargetRubyVersion: 2.7
+  TargetRubyVersion: 3.0
 
 Gemspec/DevelopmentDependencies:
   EnforcedStyle: gemspec

data/README.md

@@ -2,9 +2,11 @@
 [![Gem Version](https://badge.fury.io/rb/jekyll-kroki.svg)](https://badge.fury.io/rb/jekyll-kroki)
 
 # jekyll-kroki
+
 A [Jekyll](https://jekyllrb.com/) plugin to convert diagram descriptions into images using [Kroki](https://kroki.io/).
 
 ## Installation
+
 Add the `jekyll-kroki` Gem to the `:jekyll_plugins` group of your site's Gemfile:
 
 ```ruby
@@ -14,6 +16,7 @@ end
 ```
 
 ## Usage
+
 Kroki supports over 25 popular diagram scripting languages, including Blockdiag, D2, GraphViz, Mermaid, and PlantUML. The [examples](https://kroki.io/examples.html) page and complete list of [supported diagram languages](https://kroki.io/#support) provide a taste of what's possible.
 
 In Markdown, simply write your diagram descriptions inside a fenced code block with the language specified:
@@ -28,27 +31,32 @@ Kroki --> Jekyll: Rendered diagram in SVG format
 ```
 ````
 
-When Jekyll builds your site, the `jekyll-kroki` plugin will encode the diagrams, send them to the Kroki server for rendering, then replace the diagram descriptions in the generated HTML with the rendered images in SVG format:
+When Jekyll builds your site, the `jekyll-kroki` plugin encodes the diagram descriptions, renders them as SVG images using the Kroki server, then embeds them in the generated HTML:
 
 ![sample-diagram](https://github.com/felixvanoost/jekyll-kroki/assets/10233016/244d2ec4-b09b-4a5f-8164-3851574c3dd2)
 
-The site remains fully static as the images are directly embedded in the HTML files served by Jekyll. Jekyll only depends on the Kroki server - which can also be run locally - during the build stage, and all of the client-side processing that is normally used to render diagrams into images is eliminated.
+Since the images are rendered and embedded at build-time, the Jekyll site remains completely static and doesn't depend on access to the Kroki server later on. This also eliminates all of the client-side processing that is typically used to render diagrams into images.
 
 ### Advantages
 
 #### Consistent syntax
+
 Instead of using Liquid tags, `jekyll-kroki` leverages the same Markdown fenced code block syntax used by both [GitLab](https://docs.gitlab.com/ee/user/markdown.html#diagrams-and-flowcharts) and [GitHub](https://docs.github.com/en/get-started/writing-on-github/working-with-advanced-formatting/creating-diagrams) to display diagrams. Besides being more consistent, this means that diagram descriptions in Markdown files can be displayed consistently as images across the GitLab/GitHub UI and on GitLab/GitHub Pages sites generated using Jekyll. GitLab currently supports Mermaid and PlantUML, while GitHub only supports Mermaid.
 
 #### Seamless GitLab integration
+
 Self-managed GitLab instances can additionally enable the [Kroki integration](https://docs.gitlab.com/ee/administration/integration/kroki.html), which adds support for all the same diagram scripting languages used by `jekyll-kroki`. Furthermore, by pointing both GitLab and `jekyll-kroki` to the same Kroki instance, you can guarantee that diagrams are generated using identical versions of the diagram libraries.
 
 #### Speed
-The server-side nature of Kroki means that you don't have to deal with installing or updating any diagram library dependencies on your machine. Jekyll sites that are generated in CI/CD pipelines will thus build faster.
+
+The server-side nature of Kroki means that you don't have to deal with installing or updating any diagram library dependencies on your machine. Jekyll sites that are generated in CI/CD pipelines can bypass these steps and will thus build faster.
 
 #### Flexibility
-Kroki is available either as a free service or self-hosted using Docker. Organisations that frequently build large Jekyll sites with many diagrams or want maximum control over their data have the option of running their own Kroki instance to provide consistency and use compute resources efficiently.
+
+Kroki is available either as a free service or self-hosted using Docker. Organisations that frequently build large Jekyll sites with many diagrams or want maximum control over their data can choose to operate their own Kroki instance to ensure consistency and use compute resources efficiently. For individuals, you can also opt to run Kroki locally.
 
 ### Configuration
+
 You can specify the following parameters in the Jekyll `_config.yml` file:
 
 | Parameter | Default value | Description |
@@ -69,6 +77,7 @@ kroki:
 ```
 
 ### Security
+
 Embedding diagrams as SVGs directly within HTML files can be dangerous. You should only use a Kroki instance that you trust (or run your own!). For additional security, you can configure a [Content Security Policy (CSP)](https://developer.mozilla.org/en-US/docs/Web/HTTP/CSP) using custom Webrick headers in the Jekyll `_config.yml` file:
 
 ```yaml
@@ -78,4 +87,5 @@ webrick:
 ```
 
 ## Contributing
+
 Bug reports and pull requests are welcome! For major changes, please open an issue first to discuss what you would like to change.

data/lib/jekyll/kroki/config.rb

@@ -0,0 +1,48 @@
+# frozen_string_literal: true
+
+module Jekyll
+  class Kroki
+    # Reads, validates, and exposes the jekyll-kroki configuration from the Jekyll site config.
+    class Config
+      DEFAULT_KROKI_URL = "https://kroki.io"
+      DEFAULT_HTTP_RETRIES = 3
+      DEFAULT_HTTP_TIMEOUT = 15
+      DEFAULT_MAX_CONCURRENT_DOCS = 8
+
+      attr_reader :kroki_url, :http_retries, :http_timeout, :max_concurrent_docs
+
+      # @param [Hash] The Jekyll site configuration.
+      # @raise [TypeError] If any parameter has an incorrect type.
+      # @raise [ArgumentError] If any parameter is out of the valid range.
+      def initialize(site_config)
+        kroki_config = site_config.fetch("kroki", {})
+
+        @kroki_url = parse_url(kroki_config)
+        @http_retries = parse_integer(kroki_config, "http_retries", DEFAULT_HTTP_RETRIES, min: 0)
+        @http_timeout = parse_integer(kroki_config, "http_timeout", DEFAULT_HTTP_TIMEOUT, min: 0)
+        @max_concurrent_docs = parse_integer(kroki_config, "max_concurrent_docs", DEFAULT_MAX_CONCURRENT_DOCS, min: 1)
+
+        freeze
+      end
+
+      private
+
+      def parse_url(kroki_config)
+        param_name = "url"
+        raw = kroki_config.fetch(param_name, DEFAULT_KROKI_URL)
+        uri = URI.parse(raw)
+        raise TypeError, "'#{param_name}' is not a valid HTTP URL" unless uri.is_a?(URI::HTTP)
+
+        uri
+      end
+
+      def parse_integer(kroki_config, param_name, default, min:)
+        value = kroki_config.fetch(param_name, default)
+        raise TypeError, "'#{param_name}' must be an integer" unless value.is_a?(Integer)
+        raise ArgumentError, "'#{param_name}' must be >= #{min}" if value < min
+
+        value
+      end
+    end
+  end
+end

data/lib/jekyll/kroki/version.rb

@@ -2,6 +2,6 @@
 
 module Jekyll
   class Kroki
-    VERSION = "0.5.0"
+    VERSION = "0.6.0"
   end
 end

data/lib/jekyll/kroki.rb

@@ -1,10 +1,13 @@
 # frozen_string_literal: true
 
+require_relative "kroki/config"
 require_relative "kroki/version"
 
 require "async"
 require "async/semaphore"
 require "base64"
+require "concurrent-ruby"
+require "digest"
 require "faraday"
 require "faraday/retry"
 require "httpx/adapters/faraday"
@@ -15,10 +18,6 @@ require "zlib"
 module Jekyll
   # Converts diagram descriptions into images using Kroki.
   class Kroki
-    DEFAULT_KROKI_URL = "https://kroki.io"
-    DEFAULT_HTTP_RETRIES = 3
-    DEFAULT_HTTP_TIMEOUT = 15
-    DEFAULT_MAX_CONCURRENT_DOCS = 8
     EXPECTED_HTML_TAGS = %w[code div].freeze
     HTTP_RETRY_INTERVAL_BACKOFF_FACTOR = 2
     HTTP_RETRY_INTERVAL_RANDOMNESS = 0.5
@@ -27,27 +26,26 @@ module Jekyll
                              graphviz mermaid nomnoml nwdiag packetdiag pikchr plantuml rackdiag seqdiag structurizr
                              svgbob symbolator tikz umlet vega vegalite wavedrom wireviz].freeze
 
+    @diagram_cache = Concurrent::Map.new
+
     class << self
       # Renders and embeds all diagram descriptions in the given Jekyll site using Kroki.
       #
       # @param [Jekyll::Site] The Jekyll site to embed diagrams in.
       def embed_site(site)
-        kroki_url = get_kroki_url(site.config)
-        http_retries = get_http_retries(site.config)
-        http_timeout = get_http_timeout(site.config)
-        connection = setup_connection(kroki_url, http_retries, http_timeout)
+        config = Config.new(site.config)
+        connection = setup_connection(config.kroki_url, config.http_retries, config.http_timeout)
 
-        max_concurrent_docs = get_max_concurrent_docs(site.config)
-        rendered_diag = embed_docs_in_site(site, connection, max_concurrent_docs)
+        rendered_diag = embed_docs_in_site(site, connection, config.max_concurrent_docs)
         unless rendered_diag.zero?
-          puts "[jekyll-kroki] Rendered #{rendered_diag} diagrams using Kroki instance at '#{kroki_url}'"
+          puts "[jekyll-kroki] Rendered #{rendered_diag} diagrams using Kroki instance at '#{config.kroki_url}'"
         end
       rescue StandardError => e
-        exit(e)
+        fatal_error(e)
       end
 
       # Renders the diagram descriptions in all Jekyll pages and documents in the given Jekyll site. Pages / documents
-      # are rendered concurrently up to the limit defined by DEFAULT_MAX_CONCURRENT_DOCS.
+      # are rendered concurrently up to the limit defined by max_concurrent_docs.
       #
       # @param [Jekyll::Site] The Jekyll site to embed diagrams in.
       # @param [Faraday::Connection] The Faraday connection to use.
@@ -70,8 +68,8 @@ module Jekyll
         rendered_diag
       end
 
-      # Renders the supported diagram descriptions in a single document asynchronously, respecting the concurrency limit
-      # imposed by the provided semaphore.
+      # Renders the supported diagram descriptions in a single document. Multiple documents can be rendered concurrently
+      # up to the limit imposed by the given semaphore.
       #
       # @param [Async::Task] The parent async task to spawn a child task from.
       # @param [Async::Semaphore] A semaphore to limit concurrency.
@@ -87,8 +85,8 @@ module Jekyll
         end
       end
 
-      # Renders the supported diagram descriptions in a single document and embeds them as inline SVGs in the HTML
-      # source.
+      # Renders the supported diagram descriptions in a single document sequentially and embeds them as inline SVGs in
+      # the HTML source.
       #
       # @param [Faraday::Connection] The Faraday connection to use.
       # @param [Jekyll::Page, Jekyll::Document] The document to process.
@@ -113,26 +111,37 @@ module Jekyll
         rendered_diag
       end
 
-      # Renders a single diagram description using Kroki.
+      # Renders a single diagram description using Kroki. The rendered diagram is cached to avoid redundant HTTP
+      # requests across documents, using the diagram language and the SHA1 of the diagram description as the key.
       #
       # @param [Faraday::Connection] The Faraday connection to use.
       # @param [String] The diagram description.
       # @param [String] The language of the diagram description.
       # @return [String] The rendered diagram in SVG format.
       def render_diagram(connection, diagram_desc, language)
-        begin
-          response = connection.get("#{language}/svg/#{encode_diagram(diagram_desc.text)}")
-        rescue Faraday::Error => e
-          raise e.message
+        diagram_text = diagram_desc.text
+        cache_key = "#{language}:#{Digest::SHA1.hexdigest(diagram_text)}"
+        @diagram_cache.compute_if_absent(cache_key) do
+          begin
+            response = connection.get("#{language}/svg/#{encode_diagram(diagram_text)}")
+          rescue Faraday::Error => e
+            raise e.message
+          end
+          validate_content_type(response)
+          sanitise_diagram(response.body)
         end
+      end
+
+      # Validates that the Kroki response has the expected SVG content type.
+      #
+      # @param [Faraday::Response] The response to validate.
+      def validate_content_type(response)
         expected_content_type = "image/svg+xml"
         returned_content_type = response.headers[:content_type]
-        if returned_content_type != expected_content_type
-          raise "Kroki returned an incorrect content type: " \
-                "expected '#{expected_content_type}', received '#{returned_content_type}'"
+        return if returned_content_type == expected_content_type
 
-        end
-        sanitise_diagram(response.body)
+        raise "Kroki returned an incorrect content type: " \
+              "expected '#{expected_content_type}', received '#{returned_content_type}'"
       end
 
       # Sanitises a rendered diagram. Only <script> elements are removed, which is the most minimal / naive
@@ -175,45 +184,10 @@ module Jekyll
         end
       end
 
-      # Gets the URL of the Kroki instance to use for rendering diagrams.
-      #
-      # @param The Jekyll site configuration.
-      # @return [URI::HTTP] The URL of the Kroki instance.
-      def get_kroki_url(config)
-        url = config.fetch("kroki", {}).fetch("url", DEFAULT_KROKI_URL)
-        raise TypeError, "'url' is not a valid HTTP URL" unless URI.parse(url).is_a?(URI::HTTP)
-
-        URI(url)
-      end
-
-      # Gets the number of HTTP retries.
-      #
-      # @param The Jekyll site configuration.
-      # @return [Integer] The number of HTTP retries.
-      def get_http_retries(config)
-        config.fetch("kroki", {}).fetch("http_retries", DEFAULT_HTTP_RETRIES)
-      end
-
-      # Gets the HTTP timeout value.
-      #
-      # @param The Jekyll site configuration.
-      # @return [Integer] The HTTP timeout value in seconds.
-      def get_http_timeout(config)
-        config.fetch("kroki", {}).fetch("http_timeout", DEFAULT_HTTP_TIMEOUT)
-      end
-
-      # Gets the maximum number of documents to render concurrently.
-      #
-      # @param The Jekyll site configuration.
-      # @return [Integer] The maximum number of documents to render concurrently.
-      def get_max_concurrent_docs(config)
-        config.fetch("kroki", {}).fetch("max_concurrent_docs", DEFAULT_MAX_CONCURRENT_DOCS)
-      end
-
       # Determines whether a document may contain embeddable diagram descriptions; it is in HTML format and is either
       # a Jekyll::Page or writeable Jekyll::Document.
       #
-      # @param [Jekyll::Page or Jekyll::Document] The document to check for embeddability.
+      # @param [Jekyll::Page or Jekyll::Document] The document to check for embeddable diagrams.
       def embeddable?(doc)
         doc.output_ext == ".html" && (doc.is_a?(Jekyll::Page) || doc.write?)
       end
@@ -226,7 +200,7 @@ module Jekyll
       #              calling method. To specify the calling method's caller, pass in 2.
       #
       # Source: https://www.mslinn.com/ruby/2200-crash-exit.html
-      def exit(error, caller_index = 1)
+      def fatal_error(error, caller_index = 1)
         raise error
       rescue StandardError => e
         file, line_number, caller = e.backtrace[caller_index].split(":")

metadata

@@ -1,11 +1,11 @@
 --- !ruby/object:Gem::Specification
 name: jekyll-kroki
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.6.0
 platform: ruby
 authors:
 - Felix van Oost
-bindir: exe
+bindir: bin
 cert_chain: []
 date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
@@ -23,6 +23,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '2.25'
+- !ruby/object:Gem::Dependency
+  name: concurrent-ruby
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.3'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.3'
 - !ruby/object:Gem::Dependency
   name: faraday
   requirement: !ruby/object:Gem::Requirement
@@ -190,14 +204,13 @@ files:
 - LICENSE
 - README.md
 - Rakefile
-- jekyll-kroki.gemspec
 - lib/jekyll/kroki.rb
+- lib/jekyll/kroki/config.rb
 - lib/jekyll/kroki/version.rb
 homepage: https://github.com/felixvanoost/jekyll-kroki
 licenses:
 - MIT
 metadata:
-  homepage_uri: https://github.com/felixvanoost/jekyll-kroki
   source_code_uri: https://github.com/felixvanoost/jekyll-kroki
   rubygems_mfa_required: 'true'
 rdoc_options: []
@@ -207,14 +220,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.7.0
+      version: 3.0.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.7
+rubygems_version: 3.6.9
 specification_version: 4
 summary: A Jekyll plugin to convert diagram descriptions into images using Kroki
 test_files: []

data/jekyll-kroki.gemspec

@@ -1,45 +0,0 @@
-# frozen_string_literal: true
-
-require_relative "lib/jekyll/kroki/version"
-
-Gem::Specification.new do |spec|
-  spec.name = "jekyll-kroki"
-  spec.version = Jekyll::Kroki::VERSION
-  spec.authors = ["Felix van Oost"]
-
-  spec.summary = "A Jekyll plugin to convert diagram descriptions into images using Kroki"
-  spec.description = "A Jekyll plugin to convert diagram descriptions written in over 25 popular diagram scripting
-                      languages into images using Kroki"
-  spec.homepage = "https://github.com/felixvanoost/jekyll-kroki"
-  spec.license = "MIT"
-  spec.required_ruby_version = ">= 2.7.0"
-
-  spec.metadata["homepage_uri"] = spec.homepage
-  spec.metadata["source_code_uri"] = spec.homepage
-  spec.metadata["rubygems_mfa_required"] = "true"
-
-  # Load the files that are versioned in Git into the RubyGem.
-  spec.files = Dir.chdir(__dir__) do
-    `git ls-files -z`.split("\x0").reject do |f|
-      (File.expand_path(f) == __FILE__) ||
-        f.start_with?(*%w[bin/ test/ spec/ features/ .git appveyor Gemfile])
-    end
-  end
-  spec.bindir = "exe"
-  spec.executables = spec.files.grep(%r{\Aexe/}) { |f| File.basename(f) }
-  spec.require_paths = ["lib"]
-
-  spec.add_dependency "async", ["~> 2.25"]
-  spec.add_dependency "faraday", ["~> 2.7"]
-  spec.add_dependency "faraday-retry", ["~> 2.2"]
-  spec.add_dependency "httpx", ["~> 1.1"]
-  spec.add_dependency "jekyll", ["~> 4"]
-  spec.add_dependency "nokogiri", ["~> 1.15"]
-
-  spec.add_development_dependency "minitest", ["~> 5.0"]
-  spec.add_development_dependency "rake", ["~> 13.0"]
-  spec.add_development_dependency "rubocop", ["~> 1.21"]
-  spec.add_development_dependency "rubocop-minitest", ["~> 0.38"]
-  spec.add_development_dependency "rubocop-performance", ["~> 1.25"]
-  spec.add_development_dependency "rubocop-rake", ["~> 0.7"]
-end