proscenium 0.1.2-arm64-darwin → 0.2.0-arm64-darwin

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d20f6189b8ed53c8f8c49fdd34ff7c94e9dd664e4a270d0f19300ce2435868b2
-  data.tar.gz: 19ca20f0dc49a00ccacddc79459cfa1f9676f3d89d5f276a8c1b9c27637f2212
+  metadata.gz: 37bbbaaaf48e2acf58f2f79a62a96a2829df81c0cffb1d6de8116382ceb3cf92
+  data.tar.gz: b421eacfd9fae7883c6858aaf30e77b4a1be63451ccdb8e42ce060013fb022e1
 SHA512:
-  metadata.gz: 2703a824ce1acc0e8aa972012fe275b881d8946ff63c8498f8b52de64229faeadc4008e00acb960078c5c9c3d8c02b0219ddaeb75f0b7479f529e7f34495e12a
-  data.tar.gz: fa206820113a895af26afa184c88045a62ac307e83eaf7337c9feaff202f2d4ac01d16de9b19378f162397bb839f6ca268b0429ec515a39d21899b34b706a8a5
+  metadata.gz: 67e7d21eb3053d07f412d9004b983faf28a10707b56d5ef3dcb638ed70296094018fe419e482649669d936570b47f9c9767235a3e65866afa815c30f6d9ab1e4
+  data.tar.gz: af496cb5b9ed927983630bcbd88ce8c911e3b1df4989fd454ae5c76ca6c9e9c50ef8cdc98f59f1613ac95d315a5b250713d7ff68910d0dd8b04027bc2e66b886

data/lib/proscenium/compilers/esbuild/resolve_plugin.js

