[Document Translator] Update package metadata

common/tools/eslint-plugin-azure-sdk/src/rules/ts-package-json-name.ts

@@ -38,18 +38,20 @@ export = {
           ): void => {
             const nodeValue = node.value as Literal;
             const name = nodeValue.value as string;
+            const subScope = getSubScope(name);
 
-            if (!name.startsWith("@azure/")) {
+            // Check for a valid scope
+            if (!/^@azure(-[a-z]+)?\//.test(name)) {
               context.report({
                 node: nodeValue,
-                message: "name is not set to @azure/<service>"
+                message: "name is not set to @azure[-<subscope>]/<service>"
               });
               return;
             }
-
-            const packageDirectory = stripPath(stripFileName(fileName));
             const packageBaseName = stripPath(name);
-            if (!/^@azure\/([a-z]+-)*[a-z]+$/.test(name)) {
+            const packageDirectory = stripPath(stripFileName(fileName));
+
+            if (!/^@azure(-[a-z]+)?\/([a-z]+-)*[a-z]+$/.test(name)) {
               context.report({
                 node: nodeValue,
                 message: "service name is not in kebab-case (lowercase and separated by hyphens)"
@@ -63,14 +65,32 @@ export = {
                     "service name matches directory name, but the directory is not kebab case (lowercase and separated by hyphens)"
                 });
               }
-            } else if (name !== `@azure/${packageDirectory}`) {
+            } else if (!isValidFolder(name, packageDirectory)) {
+              const subScopeSuffix = subScope ?? "";
               context.report({
                 node: nodeValue,
-                message: `service should be named '@azure/${packageDirectory}' or should be moved to a directory called '${packageBaseName}'`
+                message: `service should be named '@azure${subScopeSuffix}/${packageDirectory}' or should be moved to a directory called '${packageBaseName}${subScopeSuffix}'`
               });
             }
           }
         } as Rule.RuleListener)
       : {};
   }
 };
