brut 0.0.13 → 0.0.21

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8e8b01889cc06839d9cca47e021c8b781e6f21417b08ba0658d55697285bb061
-  data.tar.gz: 4d1bf20dac16e8e6322402aa364f886235140028e53f8aa313ddc6d3b2de81c6
+  metadata.gz: 13ef62dae2d2a40f20fce1aa2f5df7809fc41ae99b75b2a9aefd25423ef51fb3
+  data.tar.gz: 65b9e0b2b77a441210d0047da5c6c8f77d01d97a954d64626683d11fa7f2f1d1
 SHA512:
-  metadata.gz: 77c38b80c31452da7db8ef80ce2532aa1d3193257193d70625135d23af01695cc5f0c3f46568988bcf0c02849af8b8f9f3f166f9065445efd90a160c1e3206c0
-  data.tar.gz: e36fa4f081d48518cf941d0e0323cde04a6370082e0a594da73221f65c1f319d29df29c7cbd5663f30b2da2776fa8ec24295a16cc55b41b94c6f1096590282f7
+  metadata.gz: 4a524485b78b02dd2fee75c540b8cef83fc1c79e8ffa6ec8b214995000c42c6e41db5dbfe8ab65fe28b3e9f08eb9edb63f9156e4208618750f3165bf548995ee
+  data.tar.gz: af71268d9598c889731db3cbff39c47d14173ae4112b4eb9e723ff917bf036da93ee4efb3bdffc7f0fc1136abc71b089ab64fe0f8e503226f61e1b0511ffa9f1

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    brut (0.0.13)
+    brut (0.0.21)
       concurrent-ruby
       i18n
       irb
@@ -9,15 +9,13 @@ PATH
       opentelemetry-exporter-otlp
       opentelemetry-sdk
       ostruct
+      phlex
       prism
       rack-protection
       rackup
-      rexml
       semantic_logger
       sequel
       sinatra
-      temple
-      tilt
       tzinfo
       tzinfo-data
       zeitwerk
@@ -114,6 +112,8 @@ GEM
     opentelemetry-semantic_conventions (1.10.1)
       opentelemetry-api (~> 1.0)
     ostruct (0.6.1)
+    phlex (2.2.1)
+      zeitwerk (~> 2.7)
     pp (0.6.2)
       prettyprint
     prettyprint (0.2.0)
@@ -136,7 +136,6 @@ GEM
       psych (>= 4.0.0)
     reline (0.6.0)
       io-console (~> 0.5)
-    rexml (3.3.9)
     rspec (3.13.0)
       rspec-core (~> 3.13.0)
       rspec-expectations (~> 3.13.0)
@@ -163,7 +162,6 @@ GEM
       rack-session (>= 2.0.0, < 3)
       tilt (~> 2.0)
     stringio (3.1.3)
-    temple (0.10.3)
     tilt (2.4.0)
     tzinfo (2.0.6)
       concurrent-ruby (~> 1.0)

data/brut.gemspec

@@ -39,15 +39,13 @@ Gem::Specification.new do |spec|
   spec.add_runtime_dependency "concurrent-ruby"
   spec.add_runtime_dependency "i18n"
   spec.add_runtime_dependency "nokogiri"
+  spec.add_runtime_dependency "phlex"
   spec.add_runtime_dependency "prism"
   spec.add_runtime_dependency "rack-protection"
   spec.add_runtime_dependency "rackup"
-  spec.add_runtime_dependency "rexml"
   spec.add_runtime_dependency "semantic_logger"
   spec.add_runtime_dependency "sequel"
   spec.add_runtime_dependency "sinatra"
-  spec.add_runtime_dependency "temple"
-  spec.add_runtime_dependency "tilt"
   spec.add_runtime_dependency "tzinfo"
   spec.add_runtime_dependency "tzinfo-data"
   spec.add_runtime_dependency "zeitwerk"

data/lib/brut/back_end/seed_data.rb

@@ -1,10 +1,19 @@
 require_relative "../factory_bot"
 module Brut
   module BackEnd
-    # Base class and manager of Seed Data for the app.  Seed Data is data used for development. It is not for populating e.g.
-    # reference data or other stuff in production.
+    # Base class and manager of Seed Data for the app.  Seed Data is data used for development.
+    # It is not for populating e.g. reference data or other stuff in production, nor is it for
+    # managing test data.
     #
     # Seed Data uses FactoryBot.
