Skip to content

Commit 12844cb

Browse files
author
SDKAuto
committed
CodeGen from PR 18835 in Azure/azure-rest-api-specs
Merge 9df937b084e3db1f0d7bb6639f65948b6aa94798 into 50ed15bd61ac79f2368d769df0c207a00b9e099f
1 parent 045859c commit 12844cb

File tree

8 files changed

+109
-102
lines changed

8 files changed

+109
-102
lines changed

‎common/config/rush/pnpm-lock.yaml

Lines changed: 16 additions & 17 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎sdk/mixedreality/arm-mixedreality/CHANGELOG.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# Release History
22

3-
## 4.1.0-beta.1 (2022-04-25)
3+
## 4.1.0-beta.1 (2022-05-04)
44

55
**Features**
66

‎sdk/mixedreality/arm-mixedreality/README.md

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -16,8 +16,6 @@ Mixed Reality Client
1616
- [LTS versions of Node.js](https://nodejs.org/about/releases/)
1717
- Latest versions of Safari, Chrome, Edge and Firefox.
1818

19-
See our [support policy](https://github.com/Azure/azure-sdk-for-js/blob/main/SUPPORT.md) for more details.
20-
2119
### Prerequisites
2220

2321
- An [Azure subscription][azure_sub].
@@ -51,17 +49,8 @@ For more information about how to create an Azure AD Application check out [this
5149
```javascript
5250
const { MixedRealityClient } = require("@azure/arm-mixedreality");
5351
const { DefaultAzureCredential } = require("@azure/identity");
54-
// For client-side applications running in the browser, use InteractiveBrowserCredential instead of DefaultAzureCredential. See https://aka.ms/azsdk/js/identity/examples for more details.
55-
5652
const subscriptionId = "00000000-0000-0000-0000-000000000000";
5753
const client = new MixedRealityClient(new DefaultAzureCredential(), subscriptionId);
58-
59-
// For client-side applications running in the browser, use this code instead:
60-
// const credential = new InteractiveBrowserCredential({
61-
// tenantId: "<YOUR_TENANT_ID>",
62-
// clientId: "<YOUR_CLIENT_ID>"
63-
// });
64-
// const client = new MixedRealityClient(credential, subscriptionId);
6554
```
6655

6756

‎sdk/mixedreality/arm-mixedreality/_meta.json

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
{
2-
"commit": "d967edeee8fd6af6d40bffe53cceed1bd053d7ad",
3-
"readme": "specification\\mixedreality\\resource-manager\\readme.md",
4-
"autorest_command": "autorest --version=3.7.3 --typescript --modelerfour.lenient-model-deduplication --head-as-boolean=true --license-header=MICROSOFT_MIT_NO_VERSION --generate-test --typescript-sdks-folder=D:\\work\\azure-sdk-for-js ..\\azure-rest-api-specs\\specification\\mixedreality\\resource-manager\\readme.md --use=@autorest/[email protected].19.20220408.1 --generate-sample=true",
2+
"commit": "d282151a304748b661bb065a3a164917e31c6e8c",
3+
"readme": "specification/mixedreality/resource-manager/readme.md",
4+
"autorest_command": "autorest --version=3.7.3 --typescript --modelerfour.lenient-model-deduplication --head-as-boolean=true --license-header=MICROSOFT_MIT_NO_VERSION --generate-test --typescript-sdks-folder=/mnt/vss/_work/1/s/azure-sdk-for-js ../azure-rest-api-specs/specification/mixedreality/resource-manager/readme.md --use=@autorest/[email protected].16.20220105.1",
55
"repository_url": "https://github.com/Azure/azure-rest-api-specs.git",
6-
"release_tool": "@azure-tools/js-sdk-release-tools@2.2.6",
7-
"use": "@autorest/[email protected].19.20220408.1"
8-
}
6+
"release_tool": "@azure-tools/js-sdk-release-tools@2.3.0",
7+
"use": "@autorest/[email protected].16.20220105.1"
8+
}

‎sdk/mixedreality/arm-mixedreality/package.json

Lines changed: 11 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -9,9 +9,9 @@
99
},
1010
"dependencies": {
1111
"@azure/core-paging": "^1.2.0",
12-
"@azure/core-client": "^1.5.0",
12+
"@azure/core-client": "^1.0.0",
1313
"@azure/core-auth": "^1.3.0",
14-
"@azure/core-rest-pipeline": "^1.8.0",
14+
"@azure/core-rest-pipeline": "^1.1.0",
1515
"tslib": "^2.2.0"
1616
},
1717
"keywords": [
@@ -26,14 +26,14 @@
2626
"module": "./dist-esm/src/index.js",
2727
"types": "./types/arm-mixedreality.d.ts",
2828
"devDependencies": {
29-
"@microsoft/api-extractor": "7.18.11",
30-
"@rollup/plugin-commonjs": "^21.0.1",
31-
"@rollup/plugin-json": "^4.1.0",
32-
"@rollup/plugin-multi-entry": "^4.1.0",
33-
"@rollup/plugin-node-resolve": "^13.1.3",
29+
"@microsoft/api-extractor": "^7.18.11",
30+
"@rollup/plugin-commonjs": "11.0.2",
31+
"@rollup/plugin-json": "^4.0.0",
32+
"@rollup/plugin-multi-entry": "^3.0.0",
33+
"@rollup/plugin-node-resolve": "^8.0.0",
3434
"mkdirp": "^1.0.4",
35-
"rollup": "^2.66.1",
36-
"rollup-plugin-sourcemaps": "^0.6.3",
35+
"rollup": "^1.16.3",
36+
"rollup-plugin-sourcemaps": "^0.4.2",
3737
"typescript": "~4.2.0",
3838
"uglify-js": "^3.4.9",
3939
"rimraf": "^3.0.0",
@@ -96,21 +96,5 @@
9696
"docs": "echo skipped"
9797
},
9898
"sideEffects": false,
99-
"//metadata": {
100-
"constantPaths": [
101-
{
102-
"path": "src/mixedRealityClient.ts",
103-
"prefix": "packageDetails"
104-
}
105-
]
106-
},
107-
"autoPublish": true,
108-
"//sampleConfiguration": {
109-
"productName": "",
110-
"productSlugs": [
111-
"azure"
112-
],
113-
"disableDocsMs": true,
114-
"apiRefLink": "https://docs.microsoft.com/javascript/api/@azure/arm-mixedreality?view=azure-node-preview"
115-
}
116-
}
99+
"autoPublish": true
100+
}

