opal 1.3.1 → 1.3.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3a33a6b0ec566d750d22d1bca26dcad96c9410b6c9f009c2d576d5d034d78b76
-  data.tar.gz: cc9550e4097c954b66d556ed11a1cb90a88336383b4ff1fc8d732117f773117e
+  metadata.gz: 2b87866d518e2345b4aa120b4f8d457dad6f86ed9f365974161efad2e2f0dde9
+  data.tar.gz: a3dac2be866ce053470b56377131f7615f0ff6bc3dc7a1d567ce77a8efff5278
 SHA512:
-  metadata.gz: fb47426dadf8b67198eaf7efcfb5b2d91bc1c567d34247fa170e8a4644236d308bcfabf53e8e428b64728feb81da6692325b9f4dd6d93ebf14a0f0345f020e80
-  data.tar.gz: b2bbd78addde393c3489fd4d55f3df1d259457415a1593c2a46343943ff227bde4b668a829a434ad57444284681591fab31dcd73e68c6f1acb3677b2c3496649
+  metadata.gz: 161e91aa69fc166adf4719b56724251e97239de73dba172916a3de5c744e4d7166a33b0fdc97f9fb68079c0627815a5ae00aba0cbedb137b35a6fc34191979ac
+  data.tar.gz: ec37e385c55325552d2b7a82fc4c9ec4cd2a98aa3be966a6f9c193e483d0278f58ac8241eec65e12d9e965ea4eda17454430ae284352526ce408e7bc69729ad3

data/CHANGELOG.md

@@ -15,16 +15,37 @@ Changes are grouped as follows:
 
 
 
-## [3.0.1](https://github.com/opal/opal/compare/v1.3.0...v3.0.1) - 2021-11-03
+## [1.3.2](https://github.com/opal/opal/compare/v1.3.1...v1.3.2) - 2021-11-10
 
 
 ### Fixed
 
