sass-embedded 0.7.2 → 0.7.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 420195776d51e7fcd3d510db9445f0a7bb4c926613515e1a849a865f4bd6447d
-  data.tar.gz: 518333274bb5f2b521894075bce89314a47e993b29c84fd54ebdfd38aeab33f3
+  metadata.gz: 835b2c9a1ba57c2029d2742def3168da30f6779b79f6da189cd0bb17a050b4b4
+  data.tar.gz: 781cdcecc2740ef1d817328c019df048404b40e66019e3ea4714508db42050fb
 SHA512:
-  metadata.gz: 52ab4ccf5f0ffd05554749a5669dd474a5e5c8c027e55fd93a8665070182f973a37be5e11a93066d485b89d5144ef893c0cc26fc5892b2fb31b8c6824799462a
-  data.tar.gz: 83f042430fa1e8339ea31d63752714ce842994b230e66ba8745695dac997617e9ecc28aad732c7dffc558fb3660471970ec1109d9a2f8f42e172c5ce1dc9f192
+  metadata.gz: 3ce7a3556eafb00728a5670800f9b60974fc62c81a6b4f934afc6f3a37b1a04544cbe6553fa629b2c33a31fdda8574d33943b417a0928e4d9946424b18117892
+  data.tar.gz: 23b8d42df92a6053034c3be000568dcb260c04afba8318b9b4d4f960d194b1483325720dfb4aaacec7266b05ce0455a24c16dc96122101ff8b79c52368c79f7d

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

@@ -23,7 +23,7 @@ module Sass
     #
     # @raise [ProtocolError]
     def info
-      @info ||= Version.new(@transport, next_id).message
+      @info ||= VersionContext.new(@transport, next_id).receive_message
     end
 
     # The {Embedded#render} method.
@@ -55,16 +55,16 @@ module Sass
       indent_width = parse_indent_width(indent_width)
       linefeed = parse_linefeed(linefeed)
 
