tapioca 0.16.9 → 0.16.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b2d6abd53e84e3b22cc6045597788c98d8f9571f486976baa8db4e5beb6ccd10
-  data.tar.gz: 5aa9162275489d2280f07419f74a4243c9ee813f922dd367e6260bae0fa37331
+  metadata.gz: 9b1ddf8615a19929d81d7662c300a28ee8d49d08d7ba984fdaa7644bf70d5dfb
+  data.tar.gz: 4f2a79db13ce4a0e7614178d14423f5bd0fded127a6c5b6d0503bf7e2d73df7b
 SHA512:
-  metadata.gz: 4fd946bacca4a56fe9f9befaa65404393719cacedf4877ce2e8ba0d9d5ab150d9957bea7298783a04e456170086b035b06b42b0312bc2840e1465631259801a1
-  data.tar.gz: '0825b9451bed3958b7132008993675de624753d41d5b7b267b50e8b7cf2900f344e5c666ac7726686ebfe9c997592dec6c02b5b2e1d14b144cdbaf837daeac08'
+  metadata.gz: a03a8d4866b5faf46af5cea289d4d7087540d58ee5b4eb759cb30cc23c540cc037c03bdd937cdf73e3f955407b2a47865600297fafe6b495e786e64b99f38942
+  data.tar.gz: 3a5bf689b47f198577480420b9ea0c1294a6c2170df0f5ba9a6e170a8f60b3d2de73827053f4b33002f1a6ae9f2501a73b70590c250c8306b86d52b244aa7f36

data/lib/ruby_lsp/tapioca/addon.rb

@@ -1,7 +1,7 @@
 # typed: strict
 # frozen_string_literal: true
 
-RubyLsp::Addon.depend_on_ruby_lsp!(">= 0.23.1", "< 0.24")
+RubyLsp::Addon.depend_on_ruby_lsp!(">= 0.23.10", "< 0.24")
 
 begin
   # The Tapioca add-on depends on the Rails add-on to add a runtime component to the runtime server. We can allow the
@@ -53,8 +53,11 @@ module RubyLsp
             workspace_path: @global_state.workspace_path,
           )
 
+          send_usage_telemetry("activated")
           run_gem_rbi_check
         rescue IncompatibleApiError
