jekyll-sass-converter 1.5.2 → 2.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a89449e32d89721639e197984bc0ff57632a816ca8649d787d07c0b8b7e94cc0
-  data.tar.gz: 0b4d273ad2a4463e679f70971a742fdb779376d8f37bb1ace8f7596fa3a27431
+  metadata.gz: c051522099eb9c977d893591246ad37863d0ff38feec276b8992c9f16cfa6b3b
+  data.tar.gz: 388b71b1d56dcde5f45b9860b28ba932291951cf2385e42560032b3c4e144c5f
 SHA512:
-  metadata.gz: 3268e9b3fce2074122f16fc64882b75048847930ae7329d42268bfe7e519d92b2727b4564f6c587c95922075ca5cc0d2d39287ac8abbdad5f70f7b5b16431b78
-  data.tar.gz: bfbf9cf8fc49ebfc788287ef5e5abd810c9cbf26634d96d43876375ac3ddd0923e5c381f7bed4d4a9cedb616d86e46766c7483bfb67b343a8c38e6731bfcc6da
+  metadata.gz: 0b2c4cd74ba4ee2612c92394011d98f6db389078d3668ccebe3cf96ef855c5a1a72c9dbe47057f3cc586b1dac2937379ee02356e289fdbf96869ad860e2ad1c3
+  data.tar.gz: 275e76080d121f8693b42d1ca46088a0328013b111c8eddc4a998baafd04990ed9ff6cad5ac7246198c79bf63aad7e3ff556a2bbe9d6ffd0a0582dc259daec68

data/lib/jekyll-sass-converter/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module JekyllSassConverter
-  VERSION = "1.5.2".freeze
+  VERSION = "2.0.0"
 end

data/lib/jekyll/converters/sass.rb

@@ -1,19 +1,17 @@
 # frozen_string_literal: true
 
-require "sass"
+require "sassc"
 require "jekyll/utils"
 require "jekyll/converters/scss"
 
 module Jekyll
   module Converters
     class Sass < Scss
+      EXTENSION_PATTERN = %r!^\.sass$!i.freeze
+
       safe true
       priority :low
 
-      def matches(ext)
-        ext =~ %r!^\.sass$!i
-      end
-
       def syntax
         :sass
       end

data/lib/jekyll/converters/scss.rb

@@ -1,21 +1,71 @@
 # frozen_string_literal: true
 
-require "sass"
+require "sassc"
 require "jekyll/utils"
+require "jekyll/source_map_page"
 
 module Jekyll
   module Converters
     class Scss < Converter
-      BYTE_ORDER_MARK = %r!^\xEF\xBB\xBF!
+      BYTE_ORDER_MARK = %r!^\xEF\xBB\xBF!.freeze
+      EXTENSION_PATTERN = %r!^\.scss$!i.freeze
+
       SyntaxError = Class.new(ArgumentError)
 
       safe true
       priority :low
 
+      # This hook is triggered just before the method {#convert(content)} is executed, it
+      # associates the Scss (and Sass) converters with their respective sass_page objects.
+      Jekyll::Hooks.register :pages, :pre_render do |page|
+        page.converters.each do |converter|
+          converter.associate_page(page) if converter.is_a?(Jekyll::Converters::Scss)
+        end
+      end
+
+      # This hook is triggered just after the method {#convert(content)} has been executed, it
+      # dissociates the Scss (and Sass) converters with their respective sass_page objects.
+      Jekyll::Hooks.register :pages, :post_render do |page|
+        page.converters.each do |converter|
+          converter.dissociate_page(page) if converter.is_a?(Jekyll::Converters::Scss)
+        end
+      end
+
       ALLOWED_STYLES = %w(nested expanded compact compressed).freeze
 
+      # Associate this Converter with the "page" object that manages input and output files for
+      # this converter.
+      #
+      # Note: changing the associated sass_page during the live time of this Converter instance
+      # may result in inconsistent results.
+      #
+      # @param [Jekyll:Page] page The sass_page for which this object acts as converter.
+      def associate_page(page)
+        if @sass_page
+          Jekyll.logger.debug "Sass Converter:",
+                              "sass_page re-assigned: #{@sass_page.name} to #{page.name}"
+          dissociate_page(page)
+          return
+        end
+        @sass_page = page
+      end
+
+      # Dissociate this Converter with the "page" object.
+      #
+      # @param [Jekyll:Page] page The sass_page for which this object has acted as a converter.
+      def dissociate_page(page)
+        unless page.equal?(@sass_page)
+          Jekyll.logger.debug "Sass Converter:",
+                              "dissociating a page that was never associated #{page.name}"
+        end
+
+        @source_map_page = nil
+        @sass_page = nil
+        @site = nil
+      end
+
       def matches(ext)
