Merge pull request #148 from eregon/default-cruby

Default to CRuby for https://try.ruby-lang.org/

Author: hmdne

app/ruby_engine.rb

@@ -9,6 +9,16 @@ def run(source)
     raise NotImplementedError
   end
 
+  def exception_to_string(err)
+    # Beautify the backtrace a little bit
+    backtrace = err.backtrace
+    backtrace = backtrace.select { |i| i.include? '<anonymous>' }
+    backtrace = backtrace.map { |i| i.gsub(/.*(<anonymous>)/, '\1') }
+    backtrace = ["(file)"] if backtrace.empty?
+    err.set_backtrace(backtrace)
+    err.full_message
+  end
+
   def run_with_writer(source, writer, &block)
     @writer = writer
     @dots = 0
@@ -29,8 +39,6 @@ def loading(part = nil)
   # When you update the engines, ensure that they are tested correctly.
   # Update the engine list also in spec/playground_spec.
   ENGINES = [
-    # Opal.new,
-    OpalWebWorker.new("1.7.1"),
     CRubyWASI.new(
       "https://cdn.jsdelivr.net/npm/ruby-head-wasm-wasi@0.5.0-2022-12-25-a/dist/ruby.wasm",
       "3.2.0"
@@ -41,6 +49,9 @@ def loading(part = nil)
     #   "https://cdn.jsdelivr.net/npm/ruby-head-wasm-wasi@next/dist/ruby.wasm",
     #   "3.3.0dev"
     # ),
+
+    # Opal.new,
+    OpalWebWorker.new("1.7.1"),
   ].each_with_object({}) do |engine, hash|
     hash[engine.engine_id] = engine
   end

app/ruby_engine/cruby_wasi.rb

@@ -74,8 +74,8 @@ def run(source)
         textDecoder = `new TextDecoder("utf-8")`
         %x{
           wasmFs.fs.writeSync = (fd, buffer, offset, length, position) => {
-            const text = textDecoder.decode(buffer);
             if (fd == 1 || fd == 2) {
+              const text = textDecoder.decode(buffer);
               #{@writer.print_to_output(`text`, "")};
             }
             return originalWriteSync(fd, buffer, offset, length, position);
@@ -102,11 +102,18 @@ def run(source)
         `vm.setInstance(wasmInstance)`.await
         `wasi.setMemory(wasmInstance.exports.memory)`
         `vm.initialize()`
+        set_external_encoding = "Encoding.default_external = Encoding::UTF_8"
+        `vm.eval(set_external_encoding)`
       end
 
       yield `vm.eval(source).toString()`
     rescue JS::Error => err
-      @writer.log_error(err)
+      raise err
+    end
+
+    def exception_to_string(err)
+      # "...: undefined method `reverse' for 40:Integer (NoMethodError)\n (Exception)\n"
+      super(err).sub(/\s+\(Exception\)\s*\z/, '')
     end
   end
 end

app/try_ruby.rb

@@ -14,7 +14,7 @@ class TryRuby
   RUBY
   INITIAL_TRY_RESULT = 'Welcome ' * 3
 
-  DEFAULT_RUBY_ENGINE = "opal-ww-1.7.1"
+  DEFAULT_RUBY_ENGINE = "cruby-3.2.0"
 
   def self.start
     instance
@@ -397,14 +397,14 @@ def show_result(retval)
     # Do not check the answer if there is no regexp matcher
     if @current_item && @current_item.answer
       # Get last line of output
-      value_to_check = @output_buffer.length > 0 && !@output_buffer.last.empty? ? @output_buffer.last.chomp : ''
+      value_to_check = @output_buffer.join.rstrip.lines.last
 
       # Check if output matches the defined answer regexp
       # and print status message
       print_to_output("\n")
       from = count_lines
 
-      if !value_to_check.empty? && value_to_check.chomp.match(@current_item.answer)
+      if value_to_check && !value_to_check.empty? && value_to_check.match(@current_item.answer)
         @current_item.ok.each do |line|
           print_to_output(line)
         end
@@ -431,13 +431,7 @@ def count_lines
 
   def log_error(err)
     unless err.is_a? String
-      # Beautify the backtrace a little bit
-      backtrace = err.backtrace
-      backtrace = backtrace.select { |i| i.include? '<anonymous>' }
-      backtrace = backtrace.map { |i| i.gsub(/.*(<anonymous>)/, '\1') }
-      backtrace = ["(file)"] if backtrace.empty?
-      err.set_backtrace(backtrace)
-      err = err.full_message
+      err = @engine.exception_to_string(err)
     end
 
     from = count_lines