grape_sorbet 0.0.1 → 0.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 68395f140413d15f9db05463e554cdf86ec2e60915226877fd738cf1b7d17183
-  data.tar.gz: 8c4a99e44696bd706c216c1629c18c48a359902710100542061f65fd01fae256
+  metadata.gz: afca4010268efe01d37b64b7cfd4bbd7f6266779a3a135ab04fe14869ab71daf
+  data.tar.gz: ee494b0147c0b9d32ae90a5a1fba81218687b26396131a56ff9a1bb16306d693
 SHA512:
-  metadata.gz: db6b2f00da52ea9382f4cc6fc05c38b569614e95f7f81061d684d884716ea761a2b8d23a090606dccab3c5e54faa715fcc5e4b59d598b7e66b279ec2cfae0dce
-  data.tar.gz: ef8ffe995841f9cb5ca094e36ea7ef82d0ccc05626a3a68e8a252469ff1197460cfa5556e58d93c361964f2a4617a846b3e873d3d228a58da276fbdf0eae3fc9
+  metadata.gz: 122aa7dab2572d6df1cfc79a3ec7a80b652f5ad7d5d51d87b0e9077385e3fb8041fd75b190f76ce4c58b4f1aea575dd1194d505b21c5d9ff32d80b9fca1aa33a
+  data.tar.gz: 4eb298722d0e14cb745a20917899825463fd50d6b87b0e1d9541566c9bc5961effec782c7ec767cacc9bcf37536596e0f885c3ae9eb5bdf60717f32d1833c341

data/CHANGELOG.md

@@ -0,0 +1,11 @@
+## 0.0.3 (2024-06-05)
+
+- Better support for callbacks
+
+## 0.0.2 (2024-06-05)
+
+- Added support for callbacks (`before`, `after`, etc.)
+
+## 0.0.1 (2024-06-04)
+
+- First release

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    grape_sorbet (0.0.1)
+    grape_sorbet (0.0.3)
       activesupport
       grape (~> 2.0)
       sorbet-runtime (~> 0.5.10741)

data/README.md