@@ -11,6 +11,9 @@ export default setup('resolve', (build, options) => {
   const runtimeCwdAlias = `${cwd}/proscenium-runtime`
   let bundled = false
 
+  const env = Deno.env.get('RAILS_ENV')
+  const isProd = env === 'production'
+
   return [
     {
       type: 'onResolve',
@@ -28,24 +31,7 @@ export default setup('resolve', (build, options) => {
           return { external: true }
         }
 
-        // Proscenium runtime
-        // if (args.path.startsWith('@proscenium/')) {
-        //   const result = { suffix: args.suffix }
-
-        //   if (args.queryParams?.has('bundle-all')) {
-        //     bundled = true
-        //   }
-
-        //   if (bundled || args.queryParams?.has('bundle')) {
-        //     result.path = join(runtimeDir, `${args.path.replace(/^@proscenium/, '')}/index.js`)
-        //   } else {
-        //     result.path = `${args.path.replace(/^@proscenium/, '/proscenium-runtime')}/index.js`
-        //     result.external = true
-        //   }
-
-        //   return result
-        // }
-
+        // Rewrite the path to the actual runtime directory.
         if (args.path.startsWith(runtimeCwdAlias)) {
           return { path: join(runtimeDir, args.path.slice(runtimeCwdAlias.length)) }
         }
@@ -88,11 +74,12 @@ export default setup('resolve', (build, options) => {
     // Resolve the path using esbuild's internal resolution. This allows us to import node packages
     // and extension-less paths without custom code, as esbuild with resolve them for us.
     const resolveResult = await build.resolve(result.path, {
-      // If path is a bare module (node_modules), and resolveDir is the Proscenium runtime dir, then
-      // use `cwd` as the `resolveDir`, otherwise pass it through as is. This ensures that nested
-      // node_modules are resolved correctly.
+      // If path is a bare module (node_modules), and resolveDir is the Proscenium runtime dir, or
+      // is the current working dir, then use `cwd` as the `resolveDir`, otherwise pass it through
+      // as is. This ensures that nested node_modules are resolved correctly.
       resolveDir:
-        isBareModule(result.path) && params.resolveDir.startsWith(runtimeDir)
+        isBareModule(result.path) &&
+        (!params.resolveDir.startsWith(cwd) || params.resolveDir.startsWith(runtimeDir))
           ? cwd
           : params.resolveDir,
       pluginData: {
@@ -118,9 +105,17 @@ export default setup('resolve', (build, options) => {
 
     if (resolveResult.path.startsWith(runtimeDir)) {
       result.path = '/proscenium-runtime' + resolveResult.path.slice(runtimeDir.length)
-    } else if (!resolveResult.path.startsWith(cwd)) {
-      // If resolved path does not start with cwd, then it is most likely linked, so bundle it.
-      return { ...resolveResult, suffix: '?bundle' }
+    } else if (!resolveResult.path.startsWith(cwd) && !isProd) {
+      // Resolved path is not in the current working directory. It could be linked to a file outside
+      // the CWD, or it's just invalid. If not in production, return as an outsideRoot namespaced,
+      // and externally suffixed path. This lets the Rails Proscenium::Middleware::OutsideRoot
+      // handle the import.
+      return {
+        ...resolveResult,
+        namespace: 'outsideRoot',
+        path: `${resolveResult.path}?outsideRoot`,
+        external: true
+      }
     } else {
       result.path = resolveResult.path.slice(cwd.length)
     }

data/lib/proscenium/compilers/esbuild.js

@@ -23,7 +23,7 @@ import throwCompileError from './esbuild/compile_error.js'
  *   esbuild [OPTIONS] <PATHS_ARG>...
  *
  * ARGS:
- *   <PATHS_ARG>...    One or more file paths to compile.
+ *   <PATHS_ARG>... One or more file paths or globs to compile.
  *
  * OPTIONS:
  *   --root
@@ -80,8 +80,8 @@ async function main(paths = [], options = {}) {
   const env = Deno.env.get('RAILS_ENV')
   const isProd = env === 'production'
   const isTest = env === 'test'
-
   const entryPoints = new Set()
+
   for (let i = 0; i < paths.length; i++) {
     const path = paths[i]
 
@@ -89,6 +89,10 @@ async function main(paths = [], options = {}) {
       for await (const file of expandGlob(path, { root })) {
         file.isFile && entryPoints.add(file.path)
       }
+    } else if (path.startsWith('/')) {
+      // Path is absolute, so it must be outsideRoot. Don't prefix the root.
+      // See Proscenium::Middleware::OutsideRoot.
+      entryPoints.add(path)
     } else {
       entryPoints.add(join(root, path))
     }

data/lib/proscenium/helper.rb

@@ -3,7 +3,15 @@
 module Proscenium
   module Helper
     def compute_asset_path(path, options = {})
-      return "/#{path}" if %i[javascript stylesheet].include?(options[:type])
+      if %i[javascript stylesheet].include?(options[:type])
+        result = "/#{path}"
+
+        if (qs = Proscenium.config.cache_query_string)
+          result << "?#{qs}"
+        end
+
+        return result
+      end
 
       super
     end

data/lib/proscenium/middleware/base.rb

@@ -60,6 +60,10 @@ module Proscenium
         response.content_type = content_type
         response['X-Proscenium-Middleware'] = name
 
+        if Proscenium.config.cache_query_string
+          response['Cache-Control'] = "public, max-age=#{2.days.to_i}"
+        end
+
         yield response if block_given?
 
         response.finish

data/lib/proscenium/middleware/outside_root.rb

@@ -0,0 +1,26 @@
+# frozen_string_literal: true
+
+module Proscenium
+  class Middleware
+    # Provides a way to render files outside of the Rails root during non-production. This is
+    # primarily to support linked NPM modules, for example when using `pnpm link ...`.
+    class OutsideRoot < Esbuild
+      private
+
+      # @override [Esbuild] reassigns root to '/'.
+      def renderable?
+        old_root = root
+        @root = Pathname.new('/')
+
+        super
+      ensure
+        @root = old_root
+      end
+
+      # @override [Esbuild] does not remove leading slash, ensuring it is an absolute path.
+      def path
+        @request.path
+      end
+    end
+  end
+end

data/lib/proscenium/middleware.rb

@@ -10,6 +10,7 @@ module Proscenium
     autoload :Base
     autoload :Esbuild
     autoload :Runtime
+    autoload :OutsideRoot
 
     def initialize(app)
       @app = app
@@ -38,6 +39,12 @@ module Proscenium
     def find_type(request)
       path = Pathname.new(request.path)
 
+      # Non-production only!
+      if request.query_string == 'outsideRoot'
+        return if Rails.env.production?
+        return OutsideRoot if path.fnmatch?(glob_types[:outsideRoot], File::FNM_EXTGLOB)
+      end
+
       return Runtime if path.fnmatch?(glob_types[:runtime], File::FNM_EXTGLOB)
       return Esbuild if path.fnmatch?(glob_types[:esbuild], File::FNM_EXTGLOB)
     end

data/lib/proscenium/railtie.rb

@@ -13,7 +13,8 @@ module Proscenium
   # See https://doc.deno.land/https://deno.land/std@0.145.0/path/mod.ts/~/globToRegExp
   DEFAULT_GLOB_TYPES = {
     esbuild: '/{config,app,lib,node_modules}/**.{js,mjs,jsx,css}',
-    runtime: '/proscenium-runtime/**.{js,jsx}'
+    runtime: '/proscenium-runtime/**.{js,jsx}',
+    outsideRoot: '/**/*.{js,jsx,mjs,css}'
   }.freeze
 
   class << self
@@ -27,6 +28,7 @@ module Proscenium
 
     config.proscenium = ActiveSupport::OrderedOptions.new
     config.proscenium.side_load = true
+    config.proscenium.cache_query_string = Rails.env.production? && ENV.fetch('REVISION', nil)
     config.proscenium.auto_reload = Rails.env.development?
     config.proscenium.auto_reload_paths ||= %w[lib app config]
     config.proscenium.auto_reload_extensions ||= /\.(css|jsx?)$/

data/lib/proscenium/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Proscenium
-  VERSION = '0.1.2'
+  VERSION = '0.2.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: proscenium
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.2.0
 platform: arm64-darwin
 authors:
 - Joel Moss
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-10-16 00:00:00.000000000 Z
+date: 2022-10-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: actioncable
@@ -167,6 +167,7 @@ files:
 - lib/proscenium/middleware/base.rb
 - lib/proscenium/middleware/esbuild.rb
 - lib/proscenium/middleware/lightningcss.rb
+- lib/proscenium/middleware/outside_root.rb
 - lib/proscenium/middleware/runtime.rb
 - lib/proscenium/middleware/static.rb
 - lib/proscenium/phlex.rb