fix(189pc): address review comments on torrent sidecar handling

- Only append SHA-1 piece hash when n > 0, preventing spurious piece
  for empty files (BitTorrent v1 requires 0 pieces for 0-byte files)
- Explicitly guard torrent generation with size > 0
- Log findFileByName errors in torrentFollowCopy/Move/Rename instead of
  silently discarding them
- Log WaitBatchTask failure for follow-move operation
- Rename Cloud189File.ParentId to ParentID to match Go initialism
  conventions (consistent with Cloud189Folder.ParentID, Cloud189File.ID)

Author: Copilot

drivers/189pc/driver.go

@@ -267,7 +267,7 @@ func (y *Cloud189PC) Move(ctx context.Context, srcObj, dstDir model.Obj) (model.
 	if !srcObj.IsDir() {
 		var srcFolderId string
 		if f, ok := srcObj.(*Cloud189File); ok {
-			srcFolderId = f.ParentId
+			srcFolderId = f.ParentID
 		}
 		y.torrentFollowMove(srcFolderId, srcObj.GetName(), dstDir)
 	}
@@ -311,7 +311,7 @@ func (y *Cloud189PC) Rename(ctx context.Context, srcObj model.Obj, newName strin
 
 	// 跟随重命名 torrent 文件
 	if f, ok := srcObj.(*Cloud189File); ok {
-		y.torrentFollowRename(f.ParentId, srcObj.GetName(), newName)
+		y.torrentFollowRename(f.ParentID, srcObj.GetName(), newName)
 	}
 
 	switch f := srcObj.(type) {
@@ -343,7 +343,7 @@ func (y *Cloud189PC) Copy(ctx context.Context, srcObj, dstDir model.Obj) error {
 	if !srcObj.IsDir() {
 		var srcFolderId string
 		if f, ok := srcObj.(*Cloud189File); ok {
-			srcFolderId = f.ParentId
+			srcFolderId = f.ParentID
 		}
 		y.torrentFollowCopy(srcFolderId, srcObj.GetName(), dstDir)
 	}

drivers/189pc/torrent.go

@@ -345,7 +345,7 @@ func (y *Cloud189PC) torrentFollowCopy(srcFolderId string, srcFileName string, d
 	go func() {
 		torrentFile, err := y.findFileByName(context.Background(), torrentName, srcFolderId, isFamily)
 		if err != nil {
-			// torrent 文件不存在，忽略
+			utils.Log.Debugf("查找 torrent 文件失败（跟随复制）: %v", err)
 			return
 		}
 		// 复制 torrent 文件到目标目录
@@ -383,7 +383,7 @@ func (y *Cloud189PC) torrentFollowMove(srcFolderId string, srcFileName string, d
 	go func() {
 		torrentFile, err := y.findFileByName(context.Background(), torrentName, srcFolderId, isFamily)
 		if err != nil {
-			// torrent 文件不存在，忽略
+			utils.Log.Debugf("查找 torrent 文件失败（跟随移动）: %v", err)
 			return
 		}
 		// 移动 torrent 文件到目标目录
@@ -398,7 +398,9 @@ func (y *Cloud189PC) torrentFollowMove(srcFolderId string, srcFileName string, d
 			utils.Log.Warnf("跟随移动 torrent 文件失败: %v", moveErr)
 			return
 		}
-		_ = y.WaitBatchTask("MOVE", resp.TaskID, time.Millisecond*400)
+		if err = y.WaitBatchTask("MOVE", resp.TaskID, time.Millisecond*400); err != nil {
+			utils.Log.Warnf("等待跟随移动 torrent 文件失败: %v", err)
+		}
 	}()
 }
 
@@ -420,7 +422,7 @@ func (y *Cloud189PC) torrentFollowRename(folderId string, oldFileName string, ne
 	go func() {
 		torrentFile, err := y.findFileByName(context.Background(), oldTorrentName, folderId, isFamily)
 		if err != nil {
-			// torrent 文件不存在，忽略
+			utils.Log.Debugf("查找 torrent 文件失败（跟随重命名）: %v", err)
 			return
 		}
 

drivers/189pc/types.go

@@ -170,7 +170,7 @@ type Cloud189File struct {
 	Name     string `json:"name"`
 	Size     int64  `json:"size"`
 	Md5      string `json:"md5"`
-	ParentId string `json:"-"` // 由 getFiles 设置，不从 JSON 解析
+	ParentID string `json:"-"` // 由 getFiles 设置，不从 JSON 解析
 
 	LastOpTime Time `json:"lastOpTime"`
 	CreateDate Time `json:"createDate"`

drivers/189pc/utils.go

@@ -210,7 +210,7 @@ func (y *Cloud189PC) getFiles(ctx context.Context, fileId string, isFamily bool)
 			res = append(res, &resp.FileListAO.FolderList[i])
 		}
 		for i := 0; i < len(resp.FileListAO.FileList); i++ {
-			resp.FileListAO.FileList[i].ParentId = fileId
+			resp.FileListAO.FileList[i].ParentID = fileId
 			res = append(res, &resp.FileListAO.FileList[i])
 		}
 	}
@@ -986,8 +986,8 @@ func (y *Cloud189PC) fastUpload(ctx context.Context, dstDir model.Obj, file mode
 		partInfos = append(partInfos, fmt.Sprint(i, "-", base64.StdEncoding.EncodeToString(md5Byte)))
 		sliceMd5.Reset()
 
-		// 收集 SHA-1 piece hash
-		if generateTorrent && sha1Writer != nil {
+		// 收集 SHA-1 piece hash（仅在本次分片实际写入了数据时追加）
+		if generateTorrent && n > 0 {
 			pieceSHA1Hashes = append(pieceSHA1Hashes, sha1Writer.Sum(nil)...)
 		}
 	}
@@ -1107,7 +1107,7 @@ func (y *Cloud189PC) fastUpload(ctx context.Context, dstDir model.Obj, file mode
 	}
 
 	// 生成 torrent 文件（异步，不影响上传结果）
-	if generateTorrent && len(pieceSHA1Hashes) > 0 {
+	if generateTorrent && size > 0 && len(pieceSHA1Hashes) > 0 {
 		capturedDstDir := dstDir
 		capturedIsFamily := isFamily
 		capturedFileName := file.GetName()

go.sum

@@ -27,8 +27,6 @@ github.com/Masterminds/semver/v3 v3.2.0 h1:3MEsd0SM6jqZojhjLWWeBY+Kcjy9i6MQAeY7Y
 github.com/Masterminds/semver/v3 v3.2.0/go.mod h1:qvl/7zhW3nngYb5+80sSMF+FG2BjYrf8m9wsX0PNOMQ=
 github.com/Max-Sum/base32768 v0.0.0-20230304063302-18e6ce5945fd h1:nzE1YQBdx1bq9IlZinHa+HVffy+NmVRoKr+wHN8fpLE=
 github.com/Max-Sum/base32768 v0.0.0-20230304063302-18e6ce5945fd/go.mod h1:C8yoIfvESpM3GD07OCHU7fqI7lhwyZ2Td1rbNbTAhnc=
-github.com/OpenListTeam/115-sdk-go v0.2.3 h1:nDNz0GxgliW+nT2Ds486k/rp/GgJj7Ngznc98ZBUwZo=
-github.com/OpenListTeam/115-sdk-go v0.2.3/go.mod h1:cfvitk2lwe6036iNi2h+iNxwxWDifKZsSvNtrur5BqU=
 github.com/OpenListTeam/115-sdk-go v0.2.4 h1:dVoEz3Pm996n/ZCuRckQvz36w938uNHjPrS7E/SVpBM=
 github.com/OpenListTeam/115-sdk-go v0.2.4/go.mod h1:cfvitk2lwe6036iNi2h+iNxwxWDifKZsSvNtrur5BqU=
 github.com/OpenListTeam/go-cache v0.1.0 h1:eV2+FCP+rt+E4OCJqLUW7wGccWZNJMV0NNkh+uChbAI=