+
+function isValidFolder(packageName: string, folderName: string): boolean {
+  const subScope = getSubScope(packageName);
+  if (!subScope) {
+    return RegExp(`^@azure(-[a-z]+)?\/${folderName}`).test(packageName);
+  }
+
+  // If there is a subScope, allow the  folder name to have it appended at the end i.e folder-name-subscope
+  return RegExp(`^@azure(-[a-z]+)?\/${folderName}`).test(`${packageName}${subScope}`);
+}
+
+function getSubScope(packageName: string): string | undefined {
+  // Check if there is a sub scope i.e @azure-rest
+  const [_, subScope] = packageName.match(/^@azure(-[a-z]+)?\//) ?? [];
+
+  return subScope;
+}

common/tools/eslint-plugin-azure-sdk/tests/rules/ts-package-json-name.ts

@@ -258,6 +258,11 @@ ruleTester.run("ts-package-json-name", rule, {
       code: '{"name": "@azure/service-bus"}',
       filename: "service-bus/package.json"
     },
+    {
+      // subscope
+      code: '{"name": "@azure-rest/service-bus"}',
+      filename: "service-bus-rest/package.json"
+    },
     {
       // a full example package.json (taken from https://github.com/Azure/azure-sdk-for-js/blob/master/sdk/eventhub/event-hubs/package.json with "scripts" removed for testing purposes)
       code: examplePackageGood,
@@ -295,7 +300,7 @@ ruleTester.run("ts-package-json-name", rule, {
       filename: "service-bus/package.json",
       errors: [
         {
-          message: "name is not set to @azure/<service>"
+          message: "name is not set to @azure[-<subscope>]/<service>"
         }
       ]
     },
@@ -329,13 +334,24 @@ ruleTester.run("ts-package-json-name", rule, {
         }
       ]
     },
+    {
+      // not kebab-case
+      code: '{"name": "@azure-rest/service-bus"}',
+      filename: "not-service-bus/package.json",
+      errors: [
+        {
+          message:
+            "service should be named '@azure-rest/not-service-bus' or should be moved to a directory called 'service-bus-rest'"
+        }
+      ]
+    },
     {
       // example file with name set to service-bus
       code: examplePackageBad,
       filename: "service-bus/package.json",
       errors: [
         {
-          message: "name is not set to @azure/<service>"
+          message: "name is not set to @azure[-<subscope>]/<service>"
         }
       ]
     },

sdk/core-rest/core-client/CHANGELOG.md

@@ -1,3 +1,5 @@
 # Release History
 
 ## 1.0.0-beta.1 (Unreleased)
+
+- First release of package, see README.md for details.

sdk/core-rest/core-client/package.json

@@ -9,6 +9,14 @@
     "./dist-esm/src/url.js": "./dist-esm/src/url.browser.js"
   },
   "types": "types/src/latest/core-client.d.ts",
+  "//metadata": {
+    "constantPaths": [
+      {
+        "path": "src/constants.ts",
+        "prefix": "SDK_VERSION"
+      }
+    ]
+  },
   "scripts": {
     "audit": "node ../../../common/scripts/rush-audit.js && rimraf node_modules package-lock.json && npm i --package-lock-only 2>&1 && npm audit",
     "build:browser": "npm run build:ts && cross-env ONLY_BROWSER=true rollup -c 2>&1",

sdk/core-rest/core-client/src/clientHelpers.ts

@@ -1,3 +1,6 @@
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT license.
+
 import {
   createPipelineFromOptions,
   bearerTokenAuthenticationPolicy,

sdk/core-rest/core-client/src/common.ts

@@ -1,3 +1,6 @@
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT license.
+
 import { PipelineOptions, RawHttpHeaders, PipelineRequest } from "@azure/core-rest-pipeline";
 
 /**

sdk/core-rest/core-client/src/constants.ts

@@ -0,0 +1,7 @@
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT license.
+
+/**
+ * @internal
+ */
+export const SDK_VERSION: string = "1.0.0-beta.1";

sdk/core-rest/core-client/src/getClient.ts

@@ -1,3 +1,6 @@
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT license.
+
 import { isTokenCredential, KeyCredential, TokenCredential } from "@azure/core-auth";
 import { HttpMethods, Pipeline, PipelineOptions } from "@azure/core-rest-pipeline";
 import { createDefaultPipeline } from "./clientHelpers";
@@ -19,6 +22,7 @@ export interface Client {
    * This method will be used to send request that would check the path to provide
    * strong types
    */
+  // eslint-disable-next-line @typescript-eslint/ban-types
   path: Function;
   /**
    * This method allows arbitrary paths and doesn't provide strong types
@@ -40,15 +44,15 @@ export interface Client {
 
 /**
  * Creates a client with a default pipeline
- * @param baseUrl Base endpoint for the client
- * @param options Client options
+ * @param baseUrl - Base endpoint for the client
+ * @param options - Client options
  */
 export function getClient(baseUrl: string, options?: PipelineOptions): Client;
 /**
  * Creates a client with a default pipeline
- * @param baseUrl Base endpoint for the client
- * @param credentials Credentials to authenticate the requests
- * @param options Client options
+ * @param baseUrl - Base endpoint for the client
+ * @param credentials - Credentials to authenticate the requests
+ * @param options - Client options
  */
 export function getClient(
   baseUrl: string,
@@ -57,18 +61,16 @@ export function getClient(
 ): Client;
 export function getClient(
   baseUrl: string,
-  credentialsOrPipelineOptions?: (TokenCredential | KeyCredential) | PipelineOptions,
-  options: ClientOptions = {}
+  credentialsOrPipelineOptions?: (TokenCredential | KeyCredential) | ClientOptions,
+  clientOptions: ClientOptions = {}
 ): Client {
   let credentials: TokenCredential | KeyCredential | undefined;
-  let clientOptions = options;
 
   if (credentialsOrPipelineOptions) {
     if (isCredential(credentialsOrPipelineOptions)) {
       credentials = credentialsOrPipelineOptions;
-      clientOptions = options;
     } else {
-      clientOptions = credentialsOrPipelineOptions || {};
+      clientOptions = credentialsOrPipelineOptions ?? {};
     }
   }
 
@@ -116,7 +118,7 @@ function buildSendRequest(
   pipeline: Pipeline,
   requestOptions: RequestParameters = {},
   args: string[] = []
-) {
+): Promise<HttpResponse> {
   // If the client has an api-version and the request doesn't specify one, inject the one in the client options
   if (!requestOptions.queryParameters?.["api-version"] && clientOptions.apiVersion) {
     if (!requestOptions.queryParameters) {

sdk/core-rest/core-client/src/index.ts

@@ -1,3 +1,6 @@
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT license.
+
 /**
  * Azure Rest Core Client library for JavaScript
  * @packageDocumentation

sdk/core-rest/core-client/src/keyCredentialAuthenticationPolicy.ts

@@ -1,3 +1,6 @@
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT license.
+
 import { KeyCredential } from "@azure/core-auth";
 import {
   PipelinePolicy,

sdk/core-rest/core-client/src/pathClientTypes.ts

@@ -1,3 +1,6 @@
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT license.
+
 import { RawHttpHeaders } from "@azure/core-rest-pipeline";
 
 /**
@@ -32,7 +35,7 @@ export type RequestParameters = {
 
 /**
  * Helper type used to detect parameters in a path template
- * keys surounded by {} will be considered a path parameter
+ * keys surounded by \{\} will be considered a path parameter
  */
 export type RouteParams<TRoute extends string> = TRoute extends `{${infer _Param}}/${infer Tail}`
   ? [pathParam: string, ...pathParams: RouteParams<Tail>]

sdk/core-rest/core-client/src/sendRequest.ts

@@ -1,3 +1,6 @@
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT license.
+
 import {
   createHttpHeaders,
   createPipelineRequest,
@@ -11,10 +14,10 @@ import { HttpResponse } from "./common";
 
 /**
  * Helper function to send request used by the client
- * @param method method to use to send the request
- * @param url url to send the request to
- * @param pipeline pipeline with the policies to run when sending the request
- * @param options request options
+ * @param method - method to use to send the request
+ * @param url - url to send the request to
+ * @param pipeline - pipeline with the policies to run when sending the request
+ * @param options - request options
  * @returns returns and HttpResponse
  */
 export async function sendRequest(
@@ -65,7 +68,7 @@ export async function sendRequest(
 /**
  * Function to determine the content-type of a body
  * this is used if an explicit content-type is not provided
- * @param body body in the request
+ * @param body - body in the request
  * @returns returns the content-type
  */
 function getContentType(body: any): string {

sdk/core-rest/core-client/src/urlHelpers.ts

@@ -1,12 +1,15 @@
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT license.
+
 import { RequestParameters } from "./pathClientTypes";
 import { URL } from "./url";
 
 /**
  * Builds the request url, filling in query and path parameters
- * @param baseUrl base url which can be a template url
- * @param routePath path to append to the baseUrl
- * @param pathParameters values of the path parameters
- * @param options request parameters including query parameters
+ * @param baseUrl - base url which can be a template url
+ * @param routePath - path to append to the baseUrl
+ * @param pathParameters - values of the path parameters
+ * @param options - request parameters including query parameters
  * @returns a full url with path and query parameters
  */
 export function buildRequestUrl(
@@ -22,7 +25,7 @@ export function buildRequestUrl(
   }
 
   for (const pathParam of pathParameters) {
-    path = path.replace(/{([^\/]+)}/, pathParam);
+    path = path.replace(/{([^/]+)}/, pathParam);
   }
 
   const url = new URL(`${baseUrl}/${path}`);

sdk/core-rest/core-client/test/clientHelpers.spec.ts

@@ -1,3 +1,6 @@
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT license.
+
 import { createDefaultPipeline } from "../src/clientHelpers";
 import { assert } from "chai";
 import { bearerTokenAuthenticationPolicyName } from "@azure/core-rest-pipeline";

sdk/core-rest/core-client/test/sendRequest.spec.ts

@@ -1,3 +1,6 @@
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT license.
+
 import { sendRequest } from "../src/sendRequest";
 import { assert } from "chai";
 import {

sdk/core-rest/core-client/test/urlHelpers.spec.ts

@@ -1,3 +1,6 @@
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT license.
+
 import { buildRequestUrl } from "../src/urlHelpers";
 import { assert } from "chai";
 describe("urlHelpers", () => {

sdk/documenttranslator/ai-document-translator-rest/.eslintrc.json

@@ -0,0 +1,7 @@
+{
+  "plugins": ["@azure/azure-sdk"],
+  "extends": ["plugin:@azure/azure-sdk/azure-sdk-base"],
+  "rules": {
+    "@azure/azure-sdk/ts-modules-only-named": "warn"
+  }
+}

sdk/documenttranslator/ai-document-translator-rest/CHANGELOG.md

@@ -1,3 +1,5 @@
 # Release History
 
 ## 1.0.0-beta.1 (Unreleased)
+
+- First release of package, see README.md for details.

sdk/documenttranslator/ai-document-translator-rest/package.json

@@ -33,6 +33,10 @@
       {
         "path": "src/constants.ts",
         "prefix": "SDK_VERSION"
+      },
+      {
+        "path": "swagger/README.md",
+        "prefix": "package-version"
       }
     ]
   },
@@ -44,7 +48,7 @@
     "productSlugs": [
       "azure",
       "azure-cognitive-services",
-      "azure-document-translator"
+      "azure-translator"
     ],
     "requiredResources": {
       "Azure Cognitive Services instance": "https://docs.microsoft.com/azure/cognitive-services/cognitive-services-apis-create-account"

sdk/documenttranslator/ai-document-translator-rest/samples/v1/javascript/README.md

@@ -5,7 +5,7 @@ languages:
 products:
   - azure
   - azure-cognitive-services
-  - azure-document-translator
+  - azure-translator
 urlFragment: ai-document-translator-javascript
 disableDocsMs: true
 ---

sdk/documenttranslator/ai-document-translator-rest/samples/v1/typescript/README.md

@@ -5,7 +5,7 @@ languages:
 products:
   - azure
   - azure-cognitive-services
-  - azure-document-translator
+  - azure-translator
 urlFragment: ai-document-translator-typescript
 disableDocsMs: true
 ---

sdk/documenttranslator/ai-document-translator-rest/src/constants.ts

@@ -0,0 +1,7 @@
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT license.
+
+/**
+ * @internal
+ */
+export const SDK_VERSION: string = "1.0.0-beta.1";

sdk/documenttranslator/ai-document-translator-rest/test/public/utils/recordedClient.ts

@@ -34,7 +34,7 @@ export const environmentSetup: RecorderEnvironmentSetup = {
 };
 
 export function createClient(options?: ClientOptions): DocumentTranslatorClient {
-  let credential = { key: env.DOCUMENT_TRANSLATOR_API_KEY };
+  const credential = { key: env.DOCUMENT_TRANSLATOR_API_KEY };
   return DocumentTranslator(env.ENDPOINT, credential, options);
 }