sass-embedded 0.6.6 → 0.7.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d4a71f28bfa6faf9c0e4d39587a78e05d18f907a412b689283fca8b8184195f5
-  data.tar.gz: d7b2f473c517e43f104b81b7818ed709d74f20c8c833103df1a1c6bd3718a563
+  metadata.gz: 65d80ba547e32d13dc4a4b9f0760e8203df64c5481dea662564e1872bff6bec9
+  data.tar.gz: 363baff44d6a57b5ad5cd6e8963c915b582a147b2c360182e8218ce4ae7a7338
 SHA512:
-  metadata.gz: 74eb4043c28ddba5c4b189454872057cecccb9d536a637da8345f3b8934e2496cc7f5ff56d7371dda9d794e5610fbc1eda47f1dfde92e16d135f48c6a9f02f6a
-  data.tar.gz: ad9631c22232163d842c5e7a688ea50fd990a54ef199dcb46893bdb6572622b6cc05e71b8f75b4108803cfeea8d00f96acc07825d7bd9de598d743c817e1ebef
+  metadata.gz: 12f310e82141b47c7fe61b06a03df7abd00ec06cb9b268aa6c29bd28a523d7182014caaabdd314bebcaeeddad7c62517cb0fda180a81a4ca276862207f8f796b
+  data.tar.gz: 5d4a397ec28e27cedc67c3467a2ffa91b9d8fe0e3ae02b0939fb29fa05a61a0156a7a5a3a6c6eceb6a8049b23867136a94b3962e360db29dd8d3427032f2a52e

data/README.md

@@ -22,7 +22,7 @@ Sass.render(file: "style.scss")
 
 ## Options
 
