parklife 0.7.0 → 0.8.0.beta1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 716fff5f3d7ad38f43c2fd071de3a69dc58363b05fb7fd3d997311855a207f04
-  data.tar.gz: b153dc13fb74648185401f3a512c34eee2f0ad62659723937c0354d9c7547ecd
+  metadata.gz: 5fee28b7e4a5dd221a56dc23db922173062e06b22888b2daf3a8bc05136525e2
+  data.tar.gz: '02408d7f687ea75db42814178c9ae168595f43d16e287ce30e558baa0abb5d9a'
 SHA512:
-  metadata.gz: 7a1a022c68a6a7c77277440c926cd430ebf857a7757f42eab76b8233af84bdc4dd9407952a9465720a6a74e43d7706b8acb41a25705ebb3d0e34c9a75a11b441
-  data.tar.gz: 80af344c3209f6a57f147714c789d6c3d21a42cd3131186e9dbebb0e267d19258cdb297f13f273baefdb4bbd4bcba955c2c1e726ee9e218d8febaf50ec4c26c8
+  metadata.gz: c6d913ec8bb296a0ad792b75dd4460a65cb94bf07a47d02db3f0db5c4ec5070de63033a8c4333eda08dfa1655c743f67bc615eef89ee5804d08502df75524ddb
+  data.tar.gz: 10fd0665002537622e0229dd0c366cd3ff32ad6234f18a0da80a8fda4cfc64c296d106346193e3ec1ef4aad1c0bd66eff17373a0a402dcffa33842a19d459ec5

data/.github/workflows/examples.yml

@@ -18,21 +18,6 @@ jobs:
     - run: test -f build/index.html
       working-directory: examples/rack
 
-  rails:
-    runs-on: ubuntu-latest
-    name: Rails example
-    steps:
-    - uses: actions/checkout@v3
-    - uses: ruby/setup-ruby@v1
-      with:
-        bundler-cache: true
-        ruby-version: '3.2'
-        working-directory: examples/rails
-    - run: bin/static-build
-      working-directory: examples/rails
-    - run: test -f build/index.html
-      working-directory: examples/rails
-
   roda:
     runs-on: ubuntu-latest
     name: Roda example

data/.github/workflows/tests.yml

@@ -3,52 +3,6 @@ name: Tests
 on: push
 
 jobs:
-  rails:
-    runs-on: ubuntu-latest
-    strategy:
-      fail-fast: false
-      matrix:
-        rails:
-          - '7.0'
-          - '7.1'
-          - '7.2'
-          - '8.0'
-        ruby:
-          - '2.7'
-          - '3.0'
-          - '3.1'
-          - '3.2'
-          - '3.3'
-          - '3.4'
-        exclude:
-          # Rails 8.0 requires Ruby 3.2.
-          - rails: '8.0'
-            ruby: '3.1'
-          - rails: '8.0'
-            ruby: '3.0'
-          - rails: '8.0'
-            ruby: '2.7'
-
-          # Rails 7.2 requires Ruby 3.1.
-          - rails: '7.2'
-            ruby: '3.0'
-          - rails: '7.2'
-            ruby: '2.7'
-
-          # Rails 7.0 doesn't work out of the box with Ruby 3.4.
-          - rails: '7.0'
-            ruby: '3.4'
-    env:
-      BUNDLE_GEMFILE: gemfiles/rails_${{ matrix.rails }}.gemfile
-    name: Rails (Ruby ${{ matrix.ruby }} / Rails ${{ matrix.rails }})
-    steps:
-    - uses: actions/checkout@v4
-    - uses: ruby/setup-ruby@v1
-      with:
-        bundler-cache: true
-        ruby-version: ${{ matrix.ruby }}
-    - run: bundle exec rspec --pattern "spec/**/*rails*"
-
   rspec:
     runs-on: ubuntu-latest
     strategy:
@@ -68,7 +22,7 @@ jobs:
       with:
         bundler-cache: true
         ruby-version: ${{ matrix.ruby }}
-    - run: bundle exec rspec --exclude-pattern "spec/**/*rails*,spec/**/*sinatra*"
+    - run: bundle exec rspec --exclude-pattern "spec/**/*sinatra*"
 
   rubocop:
     runs-on: ubuntu-latest
@@ -99,7 +53,7 @@ jobs:
           - '4.1'
     env:
       BUNDLE_GEMFILE: gemfiles/sinatra_${{ matrix.sinatra }}.gemfile
-    name: Rails (Ruby ${{ matrix.ruby }} / Sinatra ${{ matrix.sinatra }})
+    name: Ruby ${{ matrix.ruby }} / Sinatra ${{ matrix.sinatra }}
     steps:
     - uses: actions/checkout@v4
     - uses: ruby/setup-ruby@v1

data/Appraisals