-      message = Render.new(@transport, next_id,
-                           data: data,
-                           file: file,
-                           indented_syntax: indented_syntax,
-                           include_paths: include_paths,
-                           output_style: output_style,
-                           source_map: source_map,
-                           out_file: out_file,
-                           functions: functions,
-                           importer: importer).message
+      message = RenderContext.new(@transport, next_id,
+                                  data: data,
+                                  file: file,
+                                  indented_syntax: indented_syntax,
+                                  include_paths: include_paths,
+                                  output_style: output_style,
+                                  source_map: source_map,
+                                  out_file: out_file,
+                                  functions: functions,
+                                  importer: importer).receive_message
 
       if message.failure
         raise RenderError.new(
@@ -75,7 +75,7 @@ module Sass
           elsif message.failure.span.url == ''
             'stdin'
           else
-            Util.path(message.failure.span.url)
+            Util.path_from_file_uri(message.failure.span.url)
           end,
           message.failure.span ? message.failure.span.start.line + 1 : nil,
           message.failure.span ? message.failure.span.start.column + 1 : nil,
@@ -138,7 +138,7 @@ module Sass
       source_map_dir = File.dirname(source_map_path)
 
       if out_file
-        map_data['file'] = Util.relative(source_map_dir, out_file)
+        map_data['file'] = Util.relative_path(source_map_dir, out_file)
       elsif file
         ext = File.extname(file)
         map_data['file'] = "#{file[0..(ext.empty? ? -1 : -ext.length - 1)]}.css"
@@ -149,10 +149,10 @@ module Sass
       map_data['sourcesContent'] = [] if source_map_contents
 
       map_data['sources'].map! do |source|
-        if source.start_with? Util::FILE_PROTOCOL
-          path = Util.path(source)
+        if source.start_with? 'file://'
+          path = Util.path_from_file_uri(source)
           map_data['sourcesContent'].push(File.read(path)) if source_map_contents
-          Util.relative(source_map_dir, path)
+          Util.relative_path(source_map_dir, path)
         else
           map_data['sourcesContent'].push(nil) if source_map_contents
           source
@@ -184,7 +184,7 @@ module Sass
         url = if source_map_embed
                 "data:application/json;base64,#{Base64.strict_encode64(map)}"
               elsif out_file
-                Util.relative(File.dirname(out_file), source_map)
+                Util.relative_path(File.dirname(out_file), source_map)
               else
                 source_map
               end

data/lib/sass/info.rb

@@ -2,7 +2,7 @@
 
 module Sass
   # The {Observer} for {Embedded#info}.
-  class Version
+  class VersionContext
     include Observer
 
     def initialize(transport, id)
@@ -17,8 +17,6 @@ module Sass
       raise error unless error.nil?
 
       case message
-      when EmbeddedProtocol::ProtocolError
-        raise ProtocolError, message.message
       when EmbeddedProtocol::OutboundMessage::VersionResponse
         return unless message.id == @id
 

data/lib/sass/observer.rb

@@ -12,7 +12,7 @@ module Sass
       @transport.add_observer self
     end
 
-    def message
+    def receive_message
       @mutex.synchronize do
         @condition_variable.wait(@mutex) if @error.nil? && @message.nil?
       end

data/lib/sass/render.rb

@@ -2,7 +2,7 @@
 
 module Sass
   # The {Observer} for {Embedded#render}.
-  class Render
+  class RenderContext
     include Observer
 
     def initialize(transport, id,
@@ -41,8 +41,6 @@ module Sass
       raise error unless error.nil?
 
       case message
-      when EmbeddedProtocol::ProtocolError
-        raise ProtocolError, message.message
       when EmbeddedProtocol::OutboundMessage::CompileResponse
         return unless message.id == @id
 
@@ -97,7 +95,7 @@ module Sass
     end
 
     def canonicalize_response(canonicalize_request)
-      url = Util.file_uri(File.absolute_path(canonicalize_request.url, (@file.nil? ? 'stdin' : @file)))
+      url = Util.file_uri_from_path(File.absolute_path(canonicalize_request.url, (@file.nil? ? 'stdin' : @file)))
 
       begin
         result = @importer[canonicalize_request.importer_id].call canonicalize_request.url, @file
@@ -123,7 +121,7 @@ module Sass
           url: url
         )
       elsif result&.key? :file
-        canonicalized_url = Util.file_uri(result[:file])
+        canonicalized_url = Util.file_uri_from_path(result[:file])
 
         # TODO: FileImportRequest is not supported yet.
         # Workaround by reading contents and return it when server asks
@@ -185,7 +183,7 @@ module Sass
     def url
       return if @file.nil?
 
-      Util.file_uri @file
+      Util.file_uri_from_path @file
     end
 
     def string

data/lib/sass/transport.rb

@@ -24,12 +24,18 @@ module Sass
       [field_descriptor.subtype, field_descriptor.name]
     end.to_h
 
+    private_constant :ONEOF_MESSAGE
+
     def initialize
       @observerable_mutex = Mutex.new
       @stdin_mutex = Mutex.new
       @stdin, @stdout, @stderr, @wait_thread = Open3.popen3(DART_SASS_EMBEDDED)
-      pipe @stderr, $stderr
-      read @stdout
+      poll do
+        $stderr.write @stderr.readline
+      end
+      poll do
+        receive_proto read
+      end
     end
 
     def add_observer(*args)
@@ -58,44 +64,43 @@ module Sass
 
     private
 
-    def receive_message(error, message)
-      @observerable_mutex.synchronize do
-        changed
-        notify_observers error, message
-      end
-    end
-
-    def pipe(readable, writeable)
+    def poll
       Thread.new do
         loop do
-          writeable.write readable.readline
+          yield
         rescue Interrupt
           break
         rescue IOError => e
-          receive_message(e, nil)
+          notify_observers(e, nil)
           close
           break
         end
       end
     end
 
-    def read(readable)
-      Thread.new do
-        loop do
-          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
-          receive_message(e, nil)
-          close
-          break
-        end
+    def notify_observers(*args)
+      @observerable_mutex.synchronize do
+        changed
+        super(*args)
+      end
+    end
+
+    def receive_proto(proto)
+      payload = EmbeddedProtocol::OutboundMessage.decode(proto)
+      message = payload[payload.message.to_s]
+      case message
+      when EmbeddedProtocol::ProtocolError
+        notify_observers(ProtocolError.new(message.message), nil)
+      else
+        notify_observers(nil, message)
       end
     end
 
+    def read
+      length = read_varint(@stdout)
+      @stdout.read(length)
+    end
+
     def write(payload)
       @stdin_mutex.synchronize do
         write_varint(@stdin, payload.length)

data/lib/sass/util.rb

@@ -1,39 +1,30 @@
 # frozen_string_literal: true
 
 require 'pathname'
+require 'uri'
 
 module Sass
   # The {Util} module.
   module Util
     module_function
 
-    FILE_PROTOCOL = 'file://'
+    URI_PARSER = URI::Parser.new({ RESERVED: ';/?:@&=+$,' })
 
-    def file_uri(path)
+    private_constant :URI_PARSER
+
+    def file_uri_from_path(path)
       absolute_path = File.absolute_path(path)
 
-      unless absolute_path.start_with? File::SEPARATOR
-        components = absolute_path.split File::SEPARATOR
-        components[0] = components[0][0].downcase
-        absolute_path = components.join File::SEPARATOR
-      end
+      absolute_path = File::SEPARATOR + absolute_path unless absolute_path.start_with? File::SEPARATOR
 
-      "#{FILE_PROTOCOL}#{absolute_path}"
+      URI_PARSER.escape("file://#{absolute_path}")
     end
 
-    def path(file_uri)
-      absolute_path = file_uri[FILE_PROTOCOL.length..]
-
-      unless absolute_path.start_with? File::SEPARATOR
-        components = absolute_path.split File::SEPARATOR
-        components[0] = "#{components[0].upcase}:"
-        absolute_path = components.join File::SEPARATOR
-      end
-
-      absolute_path
+    def path_from_file_uri(file_uri)
+      URI_PARSER.unescape(file_uri)[7..]
     end
 
-    def relative(from, to)
+    def relative_path(from, to)
       Pathname.new(File.absolute_path(to)).relative_path_from(Pathname.new(File.absolute_path(from))).to_s
     end
 

data/lib/sass/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sass-embedded
 version: !ruby/object:Gem::Version
-  version: 0.7.2
+  version: 0.7.7
 platform: ruby
 authors:
 - なつき
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-05-30 00:00:00.000000000 Z
+date: 2021-06-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: google-protobuf
@@ -165,7 +165,7 @@ homepage: https://github.com/ntkme/sass-embedded-host-ruby
 licenses:
 - MIT
 metadata:
-  source_code_uri: https://github.com/ntkme/sass-embedded-host-ruby/tree/v0.7.2
+  source_code_uri: https://github.com/ntkme/sass-embedded-host-ruby/tree/v0.7.7
 post_install_message:
 rdoc_options: []
 require_paths: