Type docs fix

sdk/servicebus/service-bus/review/service-bus.api.md

@@ -120,7 +120,8 @@ export interface CreateQueueOptions extends OperationOptions {
 // @public
 export interface CreateReceiverOptions<ReceiveModeT extends ReceiveMode> {
     receiveMode?: ReceiveModeT;
-    subQueue?: SubQueue;
+    // @internal
+    _subQueue?: _SubQueue;
 }
 
 // @public
@@ -478,8 +479,8 @@ export interface SqlRuleFilter {
     };
 }
 
-// @public
-export type SubQueue = "deadLetter" | "transferDeadLetter";
+// @internal
+export type _SubQueue = "deadLetter" | "transferDeadLetter";
 
 // @public
 export interface SubscribeOptions extends MessageHandlerOptions {

sdk/servicebus/service-bus/src/index.ts

@@ -28,7 +28,7 @@ export {
   ReceiveMessagesOptions,
   ReceiveMode,
   SessionSubscribeOptions,
-  SubQueue,
+  _SubQueue,
   SubscribeOptions
 } from "./models";
 export { OperationOptionsBase } from "./modelsToBeSharedWithEventHubs";

sdk/servicebus/service-bus/src/log.ts

@@ -6,6 +6,8 @@ import { createClientLogger } from "@azure/logger";
 /**
  * The @azure/logger configuration for this package.
  * This will output logs using the `azure:event-hubs` namespace prefix.
+ * @ignore
+ * @internal
  */
 export const logger = createClientLogger("service-bus");
 

sdk/servicebus/service-bus/src/models.ts

@@ -44,8 +44,10 @@ export type ReceiveMode = "peekLock" | "receiveAndDelete";
  * Represents the sub queue that is applicable for any queue or subscription.
  * Valid values are "deadLetter" and "transferDeadLetter". To learn more about dead letter queues,
  * see https://docs.microsoft.com/azure/service-bus-messaging/service-bus-dead-letter-queues
+ * @internal
+ * @ignore
  */
-export type SubQueue = "deadLetter" | "transferDeadLetter";
+export type _SubQueue = "deadLetter" | "transferDeadLetter";
 
 /**
  *
@@ -78,8 +80,10 @@ export interface CreateReceiverOptions<ReceiveModeT extends ReceiveMode> {
    * Represents the sub queue that is applicable for any queue or subscription.
    * Valid values are "deadLetter" and "transferDeadLetter". To learn more about dead letter queues,
    * see https://docs.microsoft.com/azure/service-bus-messaging/service-bus-dead-letter-queues
+   * @ignore
+   * @internal
    */
-  subQueue?: SubQueue;
+  _subQueue?: _SubQueue;
 }
 
 /**

sdk/servicebus/service-bus/src/serviceBusClient.ts

@@ -219,8 +219,8 @@ export class ServiceBusClient {
     );
 
     let entityPathWithSubQueue = entityPath;
-    if (options?.subQueue) {
-      switch (options?.subQueue) {
+    if (options?._subQueue) {
+      switch (options?._subQueue) {
         case "deadLetter":
           entityPathWithSubQueue += "/$DeadLetterQueue";
           break;
@@ -229,7 +229,7 @@ export class ServiceBusClient {
           break;
         default:
           throw new Error(
-            `Invalid subQueue '${options?.subQueue}' provided. Valid values are 'deadLetter' and 'transferDeadLetter'`
+            `Invalid subQueue '${options?._subQueue}' provided. Valid values are 'deadLetter' and 'transferDeadLetter'`
           );
       }
     }

sdk/servicebus/service-bus/src/serviceBusMessage.ts

@@ -826,6 +826,8 @@ export function fromAmqpMessage(
  * @export
  * @param {AmqpMessage} msg
  * @returns {AmqpAnnotatedMessage}
+ * @internal
+ * @ignore
  */
 export function toAmqpAnnotatedMessage(msg: AmqpMessage): AmqpAnnotatedMessage {
   const messageHeader = MessageHeader.fromAmqpMessageHeader(msg);

sdk/servicebus/service-bus/src/util/errors.ts

@@ -7,10 +7,10 @@ import { ConnectionContext } from "../connectionContext";
 import { AmqpError } from "rhea-promise";
 
 /**
- * Error message to use when EntityPath in connection string does not match the 
+ * Error message to use when EntityPath in connection string does not match the
  * queue or topic name passed to the methods in the ServiceBusClient that create
  * senders and receivers.
- * 
+ *
  * @internal
  * @ignore
  */
@@ -217,6 +217,10 @@ export function logError(err: Error | AmqpError | undefined, ...args: any[]): vo
   logErrorStackTrace(err);
 }
 