‎sdk/mixedreality/arm-mixedreality/rollup.config.js

Lines changed: 72 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -14,14 +14,62 @@ import json from "@rollup/plugin-json";
1414

1515
import nodeBuiltins from "builtin-modules";
1616

17+
/**
18+
* Gets the proper configuration needed for rollup's commonJS plugin for @opentelemetry/api.
19+
*
20+
* NOTE: this manual configuration is only needed because OpenTelemetry uses an
21+
* __exportStar downleveled helper function to declare its exports which confuses
22+
* rollup's automatic discovery mechanism.
23+
*
24+
* @returns an object reference that can be `...`'d into your cjs() configuration.
25+
*/
26+
export function openTelemetryCommonJs() {
27+
const namedExports = {};
28+
29+
for (const key of [
30+
"@opentelemetry/api",
31+
"@azure/core-tracing/node_modules/@opentelemetry/api"
32+
]) {
33+
namedExports[key] = [
34+
"SpanKind",
35+
"TraceFlags",
36+
"getSpan",
37+
"setSpan",
38+
"SpanStatusCode",
39+
"getSpanContext",
40+
"setSpanContext"
41+
];
42+
}
43+
44+
const releasedOpenTelemetryVersions = ["0.10.2", "1.0.0-rc.0"];
45+
46+
for (const version of releasedOpenTelemetryVersions) {
47+
namedExports[
48+
// 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.
49+
`../../../common/temp/node_modules/.pnpm/@opentelemetry/api@${version}/node_modules/@opentelemetry/api/build/src/index.js`
50+
] = [
51+
"SpanKind",
52+
"TraceFlags",
53+
"getSpan",
54+
"setSpan",
55+
"StatusCode",
56+
"CanonicalCode",
57+
"getSpanContext",
58+
"setSpanContext"
59+
];
60+
}
61+
62+
return namedExports;
63+
}
64+
1765
// #region Warning Handler
1866

1967
/**
20-
* A function that can determine whether a rollup warning should be ignored. If
68+
* A function that can determine whether a rollupwarning should be ignored. If
2169
* the function returns `true`, then the warning will not be displayed.
2270
*/
2371

