jets 4.0.10 → 5.0.0

data/lib/jets/{resource → cfn/resource}/api_gateway/base_path/mapping.rb

@@ -16,28 +16,29 @@
 #   * Also tried to change the domain name of to something like demo-dev-[random].mydomain.com
 #   That does not work because the domain name has to match the route53 record exactly.
 #
-module Jets::Resource::ApiGateway::BasePath
-  class Mapping < Jets::Resource::Base
+module Jets::Cfn::Resource::ApiGateway::BasePath
+  class Mapping < Jets::Cfn::Base
     def definition
       function_logical_id = "BasePathFunction" # lambda function that supports custom resource
       {
-        base_path_mapping: {
-          type: "Custom::BasePathMapping",
-          properties: {
-            service_token: "!GetAtt #{function_logical_id}.Arn",
-            # base_path: '', # empty path represents root
-            # domain_name: "!Ref DomainName",
-            # rest_api_id: "!Ref RestApi", # since this is in the Deployment template
-            # stage: Deployment.stage_name,
+        BasePathMapping: {
+          Type: "Custom::BasePathMapping",
+          Properties: {
+            ServiceToken: "!GetAtt #{function_logical_id}.Arn",
+            # A change to any of these properties updates the CloudFormation Custom Resource
+            # IE: It runs the Lambda function that implements the custom resource
+            BasePath: Jets.config.domain.base_path, # '' empty path represents root
+            DomainName: "!Ref DomainName",
+            RestApiId: "!Ref RestApi",
+            Stage: Jets::Cfn::Resource::ApiGateway::Deployment.stage_name,
           },
-          depends_on: Jets::Resource::ApiGateway::Deployment.logical_id,
         }
       }
     end
 
     def outputs
       {
-        "BasePathMapping" => "!Ref BasePathMapping",
+        BasePathMapping: "!Ref BasePathMapping",
       }
     end
   end

data/lib/jets/cfn/resource/api_gateway/base_path/role.rb

@@ -0,0 +1,70 @@
+module Jets::Cfn::Resource::ApiGateway::BasePath
+  class Role < Jets::Cfn::Base
+    extend Memoist
+    include Jets::AwsServices
+
+    def definition
+      {
+        BasePathRole: {
+          Type: "AWS::IAM::Role",
+          Properties: {
+            # RoleName: role_name,
+            Path: "/",
+            AssumeRolePolicyDocument: {
+              Version: "2012-10-17",
+              Statement: [{
+                Effect: "Allow",
+                Principal: {Service: ["lambda.amazonaws.com"]},
+                Action: ["sts:AssumeRole"]}
+              ]
+            },
+            Policies: [
+              PolicyName: "base-path-mapping-policy", # cannot be empty
+              PolicyDocument: policy_document,
+            ]
+          },
+        }
+      }
+    end
+
+    def policy_document
+      project_namespace = Jets.project_namespace
+      default_policy_statements = Jets.application.config.default_iam_policy # Array of Hashes
+      apigateway = [{
+        Action: [ "apigateway:*" ],
+        Effect: "Allow",
+        Resource: "arn:aws:apigateway:#{Jets.aws.region}::/restapis/*", # scoped to all restapis because this changes
+      },{
+        Action: [ "apigateway:*" ],
+        Effect: "Allow",
+        Resource: "arn:aws:apigateway:#{Jets.aws.region}::/domainnames/*", # scoped to all restapis because this changes
+      }]
+      cloudformation = [{
+        Action: ["cloudformation:DescribeStacks"],
+        Effect: "Allow",
+        Resource: "arn:aws:cloudformation:#{Jets.aws.region}:#{Jets.aws.account}:stack/#{project_namespace}*",
+      }]
+
+      # Combine the statements
+      {
+        Version: '2012-10-17',
+        Statement: default_policy_statements + apigateway + cloudformation
+      }
+    end
+
+    # Duplicated in rest_api/change_detection.rb, base_path/role.rb, rest_api/routes.rb
+    def rest_api_id
+      stack_name = Jets::Names.parent_stack_name
+      return "RestApi" unless stack_exists?(stack_name)
+
+      stack = cfn.describe_stacks(stack_name: stack_name).stacks.first
+
+      api_gateway_stack_arn = lookup(stack[:outputs], "ApiGateway")
+
+      # resources = cfn.describe_stack_resources(stack_name: api_gateway_stack_arn).stack_resources
+      stack = cfn.describe_stacks(stack_name: api_gateway_stack_arn).stacks.first
+      rest_api_id = lookup(stack[:outputs], "RestApi")
+    end
+    memoize :rest_api_id
+  end
+end

data/lib/jets/{resource → cfn/resource}/api_gateway/deployment.rb

@@ -1,45 +1,38 @@
-module Jets::Resource::ApiGateway
-  class Deployment < Jets::Resource::Base
+module Jets::Cfn::Resource::ApiGateway
+  class Deployment < Jets::Cfn::Base
     def definition
       {
         deployment_logical_id => {
-          type: "AWS::ApiGateway::Deployment",
-          properties: {
-            description: "Version #{timestamp} deployed by jets",
-            rest_api_id: "!Ref #{RestApi.logical_id}",
-            stage_name: stage_name,
+          Type: "AWS::ApiGateway::Deployment",
+          Properties: {
+            Description: "Version #{timestamp} deployed by jets",
+            RestApiId: "!Ref #{RestApi.logical_id}",
+            StageName: stage_name,
           }
         }
       }
     end
 
     def parameters
-      p = {
-        "GemLayer" => "GemLayer",
-        "IamRole" => "IamRole",
-        "RestApi" => "RestApi",
-        "S3Bucket" => "S3Bucket",
+      {
+        RestApi: "RestApi",
       }
-      p[:DomainName] = "DomainName" if Jets.custom_domain?
-      p[:BasePath] = "BasePath" unless Jets.config.domain.base_path.nil?
-      p
     end
 
     def outputs(internal=false)
       rest_api = internal ? RestApi.internal_logical_id : "RestApi"
       {
-        "RestApiUrl" => "!Sub 'https://${#{rest_api}}.execute-api.${AWS::Region}.amazonaws.com/#{stage_name}/'",
+        RestApiUrl: "!Sub 'https://${#{rest_api}}.execute-api.${AWS::Region}.amazonaws.com/#{stage_name}/'",
       }
     end
 
     def depends_on
-      expression = "#{Jets::Names.template_path_prefix}-*_controller*"
+      expression = "#{Jets::Names.templates_folder}/*_controller*"
       controller_logical_ids = []
       Dir.glob(expression).each do |path|
         next unless File.file?(path)
 
-        regexp = Regexp.new(".*#{Jets.config.project_namespace}-")
-        controller_name = path.sub(regexp, '').sub('.yml', '')
+        controller_name = path.sub("#{Jets::Names.templates_folder}/", '').sub('.yml', '')
         # map the path to a camelized logical_id. Example:
         #   /tmp/jets/demo/templates/demo-dev-2-posts_controller.yml to
         #   PostsController
@@ -57,7 +50,7 @@ module Jets::Resource::ApiGateway
 
     def self.stage_name
       # Stage name only allows a-zA-Z0-9_
-      [Jets.config.short_env, Jets.config.extra].compact.join('_').gsub('-','_')
+      [Jets.short_env, Jets.extra].compact.join('_').gsub('-','_')
     end
 
     def timestamp

data/lib/jets/{resource → cfn/resource}/api_gateway/domain_name.rb

@@ -8,34 +8,34 @@
 #       DomainName: api.mydomain.com
 #       CertificateArn: arn:aws:acm:us-east-1:111122223333:certificate/fb1b9770-a305-495d-aefb-27e5e101ff3
 #
-module Jets::Resource::ApiGateway
-  class DomainName < Jets::Resource::Base
+module Jets::Cfn::Resource::ApiGateway
+  class DomainName < Jets::Cfn::Base
     def definition
       properties = {
-        domain_name: domain_name,
-        endpoint_configuration: {
-          types: endpoint_types
+        DomainName: domain_name,
+        EndpointConfiguration: {
+          Types: endpoint_types
         }
       }
       # Can really only be REGIONAL or EDGE
       if endpoint_types.include?("REGIONAL")
-        properties[:regional_certificate_arn] = cert_arn
+        properties[:RegionalCertificateArn] = cert_arn
       end
       if endpoint_types.include?("EDGE")
-        properties[:certificate_arn] = cert_arn
+        properties[:CertificateArn] = cert_arn
       end
 
       {
-        domain_name: {
-          type: "AWS::ApiGateway::DomainName",
-          properties: properties
+        DomainName: {
+          Type: "AWS::ApiGateway::DomainName",
+          Properties: properties
         }
       }
     end
 
     def outputs
       {
-        "DomainName" => "!Ref DomainName",
+        DomainName: "!Ref DomainName",
       }
     end
 

data/lib/jets/{resource → cfn/resource}/api_gateway/method/authorization.rb

@@ -1,4 +1,4 @@
-class Jets::Resource::ApiGateway::Method
+class Jets::Cfn::Resource::ApiGateway::Method
   module Authorization
   private
     def authorizer_id

data/lib/jets/cfn/resource/api_gateway/method.rb

@@ -0,0 +1,92 @@
+# Converts a Jets::Route to a CloudFormation Jets::Cfn::Resource::ApiGateway::Method resource
+module Jets::Cfn::Resource::ApiGateway
+  class Method < Jets::Cfn::Base
+    include Authorization
+
+    # route - Jets::Route
+    def initialize(route)
+      @route = route
+    end
+
+    def definition
+      {
+        method_logical_id => {
+          Type: "AWS::ApiGateway::Method",
+          Properties: props
+        }
+      }
+    end
+
+    # Note: The {namespace} in
+    #   functions/${{namespace}LambdaFunction.Arn}/invocations
+    # is replaced by Jets::Cfn::Resource::Replacer
+    #
+    #   Jets::Cfn::Resource::ApiGateway::Method
+    #     Jets::Cfn::Resource (attributes delegate to resource)
+    #     Jets::Cfn::Resource => replacer
+    #
+    def props
+      function_logical_id = Jets.one_lambda_for_all_controllers? ?
+          "JetsControllerLambdaFunction" :
+          "{namespace}LambdaFunction"
+
+      resource_id = ResourceId.new(@route.path).resource_id
+      props = {
+        ResourceId: "!Ref #{resource_id}",
+        RestApiId: "!Ref #{RestApi.logical_id}",
+        HttpMethod: @route.http_method,
+        RequestParameters: {},
+        AuthorizationType: authorization_type,
+        ApiKeyRequired: api_key_required?,
+        Integration: {
+          IntegrationHttpMethod: "POST",
+          Type: "AWS_PROXY",
+          Uri: "!Sub arn:aws:apigateway:${AWS::Region}:lambda:path/2015-03-31/functions/${#{function_logical_id}}/invocations"
+        },
+        MethodResponses: []
+      }
+      props[:AuthorizerId] = authorizer_id if authorizer_id
+      props[:AuthorizationScopes] = authorization_scopes if authorization_scopes
+
+      props
+    end
+
+    def method_logical_id
+      # https://stackoverflow.com/questions/6104240/how-do-i-strip-non-alphanumeric-characters-from-a-string-and-keep-spaces
+      # Add path to the logical id to allow 2 different paths to be connected to the same controller action.
+      # Example:
+      #
+      #   root "jets/public#show"
+      #   any "*catchall", to: "jets/public#show"
+      #
+      # Without the path in the logical id, the logical id would be ShowApiMethod for both routes and only the
+      # last one would be created in the CloudFormation template.
+      path = @route.path.gsub('*','')
+              .gsub(/[^0-9a-z]/i, ' ')
+              .gsub(/\s+/, '_')
+      path = nil if path == ''
+      http_verb = @route.http_method.downcase
+      [http_verb, path, "api_method"].compact.join('_')
+    end
+
+    def replacements
+      # Mimic task to grab replacements
+      # Use functions/${namespace} in Uri
+      resources = [definition]
+      action_name = Jets.one_lambda_per_controller? ? "lambda_handler" : @route.action_name
+      task = Jets::Lambda::Definition.new(@route.controller_name, action_name,
+               resources: resources)
+      task.replacements
+    end
+
+  private
+
+    def controller_klass
+      @controller_klass ||= "#{controller_name}_controller".camelize.constantize
+    end
+
+    def controller_name
+      @controller_name ||= @route.to.split('#').first
+    end
+  end
+end

data/lib/jets/{resource → cfn/resource}/api_gateway/resource.rb

@@ -1,18 +1,20 @@
-module Jets::Resource::ApiGateway
-  class Resource < Jets::Resource::Base
+module Jets::Cfn::Resource::ApiGateway
+  class Resource < Jets::Cfn::Base
     def initialize(path, internal: false)
-      @path = path # Examples: "posts/:id/edit" or "posts"
+      # The original implementation uses path without the leading slash.
+      # Remove it here so the path_part is calculated correctly.
+      @path = path.delete_prefix('/') # Examples: "posts/:id/edit" or "posts"
       @internal = internal
     end
 
     def definition
       {
         resource_logical_id => {
-          type: "AWS::ApiGateway::Resource",
-          properties: {
-            parent_id: parent_id,
-            path_part: path_part,
-            rest_api_id: "!Ref #{RestApi.logical_id(@internal)}",
+          Type: "AWS::ApiGateway::Resource",
+          Properties: {
+            ParentId: parent_id,
+            PathPart: path_part,
+            RestApiId: "!Ref #{RestApi.logical_id(@internal)}",
           }
         }
       }
@@ -26,9 +28,11 @@ module Jets::Resource::ApiGateway
 
     def resource_logical_id
       if @path == ''
+        # Not including ApiResource in the logical id so it doesn't collide with a
+        # user-defined ApiResource that happens to be named RootResourceId.
         "RootResourceId"
       else
-        Jets::Resource.truncate_id path_logical_id(@path), "ApiResource"
+        Jets::Cfn::Resource.truncate_id(path_logical_id(@path), "ApiResource")
       end
     end
 
@@ -41,7 +45,7 @@ module Jets::Resource::ApiGateway
       if @path.include?('/') # posts/:id or posts/:id/edit
         parent_path = @path.split('/')[0..-2].join('/')
         parent_logical_id = path_logical_id(parent_path)
-        Jets::Resource.truncate_id(parent_logical_id, "ApiResource")
+        Jets::Cfn::Resource.truncate_id(parent_logical_id, "ApiResource")
       else
         "RootResourceId"
       end

data/lib/jets/cfn/resource/api_gateway/resource_id.rb

@@ -0,0 +1,25 @@
+module Jets::Cfn::Resource::ApiGateway
+  class ResourceId
+    def initialize(path)
+      @path = path
+    end
+
+    # Used by Method and Cors
+    def resource_id
+      @path == '/' ?
+        "RootResourceId" :
+        resource_logical_id.camelize + "ApiResource"
+    end
+
+    # Example: Posts
+    def resource_logical_id
+      camelized_path.underscore
+    end
+
+    def camelized_path
+      path = @path
+      path = "homepage" if path == '/'
+      path.gsub('/','_').gsub(':','').gsub('*','').gsub('.','').camelize
+    end
+  end
+end

data/lib/jets/{resource → cfn/resource}/api_gateway/rest_api/change_detection.rb

@@ -1,4 +1,4 @@
-class Jets::Resource::ApiGateway::RestApi
+class Jets::Cfn::Resource::ApiGateway::RestApi
   class ChangeDetection
     extend Memoist
     include Jets::AwsServices

data/lib/jets/{resource → cfn/resource}/api_gateway/rest_api/logical_id/message.rb

@@ -1,4 +1,4 @@
-class Jets::Resource::ApiGateway::RestApi::LogicalId
+class Jets::Cfn::Resource::ApiGateway::RestApi::LogicalId
   class Message
     def routes_changed
       <<~EOL

data/lib/jets/{resource → cfn/resource}/api_gateway/rest_api/logical_id.rb

@@ -1,4 +1,4 @@
-class Jets::Resource::ApiGateway::RestApi
+class Jets::Cfn::Resource::ApiGateway::RestApi
   class LogicalId
     extend Memoist
     include Jets::AwsServices

data/lib/jets/{resource → cfn/resource}/api_gateway/rest_api/routes/change/base.rb

@@ -1,4 +1,4 @@
-class Jets::Resource::ApiGateway::RestApi::Routes::Change
+class Jets::Cfn::Resource::ApiGateway::RestApi::Routes::Change
   class Base
     extend Memoist
     include Jets::AwsServices
@@ -14,11 +14,13 @@ class Jets::Resource::ApiGateway::RestApi::Routes::Change
       return [] if data.nil?
 
       data.map do |item|
-        method = item.dig('options','method') || item.dig('options','http_method')
+        http_method = item.dig('options','method') || item.dig('options','http_method')
         Jets::Router::Route.new(
+          http_method: http_method,
           path: item['path'],
-          method: method,
           to: item['to'],
+          engine: item['engine'],
+          internal: item['internal'],
         )
       end
     end

data/lib/jets/{resource → cfn/resource}/api_gateway/rest_api/routes/change/media_types.rb

@@ -1,11 +1,11 @@
-class Jets::Resource::ApiGateway::RestApi::Routes::Change
+class Jets::Cfn::Resource::ApiGateway::RestApi::Routes::Change
   class MediaTypes < Base
     def changed?
       current_binary_media_types != new_binary_media_types
     end
 
     def new_binary_media_types
-      rest_api = Jets::Resource::ApiGateway::RestApi.new
+      rest_api = Jets::Cfn::Resource::ApiGateway::RestApi.new
       rest_api.binary_media_types
     end
     memoize :new_binary_media_types

data/lib/jets/{resource → cfn/resource}/api_gateway/rest_api/routes/change/page.rb

@@ -1,4 +1,4 @@
-class Jets::Resource::ApiGateway::RestApi::Routes::Change
+class Jets::Cfn::Resource::ApiGateway::RestApi::Routes::Change
   class Page < Base
     def changed?
       route_page_moved? || old_api_template?
@@ -33,15 +33,15 @@ class Jets::Resource::ApiGateway::RestApi::Routes::Change
     # logical id to page map
     # Important: In Cfn::Builders::ApiGatewayBuilder, the add_gateway_routes and ApiResourcesBuilder needs to run
     # before the parent add_gateway_rest_api method.
-    def local_logical_ids_map(path_expression="#{Jets::Names.template_path_prefix}-api-resources-*.yml")
+    def local_logical_ids_map(path_expression="#{Jets::Names.templates_folder}/api-resources-*.yml")
       logical_ids = {} # logical id => page number
 
       Dir.glob(path_expression).each do |path|
         md = path.match(/-api-resources-(\d+).yml/)
         page_number = md[1]
 
-        template = Jets::Cfn::BuiltTemplate.get(path)
-        template['Resources'].keys.each do |logical_id|
+        template = Jets::Cfn::Template.load_file(path)
+        template[:Resources].keys.each do |logical_id|
           logical_ids[logical_id] = page_number
         end
       end
@@ -55,7 +55,7 @@ class Jets::Resource::ApiGateway::RestApi::Routes::Change
 
       parent_resources.each do |resource|
         stack_name = resource.physical_resource_id # full physical id can be used as stack name also
-        regexp = Regexp.new("#{Jets.config.project_namespace}-ApiResources(\\d+)-") # tricky to escape \d pattern
+        regexp = Regexp.new("#{Jets.project_namespace}-ApiResources(\\d+)-") # tricky to escape \d pattern
         md = stack_name.match(regexp)
         if md
           page_number = md[1]

data/lib/jets/{resource → cfn/resource}/api_gateway/rest_api/routes/change/to.rb

@@ -1,10 +1,15 @@
 # Detects route to changes
-class Jets::Resource::ApiGateway::RestApi::Routes::Change
+class Jets::Cfn::Resource::ApiGateway::RestApi::Routes::Change
   class To < Base
     def changed?
       deployed_routes.each do |deployed_route|
+        next if deployed_route.engine   # skip engine routes
+        next if deployed_route.internal # skip internal routes
+
         new_route = find_comparable_route(deployed_route)
-        if new_route && new_route.to != deployed_route.to
+        next unless new_route
+
+        if new_route.to != deployed_route.to
           # change in already deployed route has been detected, requires bluegreen deploy
           return true
         end
@@ -18,7 +23,7 @@ class Jets::Resource::ApiGateway::RestApi::Routes::Change
     def find_comparable_route(deployed_route)
       new_routes.find do |new_route|
         new_route.path == deployed_route.path &&
-        new_route.method == deployed_route.method
+        new_route.http_method == deployed_route.http_method
       end
     end
   end

data/lib/jets/{resource → cfn/resource}/api_gateway/rest_api/routes/change/variable.rb

@@ -1,5 +1,5 @@
 # Detects route variable changes
-class Jets::Resource::ApiGateway::RestApi::Routes::Change
+class Jets::Cfn::Resource::ApiGateway::RestApi::Routes::Change
   class Variable < Base
     def changed?
       changed = false
@@ -29,7 +29,7 @@ class Jets::Resource::ApiGateway::RestApi::Routes::Change
     end
 
     def collision
-      @collision ||= Jets::Resource::ApiGateway::RestApi::Routes::Collision.new
+      @collision ||= Jets::Cfn::Resource::ApiGateway::RestApi::Routes::Collision.new
     end
   end
 end

data/lib/jets/cfn/resource/api_gateway/rest_api/routes/change.rb

@@ -0,0 +1,27 @@
+# Detects route changes
+class Jets::Cfn::Resource::ApiGateway::RestApi::Routes
+  class Change
+    include Jets::AwsServices
+
+    def changed?
+      return false unless parent_stack_exists?
+      return true if reset?
+
+      # Note: Variable.changed? will likely always true in one_apigw_method_for_all_routes mode
+      # since parent variables are allowed to vary in Jets v5.
+      if Jets.config.cfn.build.routes == "one_apigw_method_for_all_routes"
+        MediaTypes.changed? || To.changed?
+      else
+        MediaTypes.changed? || To.changed? || Variable.changed? || Page.changed?
+      end
+    end
+
+    def reset?
+      ENV['JETS_RESET'] || ENV['JETS_API_REPLACE'] || ENV['JETS_REPLACE_API']
+    end
+
+    def parent_stack_exists?
+      stack_exists?(Jets::Names.parent_stack_name)
+    end
+  end
+end

data/lib/jets/{resource → cfn/resource}/api_gateway/rest_api/routes/collision/variable_exception.rb

@@ -1,4 +1,4 @@
-class Jets::Resource::ApiGateway::RestApi::Routes::Collision
+class Jets::Cfn::Resource::ApiGateway::RestApi::Routes::Collision
   class VariableException < RuntimeError
     def initialize(message="Route variable collisions")
       super(message)

data/lib/jets/{resource → cfn/resource}/api_gateway/rest_api/routes/collision.rb

@@ -1,5 +1,5 @@
 # Detects path variable collisions
-class Jets::Resource::ApiGateway::RestApi::Routes
+class Jets::Cfn::Resource::ApiGateway::RestApi::Routes
   class Collision
     attr_reader :collisions
     def initialize

data/lib/jets/{resource → cfn/resource}/api_gateway/rest_api/routes.rb

@@ -1,4 +1,4 @@
-class Jets::Resource::ApiGateway::RestApi
+class Jets::Cfn::Resource::ApiGateway::RestApi
   class Routes
     def self.changed?
       Change.new.changed?

data/lib/jets/{resource → cfn/resource}/api_gateway/rest_api.rb

@@ -1,18 +1,18 @@
-module Jets::Resource::ApiGateway
-  class RestApi < Jets::Resource::Base
+module Jets::Cfn::Resource::ApiGateway
+  class RestApi < Jets::Cfn::Base
     def definition
       properties = {
-        name: Jets::Names.gateway_api_name,
-        endpoint_configuration: { types: endpoint_types }
+        Name: Jets::Names.gateway_api_name,
+        EndpointConfiguration: { Types: endpoint_types }
       }
-      properties[:endpoint_configuration][:vpc_endpoint_ids] = vpce_ids if vpce_ids
-      properties[:binary_media_types] = binary_media_types if binary_media_types
-      properties[:policy] = endpoint_policy if endpoint_policy
+      properties[:EndpointConfiguration][:VpcEndpointIds] = vpce_ids if vpce_ids
+      properties[:BinaryMediaTypes] = binary_media_types if binary_media_types
+      properties[:Policy] = endpoint_policy if endpoint_policy
 
       {
         internal_logical_id => {
-          type: "AWS::ApiGateway::RestApi",
-          properties: properties
+          Type: "AWS::ApiGateway::RestApi",
+          Properties: properties
         }
       }
     end
@@ -32,9 +32,9 @@ module Jets::Resource::ApiGateway
 
     def outputs
       {
-        "RestApi" => "!Ref #{internal_logical_id}",
-        "Region" => "!Ref AWS::Region",
-        "RootResourceId" => "!GetAtt #{internal_logical_id}.RootResourceId",
+        RestApi: "!Ref #{internal_logical_id}",
+        Region: "!Ref AWS::Region",
+        RootResourceId: "!GetAtt #{internal_logical_id}.RootResourceId",
       }
     end
 

data/lib/jets/{resource → cfn/resource}/associated.rb

@@ -1,6 +1,6 @@
 # Does not do full expansion, mainly a container that holds the definition and
 # standardizes it without camelizing it.
-class Jets::Resource
+class Jets::Cfn::Resource
   class Associated
     extend Memoist