typescript-rails 0.3.0 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d275ff8fd90b13580de3f11a3798e0cbd9aec47c
-  data.tar.gz: 2942656ec7afda0c974fb05f3300058747862e58
+  metadata.gz: 18c02ddb2925e76f99d15526b4d68d12ebce5a3d
+  data.tar.gz: 8f743d5be68031c673151ee590623f64b94377e7
 SHA512:
-  metadata.gz: 55f7d3ee2d3e65b07e65fa0e3c7d2a742abbff70679eb00cd3d20c9267532be40d4909b9883d10d905b0f081afca82cb8c1678ad918142be74322813abfb4530
-  data.tar.gz: bee501dd861c693c090563015b480e5aac5d161390c301a742b26fde02ed96d1dbd5269e24228dae9a807d3a7d58828906b78ab05919d687ab7517dec2b1a7f6
+  metadata.gz: 4fd56936bee817a81102ff8f5353bf7d25ad549eec5e30339b87b3aaf95d1e0cbdf1df1f1b83c7aba0f88dd58f13b77d10e7b2a53f7324c11dcc62d9bdd6e8d7
+  data.tar.gz: 8d8884552159bcce98b011afc344b23bdc1362d0a589baa6f12a5ee127882e2a09e251324234647629bc077bc85e79b8517d0322f7eec6a4e952e7ae532195ed

data/.travis.yml

@@ -1,7 +1,7 @@
 rvm:
  - 1.9.3
  - 2.0.0
- - 2.1.0
+ - 2.1.1
  - jruby
 notifications:
   email: false

data/CHANGES.md

@@ -1,3 +1,10 @@
+## v0.4.0 2014-08-11 23:04:20+0900
+
+* Set `--noImplicitAny` by default to tsc compiler
+* Typescript::Rails::Compiler.default_options to provide tsc compiler with otpions
+    * default values: `--target ES5 --noImplicitAny`
+* A lot of refactoring
+
 ## v0.3.0 2014-08-09 10:27:54+0900
 
 * Compiles TypeScript files in ES5 mode

data/lib/typescript-rails.rb

@@ -1,4 +1,6 @@
+require 'typescript/rails'
 require 'typescript/rails/railtie'
 require 'typescript/rails/engine'
+require 'typescript/rails/template'
 require 'typescript/rails/template_handler'
 require 'typescript/rails/version'

data/lib/typescript/rails.rb

@@ -0,0 +1,5 @@
+module Typescript
+  module Rails
+    # just to define the module Typescript::Rails
+  end
+end

data/lib/typescript/rails/compiler.rb