-        ext =~ %r!^\.scss$!i
+        ext =~ self.class::EXTENSION_PATTERN
       end
 
       def output_ext(_ext)
@@ -27,21 +77,18 @@ module Jekyll
       end
 
       def jekyll_sass_configuration
-        options = @config["sass"] || {}
-        unless options["style"].nil?
-          options["style"] = options["style"].to_s.gsub(%r!\A:!, "").to_sym
+        @jekyll_sass_configuration ||= begin
+          options = @config["sass"] || {}
+          unless options["style"].nil?
+            options["style"] = options["style"].to_s.gsub(%r!\A:!, "").to_sym
+          end
+          options
         end
-        options
       end
 
       def sass_build_configuration_options(overrides)
         if safe?
-          {
-            :load_paths => sass_load_paths,
-            :syntax     => syntax,
-            :style      => sass_style,
-            :cache      => false,
-          }
+          overrides
         else
           Jekyll::Utils.symbolize_hash_keys(
             Jekyll::Utils.deep_merge_hashes(
@@ -58,6 +105,7 @@ module Jekyll
 
       def sass_dir
         return "_sass" if jekyll_sass_configuration["sass_dir"].to_s.empty?
+
         jekyll_sass_configuration["sass_dir"]
       end
 
@@ -74,8 +122,10 @@ module Jekyll
         Jekyll.sanitized_path(site_source, sass_dir)
       end
 
+      # rubocop:disable Metrics/AbcSize
       def sass_load_paths
         paths = user_sass_load_paths + [sass_dir_relative_to_site_source]
+        paths << site.theme.sass_path if site.theme&.sass_path
 
         if safe?
           # Sanitize paths to prevent any attack vectors (.e.g. `/**/*`)
@@ -84,7 +134,7 @@ module Jekyll
 
         # Expand file globs (e.g. `node_modules/*/node_modules` )
         Dir.chdir(site_source) do
-          paths = paths.map { |path| Dir.glob(path) }.flatten.uniq
+          paths = paths.flat_map { |path| Dir.glob(path) }.uniq
 
           paths.map! do |path|
             if safe?
@@ -98,6 +148,7 @@ module Jekyll
 
         paths.select { |path| File.directory?(path) }
       end
+      # rubocop:enable Metrics/AbcSize
 
       def allow_caching?
         !safe?
@@ -108,24 +159,125 @@ module Jekyll
       end
 
       def sass_configs
-        sass_build_configuration_options({
-          "syntax"     => syntax,
-          "cache"      => allow_caching?,
-          "load_paths" => sass_load_paths,
-        })
+        sass_build_configuration_options(
+          :style                => sass_style,
+          :syntax               => syntax,
+          :filename             => filename,
+          :output_path          => output_path,
+          :source_map_file      => source_map_file,
+          :load_paths           => sass_load_paths,
+          :omit_source_map_url  => !sourcemap_required?,
+          :source_map_contents  => true,
+          :line_comments_option => line_comments_option
+        )
       end
 
       def convert(content)
-        output = ::Sass.compile(content, sass_configs)
+        config = sass_configs
+        engine = SassC::Engine.new(content.dup, config)
+        output = engine.render
+        generate_source_map(engine) if sourcemap_required?
         replacement = add_charset? ? '@charset "UTF-8";' : ""
         output.sub(BYTE_ORDER_MARK, replacement)
-      rescue ::Sass::SyntaxError => e
-        raise SyntaxError, "#{e} on line #{e.sass_line}"
+      rescue SassC::SyntaxError => e
+        raise SyntaxError, e.to_s
       end
 
       private
+
+      # The Page instance for which this object acts as a converter.
+      attr_reader :sass_page
+
+      def associate_page_failed?
+        !sass_page
+      end
+
+      # The name of the input scss (or sass) file. This information will be used for error
+      # reporting and will written into the source map file as main source.
+      #
+      # Returns the name of the input file or "stdin" if #associate_page failed
+      def filename
+        return "stdin" if associate_page_failed?
+
+        sass_page.name
+      end
+
+      # The value of the `line_comments` option.
+      # When set to `true` causes the line number and filename of the source be emitted into the
+      # compiled CSS-file. Useful for debugging when the source-map is not available.
+      #
+      # Returns the value of the `line_comments`-option chosen by the user or 'false' by default.
+      def line_comments_option
+        jekyll_sass_configuration.fetch("line_comments", false)
+      end
+
+      # The value of the `sourcemap` option chosen by the user.
+      #
+      # This option controls when sourcemaps shall be generated or not.
+      #
+      # Returns the value of the `sourcemap`-option chosen by the user or ':always' by default.
+      def sourcemap_option
+        jekyll_sass_configuration.fetch("sourcemap", :always).to_sym
+      end
+
+      # Determines whether a sourcemap shall be generated or not.
+      #
+      # Returns `true` if a sourcemap shall be generated, `false` otherwise.
+      def sourcemap_required?
+        return false if associate_page_failed? || sourcemap_option == :never
+        return true  if sourcemap_option == :always
+
+        !(sourcemap_option == :development && Jekyll.env != "development")
+      end
+
+      # The name of the generated css file. This information will be written into the source map
+      # file as a backward reference to the input.
+      #
+      # Returns the name of the css file or "stdin.css" if #associate_page failed
+      def output_path
+        return "stdin.css" if associate_page_failed?
+
+        sass_page.basename + ".css"
+      end
+
+      # The name of the generated source map file. This information will be written into the
+      # css file to reference to the source map.
+      #
+      # Returns the name of the css file or "" if #associate_page failed
+      def source_map_file
+        return "" if associate_page_failed?
+
+        sass_page.basename + ".css.map"
+      end
+
+      def source_map_page
+        return if associate_page_failed?
+
+        @source_map_page ||= SourceMapPage.new(sass_page)
+      end
+
+      # Reads the source-map from the engine and adds it to the source-map-page.
+      #
+      # @param [::SassC::Engine] engine The sass Compiler engine.
+      def generate_source_map(engine)
+        return if associate_page_failed?
+
+        source_map_page.source_map(engine.source_map)
+        site.pages << source_map_page
+      rescue ::SassC::NotRenderedError => e
+        Jekyll.logger.warn "Could not generate source map #{e.message} => #{e.cause}"
+      end
+
+      def site
+        if associate_page_failed?
+          Jekyll.sites.last
+        else
+          sass_page.site
+        end
+      end
+
       def site_source
-        @site_source ||= File.expand_path(@config["source"]).freeze
+        site.source
       end
     end
   end

data/lib/jekyll/source_map_page.rb

@@ -0,0 +1,37 @@
+# frozen_string_literal: true
+
+module Jekyll
+  # A Jekyll::Page subclass to manage the source map file associated with
+  # a given scss / sass page.
+  class SourceMapPage < Page
+    # Initialize a new SourceMapPage.
+    #
+    # @param [Jekyll::Page] css_page The Page object that manages the css file.
+    def initialize(css_page)
+      @site = css_page.site
+      @dir  = css_page.dir
+      @data = css_page.data
+      @name = css_page.basename + ".css.map"
+
+      process(@name)
+      Jekyll::Hooks.trigger :pages, :post_init, self
+    end
+
+    def source_map(map)
+      self.content = map
+    end
+
+    def ext
+      ".map"
+    end
+
+    def asset_file?
+      true
+    end
+
+    # @return[String] the object as a debug String.
+    def inspect
+      "#<#{self.class} @name=#{name.inspect}>"
+    end
+  end
+end

metadata

@@ -1,57 +1,49 @@
 --- !ruby/object:Gem::Specification
 name: jekyll-sass-converter
 version: !ruby/object:Gem::Version
-  version: 1.5.2
+  version: 2.0.0
 platform: ruby
 authors:
 - Parker Moore
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-02-03 00:00:00.000000000 Z
+date: 2019-08-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: sass
+  name: sassc
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">"
+      - !ruby/object:Gem::Version
+        version: 2.0.1
+    - - "<"
       - !ruby/object:Gem::Version
-        version: '3.4'
+        version: '3.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.4'
-- !ruby/object:Gem::Dependency
-  name: bundler
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
+    - - ">"
       - !ruby/object:Gem::Version
-        version: '1.5'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
+        version: 2.0.1
+    - - "<"
       - !ruby/object:Gem::Version
-        version: '1.5'
+        version: '3.0'
 - !ruby/object:Gem::Dependency
-  name: jekyll
+  name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -81,19 +73,19 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rubocop
+  name: rubocop-jekyll
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.51'
+        version: '0.4'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.51'
+        version: '0.4'
 description: 
 email:
 - parkrmoore@gmail.com
@@ -105,6 +97,7 @@ files:
 - lib/jekyll-sass-converter/version.rb
 - lib/jekyll/converters/sass.rb
 - lib/jekyll/converters/scss.rb
+- lib/jekyll/source_map_page.rb
 homepage: https://github.com/jekyll/jekyll-sass-converter
 licenses:
 - MIT
@@ -117,15 +110,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 2.4.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.4
+rubygems_version: 3.0.4
 signing_key: 
 specification_version: 4
 summary: A basic Sass converter for Jekyll.