small linting changes

Author: AbanoubGhadban

lib/react_on_rails/helper.rb

@@ -477,7 +477,10 @@ def registered_stores_including_deferred
 
     def create_render_options(react_component_name, options)
       # If no store dependencies are passed, default to all registered stores up till now
-      options[:store_dependencies] ||= registered_stores_including_deferred.map { |store| store[:store_name] }
+      unless options.key?(:store_dependencies)
+        store_dependencies = registered_stores_including_deferred.map { |store| store[:store_name] }
+        options = options.merge(store_dependencies: store_dependencies.presence)
+      end
       ReactOnRails::ReactComponent::RenderOptions.new(react_component_name: react_component_name,
                                                       options: options)
     end
@@ -641,7 +644,7 @@ def internal_react_component(react_component_name, options = {})
                                                 "data-component-name" => render_options.react_component_name,
                                                 "data-trace" => (render_options.trace ? true : nil),
                                                 "data-dom-id" => render_options.dom_id,
-                                                "data-store-dependencies" => render_options.store_dependencies.to_json,
+                                                "data-store-dependencies" => render_options.store_dependencies&.to_json,
                                                 "data-force-load" => (render_options.force_load ? true : nil))
 
       if render_options.force_load

node_package/src/ClientSideRenderer.ts

@@ -116,7 +116,7 @@ class ComponentRenderer {
         }
       }
     } catch (e: unknown) {
-      const error = e instanceof Error ? e : new Error('Unknown error');
+      const error = e instanceof Error ? e : new Error(e?.toString() ?? 'Unknown error');
       console.error(error.message);
       error.message = `ReactOnRails encountered an error while rendering component: ${name}. See above error message.`
       throw error;

node_package/src/transformRSCStreamAndReplayConsoleLogs.ts

@@ -24,10 +24,10 @@ export default function transformRSCStreamAndReplayConsoleLogs(stream: ReadableS
           });
 
         for (const jsonChunk of jsonChunks) {
-          const { html, consoleReplayScript } = jsonChunk;
+          const { html, consoleReplayScript = '' } = jsonChunk;
           controller.enqueue(encoder.encode(html));
 
-          const replayConsoleCode = consoleReplayScript?.trim().replace(/^<script.*>/, '').replace(/<\/script>$/, '');
+          const replayConsoleCode = consoleReplayScript.trim().replace(/^<script.*>/, '').replace(/<\/script>$/, '');
           if (replayConsoleCode?.trim() !== '') {
             const scriptElement = document.createElement('script');
             scriptElement.textContent = replayConsoleCode;

node_package/src/turbolinksUtils.ts

@@ -14,7 +14,7 @@ export function debugTurbolinks(...msg: string[]): void {
   }
 
   const context = reactOnRailsContext();
-  if (context.ReactOnRails && context.ReactOnRails.option('traceTurbolinks')) {
+  if (context.ReactOnRails?.option('traceTurbolinks')) {
     console.log('TURBO:', ...msg);
   }
 }