Merge branch 'main' into pr/db35a96-fn-newline-metadata

Author: mkaput

.formatter.exs

@@ -1,5 +1,6 @@
 # Used by "mix format"
 [
   plugins: [Styler],
+  import_deps: [:stream_data],
   inputs: ["{mix,.formatter}.exs", "{config,lib,test}/**/*.{ex,exs}"]
 ]

.github/property_failures.exs

@@ -0,0 +1,97 @@
+#! /usr/bin/env elixir
+
+defmodule Main do
+  @moduledoc false
+  require Logger
+
+  @repo System.get_env("REPO", "elixir-tools/spitfire")
+  def run do
+    input =
+      with :eof <- IO.read(:eof) do
+        Logger.error("Stdin is empty")
+        System.halt(2)
+      end
+
+    results =
+      case JSON.decode(input) do
+        {:ok, results} ->
+          results
+
+        {:error, reason} ->
+          case reason do
+            {_, offset, byte} ->
+              Logger.error(inspect(byte))
+              Logger.error(binary_slice(input, offset, byte_size(input)))
+
+            _ ->
+              :ok
+          end
+
+          Logger.error(inspect(reason))
+          raise "Malformed JSON: " <> input
+      end
+
+    failures =
+      for failures <- get_in(results, ["tests", Access.all(), "failures"]), f <- failures do
+        [match] = Regex.run(~r/.*Clause:.*\s+Generated:.*\n+(.*)/, f["message"], capture: :all_but_first)
+
+        JSON.decode!(String.trim(match))
+      end
+
+    failures = Enum.uniq_by(failures, & &1["code"])
+
+    if failures == [] do
+      IO.write(:stderr, "No failures!")
+    end
+
+    for fail <- failures do
+      {issues, 0} =
+        gh(
+          ~w(issue list --label property-failure --json title) ++
+            ["--jq", ~s<. | map(. | select(.title == "#{fail["code"]}"))>]
+        )
+
+      case JSON.decode!(issues) do
+        [] ->
+          IO.write(:stderr, "Making new issue for #{fail["code"]}\n")
+
+          gh(
+            ~w(issue create --label property-failure) ++
+              [
+                "--title",
+                fail["code"],
+                "--body",
+                """
+                Type: #{fail["type"]}
+                Context: #{fail["context"]}
+
+                Code:
+                ```elixir
+                #{fail["code"]}
+                ```
+
+                Elixir AST:
+                ```elixir
+                #{fail["elixir"]}
+                ```
+
+                Spitfire AST:
+                ```elixir
+                #{fail["spitfire"] || "N/A"}
+                ```
+                """
+              ]
+          )
+
+        _ ->
+          IO.write(:stderr, "Issue already exists for #{fail["code"]}\n")
+      end
+    end
+  end
+
+  defp gh(args) do
+    System.cmd("gh", args ++ ~w(--repo #{@repo}))
+  end
+end
+
+Main.run()

.github/workflows/property-tests.yaml

@@ -0,0 +1,45 @@
+name: Property Tests
+on:
+  # schedule:
+  #   - cron: "0 0 * * *"
+  workflow_dispatch:
+
+permissions:
+  contents: read
+  issues: write
+
+concurrency:
+  group: property-test
+  cancel-in-progress: false
+
+jobs:
+  proptest:
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v4
+      - uses: DeterminateSystems/nix-installer-action@main
+      - uses: cachix/cachix-action@v14
+        with:
+          name: elixir-tools
+          authToken: "${{ secrets.CACHIX_AUTH_TOKEN }}"
+
+      - uses: actions/cache@v4
+        with:
+          path: |
+            deps
+            _build
+          key: ${{ runner.os }}-mix-${{ hashFiles('**/.mise.toml') }}-${{ hashFiles('**/mix.lock') }}
+          restore-keys: |
+            ${{ runner.os }}-mix-${{ hashFiles('**/.mise.toml') }}-
+
+      - name: Install Dependencies
+        run: nix develop --command bash -c "mix deps.get"
+
+      - name: Compile
+        run: nix develop --command bash -c "MIX_ENV=test mix compile"
+
+      - name: Run property tests
+        env:
+          GH_TOKEN: ${{ github.token }}
+        run: nix develop --command bash -c "MIX_ENV=test mix test.json --only property | .github/property_failures.exs"

.github/workflows/proptest.yaml

@@ -1,3 +1,3 @@
 {
-  ".": "0.3.6"
+  ".": "0.3.7"
 }

.release-please-manifest.json

@@ -1,5 +1,15 @@
 # Changelog
 
+## [0.3.7](https://github.com/elixir-tools/spitfire/compare/v0.3.6...v0.3.7) (2026-02-19)
+
+
+### Bug Fixes
+
+* allow list literals in struct type parsing ([#111](https://github.com/elixir-tools/spitfire/issues/111)) ([f2eabee](https://github.com/elixir-tools/spitfire/commit/f2eabee78f740f6109609724179fbea1a9537558))
+* parse assoc-map values outside key context ([#102](https://github.com/elixir-tools/spitfire/issues/102)) ([6751c15](https://github.com/elixir-tools/spitfire/commit/6751c15bd1a7aec63fa2d81ff81bc8a9b3905f3f))
+* parse nested struct/map targets in struct literals ([#105](https://github.com/elixir-tools/spitfire/issues/105)) ([3d1cc61](https://github.com/elixir-tools/spitfire/commit/3d1cc61120c36f2c686ab32bbfa307407991bc17))
+* support dot-call struct targets like %e.(){} ([#110](https://github.com/elixir-tools/spitfire/issues/110)) ([706f0b0](https://github.com/elixir-tools/spitfire/commit/706f0b01c1a6440be5ea3618ecedfab0f5452353))
+
 ## [0.3.6](https://github.com/elixir-tools/spitfire/compare/v0.3.5...v0.3.6) (2026-02-16)
 
 

CHANGELOG.md

@@ -13,4 +13,17 @@ if config_env() == :test do
   # END: parse_program
 
   config :spitfire, trace: false
+
+  is_ci = "CI" |> System.get_env("false") |> String.to_existing_atom()
+
+  config :spitfire, is_ci: is_ci
+
+  max_runs =
+    if is_ci do
+      500_000
+    else
+      1000
+    end
+
+  config :stream_data, max_runs: max_runs
 end

config/config.exs

@@ -17,7 +17,6 @@
             packages = with pkgs; [
               beam.packages.erlang_27.erlang
               beam.packages.erlang_27.elixir_1_18
-              act
             ];
           };
         };

flake.nix

@@ -1195,28 +1195,9 @@ defmodule Spitfire do
               nl -> [newlines: nl]
             end
 
-          # Check if we have a semicolon right after -> (possibly after eol)
-          # Check if there's a leading semicolon right after ->
-          # Handles both ";expr" and "\n;expr" patterns
-          has_leading_semicolon =
-            case peek_token(parser) do
-              :";" ->
-                true
-
-              :eol ->
-                # Peek at the next token after eol without consuming
-                # We need to manually check the token sequence
-                with {:eol, _} <- parser.current_token,
-                     # Look at the tokens list to find what comes after eol
-                     [{:";", _} | _] <- parser.tokens do
-                  true
-                else
-                  _ -> false
-                end
-
-              _ ->
-                false
-            end
+          # A semicolon immediately after `->` (with optional newlines in between)
+          # starts the clause body with an implicit nil expression.
+          has_leading_semicolon = peek_token_skip_eol(parser) == :";"
 
           parser = eat_eoe_at(parser, 1)
 
@@ -1423,8 +1404,28 @@ defmodule Spitfire do
             {rhs, parser}
         end
 
+      parser =
+        if token == :"//" and not match?({:.., _, [_, _]}, lhs) do
+          put_error(
+            pre_parser,
+            {meta,
+             "the range step operator (//) must immediately follow the range definition operator (..), for example: 1..9//2. If you wanted to define a default argument, use (\\\\) instead. Syntax error before: '//'"}
+          )
+        else
+          parser
+        end
+
       ast =
         case token do
+          :"//" ->
+            case lhs do
+              {:.., lhs_meta, [left, middle]} ->
+                {:..//, lhs_meta, [left, middle, rhs]}
+
+              _ ->
+                {token, newlines ++ meta, [lhs, rhs]}
+            end
+
           :"not in" ->
             {:not, meta, [{:in, meta, [lhs, rhs]}]}
 
@@ -2941,7 +2942,7 @@ defmodule Spitfire do
       if MapSet.member?(@terminals_with_comma, peek_token(parser)) or
            peek_token(parser) == :";" or
            peek in [:stab_op, :do, :end, :block_identifier] or
-           (is_binary_op?(peek) and peek != :dual_op) do
+           (is_binary_op?(peek) and peek not in [:dual_op, :ternary_op]) do
         {{:..., current_meta(parser), []}, parser}
       else
         meta = current_meta(parser)

lib/spitfire.ex

@@ -7,7 +7,7 @@ defmodule Spitfire.MixProject do
     [
       app: :spitfire,
       description: "Error resilient parser for Elixir",
-      version: "0.3.6",
+      version: "0.3.7",
       elixir: "~> 1.15",
       start_permanent: Mix.env() == :prod,
       deps: deps(),
@@ -35,7 +35,8 @@ defmodule Spitfire.MixProject do
       {:styler, "~> 0.11", only: [:dev, :test]},
       {:credo, "~> 1.7", only: :dev},
       {:dialyxir, "~> 1.0", only: :dev},
-      {:stream_data, "~> 1.0", only: [:dev, :test]}
+      {:stream_data, "~> 1.0", only: [:dev, :test]},
+      {:ex_unit_json, "~> 0.4.1", only: [:dev, :test]}
     ]
   end