-* Fix REPL if bundler environment isn't set ([#2338](https://github.com/opal/opal/pull/2338))
-* Fix Chrome runner if bundler environment isn't set and make it work on other Unixes ([#2339](https://github.com/opal/opal/pull/2339))
-* `Proc#binding` to return a binding if `Binding` is defined (#2341, #2340)
-* `Array#zip` to correctly `yield` (#2342, #1611)
-* `String#scan` to correctly `yield` (#2342, #1660)
+- Update documentation ([#2350](https://github.com/opal/opal/pull/2350))
+- Fix `IO#gets` getting an extra char under some circumstances ([#2349](https://github.com/opal/opal/pull/2349))
+- Raise a `TypeError` instead of `UndefinedMethod` if not a string is passed to `__send__` ([#2346](https://github.com/opal/opal/pull/2346))
+- Do not modify `$~` when calling `String#scan` from internal methods ([#2353](https://github.com/opal/opal/pull/2353))
+- Stop interpreting falsey values as a missing constant in `Module#const_get` ([#2354](https://github.com/opal/opal/pull/2354))
+
+<!--
+### Changed
+### Deprecated
+### Removed
+### Internal
+-->
+
+
+
+
+## [1.3.1](https://github.com/opal/opal/compare/v1.3.0...v1.3.1) - 2021-11-03
+
+
+### Fixed
+
+- Fix REPL if bundler environment isn't set ([#2338](https://github.com/opal/opal/pull/2338))
+- Fix Chrome runner if bundler environment isn't set and make it work on other Unixes ([#2339](https://github.com/opal/opal/pull/2339))
+- `Proc#binding` to return a binding if `Binding` is defined (#2341, #2340)
+- `Array#zip` to correctly `yield` (#2342, #1611)
+- `String#scan` to correctly `yield` (#2342, #1660)
 
 
 
@@ -94,7 +115,7 @@ Changes are grouped as follows:
 ### Fixed
 
 - Fixed multiple line `Regexp` literal to not generate invalid syntax as JavaScript ([#1616](https://github.com/opal/opal/pull/1616))
-- Fix `Kernel#{try,catch}` along with `UncaughtThrowError` ([#2264](https://github.com/opal/opal/pull/2264))
+- Fix `Kernel#{throw,catch}` along with `UncaughtThrowError` ([#2264](https://github.com/opal/opal/pull/2264))
 - Update source-map-support to fix an off-by-one error ([#2264](https://github.com/opal/opal/pull/2264))
 - Source map: lines should start from 1, not 0 ([#2273](https://github.com/opal/opal/pull/2273))
 - Allow for multiple underscored args with the same name in strict mode ([#2292](https://github.com/opal/opal/pull/2292))
@@ -123,10 +144,6 @@ Changes are grouped as follows:
 - `Process` is now a Module, not a Class - just like in MRI ([#2332](https://github.com/opal/opal/pull/2332))
 - `s = StringIO.new("a"); s << "b"; s.string` now returns "b", like MRI, but Opal used to return "ab" ([#2309](https://github.com/opal/opal/pull/2309))
 
-### Deprecated
-
-### Removed
-
 ### Internal
 
 - Switch from jshint to ESLint ([#2289](https://github.com/opal/opal/pull/2289))

data/UNRELEASED.md

@@ -1,13 +1,7 @@
-### Added
+<!--
 ### Fixed
-
-* Fix REPL if bundler environment isn't set (#2338)
-* Fix Chrome runner if bundler environment isn't set and make it work on other Unixes (#2339)
-* `Proc#binding` to return a binding if `Binding` is defined (#2341, #2340)
-* `Array#zip` to correctly `yield` (#2342, #1611)
-* `String#scan` to correctly `yield` (#2342, #1660)
-
 ### Changed
 ### Deprecated
 ### Removed
 ### Internal
+-->

data/docs/compiled_ruby.md

@@ -30,8 +30,8 @@ to convert values to opal specific values.
 
 NOTE: Because `true` and `false` compile to their native
 javascript equivalents, they must share the same class: `Boolean`.
-For this reason, they do not belong to their respective `TrueClass`
-and `FalseClass` classes from ruby.
+Thru some level of hackery, we make them pseudo-members of the appropriate
+`TrueClass` and `FalseClass`.
 
 #### Strings & Symbols
 
@@ -55,11 +55,11 @@ class. Symbols and Strings can therefore be used interchangeably.
 
 #### Numbers
 
-In Opal there is a single class for numbers; `Numeric`. To keep Opal
+In Opal there is a single class for numbers; `Number`. To keep Opal
 as performant as possible, Ruby numbers are mapped to native numbers.
 This has the side effect that all numbers must be of the same class.
-Most relevant methods from `Integer`, `Float` and `Numeric` are
-implemented on this class.
+Most relevant methods from `Integer` and `Float` are implemented on
+this class.
 
 ```ruby
 42        # => 42
@@ -104,7 +104,7 @@ else is a truthy value including `""`, `0` and `[]`. This differs from
 JavaScript as these values are also treated as false.
 
 For this reason, most truthy tests must check if values are `false` or
-`nil`.
+`nil` (we also check for `null` and `undefined`).
 
 Taking the following test:
 
@@ -204,16 +204,16 @@ obvious what went wrong.
 
 As Opal just generates JavaScript, it is useful to use a native
 debugger to work through JavaScript code. To use a debugger, simply
-add an x-string similar to the following at the place you wish to
-debug:
+add a `debugger` statement:
 
 ```ruby
 # .. code
-`debugger`
+debugger
 # .. more code
 ```
-The x-strings just pass the debugger statement straight through to the
-JavaScript output.
+
+The `debugger` statement is compiled to become a JavaScript `debugger`
+statement. This statement breaks the code if you have your Inspector open.
 
 NOTE: All local variables and method/block arguments also keep their Ruby
 names except in the rare cases when the name is reserved in JavaScript.
@@ -246,7 +246,7 @@ in the browser.
 }
 
 # => opal version is:
-# => 0.6.0
+# => 1.3.1
 ```
 
 Even interpolations are supported, as seen here.
@@ -280,6 +280,9 @@ require 'native'
 win = Native(`window`) # equivalent to Native::Object.new(`window`)
 ```
 
+To access a Native-wrapped global JavaScript object, we can also use `$$`, after
+we have the `native` module required.
+
 Now what if we want to access one of its properties?
 
 ```ruby
@@ -514,10 +517,10 @@ Opal.BAZ;          // => 789
 To reach nested constants the safest way is to call `#const_get` on `Object`:
 
 ```javascript
-Opal.Object.$$const_get('Bar::BAR'); // => 123
-Opal.Object.$$const_get('Foo::BAR'); // => 123
-Opal.Object.$$const_get('Foo::FOO'); // => 456
-Opal.Object.$$const_get('BAZ');      // => 789
+Opal.Object.$const_get('Bar::BAR'); // => 123
+Opal.Object.$const_get('Foo::BAR'); // => 123
+Opal.Object.$const_get('Foo::FOO'); // => 456
+Opal.Object.$const_get('BAZ');      // => 789
 ```
 
 Constants can also be navigated using the `$$` property, although this is limited to constants defined directly under the current object:
@@ -528,6 +531,14 @@ Opal.Foo.$$.FOO // => 456
 Opal.Foo.$$.BAR // => undefined
 ```
 
+A later feature also allows you to skip the `$$` property:
+
+```javascript
+Opal.Bar.BAR // => 123
+Opal.Foo.FOO // => 456
+Opal.Foo.BAR // => undefined
+```
+
 
 ### Calling methods
 

data/docs/faq.md

@@ -10,7 +10,7 @@ We run opal against the [ruby spec](https://github.com/ruby/spec) as our primary
 
 ### What version of ruby does Opal target?
 
-We are running tests under ruby 2.0.0 conditions, but are mostly compatible with 1.9 level features.
+We are running tests under ruby 3.0.0 conditions, but are mostly compatible with 2.6 level features.
 
 ### Why doesn't Opal support mutable strings?
 

data/docs/headless_chrome.md

@@ -3,7 +3,6 @@
 ## Requirements
 
 First of all, make sure that you have Chrome at least 59.0 installed.
-Also for now it's supported only on Mac and Linux. (version 60 may get support on Windows)
 
 ## Using the runner
 
@@ -22,11 +21,9 @@ The runner also listens for any exceptions and prints formatted stracktraces bac
     raising_method
     "
 
-    RuntimeError : test error
-        at $$raise (file:///tmp/chrome-opal.js:4996:6)
-        at $$raising_method (file:///tmp/chrome-opal.js:21144:16)
-        at  (file:///tmp/chrome-opal.js:21146:14)
-        at  (file:///tmp/chrome-opal.js:21147:2)
+    RuntimeError: test error
+      from <internal:corelib/…>:2693:7:in `<main>'
+      from -e:1:1:in `undefined'
 
 ## Using exit codes
 
@@ -36,10 +33,9 @@ By default headless chrome runner explicitly sets exit code to 1 when there was
     0
 
     $ opal -Rchrome -e "raise 'error'"; echo $?
-    RuntimeError : error
-        at $$raise (file:///tmp/chrome-opal.js:4996:6)
-        at  (file:///tmp/chrome-opal.js:21139:14)
-        at  (file:///tmp/chrome-opal.js:21140:2)
+    RuntimeError: error
+      from <internal:corelib/kerne…>:2693:7:in `<main>'
+      from -e:1:1:in `undefined'
     1
 
 You can change final exit code by using `Kernel#exit`, but make sure to require `opal/platform` in your code.
@@ -69,20 +65,12 @@ or runs the server on its own. It detects your platform and uses a default path
 (`Opal::CliRunners::Chrome#chrome_executable`) but you can override it by specifying `GOOGLE_CHROME_BINARY` environment
 variable.
 
-When the server is up and running it passes compiled js code to `node lib/opal/cli_runners/chrome.js`
+When the server is up and running it passes compiled js code to `lib/opal/cli_runners/chrome_cdp_interface.rb`
 as a plain input using stdin (basically, it's a second part of the runner).
-`chrome.js` is a node js script that does the main job. It runs any provided code on the running chrome server,
+`chrome_cdp_interface.rb` is a node js + Opal script that does the main job. It runs any provided code on the running chrome server,
 catches errors and forwards console messages.
 
 
-Moreover, you can actually call any js using headless chrome by running
-
-      $ echo "console.log('Hello, Opal')" | node lib/opal/cli_runners/chrome.js
-
-NOTE: to run it you need to have a chrome server running on `localhost:9222` (usually `chrome.rb` does it for you)
-
-      $ chrome --disable-gpu --headless --remote-debugging-port=9222
-
 ## Using a remote chrome server
 
 If you want to change a default chrome port or your chrome server is running on a different host:port
@@ -99,7 +87,9 @@ NOTE: `CHROME_HOST` requires a chrome server to be started. You can't start remo
 
 If you need to pass additional CLI options to the Chrome executable you can do so by setting the `CHROME_OPTS` environment variable:
 
-      $ CHROME_OPS="--window-size=412,732" opal -Rchrome -e "puts 42"
+      $ CHROME_OPTS="--window-size=412,732" opal -Rchrome -e "puts 42"
       42
 
+Docker users may need `CHROME_OPTS="--no-sandbox"` due to the user namespaces limitations.
+
 _For a list of additional options see https://developers.google.com/web/updates/2017/04/headless-chrome_

data/docs/jquery.md

@@ -1,8 +1,6 @@
 # JQuery
 
-`opal-jquery` offers a nicer ruby-like syntax for JQuery (and Zepto). It is
-useful for projects which cannot use `opal-browser` due to a reliance on jquery
-for plugins or other libraries.
+`opal-jquery` offers a nicer ruby-like syntax for JQuery. It is useful for projects which cannot use `opal-browser` due to a reliance on jquery for plugins or other libraries.
 
 ```ruby
 foos = Element.find('.foo')
@@ -91,9 +89,6 @@ bridged instances of jquery objects. Just like ruby arrays are just javascript
 arrays, `Element` instances are just jquery objects. This makes interaction
 with jquery plugins much easier.
 
-Also, `Element` will try to bridge with Zepto if it cannot find jQuery loaded,
-making it ideal for mobile applications as well.
-
 ### Interacting with the DOM
 
 #### Finding Elements

data/docs/opal_parser.md

@@ -8,12 +8,14 @@ Opal is able to compile its – pure Ruby – compiler to JavaScript (how cool i
 require 'opal-parser'
 ```
 
+_Note: For the best performance and application load times, it is strongly recommended to design your application so that it won't need the parser. A lot of methods described in this document are more fun hacks than robust solutions. But if you really want or need to use them, for example so that you can implement a Ruby REPL or an interactive Ruby playground - we have you covered, but for all other cases, we strongly discourage you to take an advice from this guide._
+
 
 ## Features
 
 ### `Kernel#eval`
 
-`opal-parser` provides a partial implementation of `Kernel#eval` that only support the first parameter (String) and refuse any passed `binding`, `lineno` or `filename`.
+`opal-parser` provides a partial implementation of `Kernel#eval`.
 
 Example:
 

data/docs/promises.md

@@ -8,6 +8,8 @@ It can be required inside any Opal applicaton:
 require 'promise'
 ```
 
+_Please also take a look at the Asynchronous code guide - we are in the process of modernizing the Promises, along with supporting async/await_
+
 ## Usage
 
 This example shows how to use a `HTTP` request from `opal-jquery` from a callback style, into a promise style handler.

data/docs/releasing.md

@@ -16,6 +16,10 @@ _This guide is a work-in-progress._
 
 - Commit the updated changelog along with the version bump using this commit message:
   "Release v1.2.3"
+- Push the commit and run `bin/rake release` to release the new version to Rubygems
+- Go to GitHub releases and create a new release from the latest tag pasting the contents of UNRELEASED.md
+- Empty UNRELEASED.md and run `bin/rake changelog`
+- Push the updated changelog
 
 ## Opal docs
 

data/docs/roda-sprockets.md

@@ -23,7 +23,6 @@ class App < Roda
    plugin :sprockets, precompile: %w(application.js),
                       prefix: %w(app/),
                       opal: true,
-                      debug: ENV['RACK_ENV'] != 'production'
    plugin :public
 
    route do |r|

data/docs/source_maps.md

@@ -1,10 +1,6 @@
 # Source maps
 
-Source maps are available (on current stable release, v0.6.x) even without explicit support from Sprockets in a sort of hackish way.
-
-_As such even if they generally work fine there are some limitations and edge case issues._
-
-NOTE: Currently on `master` branch sourcemaps are work-in-progress and probably will integrate with the upcoming Sprockets 4 that has integrated support for them.
+Source maps are available on most environments we support.
 
 #### Processor `source_map_enabled` flag
 To enable sourcemaps in the Sprockets processor you need to turn on the relative flag:
@@ -16,8 +12,7 @@ Opal::Config.source_map_enabled = true # default
 
 #### Sprockets debug mode
 
-The  sourcemaps only work with Sprockets in debug mode because they are generated just for single files.
-
+The source maps only work with Sprockets in debug mode - this is a limitation of Sprockets.
 
 ## Enable source maps
 
@@ -37,13 +32,17 @@ config.assets.debug = true
 
 ### Sinatra
 
-You can add `Opal::Server` as in the official example: [sinatra/config.ru](https://github.com/opal/opal/blob/0-6-stable/examples/sinatra/config.ru).
+You can add `Opal::Server` as in the official example: [sinatra/config.ru](https://github.com/opal/opal/blob/master/examples/sinatra/config.ru).
 
 ### Opal::Server
 
-`Opal::Server` implements sourcemaps and can be used alone or with `Rack::Cascade` in conjunction with other apps.
+`Opal::Server` (which is based on Sprockets) implements sourcemaps and can be used alone or with `Rack::Cascade` in conjunction with other apps.
+
+### Opal::SimpleServer
+
+Like `Opal::Server`, `Opal::SimpleServer` (which isn't based on Sprockets) implements sourcemaps properly.
 
-### Opal::Environment
+### Opal::Builder
 
-`Opal::Environment` is a bit lower level and doesn't support source maps by itself.
+`Opal::Builder` is a bit lower level and doesn't support source maps by itself. It requires you to call in a specific method to generate them yourself.
 

data/docs/static_applications.md

@@ -2,7 +2,7 @@
 
 `opal` makes it easy to build static opal applications. Opal uses internal load
 paths to make it easy to handle resolving requirements during building. `opal`
-forms the basis of rails support, so anything you can do there, you can do
+forms the basis of Rails support, so anything you can do there, you can do
 standalone as well.
 
 ## Overview
@@ -11,7 +11,7 @@ First, install `opal` adding them to a `Gemfile`:
 
 ```ruby
 # Gemfile
-gem "opal", "~> 0.7.0"
+gem "opal"
 ```
 
 Next, we want to add our main app code. Keep all opal code inside `app/`

data/docs/unsupported_features.md

@@ -6,6 +6,10 @@ Opal does not support some language/runtime features of ruby. These are document
 
 For performance and ease of runtime features, all strings in Opal are immutable, i.e. `#<<`, `#gsub!`, etc. do not exist. Also, symbols are just strings. There is no class, runtime or feature difference between Symbols and Strings. Their syntaxes can be used interchangeably.
 
+#### Regexp differences ####
+
+We are using JavaScript regular expressions. While we do translate a few of Ruby specific instructions like `\A` or `\z`, there are a lot of incompatibilities that you should be aware of - for example `$` and `^` don't match newlines like they do in Ruby. Support for features like named matches or lookahead/lookbehind is dependent on the JavaScript environment support for those. To support everything, we would need to [compile in the entire Ruby's regular expression engine](https://opalrb.com/blog/2021/06/26/webassembly-and-advanced-regexp-with-opal/) which is unfeasible at the current time.
+
 #### Integer / Float difference ####
 
 In Opal, both integers and floats belong to same class `Number` (using JavaScript native numbers). So `1 / 4` is `0.25` (not `0`) and `4.0 / 2` is `2` (not `2.0`).

data/lib/opal/version.rb

@@ -3,5 +3,5 @@
 module Opal
   # WHEN RELEASING:
   # Remember to update RUBY_ENGINE_VERSION in opal/corelib/constants.rb too!
-  VERSION = '1.3.1'
+  VERSION = '1.3.2'
 end

data/opal/corelib/basic_object.rb

@@ -24,7 +24,11 @@ class BasicObject
 
   def __send__(symbol, *args, &block)
     %x{
-      var func = self['$' + symbol]
+      if (!symbol.$$is_string) {
+        #{raise ::TypeError, "#{inspect} is not a symbol nor a string"}
+      }
+
+      var func = self['$' + symbol];
 
       if (func) {
         if (block !== nil) {

data/opal/corelib/constants.rb

@@ -1,8 +1,8 @@
 RUBY_PLATFORM       = 'opal'
 RUBY_ENGINE         = 'opal'
 RUBY_VERSION        = '3.0.2'
-RUBY_ENGINE_VERSION = '1.3.1'
-RUBY_RELEASE_DATE   = '2021-11-03'
+RUBY_ENGINE_VERSION = '1.3.2'
+RUBY_RELEASE_DATE   = '2021-11-10'
 RUBY_PATCHLEVEL     = 0
 RUBY_REVISION       = '0'
 RUBY_COPYRIGHT      = 'opal - Copyright (C) 2013-2021 Adam Beynon and the Opal contributors'

data/opal/corelib/io.rb

@@ -124,7 +124,7 @@ class IO
     sep = sep.to_str unless orig_sep == ''
 
     # Try to deduce length of a regexp
-    seplen = `orig_sep == '' ? 2 : sep.length`
+    seplen = orig_sep == '' ? 2 : sep.length
 
     sep = / / if sep == ' ' # WTF is this, String#split(" ") matches all whitespaces???
 

data/opal/corelib/regexp.rb

@@ -245,16 +245,16 @@ class Regexp < `RegExp`
   end
 
   def names
-    source.scan(/\(?<(\w+)>/).map(&:first).uniq
+    source.scan(/\(?<(\w+)>/, no_matchdata: true).map(&:first).uniq
   end
 
   def named_captures
-    source.scan(/\(?<(\w+)>/)        # Scan for capture groups
-          .map(&:first)              # Get the first regexp match (\w+)
-          .each_with_index           # Add index to an iterator
-          .group_by(&:first)         # Group by the capture group names
-          .transform_values do |i|   # Convert hash values
-            i.map { |j| j.last + 1 } # Drop the capture group names; increase indexes by 1
+    source.scan(/\(?<(\w+)>/, no_matchdata: true) # Scan for capture groups
+          .map(&:first)                           # Get the first regexp match (\w+)
+          .each_with_index                        # Add index to an iterator
+          .group_by(&:first)                      # Group by the capture group names
+          .transform_values do |i|                # Convert hash values
+            i.map { |j| j.last + 1 }              # Drop the capture group names; increase indexes by 1
           end
   end
 

data/opal/corelib/runtime.js

@@ -244,7 +244,7 @@
         throw e;
       }
       cref.$$autoload[name].required = true;
-      if (cref.$$const[name]) {
+      if (cref.$$const[name] != null) {
         cref.$$autoload[name].success = true;
         return cref.$$const[name];
       }
@@ -268,7 +268,7 @@
   // Get the constant in the scope of the current cref
   function const_get_name(cref, name) {
     if (cref) {
-      if (cref.$$const[name]) { return cref.$$const[name]; }
+      if (cref.$$const[name] != null) { return cref.$$const[name]; }
       if (cref.$$autoload && cref.$$autoload[name]) {
         return handle_autoload(cref, name);
       }

data/opal/corelib/string.rb

@@ -992,7 +992,7 @@ class String < `String`
     `self.replace(/[\s\u0000]*$/, '')`
   end
 
-  def scan(pattern, &block)
+  def scan(pattern, no_matchdata: false, &block)
     %x{
       var result = [],
           match_data = nil,
@@ -1017,7 +1017,7 @@ class String < `String`
         }
       }
 
-      #{$~ = `match_data`}
+      if (!no_matchdata) #{$~ = `match_data`};
 
       return (block !== nil ? self : result);
     }

data/spec/filters/bugs/basicobject.rb

@@ -1,7 +1,6 @@
 # NOTE: run bin/format-filters after changing this file
 opal_filter "BasicObject" do
   fails "BasicObject raises NoMethodError for nonexistent methods after #method_missing is removed"
-  fails "BasicObject#__send__ raises a TypeError if the method name is not a string or symbol" # NoMethodError: undefined method `' for SendSpecs
   fails "BasicObject#initialize does not accept arguments"
   fails "BasicObject#instance_eval evaluates string with given filename and linenumber"
   fails "BasicObject#instance_eval evaluates string with given filename and negative linenumber" # Expected ["RuntimeError"] to equal ["b_file", "-98"]

data/spec/opal/core/io/read_spec.rb

@@ -0,0 +1,69 @@
+require 'spec_helper'
+
+describe "IO reading methods" do
+  examples = [
+    "abc\n|def\n|ghi\n",
+    "ab|c\nd|ef\n|ghi\n",
+    "©©©\n|ŋææ\n|æπ®\n",
+    "🫃🫃🫃\n|🫃🫃🫃\n|🫃🫃🫃\n",
+    "efhasdfhasf|asdfasdfasdf|asdfasdfasdf",
+    "gsdfgsdg🫃|🫃🫃\n|🫃",
+    "a\nb\nc\nd\ne\nf",
+    "abcððefsdf🫃|s\nd",
+    "a|b|c|d|e|f|g|h|i|\n|j|k|l|\n",
+    "a b\n|c| d\n|e f",
+    "",
+    "asdfasdfasdfasdfasdfasdfasdfasdfasdfasdfasdfasdfasdfsafsdsdf\nsadfasdf"
+  ]
+  examples += examples.map { |i| i.gsub("\n", "\r\n") }
+  examples += examples.map { |i| i.gsub("\n", "\r|\n") }
+  examples = examples.uniq
+
+  prepare_io_for = proc do |example|
+    example_lines = example.split("|")
+    io = IO.new(99)
+    io.read_proc = proc do |_length|
+      example_lines.shift
+    end
+    io
+  end
+
+  describe "#readlines" do
+    examples.each do |example|
+      it "correctly splits messages for input #{example.inspect}" do
+        io = prepare_io_for.(example)
+        expected_output = example.gsub("|", "").split(/(?<=\n)/)
+        io.readlines.should == expected_output
+      end
+    end
+  end
+
+  describe "#readline" do
+    examples.each do |example|
+      it "correctly splits messages for input #{example.inspect}" do
+        io = prepare_io_for.(example)
+        expected_output = example.gsub("|", "").split(/(?<=\n)/)
+        loop do
+          expected_output.shift.should == io.readline
+        rescue EOFError
+          expected_output.should == []
+          break
+        end
+      end
+    end
+  end
+
+  describe "#gets" do
+    examples.each do |example|
+      it "correctly splits messages for input #{example.inspect}" do
+        io = prepare_io_for.(example)
+        expected_output = example.gsub("|", "").split(/(?<=\n)/)
+        loop do
+          line = io.gets
+          expected_output.shift.should == line
+          break unless line
+        end
+      end
+    end
+  end
+end

data/spec/opal/core/language_spec.rb

@@ -27,3 +27,17 @@ describe "Bridging" do
     `typeof(String.call)`.should == "function"
   end
 end
+
+describe "Constants" do
+  it "doesn't raise error when a JS falsey constant is referenced" do
+    z = Class.new {
+      C1 = 0
+      C2 = nil
+      C3 = false
+      C4 = ''
+      C5 = C3
+    }
+
+    [z::C1, z::C2, z::C3, z::C4, z::C5].should == [0, nil, false, '', false]
+  end
+end

data/spec/opal/core/string/gsub_spec.rb

@@ -23,6 +23,14 @@ describe 'String' do
       expect("test".gsub(/$/, '2')).to eq "test2"
       expect("test".gsub(/\b/, '2')).to eq "2test2"
     end
+
+    it "doesn't override $~ when it's inspected" do
+      'a:b'.gsub(/([a-z]):([a-z])/) do
+        $~.inspect
+        target, content = $1, $2
+        expect([target, content]).to eq(['a', 'b'])
+      end
+    end
   end
 
   describe '#sub' do

data/stdlib/quickjs/io.rb

@@ -1,5 +1,3 @@
-`/* global std */`
-
 %x{
   Opal.gvars.stdout.write_proc = function(s) {
     std.out.printf("%s", s);

data/stdlib/quickjs/kernel.rb

@@ -1,5 +1,3 @@
-`/* global std, scriptArgs */`
-
 ARGV = `scriptArgs`
 
 `Opal.exit = std.exit`

data/stdlib/quickjs.rb

@@ -1,2 +1,4 @@
+`/* global std, scriptArgs */`
+
 require 'quickjs/io'
 require 'quickjs/kernel'

data/tasks/linting-parse-eslint-results.js

@@ -37,3 +37,4 @@ results.forEach(
 )
 
 puts(`\nFailed with ${count} error${count === 1 ? '' : 's'}.`)
+process.exit(count || 1)

data/tasks/linting.rake

@@ -25,16 +25,6 @@ namespace :lint do
           puts "Successful."
         else
           sh 'node tasks/linting-parse-eslint-results.js'
-          # results = JSON.parse File.read(result_path), symbolize_names: true
-          # results.each do |data|
-          #   next if data[:messages].empty?
-          #
-          #   relative_path = Pathname(data[:filePath]).relative_path_from(Pathname(dir).expand_path)
-          #   puts "* #{relative_path}"
-          #   data[:messages].each do |message|
-          #     puts "  - #{relative_path}:#{message[:line]}:#{message[:column]}-#{message[:endLine]}:#{message[:endColumn]} #{message[:message]}"
-          #   end
-          # end
         end
       end
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: opal
 version: !ruby/object:Gem::Version
-  version: 1.3.1
+  version: 1.3.2
 platform: ruby
 authors:
 - Elia Schito
@@ -10,7 +10,7 @@ authors:
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2021-11-03 00:00:00.000000000 Z
+date: 2021-11-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ast
@@ -824,6 +824,7 @@ files:
 - spec/opal/core/hash/internals_spec.rb
 - spec/opal/core/hash_spec.rb
 - spec/opal/core/helpers_spec.rb
+- spec/opal/core/io/read_spec.rb
 - spec/opal/core/iterable_props_spec.rb
 - spec/opal/core/kernel/at_exit_spec.rb
 - spec/opal/core/kernel/freeze_spec.rb
@@ -1123,10 +1124,10 @@ licenses:
 metadata:
   homepage_uri: https://opalrb.com/
   bug_tracker_uri: https://github.com/opal/opal/issues
-  changelog_uri: https://github.com/opal/opal/blob/v1.3.1/CHANGELOG.md
-  readme_uri: https://github.com/opal/opal/blob/v1.3.1/README.md
-  api_documentation_uri: http://opalrb.com/docs/api/v1.3.1/index.html
-  guides_uri: http://opalrb.com/docs/guides/v1.3.1/index.html
+  changelog_uri: https://github.com/opal/opal/blob/v1.3.2/CHANGELOG.md
+  readme_uri: https://github.com/opal/opal/blob/v1.3.2/README.md
+  api_documentation_uri: http://opalrb.com/docs/api/v1.3.2/index.html
+  guides_uri: http://opalrb.com/docs/guides/v1.3.2/index.html
   chat_uri: https://gitter.im/opal/opal
   source_code_uri: https://github.com/opal/opal
 post_install_message: