RubyGems - ruact - Versions diffs - 0.0.11 → 0.0.12 - Mend

ruact 0.0.11 → 0.0.12

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (116) hide show

checksums.yaml +4 -4
data/CHANGELOG.md +110 -48
data/CONTRIBUTING.md +233 -0
data/README.md +14 -4
data/RELEASING.md +125 -139
data/lib/ruact/packaging.rb +68 -0
data/lib/ruact/version.rb +1 -1
data/vendor/javascript/vite-plugin-ruact/flight-client.test.mjs +321 -0
data/vendor/javascript/vite-plugin-ruact/package-lock.json +11 -0
data/vendor/javascript/vite-plugin-ruact/package.json +1 -0
data/vendor/javascript/vite-plugin-ruact/runtime/flight-client.js +14 -3
metadata +6 -107
data/.codecov.yml +0 -31
data/.github/workflows/ci.yml +0 -284
data/.github/workflows/server-functions-bench.yml +0 -54
data/.rubocop.yml +0 -107
data/.rubocop_todo.yml +0 -63
data/Rakefile +0 -10
data/bench/server_functions_dispatch_bench.rb +0 -276
data/bench/server_functions_dispatch_bench.results.md +0 -150
data/docs/internal/README.md +0 -9
data/docs/internal/decisions/server-functions-api.md +0 -2236
data/spec/benchmarks/baseline.json +0 -12
data/spec/benchmarks/render_pipeline_benchmark_spec.rb +0 -109
data/spec/fixtures/flight/README.md +0 -136
data/spec/fixtures/flight/array.txt +0 -1
data/spec/fixtures/flight/as_json_object.txt +0 -2
data/spec/fixtures/flight/bigint.txt +0 -1
data/spec/fixtures/flight/boolean_false.txt +0 -1
data/spec/fixtures/flight/boolean_true.txt +0 -1
data/spec/fixtures/flight/client_component_with_props.txt +0 -2
data/spec/fixtures/flight/client_reference.txt +0 -2
data/spec/fixtures/flight/hash.txt +0 -1
data/spec/fixtures/flight/infinity.txt +0 -1
data/spec/fixtures/flight/nan.txt +0 -1
data/spec/fixtures/flight/negative_infinity.txt +0 -1
data/spec/fixtures/flight/nil.txt +0 -1
data/spec/fixtures/flight/number_float.txt +0 -1
data/spec/fixtures/flight/number_integer.txt +0 -1
data/spec/fixtures/flight/react_element_no_props.txt +0 -1
data/spec/fixtures/flight/redirect_row.txt +0 -1
data/spec/fixtures/flight/serializable_object.txt +0 -2
data/spec/fixtures/flight/string_basic.txt +0 -1
data/spec/fixtures/flight/string_dollar_escape.txt +0 -1
data/spec/fixtures/flight/undefined.txt +0 -1
data/spec/fixtures/readme/children-error.html.erb +0 -3
data/spec/fixtures/readme/children-error.txt +0 -1
data/spec/fixtures/story_7_9_views/controller_request_spec_support/demo/show.html.erb +0 -3
data/spec/fixtures/story_7_9_views/controller_request_spec_support/errors_demo/new.html.erb +0 -3
data/spec/fixtures/story_7_9_views/controller_request_spec_support/exploding_layout_demo/show.html.erb +0 -3
data/spec/fixtures/story_7_9_views/controller_request_spec_support/ghost_layout_demo/show.html.erb +0 -3
data/spec/fixtures/story_7_9_views/controller_request_spec_support/layout_demo/show.html.erb +0 -3
data/spec/fixtures/story_7_9_views/controller_request_spec_support/rootless_layout_demo/show.html.erb +0 -3
data/spec/fixtures/story_7_9_views/controller_request_spec_support/unwired_layout_demo/show.html.erb +0 -3
data/spec/fixtures/story_7_9_views/layouts/bare_host.html.erb +0 -16
data/spec/fixtures/story_7_9_views/layouts/exploding_host.html.erb +0 -24
data/spec/fixtures/story_7_9_views/layouts/rootless_host.html.erb +0 -15
data/spec/fixtures/story_7_9_views/layouts/ruact_host.html.erb +0 -17
data/spec/readme_demo_message_spec.rb +0 -67
data/spec/readme_spec.rb +0 -282
data/spec/ruact/client_manifest_spec.rb +0 -270
data/spec/ruact/component_contract_spec.rb +0 -119
data/spec/ruact/configuration_spec.rb +0 -518
data/spec/ruact/controller_request_spec.rb +0 -671
data/spec/ruact/controller_spec.rb +0 -343
data/spec/ruact/doctor_spec.rb +0 -769
data/spec/ruact/erb_preprocessor_hook_spec.rb +0 -55
data/spec/ruact/erb_preprocessor_spec.rb +0 -361
data/spec/ruact/errors_spec.rb +0 -93
data/spec/ruact/flight/renderer_spec.rb +0 -133
data/spec/ruact/flight/serializer_spec.rb +0 -494
data/spec/ruact/html_converter_spec.rb +0 -375
data/spec/ruact/install_generator_spec.rb +0 -1549
data/spec/ruact/layout_source_spec.rb +0 -108
data/spec/ruact/manifest_resolver_spec.rb +0 -174
data/spec/ruact/query_request_spec.rb +0 -706
data/spec/ruact/query_spec.rb +0 -105
data/spec/ruact/railtie_spec.rb +0 -155
data/spec/ruact/render_context_spec.rb +0 -58
data/spec/ruact/render_pipeline_concurrency_spec.rb +0 -78
data/spec/ruact/render_pipeline_spec.rb +0 -928
data/spec/ruact/scaffold_generator_spec.rb +0 -1849
data/spec/ruact/serializable_spec.rb +0 -179
data/spec/ruact/server_bucket_request_spec.rb +0 -785
data/spec/ruact/server_function_name_spec.rb +0 -53
data/spec/ruact/server_functions/backtrace_cleaner_spec.rb +0 -63
data/spec/ruact/server_functions/bucket_two_payload_spec.rb +0 -200
data/spec/ruact/server_functions/codegen_spec.rb +0 -397
data/spec/ruact/server_functions/error_payload_spec.rb +0 -222
data/spec/ruact/server_functions/error_suggestion_spec.rb +0 -79
data/spec/ruact/server_functions/introspection_spec.rb +0 -135
data/spec/ruact/server_functions/name_bridge_spec.rb +0 -212
data/spec/ruact/server_functions/query_context_spec.rb +0 -72
data/spec/ruact/server_functions/query_source_spec.rb +0 -193
data/spec/ruact/server_functions/railtie_integration_spec.rb +0 -215
data/spec/ruact/server_functions/rake_spec.rb +0 -86
data/spec/ruact/server_functions/route_source_spec.rb +0 -202
data/spec/ruact/server_functions/snapshot_spec.rb +0 -96
data/spec/ruact/server_functions/snapshot_writer_spec.rb +0 -71
data/spec/ruact/server_rescue_request_spec.rb +0 -416
data/spec/ruact/server_spec.rb +0 -179
data/spec/ruact/server_upload_request_spec.rb +0 -311
data/spec/ruact/signed_references_spec.rb +0 -164
data/spec/ruact/string_distance_spec.rb +0 -38
data/spec/ruact/tasks_json_introspection_spec.rb +0 -141
data/spec/ruact/testing/have_ruact_component_spec.rb +0 -170
data/spec/ruact/testing/no_production_load_spec.rb +0 -41
data/spec/ruact/validation_errors_spec.rb +0 -116
data/spec/ruact/view_helper_spec.rb +0 -131
data/spec/spec_helper.rb +0 -77
data/spec/support/fixtures/pixel.png +0 -0
data/spec/support/flight_wire_parser.rb +0 -21
data/spec/support/flight_wire_parser_spec.rb +0 -93
data/spec/support/matchers/flight_fixture_matcher.rb +0 -130
data/spec/support/matchers/flight_fixture_matcher_spec.rb +0 -250
data/spec/support/rails_stub.rb +0 -115