@@ -0,0 +1,45 @@
+require 'typescript/rails'
+require 'typescript-node'
+
+module Typescript::Rails::Compiler
+  class << self
+    # @!scope class
+    cattr_accessor :default_options
+
+    # Replace relative paths specified in /// <reference path="..." /> with absolute paths.
+    #
+    # @param [String] ts_path Source .ts path
+    # @param [String] source. It might be pre-processed by erb.
+    # @return [String] replaces source
+    def replace_relative_references(ts_path, source)
+      ts_dir = File.dirname(File.expand_path(ts_path))
+      escaped_dir = ts_dir.gsub(/["\\]/, '\\\\\&') # "\"" => "\\\"", '\\' => '\\\\'
+
+      # Why don't we just use gsub? Because it display odd behavior with File.join on Ruby 2.0
+      # So we go the long way around.
+      output = ''
+      source.each_line do |l|
+        if l.starts_with?('///') && !(m = %r!^///\s*<reference\s+path="([^"]+)"\s*/>\s*!.match(l)).nil?
+          l = l.sub(m.captures[0], File.join(escaped_dir, m.captures[0]))
+        end
+        output = output + l + $/
+      end
+
+      output
+    end
+
+    # @param [String] ts_path
+    # @param [String] source TypeScript source code
+    # @return [String] compiled JavaScript source code
+    def compile(ts_path, source, *options)
+      s = replace_relative_references(ts_path, source)
+      ::TypeScript::Node.compile(s, *default_options, *options)
+    end
+
+  end
+
+  self.default_options = [
+      '--target', 'ES5',
+      '--noImplicitAny'
+  ]
+end

data/lib/typescript/rails/engine.rb

@@ -1,10 +1,6 @@
 require 'rails/engine'
 
-module Typescript
-  module Rails
-    class Engine < ::Rails::Engine
-      # For now, let's not be the default generator ...
-      # config.app_generators.javascript_engine :ts
-    end
-  end
+class Typescript::Rails::Engine < Rails::Engine
+  # For now, let's not be the default generator ...
+  # config.app_generators.javascript_engine :ts
 end

data/lib/typescript/rails/railtie.rb

@@ -1,15 +1,11 @@
-module Typescript
-  module Rails
-    class Railtie < ::Rails::Railtie
-      config.before_initialize do |app|
-        require 'typescript-rails'
+require 'typescript/rails'
 
-        if ::Rails::VERSION::MAJOR >= 4 || app.config.assets.enabled
-          require 'sprockets'
-          require 'sprockets/engines'
-          Sprockets.register_engine '.ts', Typescript::Rails::TypeScriptTemplate
-        end
-      end
+class Typescript::Rails::Railtie < ::Rails::Railtie
+  config.before_initialize do |app|
+    if ::Rails::VERSION::MAJOR >= 4 || app.config.assets.enabled
+      require 'typescript/rails/template'
+      require 'sprockets'
+      Sprockets.register_engine '.ts', Typescript::Rails::Template
     end
   end
-end
+end

data/lib/typescript/rails/template.rb

@@ -0,0 +1,32 @@
+require 'typescript/rails'
+require 'tilt/template'
+
+class Typescript::Rails::Template < ::Tilt::Template
+  self.default_mime_type = 'application/javascript'
+
+  # @!scope class
+  class_attribute :default_bare
+
+  def self.engine_initialized?
+    defined? ::Typescript::Rails::Compiler
+  end
+
+  def initialize_engine
+    require_template_library 'typescript/rails/compiler'
+  end
+
+  def prepare
+    if !options.key?(:bare) and !options.key?(:no_wrap)
+      options[:bare] = self.class.default_bare
+    end
+  end
+
+  def evaluate(scope, locals, &block)
+    @output ||= ::Typescript::Rails::Compiler.compile(file, source)
+  end
+
+  # @override
+  def allows_script?
+    false
+  end
+end

data/lib/typescript/rails/template_handler.rb

@@ -1,98 +1,17 @@
-require 'typescript-node'
-require 'tilt'
+require 'typescript/rails/compiler'
 
-module Typescript
-  module Rails
-
-    class TypeScriptTemplate < ::Tilt::Template
-      self.default_mime_type = 'application/javascript'
-
-      @@default_bare = false
-
-      def self.default_bare
-        @@default_bare
-      end
-
-      def self.default_bare=(value)
-        @@default_bare = value
-      end
-
-      # @deprecated
-      def self.default_no_wrap
-        @@default_bare
-      end
-
-      # @deprecated
-      def self.default_no_wrap=(value)
-        @@default_bare = value
-      end
-
-      def self.engine_initialized?
-        defined? ::TypeScript
-      end
-
-      def initialize_engine
-        require_template_library 'coffee_script'
-      end
-
-      def prepare
-        if !options.key?(:bare) and !options.key?(:no_wrap)
-          options[:bare] = self.class.default_bare
-        end
-      end
-
-      def evaluate(scope, locals, &block)
-        source = Typescript::Rails.replace_relative_references(file, data)
-        # TODO: employs source-maps
-        @output ||= TypeScript::Node.compile(source, '--target', 'ES5')
-      end
-
-      def allows_script?
-        false
-      end
+class Typescript::Rails::TemplateHandler
+  class << self
+    def erb_handler
+      @erb_handler ||= ActionView::Template.registered_template_handler(:erb)
     end
 
-    class TemplateHandler
-
-      def self.erb_handler
-        @@erb_handler ||= ActionView::Template.registered_template_handler(:erb)
-      end
-
-      def self.call(template)
-        compiled_source = erb_handler.call(template)
-        escaped_path = template.identifier.gsub(/['\\]/, '\\\\\&') # "'" => "\\'", '\\' => '\\\\'
-        <<-EOS
-        TypeScript::Node.compile(
-          Typescript::Rails.replace_relative_references(
-            '#{escaped_path}', (begin;#{compiled_source};end)
-          ),
-          '--target', 'ES5'
-        )
-        EOS
-      end
-    end
-
-    # Replace relative paths specified in /// <reference path="..." /> with absolute paths.
-    #
-    # @param [String] ts_path Source .ts path
-    # @param [String] ts source. It might be pre-processed by erb.
-    # @return [String] replaces source
-    def self.replace_relative_references(ts_path, source)
-      ts_dir = File.dirname(File.expand_path(ts_path))
-      escaped_dir = ts_dir.gsub(/["\\]/, '\\\\\&') # "\"" => "\\\"", '\\' => '\\\\'
-
-      # Why don't we just use gsub? Because it display odd behavior with File.join on Ruby 2.0
-      # So we go the long way around.
-      output = ''
-      source.each_line do |l|
-        if l.starts_with?('///') && !(m = %r!^///\s*<reference\s+path="([^"]+)"\s*/>\s*!.match(l)).nil?
-          l = l.sub(m.captures[0], File.join(escaped_dir, m.captures[0]))
-        end
-
-        output = output + l + $/
-      end
-
-      output
+    def call(template)
+      compiled_source = erb_handler.call(template)
+      path = template.identifier.gsub(/['\\]/, '\\\\\&') # "'" => "\\'", '\\' => '\\\\'
+      <<-EOS
+        ::Typescript::Rails::Compiler.compile('#{path}', (begin;#{compiled_source};end))
+      EOS
     end
   end
 end

data/lib/typescript/rails/version.rb

@@ -1,5 +1,5 @@
 module Typescript
   module Rails
-    VERSION = '0.3.0'
+    VERSION = '0.4.0'
   end
 end

data/test/assets_test.rb

@@ -9,6 +9,7 @@ class AssetsTest < ActiveSupport::TestCase
     require "sprockets/railtie"
 
     @app = Class.new(Rails::Application)
+    @app.config.eager_load = false
     @app.config.active_support.deprecation = :stderr
     @app.config.assets.enabled = true
     @app.config.assets.cache_store = [ :file_store, "#{tmp_path}/cache" ]

data/test/support/site/ref3_2.ts

@@ -1,3 +1,3 @@
-var f1 = function() {
+var f1 = function(): void {
 
 };

data/test/support/site/ref3_3.ts

@@ -1,3 +1,3 @@
-var f2 = function() {
+var f2 = function(): void {
 
 };

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: typescript-rails
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.4.0
 platform: ruby
 authors:
 - FUJI, Goro
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-08-09 00:00:00.000000000 Z
+date: 2014-08-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: typescript-node
@@ -70,8 +70,11 @@ files:
 - Rakefile
 - lib/assets/javascripts/typescript.js.erb
 - lib/typescript-rails.rb
+- lib/typescript/rails.rb
+- lib/typescript/rails/compiler.rb
 - lib/typescript/rails/engine.rb
 - lib/typescript/rails/railtie.rb
+- lib/typescript/rails/template.rb
 - lib/typescript/rails/template_handler.rb
 - lib/typescript/rails/version.rb
 - test/assets_test.rb