@azure/arm-appinsights 5.0.0-beta.1 → 5.0.0-beta.4

package/package.json

@@ -3,43 +3,57 @@
   "sdk-type": "mgmt",
   "author": "Microsoft Corporation",
   "description": "A generated SDK for ApplicationInsightsManagementClient.",
-  "version": "5.0.0-beta.1",
-  "engines": { "node": ">=12.0.0" },
+  "version": "5.0.0-beta.4",
+  "engines": {
+    "node": ">=12.0.0"
+  },
   "dependencies": {
     "@azure/core-paging": "^1.2.0",
-    "@azure/core-client": "^1.0.0",
+    "@azure/core-client": "^1.5.0",
     "@azure/core-auth": "^1.3.0",
-    "@azure/core-rest-pipeline": "^1.1.0",
+    "@azure/core-rest-pipeline": "^1.8.0",
     "tslib": "^2.2.0"
   },
-  "keywords": ["node", "azure", "typescript", "browser", "isomorphic"],
+  "keywords": [
+    "node",
+    "azure",
+    "typescript",
+    "browser",
+    "isomorphic"
+  ],
   "license": "MIT",
   "main": "./dist/index.js",
   "module": "./dist-esm/src/index.js",
   "types": "./types/arm-appinsights.d.ts",
   "devDependencies": {
-    "@microsoft/api-extractor": "^7.18.11",
-    "@rollup/plugin-commonjs": "11.0.2",
-    "@rollup/plugin-json": "^4.0.0",
-    "@rollup/plugin-multi-entry": "^3.0.0",
-    "@rollup/plugin-node-resolve": "^8.0.0",
+    "@microsoft/api-extractor": "7.18.11",
+    "@rollup/plugin-commonjs": "^21.0.1",
+    "@rollup/plugin-json": "^4.1.0",
+    "@rollup/plugin-multi-entry": "^4.1.0",
+    "@rollup/plugin-node-resolve": "^13.1.3",
     "mkdirp": "^1.0.4",
-    "rollup": "^1.16.3",
-    "rollup-plugin-sourcemaps": "^0.4.2",
+    "rollup": "^2.66.1",
+    "rollup-plugin-sourcemaps": "^0.6.3",
     "typescript": "~4.2.0",
     "uglify-js": "^3.4.9",
     "rimraf": "^3.0.0",
     "@azure/identity": "^2.0.1",
-    "@azure-tools/test-recorder": "^1.0.0",
+    "@azure-tools/test-recorder": "^2.0.0",
+    "@azure-tools/test-credential": "^1.0.0",
     "mocha": "^7.1.1",
-    "cross-env": "^7.0.2"
+    "@types/chai": "^4.2.8",
+    "chai": "^4.2.0",
+    "cross-env": "^7.0.2",
+    "@azure/dev-tool": "^1.0.0"
   },
   "homepage": "https://github.com/Azure/azure-sdk-for-js/tree/main/sdk/applicationinsights/arm-appinsights",
   "repository": {
     "type": "git",
     "url": "https://github.com/Azure/azure-sdk-for-js.git"
   },
-  "bugs": { "url": "https://github.com/Azure/azure-sdk-for-js/issues" },
+  "bugs": {
+    "url": "https://github.com/Azure/azure-sdk-for-js/issues"
+  },
   "files": [
     "dist/**/*.js",
     "dist/**/*.js.map",
@@ -81,7 +95,7 @@
     "unit-test:node": "cross-env TEST_MODE=playback npm run integration-test:node",
     "unit-test:browser": "echo skipped",
     "integration-test": "npm run integration-test:node && npm run integration-test:browser",
-    "integration-test:node": "mocha -r esm --require ts-node/register --timeout 1200000 --full-trace test/*.ts --reporter ../../../common/tools/mocha-multi-reporter.js",
+    "integration-test:node": "dev-tool run test:node-ts-input -- --timeout 1200000 'test/*.ts'",
     "integration-test:browser": "echo skipped",
     "docs": "echo skipped"
   },
@@ -89,10 +103,10 @@
   "//metadata": {
     "constantPaths": [
       {
-        "path": "src/ApplicationInsightsManagementClient.ts",
+        "path": "src/applicationInsightsManagementClient.ts",
         "prefix": "packageDetails"
       }
     ]
   },
   "autoPublish": true
-}
+}

package/review/arm-appinsights.api.md