+    #
+    # To create your own seed data:
+    #
+    # 1. Inherit from this class.  Doing so will register your class with an internal data structure
+    #    Brut will use to create all seed data.
+    # 2. Provide a no-arg initializer (although you are unlikely to need any initializer at all).
+    # 3. Implement {#seed!} to use Factory Bot to create all the seed data.  This method should be self-contained
+    #    and not rely on other seed data classes.  It need not be idempotent.
     class SeedData
       def self.inherited(seed_data_klass)
         @classes ||= []
@@ -12,10 +21,13 @@ module Brut
       end
       def self.classes = @classes || []
 
+      # Sets up anything needed before seed data can be created. Do not override this method.
       def setup!
         Brut::FactoryBot.new.setup!
       end
 
+      # Loads all seed data registered with this class.  Seed data is registered when a class
+      # extends this one.  Do not override this method.
       def load_seeds!
         DB.transaction do
           self.class.classes.each do |klass|
@@ -23,6 +35,11 @@ module Brut
           end
         end
       end
+
+      # Implement this to create your seed data.
+      def seed!
+        raise Brut::Framework::Errors::AbstractMethod
+      end
     end
   end
 end

data/lib/brut/back_end/sidekiq/middlewares/server.rb

@@ -1,3 +1,4 @@
-class Brut::BackEnd::Sidekiq::Middlewares::Server
+# Useful server middlewares for Sidekiq
+module Brut::BackEnd::Sidekiq::Middlewares::Server
   autoload(:FlushSpans, "brut/back_end/sidekiq/middlewares/server/flush_spans")
 end

data/lib/brut/back_end/sidekiq/middlewares.rb

@@ -1,3 +1,4 @@
-class Brut::BackEnd::Sidekiq::Middlewares
+# Useful middlewares for Sidekiq jobs
+module Brut::BackEnd::Sidekiq::Middlewares
   autoload(:Server, "brut/back_end/sidekiq/middlewares/server")
 end

data/lib/brut/back_end/sidekiq.rb

@@ -1,3 +1,4 @@
-class Brut::BackEnd::Sidekiq
+# Namespace for Sidekiq-related support.
+module Brut::BackEnd::Sidekiq
   autoload(:Middlewares, "brut/back_end/sidekiq/middlewares")
 end

data/lib/brut/back_end/validator.rb

@@ -1,3 +1,7 @@
-class Brut::BackEnd::Validators
+# Namespace for back-end validation support.  Note that in Brut, validators
+# are not a mechanism for ensuring data integrity.  Validators are for helping
+# a website visitor or app user to understand data entry mistakes.  To ensure
+# data integrity, use your databases constraints and other features.
+module Brut::BackEnd::Validators
   autoload(:FormValidator, "brut/back_end/validators/form_validator")
 end

data/lib/brut/back_end.rb

@@ -0,0 +1,9 @@
+# The _back end_ of a Brut app is where your app's business logic and
+# database are managed.  While the bulk of your Brut app's code
+# will be in the back end, Brut is far less prescriptive about how to manage
+# that than it is the front end.
+module Brut::BackEnd
+  autoload(:Validators, "brut/back_end/validator")
+  autoload(:Sidekiq, "brut/back_end/sidekiq")
+  # Do not put SeedData here - it must be loaded only when needed
+end

data/lib/brut/cli/apps/scaffold.rb

@@ -184,7 +184,7 @@ end}
   end
 
   class Component < Brut::CLI::Command
-    description "Create a new component, template, and associated test"
+    description "Create a new component and associated test"
     detailed_description "New components go in the `components/` folder of your app, however using --page will create a 'page private' component.  To do that, the component name must be an inner class of an existing page, for example HomePage::Welcome. This component goes in a sub-folder inside the `pages/` area of your app"
     opts.on("--page","If set, this component is for a specific page and won't go with the other components")
     args "ComponentName"
@@ -217,12 +217,10 @@ end}
       end
 
       source_path      = Pathname( (components_src_dir / relative_path).to_s + ".rb" )
-      html_source_path = Pathname( (components_src_dir / relative_path).to_s + ".html.erb" )
       spec_path        = Pathname( (components_specs_dir / relative_path).to_s + ".spec.rb" )
 
       exists = [
         source_path,
-        html_source_path,
         spec_path,
       ].select(&:exist?)
 