+/**
+ * @internal
+ * @ignore
+ */
 function isError(err: Error | AmqpError | undefined): err is Error {
   return err != null && (err as any).name != null;
 }

sdk/servicebus/service-bus/test/invalidParameters.spec.ts

@@ -233,7 +233,7 @@ describe("invalid parameters", () => {
       let errorCaught: string = "";
       try {
         // @ts-expect-error
-        sbClient.createReceiver("dummyQueue", { subQueue: 123 });
+        sbClient.createReceiver("dummyQueue", { _subQueue: 123 });
       } catch (error) {
         errorCaught = error.message;
       }

sdk/servicebus/service-bus/test/propsToModify.spec.ts

@@ -47,7 +47,7 @@ describe("dead lettering", () => {
       // receiveAndDelete since I don't care about further settlement after it's been dead lettered!
       serviceBusClient.createReceiver(entityNames.queue, {
         receiveMode: "receiveAndDelete",
-        subQueue: "deadLetter"
+        _subQueue: "deadLetter"
       })
     );
 

sdk/servicebus/service-bus/test/serviceBusClient.spec.ts

@@ -721,7 +721,7 @@ describe("entityPath on sender and receiver", async () => {
 
   it("Entity Path on Queue deadletter Receiver", () => {
     const dummyQueueName = "dummy";
-    const receiver = sbClient.createReceiver(dummyQueueName, { subQueue: "deadLetter" });
+    const receiver = sbClient.createReceiver(dummyQueueName, { _subQueue: "deadLetter" });
     should.equal(
       receiver.entityPath,
       `${dummyQueueName}/$DeadLetterQueue`,
@@ -744,7 +744,7 @@ describe("entityPath on sender and receiver", async () => {
     const dummyTopicName = "dummyTopicName";
     const dummySubscriptionName = "dummySubscriptionName";
     const receiver = sbClient.createReceiver(dummyTopicName, dummySubscriptionName, {
-      subQueue: "deadLetter"
+      _subQueue: "deadLetter"
     });
     should.equal(
       receiver.entityPath,

sdk/servicebus/service-bus/test/utils/testutils2.ts

@@ -425,9 +425,9 @@ export class ServiceBusTestHelpers {
   ): ServiceBusReceiver<ServiceBusReceivedMessageWithLock> {
     return this.addToCleanup(
       entityNames.queue
-        ? this._serviceBusClient.createReceiver(entityNames.queue, { subQueue: "deadLetter" })
+        ? this._serviceBusClient.createReceiver(entityNames.queue, { _subQueue: "deadLetter" })
         : this._serviceBusClient.createReceiver(entityNames.topic!, entityNames.subscription!, {
-            subQueue: "deadLetter"
+            _subQueue: "deadLetter"
           })
     );
   }
@@ -459,7 +459,7 @@ async function purgeForTestClientType(
     receiver = serviceBusClient.createReceiver(entityPaths.queue, "receiveAndDelete");
     deadLetterReceiver = serviceBusClient.createReceiver(entityPaths.queue, {
       receiveMode: "receiveAndDelete",
-      subQueue: "deadLetter"
+      _subQueue: "deadLetter"
     });
   } else if (entityPaths.topic && entityPaths.subscription) {
     receiver = serviceBusClient.createReceiver(entityPaths.topic, entityPaths.subscription, {
@@ -468,7 +468,7 @@ async function purgeForTestClientType(
     deadLetterReceiver = serviceBusClient.createReceiver(
       entityPaths.topic,
       entityPaths.subscription,
-      { receiveMode: "receiveAndDelete", subQueue: "deadLetter" }
+      { receiveMode: "receiveAndDelete", _subQueue: "deadLetter" }
     );
   } else {
     throw new Error(`Unsupported TestClientType for purge: ${testClientType}`);