easol-canvas 1.1.0 → 1.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5fe5e6dba572cf93a26d99560c7f36cd3f7f2188c019f5c5bd2429c6b897dd20
-  data.tar.gz: df34c9f374a73b17f47e4486345d6f6c21f4f184640d72976f664e3bf873e4ee
+  metadata.gz: 5cc9522b87ab434f7121dfd557aa987b820c9934486da6c96e8c7706ac4f92b4
+  data.tar.gz: e922ef25a0d8a916a82a7b9c4d8b4faf342088f6da65077518d72b68bf64eeab
 SHA512:
-  metadata.gz: d42d0a2cecd8040521956b2bb2a7380c08471e1aebfaf162a652b61a532bda67de5426ffba517b75bbc50449ac65eccbc6df78e009103c6ccc5e86874184c732
-  data.tar.gz: b4590073ce09eef3e258f924b66bfb0f996b88dacfbfbc72c1d0120f8fe96fafc30911015ae31df76836d665efea529db0af9bf3184cd6212747993f8c3a5d1e
+  metadata.gz: c33da855f7ad39a4dad8172fc0942c9c60d4af81acf6c677bddb9c55dcacd9759392fa9c24e65910d76228a760bad31c244b865e8f83e52623ffdf26282f99b6
+  data.tar.gz: e4d2f3ebae85a1ed14716a1f5514bd3211ae130b973475bbcaf3b52c946cb25db90cdcfbd75143db9984600e3604071d153eb1766e79388e172a2307e09abcfb

data/lib/canvas/checks/required_files_check.rb

@@ -14,11 +14,18 @@ module Canvas
 
     def run
       REQUIRED_FILES.each do |filename|
-        next unless Dir.glob(filename).empty?
+        file_paths = Dir.glob(filename)
 
-        @offenses << Offense.new(
-          message: "Missing file: #{filename}"
-        )
+
+        if file_paths.empty?
+          @offenses << Offense.new(
+            message: "Missing file: #{filename}"
+          )
+        elsif File.zero?(file_paths.first)
+          @offenses << Offense.new(
+            message: "Empty file: #{file_paths.first}"
+          )
+        end
       end
     end
   end

data/lib/canvas/checks/valid_block_schemas_check.rb

@@ -37,7 +37,8 @@ module Canvas
 
     def validate_format(filename, front_matter)
       return true if front_matter.is_a?(Hash) &&