@@ -236,22 +234,21 @@ end}
 
       if global_options.dry_run?
         out.puts "FileUtils.mkdir_p #{source_path.dirname}"
-        out.puts "FileUtils.mkdir_p #{html_source_path.dirname}"
         out.puts "FileUtils.mkdir_p #{spec_path.dirname}"
       else
         FileUtils.mkdir_p source_path.dirname
-        FileUtils.mkdir_p html_source_path.dirname
         FileUtils.mkdir_p spec_path.dirname
 
         File.open(source_path,"w") do |file|
           file.puts %{class #{class_name} < AppComponent
   def initialize
   end
+
+  def view_template
+    h2 { "Welcome to your new template" }
+  end
 end}
         end
-        File.open(html_source_path,"w") do |file|
-          file.puts "<h1>#{class_name} is ready!</h1>"
-        end
         File.open(spec_path,"w") do |file|
           file.puts %{require "spec_helper"
 
@@ -262,9 +259,8 @@ RSpec.describe #{class_name} do
 end}
         end
       end
-      out.puts "Component source is in        #{source_path.relative_path_from(Brut.container.project_root)}"
-      out.puts "Component HTML template is in #{html_source_path.relative_path_from(Brut.container.project_root)}"
-      out.puts "Component test is in          #{spec_path.relative_path_from(Brut.container.project_root)}"
+      out.puts "Component source is in #{source_path.relative_path_from(Brut.container.project_root)}"
+      out.puts "Component test is in   #{spec_path.relative_path_from(Brut.container.project_root)}"
       0
     end
   end
@@ -283,7 +279,7 @@ end}
         end
       end
     end
-    description "Create a new page, template, and associated test"
+    description "Create a new page and associated test"
     args "page_route"
     def execute
       if args.length != 1
@@ -299,14 +295,12 @@ end}
       i18n_locales_dir = Brut.container.i18n_locales_dir
 
       page_source_path     = Pathname( (pages_src_dir   / page_relative_path).to_s + ".rb" )
-      template_source_path = Pathname( (pages_src_dir   / page_relative_path).to_s + ".html.erb" )
       page_spec_path       = Pathname( (pages_specs_dir / page_relative_path).to_s + ".spec.rb" )
       app_path             = Pathname( Brut.container.app_src_dir / "app.rb" )
       app_translations     = Pathname(  i18n_locales_dir / "en" / "2_app.rb")
 
       exists = [
         page_source_path,
-        template_source_path,
         page_spec_path,
       ].select(&:exist?)
 
@@ -319,7 +313,6 @@ end}
       end
 
       FileUtils.mkdir_p page_source_path.dirname,     noop: global_options.dry_run?
-      FileUtils.mkdir_p template_source_path.dirname, noop: global_options.dry_run?
       FileUtils.mkdir_p page_spec_path.dirname,       noop: global_options.dry_run?
 
       route_code = "page \"#{route.path_template}\""
@@ -334,8 +327,11 @@ end}
       page_class_code = %{class #{page_class_name} < AppPage
   def initialize#{initializer_params_code} # add needed arguments here
   end
+
+  def page_template
+    h1 { "#{page_class_name} is ready!" }
+  end
 end}
-      template_code = %{<h1>#{page_class_name} is ready!</h1>}
       page_spec_code = %{require "spec_helper"
 
 RSpec.describe #{page_class_name} do
@@ -352,8 +348,6 @@ end}
         out.puts "will contain:\n\n#{route_code}\n\n"
         out.puts page_source_path.relative_path_from(Brut.container.project_root)
         out.puts "will contain:\n\n#{page_class_code}\n\n"
-        out.puts template_source_path.relative_path_from(Brut.container.project_root)
-        out.puts "will contain:\n\n#{template_code}\n\n"
         out.puts page_spec_path.relative_path_from(Brut.container.project_root)
         out.puts "will contain:\n\n#{page_spec_code}\n\n"
         out.puts app_translations.relative_path_from(Brut.container.project_root)
@@ -361,7 +355,6 @@ end}
       else
 
         File.open(page_source_path,"w")     { it.puts page_class_code }