@@ -1,22 +1,5 @@
 # frozen_string_literal: true
 
-appraise 'rails_7.0' do
-  gem 'concurrent-ruby', '< 1.3.5'
-  gem 'rails', '~> 7.0.0'
-end
-
-appraise 'rails_7.1' do
-  gem 'rails', '~> 7.1.0'
-end
-
-appraise 'rails_7.2' do
-  gem 'rails', '~> 7.2.0'
-end
-
-appraise 'rails_8.0' do
-  gem 'rails', '~> 8.0.0'
-end
-
 appraise 'sinatra_3.x' do
   gem 'sinatra', '~> 3.0'
 end

data/CHANGELOG.md

@@ -1,3 +1,13 @@
+## Unreleased
+
+## Version 0.8.0.beta1 - 2025-08-10
+
+- Remove out-of-the-box Rails integration in preparation for a separate gem. <https://github.com/benpickles/parklife/pull/131>
+- Include the port when automatically assigning the base from Rails `default_url_options`. <https://github.com/benpickles/parklife/pull/130>
+- Improve the HTTP redirect error message to include the request's full URL and redirect location. <https://github.com/benpickles/parklife/pull/129>
+- Empty the build directory instead of deleting it. <https://github.com/benpickles/parklife/pull/125>
+- Resurrect build callbacks (`before_build`/`after_build`). <https://github.com/benpickles/parklife/pull/124>
+
 ## Version 0.7.0 - 2025-02-03
 
 - Add support for Rails 8 and add test infrastructure to ensure future compatibility with Rails 7.0, 7.1, and 8.0. <https://github.com/benpickles/parklife/pull/115>, <https://github.com/benpickles/parklife/pull/117>, <https://github.com/benpickles/parklife/pull/121>

data/README.md

@@ -110,7 +110,7 @@ Parklife.application.config.nested_index = false
 
 ### Changing the build output directory
 
