sass-embedded 0.7.1 → 0.7.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e292d8b2c097a1d19fd12d808600c91ae39dbc93180bc34cd2c56bf0cebf494c
-  data.tar.gz: 6507777d01910a91c93e793a46efdcc6990ccc0a422881643a0e5b7f395892ff
+  metadata.gz: d934996f500e6897ada46afc22c92ec529e66df4f6cb6ced17f0067fe438fd3b
+  data.tar.gz: 4983a6a25aec431857fc6642d06b8296937d351cd6d941783fcd49e6adfa23c2
 SHA512:
-  metadata.gz: 79485c6293b91b1b709ab7630f431ab01d8c0af6033ab86c0c11fa05e27c390864fff433a6d91cc5251e5e5a77f11be0709d04eed2c2b8762acef162235ce247
-  data.tar.gz: b6cce9f847c0681df0891e32044f7a9fea3d79a4c0d1736e192537d07a30763b494750f86f1d843a640c49a13758140e22e40305ce0e07b726d0a223ecb6d28b
+  metadata.gz: 5b7062bfaf917380d740d69eedad18e1b6d66b5eca52dfa7c268138fe2921320bfea350af67aad4ed0c22623cacb06ec242c4e8e3ae17dba9f7d63718657a19a
+  data.tar.gz: 2eaf5b41c9c1e4efeb46ac544964da082beafe23d4ba6c6bd4103bbb3f9d94aead77506bb4648acab7b2adf15fbea236c3cff3a3db5d2210e6a9380b3c65056a

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)

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,
@@ -97,7 +97,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 +123,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 +185,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,37 @@ 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)
+      message = EmbeddedProtocol::OutboundMessage.decode(proto)
+      notify_observers(nil, message[message.message.to_s])
+    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.1'
+  VERSION = '0.7.6'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sass-embedded
 version: !ruby/object:Gem::Version
-  version: 0.7.1
+  version: 0.7.6
 platform: ruby
 authors:
 - なつき
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-05-30 00:00:00.000000000 Z
+date: 2021-05-31 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.6
 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]"

data/.gitignore

@@ -1,45 +0,0 @@
-*.gem
-*.rbc
-/.config
-/coverage/
-/InstalledFiles
-/pkg/
-/spec/reports/
-/spec/examples.txt
-/test/tmp/
-/test/version_tmp/
-/tmp/
-
-# Used by dotenv library to load environment variables.
-# .env
-
-# Ignore Byebug command history file.
-.byebug_history
-
-## Specific to RubyMotion:
-.dat*
-.repl_history
-build/
-*.bridgesupport
-build-iPhoneOS/
-build-iPhoneSimulator/
-
-## Documentation cache and generated files:
-/.yardoc/
-/_yardoc/
-/doc/
-/rdoc/
-
-## Environment normalization:
-/.bundle/
-/vendor/bundle
-/lib/bundler/man/
-
-# for a library or gem, you might want to ignore these files since the code is
-# intended to run in multiple environments; otherwise, check them in:
-/Gemfile.lock
-/.ruby-version
-/.ruby-gemset
-
-# unless supporting rvm < 1.11.0 or doing something fancy, ignore this:
-.rvmrc