@@ -90,7 +90,29 @@ module Twitter
 end
 ```
 
-At this point Sorbet is still reporting errors in the `get :home_timeline` block, because it doesn't know that these blocks are executed in a context where the `Helpers` module methods will be available. The Tapioca compiler provided by this gem will fix that. Run `bundle exec tapioca dsl`. This should generate a `sorbet/rbi/dsl/twitter/api.rbi` file (assuming the source file is in `lib/twitter/api.rb`). After this,
+At this point Sorbet is still reporting errors in the `get :home_timeline` block, because it doesn't know that these blocks are executed in a context where the `Helpers` module methods will be available. The Tapioca compiler provided by this gem will fix that. Run `bundle exec tapioca dsl`. This should generate a `sorbet/rbi/dsl/twitter/api.rbi` file (assuming the source file is in `lib/twitter/api.rb`).
+
+After this, Sorbet should no longer report any errors.
+
+## Limitations and known issues
+
+### Subclassing from `Grape::API::Instance` instead of `Grape::API`
+
+Grape overrides `Grape::API.new` and uses the `inherited` hook so that subclasses of `Grape::API` are really subclasses of `Grape::API::Instance`.
+
+This might be fixable in a future update of grape_sorbet, but is very low priority.
+
+### Not being able to call `helpers` with block arguments
+
+Possibly fixable in a future update.
+
+### Having to use `T.bind(self, T.any(Grape::Endpoint, <HelpersModuleName>))` in helper methods
+
+Possibly fixable in a future update.
+
+### Having to use `T.bind(self, T.any(Grape::Endpoint, <HelpersModuleName>))` in `before` and `after` callback blocks
+
+The compiler already generates proper signatures for callback methods so `T.bind` should not be needed (and is in fact unneeded for the other callback methods, `before_validation`, `after_validation` and `finally`). The reason it doesn't work for `before` and `after` is because of a [bug](https://github.com/sorbet/sorbet/issues/7950) in Sorbet itself.
 
 ## Development
 

data/lib/grape_sorbet/version.rb

@@ -2,5 +2,5 @@
 # frozen_string_literal: true
 
 module GrapeSorbet
-  VERSION = "0.0.1"
+  VERSION = "0.0.3"
 end

data/lib/tapioca/dsl/compilers/grape_endpoints.rb

@@ -68,6 +68,7 @@ module Tapioca
         def decorate
           create_classes_and_includes
           create_routing_methods
+          create_callbacks_methods
         end
 
         class << self
@@ -79,6 +80,11 @@ module Tapioca
           end
         end
 
+        CALLBACKS_METHODS = T.let(
+          [:before, :before_validation, :after_validation, :after, :finally].freeze,
+          T::Array[Symbol],
+        )
+
         HTTP_VERB_METHODS = T.let(
           [:get, :post, :put, :patch, :delete, :head, :options].freeze,
           T::Array[Symbol],
@@ -94,6 +100,14 @@ module Tapioca
           )
         end
 
+        sig { returns(RBI::Scope) }
+        def callbacks_methods_module
+          @callbacks_methods_module ||= T.let(
+            api.create_module(CallbacksMethodsModuleName),
+            T.nilable(RBI::Scope),
+          )
+        end
+
         sig { returns(RBI::Scope) }
         def routing_methods_module
           @routing_methods_module ||= T.let(
@@ -104,6 +118,7 @@ module Tapioca
 
         sig { void }
         def create_classes_and_includes
+          api.create_extend(CallbacksMethodsModuleName)
           api.create_extend(RoutingMethodsModuleName)
           create_api_class
           create_endpoint_class
@@ -135,6 +150,19 @@ module Tapioca
           end
         end
 
+        sig { void }
+        def create_callbacks_methods
+          CALLBACKS_METHODS.each do |callback|
+            callbacks_methods_module.create_method(
+              callback.to_s,
+              parameters: [
+                create_block_param("block", type: "T.proc.bind(#{EndpointClassName}).void"),
+              ],
+              return_type: "void",
+            )
+          end
+        end
+
         sig { void }
         def create_routing_methods
           HTTP_VERB_METHODS.each do |verb|
@@ -142,7 +170,7 @@ module Tapioca
               verb.to_s,
               parameters: [
                 create_rest_param("args", type: "T.untyped"),
-                create_block_param("blk", type: "T.nilable(T.proc.bind(#{EndpointClassName}).void)"),
+                create_block_param("block", type: "T.nilable(T.proc.bind(#{EndpointClassName}).void)"),
               ],
               return_type: "void",
             )
@@ -152,8 +180,8 @@ module Tapioca
             "route_param",
             parameters: [
               create_param("param", type: "Symbol"),
-              create_opt_param("options", type: "T.nilable(T::Hash[Symbol, T.untyped])", default: "nil"),
-              create_block_param("blk", type: "T.nilable(T.proc.bind(T.class_of(#{APIInstanceClassName})).void)"),
+              create_opt_param("options", type: "T::Hash[Symbol, T.untyped]", default: "{}"),
+              create_block_param("block", type: "T.nilable(T.proc.bind(T.class_of(#{APIInstanceClassName})).void)"),
             ],
             return_type: "void",
           )

data/lib/tapioca/dsl/helpers/grape_constants_helper.rb

@@ -7,6 +7,7 @@ module Tapioca
       module GrapeConstantsHelper
         extend T::Sig
 
+        CallbacksMethodsModuleName = "GeneratedCallbacksMethods"
         RoutingMethodsModuleName = "GeneratedRoutingMethods"
 
         APIInstanceClassName = "PrivateAPIInstance"

data/rbi/grape.rbi

@@ -2,11 +2,6 @@
 # frozen_string_literal: true
 
 module Grape
-  module DSL::Callbacks::ClassMethods
-    sig { params(block: T.proc.bind(Grape::Endpoint).void).void }
-    def before(&block); end
-  end
-
   module DSL::Desc
     # grape evaluates config_block in the context of a dynamically created module that implements the DSL it exposes
     # at runtime. There's no good way to represent this statically, so block is just typed as T.untyped to prevent

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: grape_sorbet
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.3
 platform: ruby
 authors:
 - Thatch Health, Inc.
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-06-04 00:00:00.000000000 Z
+date: 2024-06-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -63,6 +63,7 @@ files:
 - ".editorconfig"
 - ".rubocop.yml"
 - ".ruby-version"
+- CHANGELOG.md
 - CODE_OF_CONDUCT.md
 - Gemfile
 - Gemfile.lock