chore: Address Rubocop offenses

Author: blocknotes

lib/tiny_admin/actions/index.rb

@@ -52,7 +52,7 @@ def evaluate_options(options)
       def prepare_filters(fields)
         filters = (options[:filters] || []).map { _1.is_a?(Hash) ? _1 : { field: _1 } }
         filters = filters.each_with_object({}) { |filter, result| result[filter[:field]] = filter }
-        values = (params['q'] || {})
+        values = params['q'] || {}
         fields.each_with_object({}) do |(name, field), result|
           result[field] = { value: values[name], filter: filters[name] } if filters.key?(name)
         end

lib/tiny_admin/plugins/base_repository.rb

@@ -3,7 +3,8 @@
 module TinyAdmin
   module Plugins
     class BaseRepository
-      RecordNotFound = Class.new(StandardError)
+      class RecordNotFound < StandardError
+      end
 
       attr_reader :model
 

spec/lib/tiny_admin/actions/basic_action_spec.rb

@@ -11,35 +11,36 @@
     it "handles simple string field names" do
       result = action.attribute_options(["id", "name"])
       expect(result).to eq(
-        "id" => {field: "id"},
-        "name" => {field: "name"}
+        "id" => { field: "id" },
+        "name" => { field: "name" }
       )
     end
 
     it "handles single-entry hash with method shorthand" do
-      result = action.attribute_options([{title: "downcase, capitalize"}])
+      result = action.attribute_options([{ title: "downcase, capitalize" }])
       expect(result).to eq(
-        "title" => {field: "title", method: "downcase, capitalize"}
+        "title" => { field: "title", method: "downcase, capitalize" }
       )
     end
 
     it "handles multi-entry hash with explicit field key" do
-      result = action.attribute_options([{field: "author_id", link_to: "authors"}])
+      result = action.attribute_options([{ field: "author_id", link_to: "authors" }])
       expect(result).to eq(
-        "author_id" => {field: "author_id", link_to: "authors"}
+        "author_id" => { field: "author_id", link_to: "authors" }
       )
     end
 
     it "handles mixed input types" do
-      result = action.attribute_options([
+      opts = [
         "id",
-        {title: "upcase"},
-        {field: "created_at", method: "strftime, %Y-%m-%d"}
-      ])
+        { title: "upcase" },
+        { field: "created_at", method: "strftime, %Y-%m-%d" }
+      ]
+      result = action.attribute_options(opts)
       expect(result).to eq(
-        "id" => {field: "id"},
-        "title" => {field: "title", method: "upcase"},
-        "created_at" => {field: "created_at", method: "strftime, %Y-%m-%d"}
+        "id" => { field: "id" },
+        "title" => { field: "title", method: "upcase" },
+        "created_at" => { field: "created_at", method: "strftime, %Y-%m-%d" }
       )
     end
   end

spec/lib/tiny_admin/field_spec.rb

@@ -21,7 +21,7 @@
     end
 
     it "uses the provided options when given" do
-      options = {method: "downcase"}
+      options = { method: "downcase" }
       field = described_class.create_field(name: "name", options: options)
       expect(field.options).to eq(options)
     end
@@ -39,7 +39,7 @@
 
   describe "#apply_call_option" do
     it "chains method calls on the target" do
-      field = described_class.new(name: "title", title: "Title", type: :string, options: {call: "to_s, downcase"})
+      field = described_class.new(name: "title", title: "Title", type: :string, options: { call: "to_s, downcase" })
       expect(field.apply_call_option(42)).to eq("42")
     end
 
@@ -49,7 +49,7 @@
     end
 
     it "handles nil target safely via safe navigation" do
-      field = described_class.new(name: "title", title: "Title", type: :string, options: {call: "nonexistent"})
+      field = described_class.new(name: "title", title: "Title", type: :string, options: { call: "nonexistent" })
       expect(field.apply_call_option(nil)).to be_nil
     end
   end
@@ -66,7 +66,7 @@
     end
 
     it "applies the helper method from options" do
-      field = described_class.new(name: "name", title: "Name", type: :string, options: {method: "downcase"})
+      field = described_class.new(name: "name", title: "Name", type: :string, options: { method: "downcase" })
       allow(TinyAdmin.settings).to receive(:helper_class).and_return(TinyAdmin::Support)
       expect(field.translate_value("HELLO")).to eq("hello")
     end