@@ -150,8 +150,10 @@ export type APIKeysListResponse = ApplicationInsightsComponentAPIKeyListResult;
 // @public
 export type ApplicationInsightsComponent = ComponentsResource & {
     kind: string;
+    etag?: string;
     readonly applicationId?: string;
     readonly appId?: string;
+    readonly namePropertiesName?: string;
     applicationType?: ApplicationType;
     flowType?: FlowType;
     requestSource?: RequestSource;
@@ -166,10 +168,14 @@ export type ApplicationInsightsComponent = ComponentsResource & {
     retentionInDays?: number;
     disableIpMasking?: boolean;
     immediatePurgeDataOn30Days?: boolean;
+    workspaceResourceId?: string;
+    readonly laMigrationDate?: Date;
     readonly privateLinkScopedResources?: PrivateLinkScopedResource[];
     publicNetworkAccessForIngestion?: PublicNetworkAccessType;
     publicNetworkAccessForQuery?: PublicNetworkAccessType;
     ingestionMode?: IngestionMode;
+    disableLocalAuth?: boolean;
+    forceCustomerStorageForProfiler?: boolean;
 };
 
 // @public
@@ -393,8 +399,6 @@ export class ApplicationInsightsManagementClient extends coreClient.ServiceClien
     // (undocumented)
     myWorkbooks: MyWorkbooks;
     // (undocumented)
-    operations: Operations;
-    // (undocumented)
     proactiveDetectionConfigurations: ProactiveDetectionConfigurations;
     // (undocumented)
     subscriptionId: string;
@@ -656,6 +660,17 @@ export interface ErrorResponse {
     message?: string;
 }
 
+// @public (undocumented)
+export interface ErrorResponseComponents {
+    error?: ErrorResponseComponentsError;
+}
+
+// @public
+export interface ErrorResponseComponentsError {
+    readonly code?: string;
+    readonly message?: string;
+}
+
 // @public (undocumented)
 export interface ErrorResponseLinkedStorage {
     error?: ErrorResponseLinkedStorageError;
@@ -953,17 +968,21 @@ export enum KnownRequestSource {
 }
 
 // @public
-export enum KnownSharedTypeKind {
+export enum KnownStorageType {
     // (undocumented)
-    Shared = "shared",
+    ServiceProfiler = "ServiceProfiler"
+}
+
+// @public
+export enum KnownWorkbookSharedTypeKind {
     // (undocumented)
-    User = "user"
+    Shared = "shared"
 }
 
 // @public
-export enum KnownStorageType {
+export enum KnownWorkbookUpdateSharedTypeKind {
     // (undocumented)
-    ServiceProfiler = "ServiceProfiler"
+    Shared = "shared"
 }
 
 // @public
@@ -1158,30 +1177,12 @@ export interface OperationListResult {
 // @public
 export interface OperationLive {
     display?: OperationInfo;
+    isDataAction?: boolean;
     name?: string;
     origin?: string;
     properties?: Record<string, unknown>;
 }
 
-// @public
-export interface Operations {
-    list(options?: OperationsListOptionalParams): PagedAsyncIterableIterator<OperationLive>;
-}
-
-// @public
-export interface OperationsListNextOptionalParams extends coreClient.OperationOptions {
-}
-
-// @public
-export type OperationsListNextResponse = OperationsListResult;
-
-// @public
-export interface OperationsListOptionalParams extends coreClient.OperationOptions {
-}
-
-// @public
-export type OperationsListResponse = OperationsListResult;
-
 // @public
 export interface OperationsListResult {
     nextLink?: string;
@@ -1223,7 +1224,7 @@ export interface ProactiveDetectionConfigurationsUpdateOptionalParams extends co
 export type ProactiveDetectionConfigurationsUpdateResponse = ApplicationInsightsComponentProactiveDetectionConfiguration;
 
 // @public
-export type ProxyResource = Resource & {};
+export type ProxyResource = Resource;
 
 // @public
 export type PublicNetworkAccessType = string;
@@ -1241,9 +1242,6 @@ export interface Resource {
     readonly type?: string;
 }
 
-// @public
-export type SharedTypeKind = string;
-
 // @public
 export type StorageType = string;
 
@@ -1450,12 +1448,12 @@ export interface WorkbookInnerErrorTrace {
 // @public
 export type WorkbookResource = TrackedResource & {
     identity?: WorkbookResourceIdentity;
-    kind?: Kind;
+    kind?: WorkbookSharedTypeKind;
     etag?: string;
 };
 
 // @public
-export type WorkbookResourceIdentity = ManagedServiceIdentity & {};
+export type WorkbookResourceIdentity = ManagedServiceIdentity;
 
 // @public
 export interface Workbooks {
@@ -1483,11 +1481,15 @@ export interface WorkbooksDeleteOptionalParams extends coreClient.OperationOptio
 
 // @public
 export interface WorkbooksGetOptionalParams extends coreClient.OperationOptions {
+    canFetchContent?: boolean;
 }
 
 // @public
 export type WorkbooksGetResponse = Workbook;
 
+// @public
+export type WorkbookSharedTypeKind = string;
+
 // @public
 export interface WorkbooksListByResourceGroupNextOptionalParams extends coreClient.OperationOptions {
     canFetchContent?: boolean;
@@ -1685,7 +1687,7 @@ export interface WorkbookUpdateParameters {
     category?: string;
     description?: string;
     displayName?: string;
-    kind?: SharedTypeKind;
+    kind?: WorkbookUpdateSharedTypeKind;
     revision?: string;
     serializedData?: string;
     tags?: {
@@ -1694,6 +1696,9 @@ export interface WorkbookUpdateParameters {
     tagsPropertiesTags?: string[];
 }
 
+// @public
+export type WorkbookUpdateSharedTypeKind = string;
+
 // @public
 export interface WorkItemConfiguration {
     configDisplayName?: string;

package/rollup.config.js

@@ -14,62 +14,14 @@ import json from "@rollup/plugin-json";
 
 import nodeBuiltins from "builtin-modules";
 
-/**
- * Gets the proper configuration needed for rollup's commonJS plugin for @opentelemetry/api.
- *
- * NOTE: this manual configuration is only needed because OpenTelemetry uses an
- * __exportStar downleveled helper function to declare its exports which confuses
- * rollup's automatic discovery mechanism.
- *
- * @returns an object reference that can be `...`'d into your cjs() configuration.
- */
-export function openTelemetryCommonJs() {
-  const namedExports = {};
-
-  for (const key of [
-    "@opentelemetry/api",
-    "@azure/core-tracing/node_modules/@opentelemetry/api"
-  ]) {
-    namedExports[key] = [
-      "SpanKind",
-      "TraceFlags",
-      "getSpan",
-      "setSpan",
-      "SpanStatusCode",
-      "getSpanContext",
-      "setSpanContext"
-    ];
-  }
-
-  const releasedOpenTelemetryVersions = ["0.10.2", "1.0.0-rc.0"];
-
-  for (const version of releasedOpenTelemetryVersions) {
-    namedExports[
-      // working around a limitation in the rollup common.js plugin - it's not able to resolve these modules so the named exports listed above will not get applied. We have to drill down to the actual path.
-      `../../../common/temp/node_modules/.pnpm/@opentelemetry/api@${version}/node_modules/@opentelemetry/api/build/src/index.js`
-    ] = [
-      "SpanKind",
-      "TraceFlags",
-      "getSpan",
-      "setSpan",
-      "StatusCode",
-      "CanonicalCode",
-      "getSpanContext",
-      "setSpanContext"
-    ];
-  }
-
-  return namedExports;
-}
-
 // #region Warning Handler
 
 /**
- * A function that can determine whether a rollupwarning should be ignored. If
+ * A function that can determine whether a rollup warning should be ignored. If
  * the function returns `true`, then the warning will not be displayed.
  */
 
-function ignoreNiseSinonEvalWarnings(warning) {
+function ignoreNiseSinonEval(warning) {
   return (
     warning.code === "EVAL" &&
     warning.id &&
@@ -78,17 +30,14 @@ function ignoreNiseSinonEvalWarnings(warning) {
   );
 }
 
-function ignoreChaiCircularDependencyWarnings(warning) {
+function ignoreChaiCircularDependency(warning) {
   return (
     warning.code === "CIRCULAR_DEPENDENCY" &&
     warning.importer && warning.importer.includes("node_modules/chai") === true
   );
 }
 
-const warningInhibitors = [
-  ignoreChaiCircularDependencyWarnings,
-  ignoreNiseSinonEvalWarnings
-];
+const warningInhibitors = [ignoreChaiCircularDependency, ignoreNiseSinonEval];
 
 /**
  * Construct a warning handler for the shared rollup configuration
@@ -122,22 +71,7 @@ function makeBrowserTestConfig() {
       nodeResolve({
         mainFields: ["module", "browser"]
       }),
-      cjs({
-        namedExports: {
-          // Chai's strange internal architecture makes it impossible to statically
-          // analyze its exports.
-          chai: [
-            "version",
-            "use",
-            "util",
-            "config",
-            "expect",
-            "should",
-            "assert"
-          ],
-          ...openTelemetryCommonJs()
-        }
-      }),
+      cjs(),
       json(),
       sourcemaps()
       //viz({ filename: "dist-test/browser-stats.html", sourcemap: true })
@@ -173,7 +107,7 @@ export function makeConfig(pkg, options) {
     ],
     output: { file: "dist/index.js", format: "cjs", sourcemap: true },
     preserveSymlinks: false,
-    plugins: [sourcemaps(), nodeResolve(), cjs()]
+    plugins: [sourcemaps(), nodeResolve()]
   };
 
   const config = [baseConfig];

package/src/applicationInsightsManagementClient.ts

@@ -7,6 +7,7 @@
  */
 
 import * as coreClient from "@azure/core-client";
+import * as coreRestPipeline from "@azure/core-rest-pipeline";
 import * as coreAuth from "@azure/core-auth";
 import {
   AnnotationsImpl,
@@ -27,7 +28,6 @@ import {
   WorkbooksImpl,
   ComponentsImpl,
   ComponentLinkedStorageAccountsOperationsImpl,
-  OperationsImpl,
   LiveTokenImpl
 } from "./operations";
 import {
@@ -49,7 +49,6 @@ import {
   Workbooks,
   Components,
   ComponentLinkedStorageAccountsOperations,
-  Operations,
   LiveToken
 } from "./operationsInterfaces";
 import { ApplicationInsightsManagementClientOptionalParams } from "./models";
@@ -85,7 +84,7 @@ export class ApplicationInsightsManagementClient extends coreClient.ServiceClien
       credential: credentials
     };
 
-    const packageDetails = `azsdk-js-arm-appinsights/5.0.0-beta.1`;
+    const packageDetails = `azsdk-js-arm-appinsights/5.0.0-beta.4`;
     const userAgentPrefix =
       options.userAgentOptions && options.userAgentOptions.userAgentPrefix
         ? `${options.userAgentOptions.userAgentPrefix} ${packageDetails}`
@@ -100,9 +99,33 @@ export class ApplicationInsightsManagementClient extends coreClient.ServiceClien
       userAgentOptions: {
         userAgentPrefix
       },
-      baseUri: options.endpoint || "https://management.azure.com"
+      baseUri:
+        options.endpoint ?? options.baseUri ?? "https://management.azure.com"
     };
     super(optionsWithDefaults);
+
+    if (options?.pipeline && options.pipeline.getOrderedPolicies().length > 0) {
+      const pipelinePolicies: coreRestPipeline.PipelinePolicy[] = options.pipeline.getOrderedPolicies();
+      const bearerTokenAuthenticationPolicyFound = pipelinePolicies.some(
+        (pipelinePolicy) =>
+          pipelinePolicy.name ===
+          coreRestPipeline.bearerTokenAuthenticationPolicyName
+      );
+      if (!bearerTokenAuthenticationPolicyFound) {
+        this.pipeline.removePolicy({
+          name: coreRestPipeline.bearerTokenAuthenticationPolicyName
+        });
+        this.pipeline.addPolicy(
+          coreRestPipeline.bearerTokenAuthenticationPolicy({
+            scopes: `${optionsWithDefaults.baseUri}/.default`,
+            challengeCallbacks: {
+              authorizeRequestOnChallenge:
+                coreClient.authorizeRequestOnClaimChallenge
+            }
+          })
+        );
+      }
+    }
     // Parameter assignments
     this.subscriptionId = subscriptionId;
 
@@ -134,7 +157,6 @@ export class ApplicationInsightsManagementClient extends coreClient.ServiceClien
     this.componentLinkedStorageAccountsOperations = new ComponentLinkedStorageAccountsOperationsImpl(
       this
     );
-    this.operations = new OperationsImpl(this);
     this.liveToken = new LiveTokenImpl(this);
   }
 
@@ -156,6 +178,5 @@ export class ApplicationInsightsManagementClient extends coreClient.ServiceClien
   workbooks: Workbooks;
   components: Components;
   componentLinkedStorageAccountsOperations: ComponentLinkedStorageAccountsOperations;
-  operations: Operations;
   liveToken: LiveToken;
 }