add one common problem; code review updates

Signed-off-by: Merlin Ran <[email protected]>

README.md

@@ -203,6 +203,8 @@ Problems can arise, especially when you are new to bidbot. Here are some common
     * `--running-bytes-limit` to limit the total bytes bidbot processes in a period of time. 
     * `--sealing-sectors-limit` to pause bidding if the miner node has more than desired sectors in sealing.
 
+1. We've seen a case when bidbot importing data long after winning the bids, it got errors like "normal shutdown of state machine" or "given data does not match expected commP ..." even though the real problem was the deal start epoch being exceeded. When in doubt, check your `lotus-miner` logs around that time for the real reason.
+
 It is also suggested to set `--log-debug=true` and keep the logs when you want to seek help from Textile or the community.
 
 ## Contributing

main.go

@@ -520,7 +520,7 @@ var dealsListCmd = &cobra.Command{
 }
 
 var dealsShowCmd = &cobra.Command{
-	Use:   "show <id>",
+	Use:   "show <bid-id>",
 	Short: "Show details of one deal",
 	Long:  `Show details of one deal, specified by the bid ID, which can be obtained by 'bidbot deals list'`,
 	Args:  cobra.ExactArgs(1),
@@ -548,9 +548,9 @@ var dealsShowCmd = &cobra.Command{
 }
 
 var downloadCmd = &cobra.Command{
-	Use:   "download <cid> <uri>",
+	Use:   "download <payload-cid> <data-uri>",
 	Short: "Download and write storage deal data to disk",
-	Long: `Downloads and writes storage deal data to disk by cid and uri.
+	Long: `Downloads and writes storage deal data to disk by the payload cid and data uri.
 
 Deal data is written to BIDBOT_DEAL_DATA_DIRECTORY in CAR format.
 `,

service/lotusclient/lotusclient.go

@@ -42,7 +42,7 @@ type Client struct {
 
 // New returns a new *LotusClient.
 func New(maddr string, authToken string, marketMaddr string, marketAuthToken string,
-            connRetries int, fakeMode bool) (*Client, error) {
+	connRetries int, fakeMode bool) (*Client, error) {
 	fin := finalizer.NewFinalizer()
 	ctx, cancel := context.WithCancel(context.Background())
 	fin.Add(finalizer.NewContextCloser(cancel))