@@ -81,15 +81,15 @@ def self.upcase(value, options: [])
 
       field = described_class.new(
         name: "name", title: "Name", type: :string,
-        options: {method: "upcase", converter: "TestConverter"}
+        options: { method: "upcase", converter: "TestConverter" }
       )
       expect(field.translate_value("hello")).to eq("HELLO")
     end
 
     it "passes additional args to the method" do
       field = described_class.new(
         name: "value", title: "Value", type: :float,
-        options: {method: "round, 1"}
+        options: { method: "round, 1" }
       )
       allow(TinyAdmin.settings).to receive(:helper_class).and_return(TinyAdmin::Support)
       expect(field.translate_value(3.456)).to eq(3.5)

spec/lib/tiny_admin/plugins/active_record_repository_spec.rb

@@ -31,7 +31,7 @@
     end
 
     it "returns only specified fields when options given" do
-      options = {"name" => {}, "email" => {}}
+      options = { "name" => {}, "email" => {} }
       fields = repository.fields(options: options)
       expect(fields.keys).to eq(["name", "email"])
     end
@@ -72,7 +72,7 @@
     end
 
     it "returns only specified fields when fields given", :aggregate_failures do
-      attrs = repository.index_record_attrs(author, fields: {"name" => nil, "email" => nil})
+      attrs = repository.index_record_attrs(author, fields: { "name" => nil, "email" => nil })
       expect(attrs.keys).to eq(["name", "email"])
       expect(attrs["name"]).to eq(author.name)
     end
@@ -93,7 +93,7 @@
     end
 
     it "sorts when sort option given" do
-      records, = repository.list(page: 1, limit: 10, sort: {name: :desc})
+      records, = repository.list(page: 1, limit: 10, sort: { name: :desc })
       names = records.map(&:name)
       expect(names).to eq(names.sort.reverse)
     end
@@ -104,7 +104,7 @@
 
     it "filters string fields with LIKE" do
       title_field = TinyAdmin::Field.new(name: "title", title: "Title", type: :string)
-      filters = {title_field => {value: "post 1"}}
+      filters = { title_field => { value: "post 1" } }
       results = post_repository.apply_filters(Post.all, filters)
       results.each do |post|
         expect(post.title.downcase).to include("post 1")
@@ -114,7 +114,7 @@
     it "filters non-string fields with equality" do
       author = Author.first
       author_field = TinyAdmin::Field.new(name: "author_id", title: "Author", type: :integer)
-      filters = {author_field => {value: author.id}}
+      filters = { author_field => { value: author.id } }
       results = post_repository.apply_filters(Post.all, filters)
       results.each do |post|
         expect(post.author_id).to eq(author.id)
@@ -123,14 +123,14 @@
 
     it "skips filters with nil or empty values" do
       title_field = TinyAdmin::Field.new(name: "title", title: "Title", type: :string)
-      filters = {title_field => {value: nil}}
+      filters = { title_field => { value: nil } }
       results = post_repository.apply_filters(Post.all, filters)
       expect(results.count).to eq(Post.count)
     end
 
     it "sanitizes SQL LIKE input" do
       title_field = TinyAdmin::Field.new(name: "title", title: "Title", type: :string)
-      filters = {title_field => {value: "100%"}}
+      filters = { title_field => { value: "100%" } }
       # Should not raise or cause SQL injection
       expect { post_repository.apply_filters(Post.all, filters).to_a }.not_to raise_error
     end

spec/lib/tiny_admin/section_spec.rb

@@ -20,8 +20,8 @@
     end
 
     it "stores provided options" do
-      section = described_class.new(name: "Link", slug: "link", options: {target: "_blank"})
-      expect(section.options).to eq({target: "_blank"})
+      section = described_class.new(name: "Link", slug: "link", options: { target: "_blank" })
+      expect(section.options).to eq({ target: "_blank" })
     end
   end
 end

spec/lib/tiny_admin/settings_spec.rb

@@ -49,7 +49,7 @@
     end
 
     it "converts nested string class names to constants" do
-      settings[:authentication] = {plugin: "TinyAdmin::Plugins::NoAuth"}
+      settings[:authentication] = { plugin: "TinyAdmin::Plugins::NoAuth" }
       expect(settings[:authentication, :plugin]).to eq(TinyAdmin::Plugins::NoAuth)
     end
 

spec/lib/tiny_admin/store_spec.rb

@@ -23,7 +23,7 @@
   describe "#prepare_sections" do
     context "with a content section" do
       let(:sections) do
-        [{slug: "about", name: "About", type: :content, content: "<h1>About</h1>"}]
+        [{ slug: "about", name: "About", type: :content, content: "<h1>About</h1>" }]
       end
 
       it "adds to pages and navbar", :aggregate_failures do