-                     front_matter.values.all? { |attr| attr.is_a?(Hash) }
+                     (front_matter.key?("attributes") ? front_matter["attributes"] : front_matter).values.all? { |attr| attr.is_a?(Hash) }
+
 
       @offenses << Offense.new(
         message: "Invalid Block Schema: #{filename} - \nSchema is not in a valid format"
@@ -45,8 +46,7 @@ module Canvas
       false
     end
 
-    def validate_schema(filename, front_matter, custom_types)
-      schema = extract_schema(front_matter)
+    def validate_schema(filename, schema, custom_types)
       validator = Validator::BlockSchema.new(schema: schema, custom_types: custom_types)
       return if validator.validate
 
@@ -62,11 +62,5 @@ module Canvas
       front_matter = extractor.front_matter
       front_matter.nil? ? {} : YAML.safe_load(front_matter)
     end
-
-    def extract_schema(front_matter)
-      {
-        "attributes" => Canvas::ExpandAttributes.call(front_matter)
-      }
-    end
   end
 end

data/lib/canvas/checks/valid_sass_check.rb

@@ -0,0 +1,30 @@
+# frozen_string_literal: true
+
+module Canvas
+  # :documented:
+  #
+  # This check will find all files ending in .css, .scss or .sass
+  # and run them through the sass validator - {Canvas::Validator::Sass}.
+  class ValidSassCheck < Check
+    def run
+      sass_files.each do |filename|
+        file = File.read(filename)
+        validator = Validator::Sass.new(file)
+
+        next if validator.validate
+
+        validator.errors.each do |message|
+          @offenses << Offense.new(
+            message: "Invalid Sass: #{filename} - \n#{message}"
+          )
+        end
+      end
+    end
+
+    private
+
+    def sass_files
+      Dir.glob("**/*.{css,scss,sass}")
+    end
+  end
+end

data/lib/canvas/services/expand_attributes.rb

@@ -30,6 +30,7 @@ module Canvas
       # @return [Array<Hash>] array of hashes that represent each attribute
       def call(attributes_hash)
         return [] if attributes_hash.nil?
+        return attributes_hash if attributes_hash.is_a?(Array)
 
         attributes_hash.each_with_object([]) do |(name, attribute_hash), attrs|
           attrs << attribute_hash.merge("name" => name)

data/lib/canvas/validators/block_schema.rb

@@ -23,21 +23,24 @@ module Canvas
     #   ]
     # }
     class BlockSchema
-      PERMITTED_KEYS = %w[attributes].freeze
+      PERMITTED_KEYS = %w[attributes layout].freeze
 
       attr_reader :errors, :schema
 
       # @param schema [Hash] the schema to be validated
       # @param custom_types [Array<Hash>] a list of custom types
       def initialize(schema:, custom_types: [])
-        @schema = schema
+        @schema = normalize_schema(schema)
         @custom_types = custom_types
         @errors = []
       end
 
       def validate
+        @errors = []
+
         if ensure_valid_format
           ensure_no_unrecognized_keys
+          ensure_layout_is_valid
           ensure_attributes_are_valid
         end
 
@@ -54,6 +57,16 @@ module Canvas
         false
       end
 
+      def ensure_layout_is_valid
+        return true unless schema["layout"]
+
+        layout_validator = LayoutSchema.new(schema: @schema)
+        return true if layout_validator.validate
+
+        @errors += layout_validator.errors
+        false
+      end
+
       def ensure_no_unrecognized_keys
         unrecognized_keys = schema.keys - PERMITTED_KEYS
         return true if unrecognized_keys.empty?
@@ -81,6 +94,19 @@ module Canvas
         schema["attributes"].is_a?(Array) &&
           schema["attributes"].all? { |attr| attr.is_a?(Hash) }
       end
+
+      def normalize_schema(schema)
+        if schema.key?("attributes")
+          {
+            **schema,
+            "attributes" => Canvas::ExpandAttributes.call(schema["attributes"])
+          }
+        else
+          {
+            "attributes" => Canvas::ExpandAttributes.call(schema)
+          }
+        end
+      end
     end
   end
 end

data/lib/canvas/validators/custom_type.rb

@@ -38,6 +38,7 @@ module Canvas
           ensure_has_required_keys &&
           ensure_no_unrecognized_keys &&
           ensure_keys_are_correct_types &&
+          ensure_key_value_is_not_reserved &&
           ensure_no_duplicate_attributes &&
           ensure_attributes_are_valid &&
           ensure_first_attribute_not_array
@@ -91,6 +92,17 @@ module Canvas
         true
       end
 
+      # Ensuring the value for key doesn't clash with our primitive type keys.
+      # See {Canvas::Constants::PRIMITIVE_TYPES}
+      def ensure_key_value_is_not_reserved
+        if schema["key"] && Constants::PRIMITIVE_TYPES.include?(schema["key"].downcase)
+          @errors << "\"key\" can't be one of these reserved words: #{Constants::PRIMITIVE_TYPES.join(', ')}"
+          return false
+        end
+
+        true
+      end
+
       def ensure_no_duplicate_attributes
         attribute_names = schema["attributes"].map { |a| a["name"] }
         duplicated_names = attribute_names.select { |a| attribute_names.count(a) > 1 }.uniq

data/lib/canvas/validators/layout_schema.rb

@@ -0,0 +1,123 @@
+# frozen_string_literal: true
+
+require "json"
+require "json-schema"
+
+module Canvas
+  module Validator
+    # :documented:
+    # This class is used to validate a layout definition, part of block schema.
+    # Example of a valid layout definition:
+    # {
+    #   "layout" => [
+    #   {
+    #      "label" => "Design",
+    #      "type" => "tab",
+    #     "elements" => [
+    #       "heading",
+    #       {
+    #         "type" => "accordion",
+    #         "label" => "Logo",
+    #         "elements" => [
+    #           "description",
+    #           { "type" => "attribute", "name" => "logo_alt" },
+    #           "title"
+    #         ]
+    #       }
+    #     ]
+    #   }]
+    # }
+    class LayoutSchema
+      attr_reader :errors
+
+      def initialize(schema:)
+        @schema = schema
+        @errors = []
+      end
+
+      def validate
+        @errors = []
+
+        if ensure_valid_format
+          ensure_no_unrecognized_keys
+          ensure_no_duplicate_keys
+        end
+
+        @errors.empty?
+      end
+
+      private
+
+      def ensure_no_duplicate_keys
+        attributes = gather_attributes_from_layout_schema
+        duplicates =
+          attributes
+          .group_by { |(key)| key }
+          .filter { |key, usage| usage.size > 1 }
+
+        unless duplicates.empty?
+          duplicates.each do |attribute, usage|
+            @errors << "Duplicated attribute key `#{attribute}` found. Location: #{usage.map { |(_, location)| location }.join(", ")}"
+          end
+        end
+      end
+
+      def ensure_no_unrecognized_keys
+        attributes = gather_attributes_from_layout_schema
+        defined_attributes = @schema["attributes"]&.map { |definition| normalize_attribute(definition["name"]) } || []
+
+        attributes.each do |attribute, location|
+          @errors << "Unrecognized attribute `#{attribute}`. Location: #{location}" unless defined_attributes.include?(attribute)
+        end
+      end
+
+      def gather_attributes_from_layout_schema
+        attribute_keys = []
+
+        fetch_attribute_type = ->(node, path) {
+          if node.is_a?(Hash) && node.key?("elements")
+            node["elements"].each_with_index do |element, i|
+              current_path = "#{path}/elements/#{i}"
+              fetch_attribute_type.call(element, current_path)
+            end
+          else
+            attribute_keys << [
+              normalize_attribute(node.is_a?(Hash) ? node["name"] : node),
+              path
+            ]
+          end
+        }
+
+        layout_schema.each_with_index do |tab, i|
+          current_path = "layout/#{i}"
+          fetch_attribute_type.call(tab, current_path)
+        end
+
+        attribute_keys
+      end
+
+      def ensure_valid_format
+        result = JSON::Validator.fully_validate(schema_definition, { "layout" => layout_schema }, strict: true, clear_cache: true)
+
+        return true if result.empty?
+
+        @errors += result
+        false
+      end
+
+      def layout_schema
+        @schema["layout"] || []
+      end
+
+      def schema_definition
+        File.read(
+          File.join(File.dirname(__FILE__), "../../../", "schema_definitions", "block_layout.json")
+        )
+      end
+
+      def normalize_attribute(name)
+        name.strip.downcase
+      end
+    end
+  end
+end

data/lib/canvas/validators/sass.rb

@@ -0,0 +1,26 @@
+# frozen_string_literal: true
+
+require "sassc"
+
+module Canvas
+  module Validator
+    # :documented:
+    #
+    # This validator can be used to validate Sass.
+    class Sass
+      attr_reader :errors
+
+      def initialize(file)
+        @file = file
+      end
+
+      def validate
+        SassC::Engine.new(@file, style: :compressed).render
+        true
+      rescue SassC::SyntaxError => e
+        @errors = [e.message]
+        false
+      end
+    end
+  end
+end

data/lib/canvas/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Canvas
-  VERSION = "1.1.0"
+  VERSION = "1.4.1"
 end

data/lib/canvas.rb

@@ -1,12 +1,14 @@
 # frozen_string_literal: true
 
-require_relative "canvas/version"
-require_relative "canvas/constants"
-require_relative "canvas/cli"
-require_relative "canvas/lint"
-require_relative "canvas/check"
-require_relative "canvas/offense"
-require_relative "canvas/checks"
+module Canvas
+  autoload :Check, "canvas/check"
+  autoload :Checks, "canvas/checks"
+  autoload :Cli, "canvas/cli"
+  autoload :Constants, "canvas/constants"
+  autoload :Lint, "canvas/lint"
+  autoload :Offense, "canvas/offense"
+  autoload :Version, "canvas/version"
+end
 
 # We need to ensure Canvas::Validator::SchemaAttribute::Base is required first
 require_relative "canvas/validators/schema_attributes/base"
@@ -26,10 +28,8 @@ Canvas::Checks.register(Canvas::RequiredFilesCheck)
 Canvas::Checks.register(Canvas::ValidHtmlCheck)
 Canvas::Checks.register(Canvas::ValidLiquidCheck)
 Canvas::Checks.register(Canvas::ValidJsonCheck)
+Canvas::Checks.register(Canvas::ValidSassCheck)
 Canvas::Checks.register(Canvas::ValidBlockSchemasCheck)
 Canvas::Checks.register(Canvas::ValidMenuSchemaCheck)
 Canvas::Checks.register(Canvas::ValidFooterSchemaCheck)
 Canvas::Checks.register(Canvas::ValidCustomTypesCheck)
-
-module Canvas
-end

data/lib/easol/canvas.rb

@@ -0,0 +1,4 @@
+# frozen_string_literal: true
+
+# Enables using this gem "as is"
+require_relative "../canvas"

data/schema_definitions/block_layout.json

@@ -0,0 +1,67 @@
+{
+  "type": "object",
+  "properties": {
+    "layout": {
+      "type": "array",
+      "items": { "$ref": "#/$defs/tab" }
+    }
+  },
+  "$defs": {
+    "tab": {
+      "type": "object",
+      "required": ["type", "label"],
+      "properties": {
+        "type": {
+          "type": "string",
+          "const": "tab"
+        },
+        "label": {
+          "type": "string"
+        },
+        "elements": {
+          "type": "array",
+          "minItems": 1,
+          "items": {
+            "oneOf": [
+              { "type": "string" },
+              { "$ref": "#/$defs/component" },
+              { "$ref": "#/$defs/attribute" }
+            ]
+          }
+        }
+      }
+    },
+    "component": {
+      "type": "object",
+      "required": ["label", "type"],
+      "properties": {
+        "type": {
+          "type": "string",
+          "enum": ["accordion"]
+        },
+        "label": {
+          "type": "string"
+        },
+        "elements": {
+          "type": "array",
+          "items": {
+            "oneOf": [{ "type": "string" }, { "$ref": "#/$defs/attribute" }]
+          }
+        }
+      }
+    },
+    "attribute": {
+      "type": "object",
+      "required": ["name", "type"],
+      "properties": {
+        "type": {
+          "type": "string",
+          "const": "attribute"
+        },
+        "name": {
+          "type": "string"
+        }
+      }
+    }
+  }
+}

metadata

@@ -1,15 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: easol-canvas
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.4.1
 platform: ruby
 authors:
 - Kyle Byrne
 - Ian Mooney
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-06-01 00:00:00.000000000 Z
+date: 2022-07-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: thor
@@ -67,6 +67,34 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '5.3'
+- !ruby/object:Gem::Dependency
+  name: sassc
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.4'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.4'
+- !ruby/object:Gem::Dependency
+  name: json-schema
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3'
 description: |
   Canvas is a command line tool to help with building themes for Easol.
   It provides tooling to check theme directories for errors and to make sure
@@ -90,6 +118,7 @@ files:
 - lib/canvas/checks/valid_json_check.rb
 - lib/canvas/checks/valid_liquid_check.rb
 - lib/canvas/checks/valid_menu_schema_check.rb
+- lib/canvas/checks/valid_sass_check.rb
 - lib/canvas/cli.rb
 - lib/canvas/constants.rb
 - lib/canvas/lint.rb
@@ -102,8 +131,10 @@ files:
 - lib/canvas/validators/footer_schema.rb
 - lib/canvas/validators/html.rb
 - lib/canvas/validators/json.rb
+- lib/canvas/validators/layout_schema.rb
 - lib/canvas/validators/liquid.rb
 - lib/canvas/validators/menu_schema.rb
+- lib/canvas/validators/sass.rb
 - lib/canvas/validators/schema_attribute.rb
 - lib/canvas/validators/schema_attributes/base.rb
 - lib/canvas/validators/schema_attributes/color.rb
@@ -118,11 +149,13 @@ files:
 - lib/canvas/validators/schema_attributes/select.rb
 - lib/canvas/validators/schema_attributes/variant.rb
 - lib/canvas/version.rb
+- lib/easol/canvas.rb
+- schema_definitions/block_layout.json
 homepage: https://rubygems.org/gems/easol-canvas
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -137,8 +170,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.6
-signing_key: 
+rubygems_version: 3.2.33
+signing_key:
 specification_version: 4
 summary: CLI to help with building themes for Easol
 test_files: []