-        File.open(template_source_path,"w") { it.puts template_code }
         File.open(page_spec_path,"w")       { it.puts page_spec_code }
 
         existing_translations = File.read(app_translations).split(/\n/)
@@ -393,11 +386,10 @@ end}
           out.puts "Please make sure everything is correct.  Here is the defintion that was not inserted:\n\n#{route_code}"
         end
       end
-      out.puts "Page source is in        #{page_source_path.relative_path_from(Brut.container.project_root)}"
-      out.puts "Page HTML template is in #{template_source_path.relative_path_from(Brut.container.project_root)}"
-      out.puts "Page test is in          #{page_spec_path.relative_path_from(Brut.container.project_root)}"
-      out.puts "Added title to           #{app_translations.relative_path_from(Brut.container.project_root)}"
-      out.puts "Added route to           #{app_path.relative_path_from(Brut.container.project_root)}"
+      out.puts "Page source is in #{page_source_path.relative_path_from(Brut.container.project_root)}"
+      out.puts "Page test is in   #{page_spec_path.relative_path_from(Brut.container.project_root)}"
+      out.puts "Added title to    #{app_translations.relative_path_from(Brut.container.project_root)}"
+      out.puts "Added route to    #{app_path.relative_path_from(Brut.container.project_root)}"
       0
     end
   end

data/lib/brut/cli.rb

@@ -6,9 +6,10 @@ module Brut
   # that your CLI will respond to. See {Brut::CLI::app}.
   module CLI
 
-    # Execute your CLI based on its command line invocation.  You would call this method inside the executable file placed in `bin/`
-    # in your project.  For example, if you have `YourApp::CLI::CleanOldFiles` and you wish to execute it via `bin/clean-files`, you'd
-    # create `bin/clean-files` like so:
+    # Execute your CLI based on its command line invocation.  You would call this method inside the
+    # executable file placed in `bin/` in your project. 
+    # For example, if you have `YourApp::CLI::CleanOldFiles` and you wish to execute
+    # it via `bin/clean-files`, you'd create `bin/clean-files` like so:
     #
     # ```
     # #!/usr/bin/env ruby

data/lib/brut/factory_bot.rb

@@ -1,7 +1,3 @@
-# Because FactoryBot 6.4.6 has a bug where it is not properly
-# requiring active support, active supporot must be required first,
-# then factory bot.  When 6.4.7 is released, this can be removed. See Gemfile
-require "active_support"
 require "factory_bot"
 require "faker"
 
@@ -17,6 +13,5 @@ class Brut::FactoryBot
       to_create { |instance| instance.save }
     end
     FactoryBot.find_definitions
-
   end
 end

data/lib/brut/framework/app.rb

@@ -1,9 +1,10 @@
 # An "App" in Brut paralance is the collection of source code and configuration that is needed to operate
 # a website. This includes everything needed to serve HTTP requests, but also includes ancillary
-# tasks and any related files required for the app to exist and function.  Your app will have an `App` class that subclasses this
-# class.
+# tasks and any related files required for the app to exist and function.
+# Your app will have an `App` class that subclasses this class.
 #
-# When your app is initialized, Brut will have been configured, but access to internal resources may not be available.  It is here
+# When your app is initialized, Brut will have been configured,
+# but access to internal resources may not be available.  It is here
 # that you can override configuration values or do any other setup before everything boots.
 class Brut::Framework::App
   include Brut::Framework::Errors
@@ -18,7 +19,8 @@ class Brut::Framework::App
   # actions where an app needs to exist inside some organizational context.
   def organization = id
 
-  # Call this in your app's definition to define your app's routes. The contents of the block will be evaluated in the context of
+  # Call this in your app's definition to define your app's routes.
+  # The contents of the block will be evaluated in the context of
   # {Brut::SinatraHelpers::ClassMethods}, and the methods there are generally the ones you should be calling.
   #
   # You can call this multiple times and the routes will be concatenated together.
@@ -31,6 +33,69 @@ class Brut::Framework::App
     end
   end
 