@@ -42,7 +42,7 @@
       end
 
       let(:sections) do
-        [{slug: "dashboard", name: "Dashboard", type: :page, page: page_class}]
+        [{ slug: "dashboard", name: "Dashboard", type: :page, page: page_class }]
       end
 
       it "adds to pages and navbar", :aggregate_failures do
@@ -56,7 +56,7 @@
 
     context "with a resource section" do
       let(:sections) do
-        [{slug: "authors", name: "Authors", type: :resource, model: Author}]
+        [{ slug: "authors", name: "Authors", type: :resource, model: Author }]
       end
 
       it "adds to resources and navbar", :aggregate_failures do
@@ -71,7 +71,7 @@
 
     context "with a hidden resource section" do
       let(:sections) do
-        [{slug: "secret", name: "Secret", type: :resource, model: Author, options: [:hidden]}]
+        [{ slug: "secret", name: "Secret", type: :resource, model: Author, options: [:hidden] }]
       end
 
       it "adds to resources but not to visible navbar items", :aggregate_failures do
@@ -84,14 +84,14 @@
 
     context "with a url section" do
       let(:sections) do
-        [{slug: "google", name: "Google", type: :url, url: "https://google.com", options: {target: "_blank"}}]
+        [{ slug: "google", name: "Google", type: :url, url: "https://google.com", options: { target: "_blank" } }]
       end
 
       it "adds to navbar with the url as path", :aggregate_failures do
         store.prepare_sections(sections, logout: nil)
         expect(store.navbar.size).to eq(1)
         expect(store.navbar.first.path).to eq("https://google.com")
-        expect(store.navbar.first.options).to eq({target: "_blank"})
+        expect(store.navbar.first.options).to eq({ target: "_blank" })
       end
 
       it "does not add to pages or resources", :aggregate_failures do
@@ -105,7 +105,7 @@
       let(:section_module) do
         mod = Class.new do
           def self.to_h
-            {slug: "dynamic", name: "Dynamic", type: :content, content: "<p>Hi</p>"}
+            { slug: "dynamic", name: "Dynamic", type: :content, content: "<p>Hi</p>" }
           end
         end
         mod
@@ -143,9 +143,9 @@ def self.to_h
     context "with multiple section types" do
       let(:sections) do
         [
-          {slug: "about", name: "About", type: :content, content: "<p>Test</p>"},
-          {slug: "users", name: "Users", type: :resource, model: Author},
-          {slug: "ext", name: "External", type: :url, url: "https://example.com"}
+          { slug: "about", name: "About", type: :content, content: "<p>Test</p>" },
+          { slug: "users", name: "Users", type: :resource, model: Author },
+          { slug: "ext", name: "External", type: :url, url: "https://example.com" }
         ]
       end
 

spec/lib/tiny_admin/utils_spec.rb

@@ -9,7 +9,7 @@
     end
 
     it "converts nested hash params to bracket notation", :aggregate_failures do
-      result = utils_instance.params_to_s("q" => {"title" => "test", "author" => "john"})
+      result = utils_instance.params_to_s("q" => { "title" => "test", "author" => "john" })
       expect(result).to include("q[title]=test")
       expect(result).to include("q[author]=john")
     end
@@ -19,7 +19,7 @@
     end
 
     it "handles mixed flat and nested params", :aggregate_failures do
-      result = utils_instance.params_to_s("p" => "1", "q" => {"name" => "test"})
+      result = utils_instance.params_to_s("p" => "1", "q" => { "name" => "test" })
       expect(result).to include("p=1")
       expect(result).to include("q[name]=test")
     end

tiny_admin.gemspec

@@ -28,8 +28,8 @@ Gem::Specification.new do |spec|
   spec.files         = Dir['{app,db,lib}/**/*', 'LICENSE.txt', 'README.md']
   spec.require_paths = ['lib']
 
-  spec.add_runtime_dependency 'phlex', '~> 1', '>= 1.10.0'
-  spec.add_runtime_dependency 'roda', '~> 3'
-  spec.add_runtime_dependency 'tilt', '~> 2'
-  spec.add_runtime_dependency 'zeitwerk', '~> 2'
+  spec.add_dependency 'phlex', '~> 1', '>= 1.10.0'
+  spec.add_dependency 'roda', '~> 3'
+  spec.add_dependency 'tilt', '~> 2'
+  spec.add_dependency 'zeitwerk', '~> 2'
 end