24-
function ignoreNiseSinonEval(warning) {
72+
function ignoreNiseSinonEvalWarnings(warning) {
2573
return (
2674
warning.code === "EVAL" &&
2775
warning.id &&
@@ -30,14 +78,17 @@ function ignoreNiseSinonEval(warning) {
3078
);
3179
}
3280

33-
function ignoreChaiCircularDependency(warning) {
81+
function ignoreChaiCircularDependencyWarnings(warning) {
3482
return (
3583
warning.code === "CIRCULAR_DEPENDENCY" &&
3684
warning.importer && warning.importer.includes("node_modules/chai") === true
3785
);
3886
}
3987

40-
const warningInhibitors = [ignoreChaiCircularDependency, ignoreNiseSinonEval];
88+
const warningInhibitors = [
89+
ignoreChaiCircularDependencyWarnings,
90+
ignoreNiseSinonEvalWarnings
91+
];
4192

4293
/**
4394
* Construct a warning handler for the shared rollup configuration
@@ -71,7 +122,22 @@ function makeBrowserTestConfig() {
71122
nodeResolve({
72123
mainFields: ["module", "browser"]
73124
}),
74-
cjs(),
125+
cjs({
126+
namedExports: {
127+
// Chai's strange internal architecture makes it impossible to statically
128+
// analyze its exports.
129+
chai: [
130+
"version",
131+
"use",
132+
"util",
133+
"config",
134+
"expect",
135+
"should",
136+
"assert"
137+
],
138+
...openTelemetryCommonJs()
139+
}
140+
}),
75141
json(),
76142
sourcemaps()
77143
//viz({ filename: "dist-test/browser-stats.html", sourcemap: true })
@@ -107,7 +173,7 @@ export function makeConfig(pkg, options) {
107173
],
108174
output: { file: "dist/index.js", format: "cjs", sourcemap: true },
109175
preserveSymlinks: false,
110-
plugins: [sourcemaps(), nodeResolve()]
176+
plugins: [sourcemaps(), nodeResolve(), cjs()]
111177
};
112178

113179
const config = [baseConfig];

‎sdk/mixedreality/arm-mixedreality/src/mixedRealityClient.ts

Lines changed: 1 addition & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@
77
*/
88

99
import * as coreClient from "@azure/core-client";
10-
import * as coreRestPipeline from "@azure/core-rest-pipeline";
1110
import * as coreAuth from "@azure/core-auth";
1211
import {
1312
OperationsImpl,
@@ -78,33 +77,9 @@ export class MixedRealityClient extends coreClient.ServiceClient {
7877
userAgentOptions: {
7978
userAgentPrefix
8079
},
81-
baseUri:
82-
options.endpoint ?? options.baseUri ?? "https://management.azure.com"
80+
baseUri: options.endpoint || "https://management.azure.com"
8381
};
8482
super(optionsWithDefaults);
85-
86-
if (options?.pipeline && options.pipeline.getOrderedPolicies().length > 0) {
87-
const pipelinePolicies: coreRestPipeline.PipelinePolicy[] = options.pipeline.getOrderedPolicies();
88-
const bearerTokenAuthenticationPolicyFound = pipelinePolicies.some(
89-
(pipelinePolicy) =>
90-
pipelinePolicy.name ===
91-
coreRestPipeline.bearerTokenAuthenticationPolicyName
92-
);
93-
if (!bearerTokenAuthenticationPolicyFound) {
94-
this.pipeline.removePolicy({
95-
name: coreRestPipeline.bearerTokenAuthenticationPolicyName
96-
});
97-
this.pipeline.addPolicy(
98-
coreRestPipeline.bearerTokenAuthenticationPolicy({
99-
scopes: `${optionsWithDefaults.baseUri}/.default`,
100-
challengeCallbacks: {
101-
authorizeRequestOnChallenge:
102-
coreClient.authorizeRequestOnClaimChallenge
103-
}
104-
})
105-
);
106-
}
107-
}
10883
// Parameter assignments
10984
this.subscriptionId = subscriptionId;
11085

‎sdk/mixedreality/arm-mixedreality/tsconfig.json

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,17 +15,11 @@
1515
],
1616
"declaration": true,
1717
"outDir": "./dist-esm",
18-
"importHelpers": true,
19-
"paths": {
20-
"@azure/arm-mixedreality": [
21-
"./src/index"
22-
]
23-
}
18+
"importHelpers": true
2419
},
2520
"include": [
2621
"./src/**/*.ts",
27-
"./test/**/*.ts",
28-
"samples-dev/**/*.ts"
22+
"./test/**/*.ts"
2923
],
3024
"exclude": [
3125
"node_modules"

0 commit comments

Comments
 (0)