@@ -353,24 +353,39 @@
  # when the response has no packfile section (e.g. the walker
  # returned an error and only a flush was sent).
  defp extract_pack(response) do
    {:ok, packets, _rest} = PktLine.decode(response)
    # Split into raw pkt-line payloads BINARY-EXACTLY. We must not use
    # PktLine.decode here: it strips one trailing LF from every data packet
    # (correct for text lines, wrong for sideband pack data), so a pack chunk
    # whose last byte is 0x0A would lose it and the reassembled pack would fail
    # its checksum — a content-dependent flake. The server frames sideband data
    # with encode_raw (no LF added), so the bytes on the wire are exact.
    frames = raw_pkt_payloads(response)
    # PktLine.decode strips the trailing LF from data packets, so the
    # "packfile" marker arrives without its newline.
    case Enum.drop_while(packets, fn
           {:data, "packfile"} -> false
           _ -> true
         end) do
    case Enum.drop_while(frames, &(String.trim_trailing(&1, "\n") != "packfile")) do
      [] ->
        <<>>
      [_packfile_marker | body] ->
        body
        |> Enum.flat_map(fn
          <<1, chunk::binary>> -> [chunk]
          {:data, <<1, chunk::binary>>} -> [chunk]
          _ -> []
        end)
        |> IO.iodata_to_binary()
    end
  end
  # Length-prefixed pkt-line split that returns payloads verbatim (no LF
  # stripping). Skips flush/delim/response-end control packets.
  defp raw_pkt_payloads(<<>>), do: []
  defp raw_pkt_payloads(<<"0000", rest::binary>>), do: raw_pkt_payloads(rest)
  defp raw_pkt_payloads(<<"0001", rest::binary>>), do: raw_pkt_payloads(rest)
  defp raw_pkt_payloads(<<"0002", rest::binary>>), do: raw_pkt_payloads(rest)
  defp raw_pkt_payloads(<<hex::binary-size(4), rest::binary>>) do
    len = String.to_integer(hex, 16)
    payload_len = len - 4
    <<payload::binary-size(payload_len), tail::binary>> = rest
    [payload | raw_pkt_payloads(tail)]
  end
end