[cosmos] Fix bulk error response (Azure#34294)

### Packages impacted by this PR
@azure/cosmos

### Issues associated with this PR


### Describe the problem that is addressed by this PR
This PR fixes some parsing errors in the bulk response.


### What are the possible designs available to address the problem? If
there are more than one possible design, why was the one in this PR
chosen?


### Are there test cases added in this PR? _(If not, why?)_


### Provide a list of related PRs _(if any)_


### Command used to generate this PR:**_(Applicable only to SDK release
request PRs)_

### Checklists
- [ ] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so,
create an Issue in the
[Autorest/typescript](https://github.com/Azure/autorest.typescript)
repository and link it here)_
- [ ] Added a changelog (if necessary)

---------

Co-authored-by: Aditishree . <[email protected]>

Author: aditishree1

sdk/cosmosdb/cosmos/review/cosmos.api.md

@@ -531,7 +531,7 @@ export const Constants: {
         Location: string;
         Referer: string;
         A_IM: string;
-        PREFER_RETURN_MINIMAL: string;
+        PreferReturnMinimal: string;
         Query: string;
         IsQuery: string;
         IsQueryPlan: string;
@@ -1136,25 +1136,17 @@ export interface ErrorBody {
     message: string;
 }
 
-// @public (undocumented)
+// @public
 export class ErrorResponse extends Error {
-    // (undocumented)
     [key: string]: any;
-    // (undocumented)
     activityId?: string;
-    // (undocumented)
     body?: ErrorBody;
-    // (undocumented)
     code?: number | string;
-    // (undocumented)
     diagnostics?: CosmosDiagnostics;
-    // (undocumented)
     headers?: CosmosHeaders;
-    // (undocumented)
+    requestCharge?: number;
     retryAfterInMilliseconds?: number;
-    // (undocumented)
     retryAfterInMs?: number;
-    // (undocumented)
     substatus?: number;
 }
 

sdk/cosmosdb/cosmos/src/bulk/BulkResponse.ts

@@ -116,10 +116,12 @@ export class BulkResponse {
           const error: ErrorResponse = new ErrorResponse();
           error.code = itemResponse?.statusCode;
           error.substatus = itemResponse?.subStatusCode;
+          error.message = itemResponse?.message;
+          error.requestCharge = itemResponse?.requestCharge;
           error.body = itemResponse?.resourceBody;
           error.headers = responseMessage.headers;
           error.activityId = responseMessage.headers?.[Constants.HttpHeaders.ActivityId];
-          error.retryAfterInMs = itemResponse?.retryAfter;
+          error.retryAfterInMs = itemResponse?.retryAfterMilliseconds;
           error.diagnostics = responseMessage.diagnostics;
           results.push(error);
         }
@@ -167,6 +169,7 @@ export class BulkResponse {
       errorResponse.body = error.body;
       errorResponse.diagnostics = error.diagnostics;
       errorResponse.headers = this.headers;
+      errorResponse.requestCharge = error.requestCharge;
       return errorResponse;
     });
   }

sdk/cosmosdb/cosmos/src/bulk/Limiter.ts

@@ -148,6 +148,9 @@ export class LimiterQueue {
     this.terminated = true;
     this.terminatedValue = customValue;
     const operationsList: ItemOperation[] = [];
+    if (this.tasks.isEmpty()) {
+      return;
+    }
     while (!this.tasks.isEmpty()) {
       const queueItem = this.tasks.shift();
       if (!queueItem) break;

sdk/cosmosdb/cosmos/src/common/constants.ts

@@ -61,7 +61,7 @@ export const Constants = {
     Location: "Location",
     Referer: "referer",
     A_IM: "A-IM",
-    PREFER_RETURN_MINIMAL: "return=minimal",
+    PreferReturnMinimal: "return=minimal",
 
     // Query
     Query: "x-ms-documentdb-query",

sdk/cosmosdb/cosmos/src/request/ErrorResponse.ts

@@ -94,14 +94,29 @@ export interface GroupByAliasToAggregateType {
   [key: string]: AggregateType;
 }
 
+/**
+ * Represents an error response returned in operations.
+ */
 export class ErrorResponse extends Error {
+  /** status or error code returned */
   code?: number | string;
+  /** substatus code returned */
   substatus?: number;
+  /** body of the error response, typically including error details */
   body?: ErrorBody;
+  /** HTTP headers */
   headers?: CosmosHeaders;
+  /** unique identifier for the operation's activity */
   activityId?: string;
+  /** delay (in milliseconds) before retrying the operation. */
   retryAfterInMs?: number;
+  /** delay (in milliseconds) before retrying the operation. */
+  /** Note: Use retryAfterInMs instead */
   retryAfterInMilliseconds?: number;
+  /** any additional property */
   [key: string]: any;
+  /** Detailed diagnostic information associated with the error.*/
   diagnostics?: CosmosDiagnostics;
+  /** The request charge of the operation, representing the resource cost incurred.*/
+  requestCharge?: number;
 }

sdk/cosmosdb/cosmos/src/request/request.ts

@@ -260,7 +260,7 @@ export async function getHeaders({
     !options.contentResponseOnWriteEnabled
   ) {
     if (operationType === OperationType.Batch) {
-      headers[Constants.HttpHeaders.Prefer] = Constants.HttpHeaders.PREFER_RETURN_MINIMAL;
+      headers[Constants.HttpHeaders.Prefer] = Constants.HttpHeaders.PreferReturnMinimal;
     } else {
       throw new ErrorResponse(
         "Currently `contentResponseOnWriteEnabled` option is only supported for batch and bulk operations.",