-`Sass.render()` support the following options:
+`Sass.render(**kwargs)` supports the following options:
 
 - [`data`](https://sass-lang.com/documentation/js-api#data)
 - [`file`](https://sass-lang.com/documentation/js-api#file)
@@ -35,6 +35,7 @@ Sass.render(file: "style.scss")
 - [`source_map`](https://sass-lang.com/documentation/js-api#sourcemap)
 - [`out_file`](https://sass-lang.com/documentation/js-api#outfile)
 - [`omit_source_map_url`](https://sass-lang.com/documentation/js-api#omitsourcemapurl)
+- [`source_map_contents`](https://sass-lang.com/documentation/js-api#sourcemapcontents)
 - [`source_map_embed`](https://sass-lang.com/documentation/js-api#sourcemapembed)
 - [`source_map_root`](https://sass-lang.com/documentation/js-api#sourcemaproot)
 - [`functions`](https://sass-lang.com/documentation/js-api#functions)

data/lib/sass.rb

@@ -60,6 +60,7 @@ module Sass
   end
 end
 
+require_relative 'sass/version'
 require_relative 'sass/platform'
 require_relative 'sass/util'
 require_relative 'sass/struct'
@@ -67,6 +68,6 @@ require_relative 'sass/result'
 require_relative 'sass/error'
 require_relative 'sass/transport'
 require_relative 'sass/observer'
-require_relative 'sass/version'
+require_relative 'sass/info'
 require_relative 'sass/render'
 require_relative 'sass/embedded'

data/lib/sass/embedded.rb

@@ -49,8 +49,6 @@ module Sass
                source_map_root: '',
                functions: {},
                importer: [])
-      raise NotImplementedError, 'source_map_contents is not implemented' unless source_map_contents == false
-
       start = Util.now
 
       indent_type = parse_indent_type(indent_type)
@@ -89,6 +87,7 @@ module Sass
                                          file: file,
                                          out_file: out_file,
                                          source_map: source_map,
+                                         source_map_contents: source_map_contents,
                                          source_map_root: source_map_root)
 
       css = post_process_css(css: message.success.css,
@@ -103,7 +102,7 @@ module Sass
 
       finish = Util.now
 
-      stats = RenderResult::Stats.new(file.nil? ? 'data' : file, start, finish, finish - start)
+      stats = RenderResultStats.new(file.nil? ? 'data' : file, start, finish, finish - start)
 
       RenderResult.new(css, map, stats)
     end
@@ -122,6 +121,7 @@ module Sass
                          file:,
                          out_file:,
                          source_map:,
+                         source_map_contents:,
                          source_map_root:)
       return if map.nil? || map.empty?
 
@@ -146,10 +146,15 @@ module Sass
         map_data['file'] = 'stdin.css'
       end
 
+      map_data['sourcesContent'] = [] if source_map_contents
+
       map_data['sources'].map! do |source|
         if source.start_with? Util::FILE_PROTOCOL
-          Util.relative(source_map_dir, Util.path(source))
+          path = Util.path(source)
+          map_data['sourcesContent'].push(File.read(path)) if source_map_contents
+          Util.relative(source_map_dir, path)
         else
+          map_data['sourcesContent'].push(nil) if source_map_contents
           source
         end
       end

data/lib/sass/info.rb

@@ -2,10 +2,15 @@
 
 module Sass
   # The {Observer} for {Embedded#info}.
-  class Version < Observer
+  class Version
+    include Observer
+
     def initialize(transport, id)
-      super(transport, id)
-      @transport.send EmbeddedProtocol::InboundMessage::VersionRequest.new(id: @id)
+      @id = id
+
+      super(transport)
+
+      send_message EmbeddedProtocol::InboundMessage::VersionRequest.new(id: @id)
     end
 
     def update(error, message)

data/lib/sass/observer.rb

@@ -1,13 +1,10 @@
 # frozen_string_literal: true
 
 module Sass
-  # The {Observer} for receiving messages from {Transport}.
-  class Observer
-    def initialize(transport, id)
-      raise NotImplementedError if instance_of? Observer
-
+  # The {Observer} module for receiving messages from {Transport}.
+  module Observer
+    def initialize(transport)
       @transport = transport
-      @id = id
       @mutex = Mutex.new
       @condition_variable = ConditionVariable.new
       @error = nil
@@ -33,5 +30,11 @@ module Sass
         @condition_variable.broadcast
       end
     end
+
+    private
+
+    def send_message(message)
+      @transport.send_message(message)
+    end
   end
 end

data/lib/sass/render.rb

@@ -2,7 +2,9 @@
 
 module Sass
   # The {Observer} for {Embedded#render}.
-  class Render < Observer
+  class Render
+    include Observer
+
     def initialize(transport, id,
                    data:,
                    file:,
@@ -15,8 +17,7 @@ module Sass
                    importer:)
       raise ArgumentError, 'either data or file must be set' if file.nil? && data.nil?
 
-      super(transport, id)
-
+      @id = id
       @data = data
       @file = file
       @indented_syntax = indented_syntax
@@ -31,7 +32,9 @@ module Sass
       @importer = importer
       @import_responses = {}
 
-      @transport.send compile_request
+      super(transport)
+
+      send_message compile_request
     end
 
     def update(error, message)
@@ -54,13 +57,13 @@ module Sass
         return unless message.compilation_id == @id
 
         Thread.new do
-          @transport.send canonicalize_response message
+          send_message canonicalize_response message
         end
       when EmbeddedProtocol::OutboundMessage::ImportRequest
         return unless message.compilation_id == @id
 
         Thread.new do
-          @transport.send import_response message
+          send_message import_response message
         end
       when EmbeddedProtocol::OutboundMessage::FileImportRequest
         raise NotImplementedError, 'FileImportRequest is not implemented'
@@ -68,7 +71,7 @@ module Sass
         return unless message.compilation_id == @id
 
         Thread.new do
-          @transport.send function_call_response message
+          send_message function_call_response message
         end
       end
     rescue StandardError => e

data/lib/sass/result.rb

@@ -12,19 +12,19 @@ module Sass
       @map = map
       @stats = stats
     end
+  end
 
-    # The {Stats} of {Embedded#render}.
-    class Stats
-      include Struct
+  # The {RenderResultStats} of {Embedded#render}.
+  class RenderResultStats
+    include Struct
 
-      attr_reader :entry, :start, :end, :duration
+    attr_reader :entry, :start, :end, :duration
 
-      def initialize(entry, start, finish, duration)
-        @entry = entry
-        @start = start
-        @end = finish
-        @duration = duration
-      end
+    def initialize(entry, start, finish, duration)
+      @entry = entry
+      @start = start
+      @end = finish
+      @duration = duration
     end
   end
 end

data/lib/sass/transport.rb

@@ -5,7 +5,7 @@ require 'observer'
 require_relative '../../ext/embedded_sass_pb'
 
 module Sass
-  # The {Observable} {Transport} for low level communication with
+  # The {::Observable} {Transport} for low level communication with
   # `dart-sass-embedded` using protocol buffers via stdio. Received messages
   # can be observed by an {Observer}.
   class Transport
@@ -29,7 +29,7 @@ module Sass
       @stdin_mutex = Mutex.new
       @stdin, @stdout, @stderr, @wait_thread = Open3.popen3(DART_SASS_EMBEDDED)
       pipe @stderr, $stderr
-      receive
+      read @stdout
     end
 
     def add_observer(*args)
@@ -38,7 +38,7 @@ module Sass
       end
     end
 
-    def send(message)
+    def send_message(message)
       write EmbeddedProtocol::InboundMessage.new(
         ONEOF_MESSAGE[message.class.descriptor] => message
       ).to_proto
@@ -58,42 +58,38 @@ module Sass
 
     private
 
-    def receive
+    def receive_message(error, message)
+      @observerable_mutex.synchronize do
+        changed
+        notify_observers error, message
+      end
+    end
+
+    def pipe(readable, writeable)
       Thread.new do
         loop do
-          bits = length = 0
-          loop do
-            byte = @stdout.readbyte
-            length += (byte & 0x7f) << bits
-            bits += 7
-            break if byte <= 0x7f
-          end
-          payload = @stdout.read length
-          message = EmbeddedProtocol::OutboundMessage.decode payload
-          @observerable_mutex.synchronize do
-            changed
-            notify_observers nil, message[message.message.to_s]
-          end
+          writeable.write readable.readline
         rescue Interrupt
           break
         rescue IOError => e
-          notify_observers e, nil
+          receive_message(e, nil)
           close
           break
         end
       end
     end
 
-    def pipe(readable, writeable)
+    def read(readable)
       Thread.new do
         loop do
-          writeable.write readable.read
+          length = read_varint(readable)
+          payload = readable.read(length)
+          message = EmbeddedProtocol::OutboundMessage.decode payload
+          receive_message(nil, message[message.message.to_s])
         rescue Interrupt
           break
         rescue IOError => e
-          @observerable_mutex.synchronize do
-            notify_observers e, nil
-          end
+          receive_message(e, nil)
           close
           break
         end
@@ -102,13 +98,27 @@ module Sass
 
     def write(payload)
       @stdin_mutex.synchronize do
-        length = payload.length
-        while length.positive?
-          @stdin.write ((length > 0x7f ? 0x80 : 0) | (length & 0x7f)).chr
-          length >>= 7
-        end
+        write_varint(@stdin, payload.length)
         @stdin.write payload
       end
     end
+
+    def read_varint(readable)
+      varint = bits = 0
+      loop do
+        byte = readable.readbyte
+        varint += (byte & 0x7f) << bits
+        bits += 7
+        break if byte <= 0x7f
+      end
+      varint
+    end
+
+    def write_varint(writeable, varint)
+      while varint.positive?
+        writeable.write ((varint > 0x7f ? 0x80 : 0) | (varint & 0x7f)).chr
+        varint >>= 7
+      end
+    end
   end
 end

data/lib/sass/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Sass
-  VERSION = '0.6.6'
+  VERSION = '0.7.3'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sass-embedded
 version: !ruby/object:Gem::Version
-  version: 0.6.6
+  version: 0.7.3
 platform: ruby
 authors:
 - なつき
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-05-29 00:00:00.000000000 Z
+date: 2021-05-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: google-protobuf
@@ -136,7 +136,8 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description: Use dart-sass with Ruby!
+description: A Ruby library that will communicate with Embedded Dart Sass using the
+  Embedded Sass protocol.
 email:
 - i@ntk.me
 executables: []
@@ -144,14 +145,8 @@ extensions:
 - ext/extconf.rb
 extra_rdoc_files: []
 files:
-- ".github/workflows/build.yml"
-- ".gitignore"
-- ".rubocop.yml"
-- Gemfile
 - LICENSE
 - README.md
-- Rakefile
-- ext/.gitignore
 - ext/Makefile
 - ext/extconf.rb
 - lib/sass.rb
@@ -166,22 +161,11 @@ files:
 - lib/sass/transport.rb
 - lib/sass/util.rb
 - lib/sass/version.rb
-- sass-embedded.gemspec
-- test/concurrency_test.rb
-- test/functions_test.rb
-- test/importer_test.rb
-- test/include_paths_test.rb
-- test/indented_syntax_test.rb
-- test/input_test.rb
-- test/output_test.rb
-- test/render_error_test.rb
-- test/render_test.rb
-- test/source_maps_test.rb
-- test/test_helper.rb
 homepage: https://github.com/ntkme/sass-embedded-host-ruby
 licenses:
 - MIT
-metadata: {}
+metadata:
+  source_code_uri: https://github.com/ntkme/sass-embedded-host-ruby/tree/v0.7.3
 post_install_message:
 rdoc_options: []
 require_paths:
@@ -201,15 +185,4 @@ rubygems_version: 3.2.15
 signing_key:
 specification_version: 4
 summary: Use dart-sass with Ruby!
-test_files:
-- test/concurrency_test.rb
-- test/functions_test.rb
-- test/importer_test.rb
-- test/include_paths_test.rb
-- test/indented_syntax_test.rb
-- test/input_test.rb
-- test/output_test.rb
-- test/render_error_test.rb
-- test/render_test.rb
-- test/source_maps_test.rb
-- test/test_helper.rb
+test_files: []

data/.github/workflows/build.yml

@@ -1,44 +0,0 @@
-name: build
-
-on:
-  push:
-    branches: [ main ]
-  pull_request:
-    branches: [ main ]
-
-jobs:
-  build:
-
-    runs-on: ${{ matrix.os }}
-
-    strategy:
-      matrix:
-        os: [macos-latest, ubuntu-latest, windows-latest]
-        ruby-version: ['2.6', '2.7', '3.0']
-
-    steps:
-    - name: Checkout
-      uses: actions/checkout@v2
-
-    - name: Set up Ruby
-      uses: ruby/setup-ruby@v1
-      with:
-        ruby-version: ${{ matrix.ruby-version }}
-        bundler-cache: true
-
-    - name: Download dart-sass-embedded
-      run: bundle exec rake extconf
-      env:
-        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-
-    - name: Test
-      run: bundle exec rake
-
-    - name: Install Gem
-      run: rake install
-      env:
-        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-
-    - name: Test Gem
-      run: |-
-        ruby -r sass -e "puts Sass.render(data: 'h1 { color: black; }')[:css]"