+  # Call this to specify what happens when an unhandled exception occurs.  You may call this mulitple times,
+  # however note that if an error is caught that matches more than one block's condition, the one that is called
+  # will be the first one declared.
+  #
+  # The only deviation from this rule is when you call this
+  # without any condition.  Doing that establishes the behavior for a "catch all" handler, which is
+  # only called when no other configured block can handle the exception. You can declare
+  # this at any time. **Do note** the "catch all" handler is more of a best effort. Brut is currently
+  # based on Sinatra which provides no way to arbitrarily catch all exceptions.  What Brut does here is to 
+  # explicitly catch the range of http status codes from 400 to 999.
+  #
+  # Note that Brut will record the exception via OpenTelemetry so you should not do this in your handlers.  It
+  # would be preferable to instead record an event if you want to have observability from your error handlers.
+  #
+  # @param [Class|Integer|Range<Integer>] condition if given this specifies the conditions under which the given
+  #        block will handle the error.  If omitted, this block will handle any error that doesn't have a more
+  #        specific handler configured.  Meaning of values:
+  #        * A class - this is an exception class that, if caught, triggers the handler
+  #        * An integer - this is an HTTP status code that, if returned, triggers the handler
+  #        * A range of integers - this is a range of HTTP status codes that, if returned, triggers the handler
+  # @yield [Exception] the block is given two named parameters: `exception:` and `http_status_code:`. Your block
+  #        can declare both, either, or none.  Any that are declared will be given values.  At least one
+  #        will be non-`nil`, however are encouraged to code defensively inside this block.
+  # @yieldparam [Exception] exception: the exception that was raised. This will be `nil`
+  #             if the error was caused by an HTTP status code.
+  # @yieldparam [Integer] http_status_code: the HTTP status code that was returned. If `exception:` is
+  #             not `nil`, this value is highly likely to be 500.
+  # @yieldreturn The block should return a valid Rack response. For now.
+  def self.error(condition=:catch_all, &block)
+    @error_blocks ||= {}
+    if block.nil?
+      raise ArgumentError, "You must provide a block to error"
+    end
+    parameters = block.parameters.reject { |type,name|
+      type == :keyreq && [ :http_status_code, :exception ].include?(name)
+    }
+    if parameters.any?
+      messages = parameters.map { |type,name|
+        case type
+        when :keyreq
+          "required keyword parameter '#{name}:'"
+        when :key
+          "optional keyword parameter '#{name}:'"
+        when :rest
+          "rest parameter '#{name}'"
+        when :opt
+          "optional parameter '#{name}'"
+        when :req
+          "required parameter '#{name}'"
+        else
+          "unknown parameter '#{name}'"
+        end
+      }
+      raise ArgumentError, "Your error handler block may only accept exception: and http_status_code: as required keyword parameters.  The following parameters were found:\n  #{messages.join("\n  ")}"
+    end
+    if @error_blocks[condition]
+      raise ArgumentError, "You have already configured error handling for condition '#{condition.to_s}'"
+    end
+    @error_blocks[condition] = block
+  end
+
+  def self.error_blocks = @error_blocks || {}
+
   # Add a Rack middleware to your app. Middlewares are configured in the order in which you call this method.
   #
   # @param [Class] middleware a class that implements [Rack Middleware](https://github.com/rack/rack/blob/main/SPEC.rdoc).
@@ -81,7 +146,7 @@ class Brut::Framework::App
   # code required *after* Brut has been set up and started.  You can rely on the
   # database being available. Any attempts to override configuration values
   # may not succeed.  This is called after the framework has booted, but before
-  # your apps' routes are set up.
+  # your app's routes are set up.
   def boot!
   end
 

data/lib/brut/framework/config.rb

@@ -1,11 +1,13 @@
 require_relative "project_environment"
 require "pathname"
 
-# Holds configuration for the framework and your app.  In general, you should not interact with this class, however it's source code
-# is a good reference for what is configured by default by Brut.
+# Holds configuration for the framework and your app.  In general, you 
+# should not interact with this class, however it's source code is a good 
+# reference for what is configured by default by Brut.
 class Brut::Framework::Config
 
-  # Configures all defaults.  In general, this attempts to be lazy in setting things up, so calling this should not attempt to make a
+  # Configures all defaults.  In general, this attempts to be lazy in
+  # setting things up, so calling this should not attempt to make a
   # connection to your database.
   def configure!
     Brut.container do |c|
@@ -238,21 +240,6 @@ class Brut::Framework::Config
         config_dir / "asset_metadata.json"
       end
 