-The build directory shouldn't exist and is destroyed and recreated before each build. Defaults to `build`.
+The build directory's contents are removed before each build. Defaults to `build`.
 
 ```ruby
 Parklife.application.config.build_dir = 'my/build/dir'

data/lib/parklife/application.rb

@@ -13,16 +13,37 @@ module Parklife
     def initialize
       @config = Config.new
       @route_set = RouteSet.new
+      @after_build_callbacks = []
+      @before_build_callbacks = []
+    end
+
+    def after_build(&block)
+      @after_build_callbacks << block
+    end
+
+    def before_build(&block)
+      @before_build_callbacks << block
     end
 
     def build
       raise BuildDirNotDefinedError if config.build_dir.nil?
       raise RackAppNotDefinedError if config.app.nil?
 
-      FileUtils.rm_rf(config.build_dir)
-      Dir.mkdir(config.build_dir)
+      if Dir.exist?(config.build_dir)
+        FileUtils.rm_rf(Pathname.new(config.build_dir).children)
+      else
+        Dir.mkdir(config.build_dir)
+      end
+
+      @before_build_callbacks.each do |callback|
+        callback.call(self)
+      end
 
       crawler.start
+
+      @after_build_callbacks.each do |callback|
+        callback.call(self)
+      end
     end
 
     def configure

data/lib/parklife/browser.rb

@@ -18,6 +18,10 @@ module Parklife
       session.get(path, nil, env)
     end
 
+    def uri_for(path)
+      base.dup.tap { |uri| uri.path = path }
+    end
+
     private
       def set_env
         @env = {

data/lib/parklife/crawler.rb

@@ -51,6 +51,12 @@ module Parklife
         case response.status
         when 200
           # Continue processing the route.
+        when 301, 302
+          raise HTTPRedirectError.new(
+            response.status,
+            browser.uri_for(route.path),
+            response.headers['location']
+          )
         when 404
           case config.on_404
           when :warn

data/lib/parklife/errors.rb

@@ -11,6 +11,12 @@ module Parklife
     end
   end
 
+  class HTTPRedirectError < Error
+    def initialize(status, from, to)
+      super %Q(#{status} redirect from "#{from}" to "#{to}")
+    end
+  end
+
   class ParkfileLoadError < Error
     def initialize(path)
       super %Q(Cannot load Parkfile "#{path}")

data/lib/parklife/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Parklife
-  VERSION = '0.7.0'
+  VERSION = '0.8.0.beta1'
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: parklife
 version: !ruby/object:Gem::Version
-  version: 0.7.0
+  version: 0.8.0.beta1
 platform: ruby
 authors:
 - Ben Pickles
 bindir: exe
 cert_chain: []
-date: 2025-02-03 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri
@@ -73,10 +73,6 @@ files:
 - bin/console
 - bin/setup
 - exe/parklife
-- gemfiles/rails_7.0.gemfile
-- gemfiles/rails_7.1.gemfile
-- gemfiles/rails_7.2.gemfile
-- gemfiles/rails_8.0.gemfile
 - gemfiles/sinatra_3.x.gemfile
 - gemfiles/sinatra_4.0.gemfile
 - gemfiles/sinatra_4.1.gemfile
@@ -87,7 +83,6 @@ files:
 - lib/parklife/config.rb
 - lib/parklife/crawler.rb
 - lib/parklife/errors.rb
-- lib/parklife/rails.rb
 - lib/parklife/route.rb
 - lib/parklife/route_set.rb
 - lib/parklife/sinatra.rb
@@ -119,7 +114,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.2
+rubygems_version: 3.7.1
 specification_version: 4
 summary: Convert a Rack app into a static HTML site.
 test_files: []

data/gemfiles/rails_7.0.gemfile

@@ -1,13 +0,0 @@
-# This file was generated by Appraisal
-
-source "https://rubygems.org"
-
-gem "appraisal"
-gem "bundler"
-gem "rake"
-gem "rspec"
-gem "rubocop"
-gem "concurrent-ruby", "< 1.3.5"
-gem "rails", "~> 7.0.0"
-
-gemspec path: "../"

data/gemfiles/rails_7.1.gemfile

@@ -1,12 +0,0 @@
-# This file was generated by Appraisal
-
-source "https://rubygems.org"
-
-gem "appraisal"
-gem "bundler"
-gem "rake"
-gem "rspec"
-gem "rubocop"
-gem "rails", "~> 7.1.0"
-
-gemspec path: "../"

data/gemfiles/rails_7.2.gemfile

@@ -1,12 +0,0 @@
-# This file was generated by Appraisal
-
-source "https://rubygems.org"
-
-gem "appraisal"
-gem "bundler"
-gem "rake"
-gem "rspec"
-gem "rubocop"
-gem "rails", "~> 7.2.0"
-
-gemspec path: "../"

data/gemfiles/rails_8.0.gemfile

@@ -1,12 +0,0 @@
-# This file was generated by Appraisal
-
-source "https://rubygems.org"
-
-gem "appraisal"
-gem "bundler"
-gem "rake"
-gem "rspec"
-gem "rubocop"
-gem "rails", "~> 8.0.0"
-
-gemspec path: "../"

data/lib/parklife/rails.rb

@@ -1,69 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails'
-
-module Parklife
-  module RailsConfigRefinements
-    # When setting Parklife's base also configure the Rails app's
-    # default_url_options and relative_url_root to match.
-    def base=(value)
-      super.tap { |uri|
-        app.default_url_options = {
-          host: Utils.host_with_port(uri),
-          protocol: uri.scheme,
-        }
-
-        base_path = !uri.path.empty? && uri.path != '/' ? uri.path : nil
-        ActionController::Base.relative_url_root = base_path
-      }
-    end
-  end
-
-  module RailsRouteSetRefinements
-    def default_url_options
-      Rails.application.default_url_options
-    end
-  end
-
-  class Railtie < Rails::Railtie
-    initializer 'parklife.disable_host_authorization' do |app|
-      # The offending middleware is included in Rails (6+) development mode and
-      # rejects a request with a 403 response if its host isn't present in the
-      # allowlist (a security feature). This prevents Parklife from working in
-      # a Rails app out of the box unless you manually add the expected
-      # Parklife base to the hosts allowlist or set it to nil to disable it -
-      # both of which aren't great because they disable the security feature
-      # whenever the development server is booted.
-      #
-      # https://guides.rubyonrails.org/configuring.html#actiondispatch-hostauthorization
-      #
-      # However it's safe to remove the middleware at this point because it
-      # won't be executed in the normal Rails development flow, only via a
-      # Parkfile when parklife/rails is required.
-      if defined?(ActionDispatch::HostAuthorization)
-        app.middleware.delete(ActionDispatch::HostAuthorization)
-      end
-
-      Parklife.application.config.app = app
-
-      # Allow use of the Rails application's route helpers when defining
-      # Parklife routes in the block form.
-      Parklife.application.routes.singleton_class.include(RailsRouteSetRefinements)
-      Parklife.application.routes.singleton_class.include(app.routes.url_helpers)
-
-      Parklife.application.config.extend(RailsConfigRefinements)
-    end
-
-    config.after_initialize do |app|
-      # Read the Rails app's URL config and apply it to Parklife's so that the
-      # Rails config can be used as the single source of truth.
-      host, protocol = app.default_url_options.values_at(:host, :protocol)
-      protocol = 'https' if app.config.force_ssl
-      path = ActionController::Base.relative_url_root
-
-      Parklife.application.config.base.scheme = protocol if protocol
-      Parklife.application.config.base.host = host if host
-      Parklife.application.config.base.path = path if path
-    end
-  end
-end