data/spec/ruact/doctor_spec.rb DELETED Viewed

@@ -1,769 +0,0 @@
-# frozen_string_literal: true
-require "spec_helper"
-require "tmpdir"
-require "fileutils"
-require "socket"
-# The Rails stub (including Rails.root) is provided by spec/support/rails_stub.rb.
-require "ruact/doctor"
-RSpec.describe Ruact::Doctor do
-  let(:tmpdir) { Pathname.new(Dir.mktmpdir) }
-  before { Rails.root = tmpdir }
-  after  { FileUtils.rm_rf(tmpdir) }
-  # --- helpers ---
-  def make_controller(with_include: true)
-    dir = tmpdir.join("app", "controllers")
-    FileUtils.mkdir_p(dir)
-    content = with_include ? "include Ruact::Controller\n" : "class ApplicationController\nend\n"
-    File.write(dir.join("application_controller.rb"), content)
-  end
-  # `with_assets` is the second half of a migrated layout: the React root alone
-  # leaves ruact on its built-in (CSS-less) shell, so the two markers are
-  # independent inputs, not one sentinel.
-  def make_layout(with_sentinel: true, with_assets: true)
-    dir = tmpdir.join("app", "views", "layouts")
-    FileUtils.mkdir_p(dir)
-    content =
-      if !with_sentinel
-        "<body></body>\n"
-      elsif with_assets
-        "<%# ruact: root %>\n<div id=\"root\"></div>\n<%= ruact_js_assets %>\n"
-      else
-        "<%# ruact: root %>\n<div id=\"root\"></div>\n"
-      end
-    File.write(dir.join("application.html.erb"), content)
-  end
-  def make_manifest
-    dir = tmpdir.join("public")
-    FileUtils.mkdir_p(dir)
-    File.write(dir.join("react-client-manifest.json"), "{}")
-  end
-  # --- check_manifest ---
-  describe "#check_manifest (AC#1, #2)" do
-    subject(:doctor) { described_class.new }
-    context "when manifest file exists" do
-      before { make_manifest }
-      it "returns :pass" do
-        status, = doctor.send(:check_manifest)
-        expect(status).to eq(:pass)
-      end
-      it "message includes 'Manifest found at'" do
-        _, msg = doctor.send(:check_manifest)
-        expect(msg).to include("Manifest found at")
-      end
-    end
-    context "when manifest file is missing" do
-      it "returns :fail" do
-        status, = doctor.send(:check_manifest)
-        expect(status).to eq(:fail)
-      end
-      it "message is 'Manifest not found — run vite build'" do
-        _, msg = doctor.send(:check_manifest)
-        expect(msg).to eq("Manifest not found — run vite build")
-      end
-    end
-  end
-  # --- check_vite ---
-  describe "#check_vite (AC#1, #3)" do
-    subject(:doctor) { described_class.new }
-    context "when Vite is accessible" do
-      before { allow(TCPSocket).to receive(:new).and_return(instance_double(TCPSocket, close: nil)) }
-      it "returns :pass" do
-        status, = doctor.send(:check_vite)
-        expect(status).to eq(:pass)
-      end
-    end
-    context "when Vite is not accessible" do
-      before { allow(TCPSocket).to receive(:new).and_raise(Errno::ECONNREFUSED) }
-      it "returns :fail" do
-        status, = doctor.send(:check_vite)
-        expect(status).to eq(:fail)
-      end
-      it "message is 'Vite not accessible at localhost:5173 — run npm run dev'" do
-        _, msg = doctor.send(:check_vite)
-        expect(msg).to eq("Vite not accessible at localhost:5173 — run npm run dev")
-      end
-    end
-  end
-  # --- check_controller ---
-  describe "#check_controller (AC#1, #4)" do
-    subject(:doctor) { described_class.new }
-    context "when ApplicationController includes Ruact::Controller" do
-      before { make_controller(with_include: true) }
-      it "returns :pass" do
-        status, = doctor.send(:check_controller)
-        expect(status).to eq(:pass)
-      end
-    end
-    context "when Ruact::Controller is not included" do
-      before { make_controller(with_include: false) }
-      it "returns :fail" do
-        status, = doctor.send(:check_controller)
-        expect(status).to eq(:fail)
-      end
-      it "message is 'Ruact::Controller not included in ApplicationController'" do
-        _, msg = doctor.send(:check_controller)
-        expect(msg).to eq("Ruact::Controller not included in ApplicationController")
-      end
-    end
-  end
-  # --- check_layout ---
-  describe "#check_layout (AC#1, #5)" do
-    subject(:doctor) { described_class.new }
-    context "when the layout is wired and the app opted in" do
-      before do
-        make_layout(with_sentinel: true, with_assets: true)
-        Ruact.configure { |c| c.layout = true }
-      end
-      it "returns :pass" do
-        status, = doctor.send(:check_layout)
-        expect(status).to eq(:pass)
-      end
-    end
-    # Both halves are silent when wrong, and they are DIFFERENT fixes — "add one
-    # line to the layout" versus "flip one setting" — so they report separately.
-    context "when the layout is wired but Ruact.config.layout is false" do
-      before do
-        make_layout(with_sentinel: true, with_assets: true)
-        Ruact.configure { |c| c.layout = false }
-      end
-      it "returns :warn naming the setting, not the layout" do
-        status, message, remediation = doctor.send(:check_layout)
-        expect(status).to eq(:warn)
-        expect(message).to include("config.layout is false")
-        expect(remediation).to include("config.layout = true")
-      end
-      it "does not fail the doctor run" do
-        expect(described_class::SUCCESS_STATUSES).to include(:warn)
-      end
-    end
-    context "when the layout has the React root but never calls ruact_js_assets" do
-      before do
-        make_layout(with_sentinel: true, with_assets: false)
-        Ruact.configure { |c| c.layout = true }
-      end
-      it "fails and names the helper to add" do
-        status, _message, remediation = doctor.send(:check_layout)
-        expect(status).to eq(:fail)
-        expect(remediation).to include("ruact_js_assets")
-      end
-    end
-    # The drift this shares with the runtime and the generator: a MENTION of the
-    # helper in a comment is not a call, and reporting :pass on one sent a
-    # developer looking anywhere but at the real problem.
-    context "when the layout only mentions the helper in a comment" do
-      before do
-        dir = tmpdir.join("app", "views", "layouts")
-        FileUtils.mkdir_p(dir)
-        File.write(dir.join("application.html.erb"),
-                   "<%# ruact: root %>\n<div id=\"root\"></div>\n<%# TODO: add ruact_js_assets %>\n")
-        Ruact.configure { |c| c.layout = true }
-      end
-      it "does not report it as wired" do
-        status, = doctor.send(:check_layout)
-        expect(status).to eq(:fail)
-      end
-    end
-    context "when the layout has neither the root nor the helper" do
-      before { make_layout(with_sentinel: false) }
-      it "returns :fail" do
-        status, = doctor.send(:check_layout)
-        expect(status).to eq(:fail)
-      end
-      it "names both halves, since either could be the missing one" do
-        _, msg = doctor.send(:check_layout)
-        expect(msg).to include("React root").and include("ruact_js_assets")
-      end
-    end
-    context "when there is no application layout file at all" do
-      it "reports the file, not its contents" do
-        status, msg = doctor.send(:check_layout)
-        expect(status).to eq(:fail)
-        expect(msg).to eq("React shell missing from application.html.erb")
-      end
-    end
-  end
-  # --- check_streaming ---
-  describe "#check_streaming (AC#5)" do
-    subject(:doctor) { described_class.new }
-    after { Ruact.streaming_mode = nil }
-    it "always returns :pass" do
-      status, = doctor.send(:check_streaming)
-      expect(status).to eq(:pass)
-    end
-    context "when streaming_mode is :enabled (AC#1, #5)" do
-      before do
-        Ruact.streaming_mode = :enabled
-        stub_const("Puma", Module.new)
-      end
-      it "message includes 'enabled' and 'Puma'" do
-        _, msg = doctor.send(:check_streaming)
-        expect(msg).to include("enabled").and include("Puma")
-      end
-    end
-    context "when streaming_mode is :buffered with no known server (AC#3, #5)" do
-      before { Ruact.streaming_mode = :buffered }
-      it "message includes 'buffered'" do
-        _, msg = doctor.send(:check_streaming)
-        expect(msg).to include("buffered")
-      end
-    end
-    context "when streaming_mode is nil (not yet detected)" do
-      before { Ruact.streaming_mode = nil }
-      it "defaults to buffered in the message" do
-        _, msg = doctor.send(:check_streaming)
-        expect(msg).to include("buffered")
-      end
-    end
-  end
-  # --- check_legacy_constant ---
-  describe "#check_legacy_constant (Story 5.1)" do
-    subject(:doctor) { described_class.new }
-    # Built via Array#join so this spec file passes the gem-CI
-    # `name-propagation` guard without an exclusion (Story 5.1 review F4).
-    let(:legacy_const) { %w[Rails Rsc].join }
-    let(:legacy_gem)   { %w[rails rsc].join("_") }
-    def make_initializer(content, filename: "ruact.rb")
-      dir = tmpdir.join("config", "initializers")
-      FileUtils.mkdir_p(dir)
-      File.write(dir.join(filename), content)
-    end
-    def make_app_file(content, path:)
-      dir = tmpdir.join("app", File.dirname(path))
-      FileUtils.mkdir_p(dir)
-      File.write(tmpdir.join("app", path), content)
-    end
-    context "when no legacy references exist" do
-      it "returns :pass" do
-        status, = doctor.send(:check_legacy_constant)
-        expect(status).to eq(:pass)
-      end
-    end
-    context "when an initializer references the legacy constant" do
-      before { make_initializer("#{legacy_const}.configure do |c|\n  c.foo = 1\nend\n") }
-      it "returns :fail" do
-        status, = doctor.send(:check_legacy_constant)
-        expect(status).to eq(:fail)
-      end
-      it "message names the file:line and instructs the rename" do
-        _, msg = doctor.send(:check_legacy_constant)
-        expect(msg).to include("ruact.rb:1")
-        expect(msg).to include("Replace `#{legacy_const}` with `Ruact`")
-      end
-      it "message includes the rename documentation link (AC5)" do
-        _, msg = doctor.send(:check_legacy_constant)
-        expect(msg).to include("https://github.com/luizcg/ruact/blob/main/CHANGELOG.md#renamed")
-      end
-    end
-    context "when an app file requires the legacy gem name" do
-      before { make_app_file("require \"#{legacy_gem}\"\n", path: "models/foo.rb") }
-      it "returns :fail" do
-        status, = doctor.send(:check_legacy_constant)
-        expect(status).to eq(:fail)
-      end
-    end
-    context "when only the modern Ruact constant is referenced" do
-      before { make_initializer("Ruact.configure { |c| c.foo = 1 }\n") }
-      it "returns :pass" do
-        status, = doctor.send(:check_legacy_constant)
-        expect(status).to eq(:pass)
-      end
-    end
-    context "when a word coincidentally contains the legacy substring" do
-      # e.g. "TrailsRsce" should not trigger the regex (boundary check).
-      before { make_initializer("# documenting TrailsRsce_engine here\n") }
-      it "returns :pass" do
-        status, = doctor.send(:check_legacy_constant)
-        expect(status).to eq(:pass)
-      end
-    end
-  end
-  # --- check_serialize_only (Story 13.1, AC2 + AC4) ---
-  describe "#check_serialize_only", :story_13_1 do
-    subject(:doctor) { described_class.new(serialize_only_root: scan_root.to_s) }
-    # Injectable scan root → point the tripwire at a fixture tree.
-    let(:scan_root) { Pathname.new(Dir.mktmpdir) }
-    after { FileUtils.rm_rf(scan_root) }
-    def write_source(name, content)
-      path = scan_root.join(name)
-      FileUtils.mkdir_p(path.dirname)
-      File.write(path, content)
-    end
-    context "with a clean tree (no inbound deserializer)" do
-      before { write_source("clean.rb", "class Foo\n  def bar = 42\nend\n") }
-      it "returns :pass silently" do
-        status, msg = doctor.send(:check_serialize_only)
-        expect(status).to eq(:pass)
-        expect(msg).to include("Serialize-only invariant holds")
-      end
-    end
-    context "with an unguarded inbound Flight deserializer" do
-      before do
-        write_source("evil.rb", "class FlightDeserializer\n  def call(body)\n    parse_flight(body)\n  end\nend\n")
-      end
-      it "returns :fail" do
-        status, = doctor.send(:check_serialize_only)
-        expect(status).to eq(:fail)
-      end
-      it "names the offending file:line and points to the ADR invariant" do
-        _, msg = doctor.send(:check_serialize_only)
-        expect(msg).to include("evil.rb:1") # first offense = the *Deserializer constant
-        expect(msg).to include("CVE-2025-55182")
-        expect(msg).to include("server-functions-api.md")
-      end
-    end
-    context "with a deserializer carrying the allow annotation" do
-      before do
-        annotation = ["# ruact:allow", "flight", "deserialization"].join("-")
-        write_source("guarded.rb", "def parse_flight(body) #{annotation} reviewed legacy bridge\n  body\nend\n")
-      end
-      it "returns :pass (the escape hatch makes it a guard, not a ban)" do
-        status, = doctor.send(:check_serialize_only)
-        expect(status).to eq(:pass)
-      end
-    end
-    context "with a signal that lives in an excluded location" do
-      before do
-        # generators' client-side templates are out of scope (browser RSC)
-        write_source("lib/generators/ruact/install/templates/app.rb", "def parse_flight(b) = b\n")
-      end
-      it "returns :pass (templates are excluded from the scan)" do
-        status, = doctor.send(:check_serialize_only)
-        expect(status).to eq(:pass)
-      end
-    end
-    context "with a `decode_flight` inbound entry point (review finding R3)" do
-      before { write_source("decoder.rb", "def decode_flight(body)\n  body\nend\n") }
-      it "returns :fail (decode_flight is an inbound deserialization signal)" do
-        status, = doctor.send(:check_serialize_only)
-        expect(status).to eq(:fail)
-      end
-    end
-    context "with a deserializer in a differently-located file ALSO named doctor.rb (review finding R1)" do
-      # Only the gem's own lib/ruact/doctor.rb is excluded (by exact path), not
-      # every basename `doctor.rb` — a nested namesake must still be scanned.
-      before { write_source("ruact/flight/doctor.rb", "class FlightDeserializer; end\n") }
-      it "returns :fail (basename collision is not a free pass)" do
-        status, = doctor.send(:check_serialize_only)
-        expect(status).to eq(:fail)
-      end
-    end
-  end
-  # --- check_flight_middleware (Story 13.1, AC3 + AC4) ---
-  describe "#check_flight_middleware", :story_13_1 do
-    subject(:doctor) { described_class.new }
-    # Plain value objects (not doubles) modelling the iterable middleware stack
-    # the check reads — each entry exposes a `.name`, mirroring a real
-    # ActionDispatch::MiddlewareStack::Middleware.
-    def middleware_entry(name)
-      Struct.new(:name).new(name)
-    end
-    def stub_app(stack)
-      app = Struct.new(:middleware).new(stack)
-      allow(Rails).to receive(:application).and_return(app)
-    end
-    context "when a response-transforming middleware (Rack::Deflater) is mounted" do
-      before { stub_app([middleware_entry("Rack::Deflater")]) }
-      it "returns :warn (never :fail)" do
-        status, msg = doctor.send(:check_flight_middleware)
-        expect(status).to eq(:warn)
-        expect(msg).to include("Rack::Deflater").and include("text/x-component")
-      end
-    end
-    context "when no response-transforming middleware is mounted" do
-      before { stub_app([middleware_entry("Rack::Runtime")]) }
-      it "returns :pass" do
-        status, = doctor.send(:check_flight_middleware)
-        expect(status).to eq(:pass)
-      end
-    end
-    context "when no Rails application is present" do
-      before { allow(Rails).to receive(:application).and_return(nil) }
-      it "returns :pass (guarded edge context)" do
-        status, = doctor.send(:check_flight_middleware)
-        expect(status).to eq(:pass)
-      end
-    end
-    context "when the app is not yet booted (middleware is a non-enumerable proxy)" do
-      # Mirrors a pre-`initialize!` Rails::Configuration::MiddlewareStackProxy,
-      # which does NOT respond to :each — must be skipped, not crash on filter_map.
-      before do
-        proxy = Object.new # responds to neither :each nor :filter_map
-        app = Struct.new(:middleware).new(proxy)
-        allow(Rails).to receive(:application).and_return(app)
-      end
-      it "returns :pass without raising" do
-        expect { doctor.send(:check_flight_middleware) }.not_to raise_error
-        status, = doctor.send(:check_flight_middleware)
-        expect(status).to eq(:pass)
-      end
-    end
-  end
-  # --- :warn status semantics (Story 13.1, AC3) ---
-  describe "#format_result with :warn", :story_13_1 do
-    subject(:doctor) { described_class.new }
-    it "renders :warn with the ⚠ glyph (not ✗)" do
-      expect(doctor.send(:format_result, :warn, "heads up")).to eq("⚠ heads up")
-    end
-  end
-  describe "#run with an unexpected status (review finding R1)", :story_13_1 do
-    before do
-      make_manifest
-      make_controller(with_include: true)
-      make_layout(with_sentinel: true)
-      allow(TCPSocket).to receive(:new).and_return(instance_double(TCPSocket, close: nil))
-    end
-    it "fails the run when a check returns a status that is neither :pass nor :warn" do
-      doctor = described_class.new
-      # All other checks pass; a malformed status (rendered ✗) must NOT be
-      # silently treated as a pass — only :pass / :warn are success.
-      allow(doctor).to receive(:check_streaming).and_return([:error, "broken status"])
-      expect(doctor.run).to be false
-    end
-  end
-  describe ".run with a :warn present (Story 13.1, AC3)", :story_13_1 do
-    before do
-      make_manifest
-      make_controller(with_include: true)
-      make_layout(with_sentinel: true)
-      allow(TCPSocket).to receive(:new).and_return(instance_double(TCPSocket, close: nil))
-      app = Struct.new(:middleware).new([Struct.new(:name).new("Rack::Deflater")])
-      allow(Rails).to receive(:application).and_return(app)
-    end
-    it "still returns true — a :warn does not fail the run" do
-      expect(described_class.run).to be true
-    end
-    it "prints the warning glyph" do
-      expect { described_class.run }.to output(/⚠.*Rack::Deflater/).to_stdout
-    end
-    it "does not print the fix hint" do
-      expect { described_class.run }.not_to output(/rails generate/).to_stdout
-    end
-  end
-  # --- run / .run ---
-  describe ".run / #run (AC#1, #7)" do
-    before do
-      make_manifest
-      make_controller(with_include: true)
-      make_layout(with_sentinel: true)
-      allow(TCPSocket).to receive(:new).and_return(instance_double(TCPSocket, close: nil))
-    end
-    context "when all checks pass" do
-      it "returns true" do
-        expect(described_class.run).to be true
-      end
-      it "does not print the fix hint" do
-        expect { described_class.run }.not_to output(/rails generate/).to_stdout
-      end
-    end
-    context "when any check fails" do
-      before { allow(TCPSocket).to receive(:new).and_raise(Errno::ECONNREFUSED) }
-      it "returns false" do
-        expect(described_class.run).to be false
-      end
-      it "prints the fix hint" do
-        expect { described_class.run }
-          .to output(/Run rails generate ruact:install to fix configuration issues/).to_stdout
-      end
-    end
-  end
-  # --- JSON introspection (Story 15.3, FR107, AC1 + AC3 + AC4) ---
-  describe "#as_json (machine-readable report)", :story_15_3 do
-    subject(:doctor) { described_class.new }
-    context "when all checks pass" do
-      before do
-        make_manifest
-        make_controller(with_include: true)
-        make_layout(with_sentinel: true)
-        allow(TCPSocket).to receive(:new).and_return(instance_double(TCPSocket, close: nil))
-      end
-      it "returns a document that round-trips through JSON.parse", :aggregate_failures do
-        report = doctor.as_json
-        reparsed = JSON.parse(JSON.generate(report))
-        expect(reparsed).to eq(report)
-      end
-      it "gates the document with the EXPERIMENTAL schema_version (0)" do
-        expect(doctor.as_json["schema_version"]).to eq(0)
-        expect(doctor.as_json["schema_version"]).to eq(Ruact::Doctor::SCHEMA_VERSION)
-      end
-      it "carries every check with name/status/message/remediation keys", :aggregate_failures do
-        checks = doctor.as_json["checks"]
-        expect(checks.map { |c| c["name"] }).to eq(described_class::CHECKS.map(&:to_s))
-        checks.each do |check|
-          expect(check.keys).to contain_exactly("name", "status", "message", "remediation")
-          expect(check["status"]).to be_a(String)
-          expect(check["message"]).to be_a(String)
-        end
-      end
-      it "reports top-level status 'pass' and exits-zero semantics" do
-        expect(doctor.as_json["status"]).to eq("pass")
-      end
-      it "emits NO prose to stdout (JSON mode is the document only)" do
-        expect { doctor.as_json }.not_to output.to_stdout
-      end
-    end
-    context "when a check fails" do
-      before do
-        make_controller(with_include: true)
-        make_layout(with_sentinel: true)
-        allow(TCPSocket).to receive(:new).and_return(instance_double(TCPSocket, close: nil))
-        # manifest is missing → check_manifest fails
-      end
-      it "reports top-level status 'fail' (non-zero exit semantics)" do
-        expect(doctor.as_json["status"]).to eq("fail")
-      end
-      it "carries the separate machine-readable remediation for the failing check" do
-        manifest = doctor.as_json["checks"].find { |c| c["name"] == "manifest" }
-        expect(manifest["status"]).to eq("fail")
-        expect(manifest["remediation"]).to eq("Run vite build (or bin/dev) to generate the client manifest.")
-      end
-    end
-    context "when a check warns (Rack::Deflater mounted)" do
-      before do
-        make_manifest
-        make_controller(with_include: true)
-        make_layout(with_sentinel: true)
-        allow(TCPSocket).to receive(:new).and_return(instance_double(TCPSocket, close: nil))
-        app = Struct.new(:middleware).new([Struct.new(:name).new("Rack::Deflater")])
-        allow(Rails).to receive(:application).and_return(app)
-      end
-      it "keeps status 'pass' — a :warn does not fail — but exposes the warn check" do
-        report = doctor.as_json
-        expect(report["status"]).to eq("pass")
-        flight = report["checks"].find { |c| c["name"] == "flight_middleware" }
-        expect(flight["status"]).to eq("warn")
-        expect(flight["remediation"]).to include("Exclude text/x-component from compression")
-      end
-    end
-    it "keeps the message byte-identical to the human tuple (remediation is separate)" do
-      _status, human_message = doctor.send(:check_manifest)
-      json_message = doctor.as_json["checks"].find { |c| c["name"] == "manifest" }["message"]
-      expect(json_message).to eq(human_message)
-    end
-  end
-  describe "#results / #passed? (shared compute, Story 15.3)", :story_15_3 do
-    subject(:doctor) { described_class.new }
-    before do
-      make_manifest
-      make_controller(with_include: true)
-      make_layout(with_sentinel: true)
-      allow(TCPSocket).to receive(:new).and_return(instance_double(TCPSocket, close: nil))
-    end
-    it "returns one tuple per check, index-aligned with CHECKS" do
-      expect(doctor.results.length).to eq(described_class::CHECKS.length)
-    end
-    it "passed? is true when all checks pass/warn" do
-      expect(doctor.passed?).to be true
-    end
-    it "runs each check exactly once (as_json does not double-run check_vite's socket)" do
-      # check_vite opens a socket via TCPSocket.new; a single as_json must open it
-      # exactly once — proving #results is computed once, not per consumer.
-      doctor.as_json
-      expect(TCPSocket).to have_received(:new).once
-    end
-  end
-  describe "#run stays byte-identical after the #results refactor (Story 15.3, AC4a)", :story_15_3 do
-    subject(:doctor) { described_class.new }
-    before do
-      make_manifest
-      make_controller(with_include: true)
-      make_layout(with_sentinel: true)
-      allow(TCPSocket).to receive(:new).and_return(instance_double(TCPSocket, close: nil))
-    end
-    it "prints exactly the header + one format_result line per check (no JSON/prose leak), returns true" do
-      # Reconstruct the expected output from the SAME format_result the human
-      # path uses — proves #run still emits header + one glyph line per check
-      # (the optional 3rd remediation element is ignored) and nothing else.
-      lines = doctor.results.map { |status, message| doctor.send(:format_result, status, message) }.join("\n")
-      expected = "[ruact] Health check\n#{lines}\n"
-      expect { expect(doctor.run).to be true }.to output(expected).to_stdout
-    end
-  end
-  describe "Rake task definition (Story 5.12)", :story_5_12 do
-    # Loads gem/lib/tasks/ruact.rake into a fresh Rake::Application so the
-    # task table is isolated from any other spec that may have loaded tasks.
-    # Asserts the new namespace is discoverable AND the legacy namespace is
-    # gone — guards against a partial rename (e.g. file renamed but the
-    # namespace inside left as :rsc, or vice-versa).
-    around do |example|
-      require "rake"
-      prev = Rake.application
-      Rake.application = Rake::Application.new
-      Rake.application.add_loader("rake", Rake::DefaultLoader.new)
-      Rake.application.add_loader("rb",   Rake::DefaultLoader.new)
-      load File.expand_path("../../lib/tasks/ruact.rake", __dir__)
-      example.run
-    ensure
-      Rake.application = prev
-    end
-    let(:rake_app) { Rake.application }
-    it "defines Rake::Task['ruact:doctor']" do
-      expect(rake_app.lookup("ruact:doctor")).not_to be_nil
-    end
-    it "does NOT define the legacy Rake task under the pre-Story-5.12 namespace" do
-      # Build the legacy task name from fragments so this spec file stays
-      # grep-clean under the Story 5.12 CI guard (which rejects the literal
-      # `<legacy>:doctor` substring anywhere in tracked files).
-      legacy_task = %w[rsc doctor].join(":")
-      expect(rake_app.lookup(legacy_task)).to be_nil
-    end
-    it "the ruact:doctor task action invokes Ruact::Doctor.run" do
-      task = rake_app.lookup("ruact:doctor")
-      expect(task).not_to be_nil
-      # actions is a list of Procs; just assert at least one is attached
-      expect(task.actions).not_to be_empty
-    end
-    it "defines Rake::Task['ruact:routes'] with an action (Story 15.3)", :story_15_3 do
-      task = rake_app.lookup("ruact:routes")
-      expect(task).not_to be_nil
-      expect(task.actions).not_to be_empty
-    end
-  end
-end