-
-      c.store(
-        "layout_locator",
-        "Brut::FrontEnd::Templates::Locator",
-        "Object to use to locate templates for layouts"
-      ) do |layouts_src_dir,project_env,brut_internal_dir|
-        paths = if project_env.development?
-                  [ layouts_src_dir, brut_internal_dir / "lib" / "brut" / "front_end" / "layouts" ]
-                else
-                  layouts_src_dir
-                end
-        Brut::FrontEnd::Templates::Locator.new(paths: paths,
-                                               extension: "html.erb")
-      end
-
       c.store_required_path(
         "brut_internal_dir",
         "Location to where the Brut gem is installed."
@@ -260,44 +247,20 @@ class Brut::Framework::Config
         (Pathname(__FILE__).dirname / ".." / ".." / "..").expand_path
       end
 
-      c.store(
-        "page_locator",
-        "Brut::FrontEnd::Templates::Locator",
-        "Object to use to locate templates for pages"
-      ) do |pages_src_dir,project_env,brut_internal_dir|
-        paths = if project_env.development?
-                  [ pages_src_dir, brut_internal_dir / "lib" / "brut" / "front_end" / "pages" ]
-                else
-                  pages_src_dir
-                end
-        Brut::FrontEnd::Templates::Locator.new(paths: paths,
-                                               extension: "html.erb")
-      end
-
-      c.store(
-        "component_locator",
-        "Brut::FrontEnd::Templates::Locator",
-        "Object to use to locate templates for components"
-      ) do |components_src_dir, pages_src_dir|
-        Brut::FrontEnd::Templates::Locator.new(paths: [ components_src_dir, pages_src_dir ],
-                                               extension: "html.erb")
-      end
-
       c.store(
         "svg_locator",
-        "Brut::FrontEnd::Templates::Locator",
+        "Brut::FrontEnd::InlineSvgLocator",
         "Object to use to locate SVGs"
       ) do |svgs_src_dir|
-        Brut::FrontEnd::Templates::Locator.new(paths: svgs_src_dir,
-                                               extension: "svg")
+        Brut::FrontEnd::InlineSvgLocator.new(paths: svgs_src_dir)
       end
 
       c.store(
         "asset_path_resolver",
-        "Brut::FrontEnd::Component::AssetPathResolver",
+        "Brut::FrontEnd::AssetPathResolver",
         "Object to use to resolve logical asset paths to actual asset paths"
       ) do |asset_metadata_file|
-        Brut::FrontEnd::Component::AssetPathResolver.new(metadata_file: asset_metadata_file)
+        Brut::FrontEnd::AssetPathResolver.new(metadata_file: asset_metadata_file)
       end
 
       c.store(

data/lib/brut/framework/container.rb

@@ -21,7 +21,8 @@ end
 #
 # There is no namespacing/hierarchy.
 #
-# In general, you should not create instances of this class, but you may need to access it via {Brut.container} in order to obtain
+# In general, you should not create instances of this class, but you may
+# need to access it via {Brut.container} in order to obtain
 # configuration values or set your own.
 class Brut::Framework::Container
   def initialize
@@ -92,7 +93,7 @@ class Brut::Framework::Container
   end
 
   # Called by your app to override an existing value.  The value must be overridable (see {#store}).  Generally, you should call this
-  # in the initializer of your {Brut::Framework::App} subclass.  Calling this after the fact may not have the affect you want.
+  # in the initializer of your {Brut::Framework::App} subclass.  Calling this after the fact may not have the effect you want.
   #
   # @param [String|Symbol] name name of the value to override. Will be coerced to a String. This name must have been previously
   #                             configured.

data/lib/brut/framework/errors.rb

@@ -1,7 +1,7 @@
 module Brut
   module Framework
-    # Include this in your class to access some helpful methods that throw commonly-needed
-    # errors
+    # Namespace for Brut-specific error classes, and a holder of several error-related convienience 
+    # methods.  Include this module to gain access to those methods.
     module Errors
       autoload(:Bug,"brut/framework/errors/bug")
       autoload(:NotImplemented,"brut/framework/errors/not_implemented")
@@ -10,7 +10,12 @@ module Brut
       autoload(:MissingConfiguration,"brut/framework/errors/missing_configuration")
       autoload(:AbstractMethod,"brut/framework/errors/abstract_method")
       autoload(:NoClassForPath,"brut/framework/errors/no_class_for_path")
-      # Raises {Brut::Framework::Errors::Bug}
+      # Raises {Brut::Framework::Errors::Bug}, used to indicate a codepath is a bug.  "But, why write a
+      # bug in the first place?" you may be asking.  Sometimes, a code path exists, but external factors
+      # mean that it should never be executed.  Or, sometimes an API can be mis-used, but the current
+      # state of the system is such that it would never be misused.
+      #
+      # This method allows you to indicate such situations and provide a meaningful explanation.
       #
       # @param message Message to include in the error
       # @raise [Brut::Framework::Errors::Bug]
@@ -18,7 +23,10 @@ module Brut
         raise Brut::Framework::Errors::Bug,message
       end
 
-      # Raises {Brut::Framework::Errors::AbstractMethod}
+      # Raises {Brut::Framework::Errors::AbstractMethod}, which is useful if you need to document
+      # a method that a subclass must implement, but for which there is no useful default
+      # implementation.
+      #
       # @raise [Brut::Framework::Errors::AbstractMethod]
       def abstract_method!
         raise Brut::Framework::Errors::AbstractMethod

data/lib/brut/framework/mcp.rb

@@ -116,8 +116,69 @@ class Brut::Framework::MCP
     @sinatra_app = Class.new(Sinatra::Base)
     @sinatra_app.include(Brut::SinatraHelpers)
 
+    safely_record_exception = ->(exception,http_status_code) {
+      begin
+      if exception.nil?
+        Brut.container.instrumentation.add_event("error triggered without exception", http_status_code:)
+      else
+        Brut.container.instrumentation.record_exception(exception, http_status_code:)
+      end
+      rescue => ex
+        begin
+          SemanticLogger[self.class].error(
+            "Error recording exception",
+            original_excerption: exception, 
+            exception: ex)
+        rescue => ex2
+          $stderr.puts "While handling an error recording an exception, we get another error from SemanticLogger." + [
+            [ :original_exception,  exception,  ].join(": "),
+            [ :exception_from_recording_exception,  ex, ].join(": "),
+            [ :exception_from_semantic_logger,  ex2 ].join(": "),
+
+          ].join(", ")
+        end
+      end
+    }
+
+    @app.class.error_blocks.each do |condition,block|
+      puts "Setting up error handling for #{condition}"
+      if condition != :catch_all
+        @sinatra_app.error(condition) do
+          puts "Error block triggered"
+          exception = request.env["sinatra.error"]
+          safely_record_exception.(exception, response.status)
+          block_args = if exception
+                         puts "Exception: #{exception.class}"
+                         { exception: }
+                       else
+                         puts "HTTP: #{response.status}"
+                         { http_status_code: response.status }
+                       end
+          block.(**block_args)
+        end
+      end
+    end
+    if @app.class.error_blocks[:catch_all]
+      block = @app.class.error_blocks[:catch_all]
+      block_args = block.parameters.map { |(type,name)|
+        [ name, nil ]
+      }.to_h
+      @sinatra_app.error(400..999) do
+        exception = request.env["sinatra.error"]
+        safely_record_exception.(exception, response.status)
+        if block_args.key?(:exception)
+          block_args[:exception] = exception
+        end
+        if block_args.key?(:http_status_code)
+          block_args[:http_status_code] = response.status
+        end
+        block.(**block_args)
+      end
+    else
+    end
+
     message = if Brut.container.project_env.development?
-                "Form submission did not include an authenticity token. All forms must include one. To add one, use the `form_tag` helper, or include <%= component(Brut::FrontEnd::Components::Inputs::CsrfToken) %> somewhere inside your <form> tag"
+                "Form submission did not include an authenticity token. All forms must include one. To add one, use the `form_tag` helper, or include Brut::FrontEnd::Components::Inputs::CsrfToken somewhere inside your <form> tag"
               else
                 "Forbidden"
               end
@@ -186,7 +247,7 @@ class Brut::Framework::MCP
               end
 
               if name == :request_context
-                args[name] = Thread.current.thread_variable_get(:request_context)
+                args[name] = Brut::FrontEnd::RequestContext.current
               elsif name == :session
                 args[name] = Brut.container.session_class.new(rack_session: session)
               elsif name == :request