+          send_usage_telemetry("incompatible_api_error")
+
           # The requested version for the Rails add-on no longer matches. We need to upgrade and fix the breaking
           # changes
           @outgoing_queue << Notification.window_log_message(
@@ -75,12 +78,12 @@ module RubyLsp
 
       sig { override.returns(String) }
       def version
-        "0.1.1"
+        "0.1.2"
       end
 
       sig { params(changes: T::Array[{ uri: String, type: Integer }]).void }
       def workspace_did_change_watched_files(changes)
-        return unless T.must(@global_state).enabled_feature?(:tapiocaAddon)
+        return unless @global_state&.enabled_feature?(:tapiocaAddon)
         return unless @rails_runner_client.connected?
 
         has_route_change = T.let(false, T::Boolean)
@@ -89,11 +92,10 @@ module RubyLsp
 
         constants = changes.flat_map do |change|
           path = URI(change[:uri]).to_standardized_path
-          next if path.end_with?("_test.rb", "_spec.rb")
           next unless file_updated?(change, path)
 
-          if File.fnmatch?("**/tapioca/**/compilers/**/*.rb", path, File::FNM_PATHNAME)
-            needs_compiler_reload = true
+          if File.fnmatch("**/fixtures/**/*.yml{,.erb}", path, File::FNM_PATHNAME | File::FNM_EXTGLOB)
+            has_fixtures_change = true
             next
           end
 
@@ -102,9 +104,10 @@ module RubyLsp
             next
           end
 
-          # NOTE: We only get notification for fixtures if ruby-lsp-rails is v0.3.31 or higher
-          if File.fnmatch("**/fixtures/**/*.yml{,.erb}", path, File::FNM_PATHNAME | File::FNM_EXTGLOB)
-            has_fixtures_change = true
+          next if File.fnmatch?("**/{test,spec,features}/**/*", path, File::FNM_PATHNAME | File::FNM_EXTGLOB)
+
+          if File.fnmatch?("**/tapioca/**/compilers/**/*.rb", path, File::FNM_PATHNAME)
+            needs_compiler_reload = true
             next
           end
 
@@ -124,19 +127,22 @@ module RubyLsp
           @rails_runner_client.delegate_notification(
             server_addon_name: "Tapioca",
             request_name: "reload_workspace_compilers",
-            workspace_path: T.must(@global_state).workspace_path,
+            workspace_path: @global_state.workspace_path,
           )
         end
 
         if has_route_change
+          send_usage_telemetry("route_dsl")
           @rails_runner_client.delegate_notification(server_addon_name: "Tapioca", request_name: "route_dsl")
         end
 
         if has_fixtures_change
+          send_usage_telemetry("fixtures_dsl")
           @rails_runner_client.delegate_notification(server_addon_name: "Tapioca", request_name: "fixtures_dsl")
         end
 
         if constants.any?
+          send_usage_telemetry("dsl")
           @rails_runner_client.delegate_notification(
             server_addon_name: "Tapioca",
             request_name: "dsl",
@@ -147,6 +153,25 @@ module RubyLsp
 
       private
 
+      sig { params(feature_name: String).void }
+      def send_usage_telemetry(feature_name)
+        return unless @outgoing_queue && @global_state
+
+        # Telemetry is not captured by default even if events are produced by the server
+        # See https://github.com/Shopify/ruby-lsp/tree/main/vscode#telemetry
+        @outgoing_queue << Notification.telemetry({
+          eventName: "tapioca_addon.feature_usage",
+          type: "data",
+          data: {
+            type: "counter",
+            attributes: {
+              label: feature_name,
+              machineId: @global_state.telemetry_machine_id,
+            },
+          },
+        })
+      end
+
       sig { params(change: T::Hash[Symbol, T.untyped], path: String).returns(T::Boolean) }
       def file_updated?(change, path)
         case change[:type]

data/lib/ruby_lsp/tapioca/server_addon.rb

@@ -62,11 +62,13 @@ module RubyLsp
       def dsl(constants, *args)
         load("tapioca/cli.rb") # Reload the CLI to reset thor defaults between requests
 
+        ::Tapioca::Cli.addon_mode!
+
         # Order here is important to avoid having Thor confuse arguments. Do not put an array argument at the end before
         # the list of constants
         arguments = ["dsl"]
         arguments.concat(args)
-        arguments.push("--lsp_addon", "--workers=1")
+        arguments.push("--workers=1")
         arguments.concat(constants)
 
         ::Tapioca::Cli.start(arguments)

data/lib/tapioca/cli.rb

@@ -143,11 +143,6 @@ module Tapioca
       type: :hash,
       desc: "Options to pass to the DSL compilers",
       default: {}
-    option :lsp_addon,
-      type: :boolean,
-      desc: "Generate DSL RBIs from the LSP add-on. Internal to Tapioca and not intended for end-users",
-      default: false,
-      hide: true
     def dsl(*constant_or_paths)
       set_environment(options)
 
@@ -170,7 +165,7 @@ module Tapioca
         app_root: options[:app_root],
         halt_upon_load_error: options[:halt_upon_load_error],
         compiler_options: options[:compiler_options],
-        lsp_addon: options[:lsp_addon],
+        lsp_addon: self.class.addon_mode,
       }
 
       command = if options[:verify]
@@ -379,9 +374,22 @@ module Tapioca
     end
 
     no_commands do
+      @addon_mode = false
+
       class << self
+        extend T::Sig
+
+        # Indicates that we are running from the LSP, set using the `addon_mode!` method
+        attr_reader :addon_mode
+
+        sig { void }
+        def addon_mode!
+          @addon_mode = true
+        end
+
+        sig { returns(T::Boolean) }
         def exit_on_failure?
-          true
+          !@addon_mode
         end
       end
     end

data/lib/tapioca/version.rb

@@ -2,5 +2,5 @@
 # frozen_string_literal: true
 
 module Tapioca
-  VERSION = "0.16.9"
+  VERSION = "0.16.10"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: tapioca
 version: !ruby/object:Gem::Version
-  version: 0.16.9
+  version: 0.16.10
 platform: ruby
 authors:
 - Ufuk Kayserilioglu
@@ -10,7 +10,7 @@ authors:
 - Peter Zhu
 bindir: exe
 cert_chain: []
-date: 2025-02-04 00:00:00.000000000 Z
+date: 2025-02-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: benchmark