Fix crash when a single pipe had inner defs

Closes #39

CHANGELOG.md

@@ -2,6 +2,10 @@
 
 ## main
 
+### Fixes
+
+* Fix crash when single pipe had inner defs (h/t [@michallepicki](https://github.com/adobe/elixir-styler/issues/39))
+
 ## v0.7.5
 
 ### Fixes

lib/style/pipes.ex

@@ -39,8 +39,9 @@ defmodule Styler.Style.Pipes do
             {:cont, find_pipe_start(chain_zipper), ctx}
 
           {{:|>, _, [lhs, rhs]}, _} = single_pipe_zipper ->
-            lhs = Style.drop_line_meta(lhs)
-            {fun, meta, args} = Style.drop_line_meta(rhs)
+            {_, meta, _} = lhs
+            lhs = Style.set_line_meta_to_line(lhs, meta[:line])
+            {fun, meta, args} = Style.set_line_meta_to_line(rhs, meta[:line])
             function_call_zipper = Zipper.replace(single_pipe_zipper, {fun, meta, [lhs | args || []]})
             {:cont, function_call_zipper, ctx}
         end
@@ -94,9 +95,10 @@ defmodule Styler.Style.Pipes do
   #
   # block_result
   # |> ...
-  defp extract_start({block, _, _} = lhs) when block in @blocks do
-    variable = {:"#{block}_result", [], nil}
-    new_assignment = {:=, [], [variable, lhs]}
+  defp extract_start({block, meta, _} = lhs) when block in @blocks do
+    meta = [line: meta[:line]]
+    variable = {:"#{block}_result", meta, nil}
+    new_assignment = {:=, meta, [variable, lhs]}
     {variable, new_assignment}
   end
 

test/style/styles_test.exs

@@ -0,0 +1,38 @@
+# Copyright 2023 Adobe. All rights reserved.
+# This file is licensed to you under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License. You may obtain a copy
+# of the License at http://www.apache.org/licenses/LICENSE-2.0
+
+# Unless required by applicable law or agreed to in writing, software distributed under
+# the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR REPRESENTATIONS
+# OF ANY KIND, either express or implied. See the License for the specific language
+# governing permissions and limitations under the License.
+
+defmodule Styler.Style.StylesTest do
+  @moduledoc """
+  A place for tests that make sure our styles play nicely with each other
+  """
+  use Styler.StyleCase, async: true
+
+  describe "pipes + defs" do
+    test "pipes doesnt abuse meta and break defs" do
+        assert_style(
+          """
+          foo
+          |> bar(fn baz ->
+            def widget() do
+              :bop
+            end
+          end)
+          """,
+          """
+          bar(foo, fn baz ->
+            def widget do
+              :bop
+            end
+          end)
+          """
+        )
+      end
+  end
+end