Граф коммитов

2507 Коммитов

Автор SHA1 Сообщение Дата
ZiWei Chen 33099a058e
[mgmt] containerservicefleet release (#31405)
https://github.com/Azure/sdk-release-request/issues/5519
2024-10-25 07:58:29 +00:00
ZiWei Chen df87d3f655
[mgmt] dns release (#31408)
https://github.com/Azure/sdk-release-request/issues/5528
2024-10-25 07:50:06 +00:00
ZiWei Chen 4a15893c53
[mgmt] servicefabricmanagedcluster release (#31444)
https://github.com/Azure/sdk-release-request/issues/5554
2024-10-25 15:24:34 +08:00
ZiWei Chen 64df053b02
[mgmt] containerinstance release (#31459)
https://github.com/Azure/sdk-release-request/issues/5571
2024-10-25 14:15:11 +08:00
ZiWei Chen cae1265f1a
[mgmt] computefleet ga (#31478)
https://github.com/Azure/sdk-release-request/issues/5596?reload=1
2024-10-24 14:48:42 +08:00
Jeremy Meng af5c6a8006
[dev-tool] react to NodeJS `spawn` security fix (#29414)
Node made a breaking change in a security release for 18/20 where `spawn()` no longer loads `.cmd` files by default. nodejs/node#52554.

This PR sets the `shell: true` option when running vendored command on Windows.
2024-10-23 13:30:55 -07:00
angellan-msft e605b318e5
Get real-time notification configuration from Chat Gateway 2 (#31492)
### Packages impacted by this PR
@azure/communication-signaling

### Describe the problem that is addressed by this PR
This PR is used for retrieving Trouter Config from Chat GW rather than
hardcode it in signaling client.
There are 2 main changes.
1.Remove the "environment" from SignalingClientOptions as it should be
retrieved from Chat GW;
2.Declare a TrouterConfigClientOptions and pass it to Signaling Client
constructor since this is needed for signaling client to make http call
to Chat Gateway.


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

### Checklists
- [Yes] Added impacted package name to the issue description
- [No] 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)_
- [TODO] Added a changelog (if necessary)
2024-10-22 11:54:47 -07:00
Maor Leger b3bda3a2f9
[attestation] Migrate to ESM and tshy (#31433)
### Packages impacted by this PR

@azure/attestation

### Issues associated with this PR

#31338 

### Describe the problem that is addressed by this PR

Migrates `@azure/attestation` to ESM and tshy.

Co-authored-by: Deyaaeldeen Almahallawi <dealmaha@microsoft.com>
2024-10-22 09:39:30 -07:00
Timo van Veenendaal 31b76c94f5
[ts-http-runtime] add azure diff check tool (#31285)
### Packages impacted by this PR

- `@typespec/ts-http-runtime`

### Describe the problem that is addressed by this PR

We have had issues with the unbranded Core package becoming out of sync
with the Azure packages. This PR adds a report which is a diff between
the Azure Core packages and the unbranded package and causes the lint
step to fail if there are any differences between Azure and unbranded
that do not appear in the report. Running the `lint:fix` script will
update the diff report. This will mean that:

- Making an Azure Core change without a corresponding change to
unbranded package, either in the source code or in the diff report, will
result in a CI failure
- Changes to the diff report can be reviewed so that any differences
between Azure and unbranded implementations of a feature can be easily
compared

Why use the lint script (and lint:fix)? This is already run by CI, so by
using it we don't need to make any changes to pipeline steps to get this
working. This script is a one-off and so I just want to do the bare
minimum to get things working 😄
2024-10-21 21:44:06 +00:00
Azure SDK Bot 1c6a7bd057
[EngSys] automatic rush update --full
<p dir="auto">This is an automatic PR generated weekly with changes from running the command rush update --full</p>
2024-10-21 09:25:23 -07:00
Jeremy Meng 1f4c88c136
[EngSys] upgrade dev dependency `@typescript-eslint/*` to `~8.10.0` (#31466)
- replace version string with ~8.10.0
- rush update
- [monitor-opentelemetry-exporter] fixing linting error of unnecessary `undefine` type for the optional parameter
2024-10-18 13:22:56 -07:00
Matthew Podwysocki e008167f7a
[monitor] update to latest OTEL (#31431)
### Packages impacted by this PR

- @azure/monitor-opentelemetry-exporter
- @azure/monitor-opentelemetry

### Issues associated with this PR

- #31391

### Describe the problem that is addressed by this PR

Updates to latest packages

### 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)
2024-10-16 12:31:50 -04:00
Matthew Podwysocki fe7545b8bd
[communication] Move @azure/communication-common to ESM/vitest (#31419)
### Packages impacted by this PR

- @azure/communication-common

### Issues associated with this PR

- https://github.com/Azure/azure-sdk-for-js/issues/31338

### Describe the problem that is addressed by this PR

Updates to ESM/vitest for @azure/communication-common

### 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)
2024-10-16 11:51:06 -04:00
Matthew Podwysocki 47ecb2d6ce
[maps] Migrate @azure/maps-common to ESM/vitest (#31409)
### Packages impacted by this PR

- @azure/maps-common

### Issues associated with this PR

- #31338

### Describe the problem that is addressed by this PR


### 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)
2024-10-15 19:30:25 -04:00
EmmaZhu-MSFT d8764f9e6e
[Storage]Bump package version for STG96 features preview (#31404) 2024-10-16 01:19:57 +08:00
Matthew Podwysocki c5b1b80f4b
[appconfiguration] Upgrade to ESM/vitest (#30066)
- @azure/app-configuration ESM migration
2024-10-14 15:32:11 -07:00
Azure SDK Bot 9513058fa3
[EngSys] automatic rush update --full
<p dir="auto">This is an automatic PR generated weekly with changes from running the command rush update --full</p>
2024-10-14 09:51:09 -07:00
Maor Leger 155d4be91b
[dev-tool] Migrate package codemods (#31324)
### Packages impacted by this PR

@azure/dev-tool

### Issues associated with this PR

Resolves #31325

### Describe the problem that is addressed by this PR

In order to automate as much as possible, we're introducing a set of
codemods to dev-tool's migrate-package command which can be added to by anyone.
These codemods will run in sequence and commit their work in stages for ease
of reviewing
2024-10-14 09:01:46 -07:00
ZiWei Chen 9ccc79a350
[mgmt] hybridcompute release (#31377)
https://github.com/Azure/sdk-release-request/issues/5581
2024-10-12 17:06:00 +08:00
Minh-Anh Phan 72187a1b21
[Schema Registry Avro] ESM Migrate (#31131)
Migrate SR Avro
2024-10-11 11:23:55 -07:00
Jeremy Meng 0e2c0bd934
[ServiceBus] migrate to ESM/tshy/vitest (#31304)
- apply the result of `dev-tool admin migrate-package --package-name
@azure/service-bus`

without removing chai usages in tests because it is causing crash when
running
the command

- work around compilation error from "long" with `"skipLibCheck": true`

- fix assert/chai/etc.

- replace sinon usage with vi
2024-10-11 08:35:13 -07:00
Maor Leger 5b7a025ac1
[keyvault-keys] Migrate to ESM and tshy (#31332)
### Packages impacted by this PR

@azure/keyvault-keys

### Issues associated with this PR

Resolves #30822 

### Describe the problem that is addressed by this PR

Migrate keyvault-keys to ESM and tshy
2024-10-09 13:01:26 -07:00
Maor Leger 2ffa690a77
[dev-tool] Upgrade ts-morph to 24.x (#31326)
Resolves #31321
2024-10-07 17:25:07 +00:00
Azure SDK Bot be5f03aa40
[EngSys] automatic rush update --full
<p dir="auto">This is an automatic PR generated weekly with changes from running the command rush update --full</p>
2024-10-07 09:44:25 -07:00
Maor Leger ff53ed55b8
[keyvault-secrets] Migrate to ESM and tshy (#31254)
### Packages impacted by this PR

@azure/keyvault-secrets

### Issues associated with this PR

#30822

### Describe the problem that is addressed by this PR

Migrates keyvault-secrets to ESM and tshy
2024-10-04 08:29:46 -07:00
Matthew Podwysocki 4d4b874244
[template] Add supportsTracing back (#31300)
### Packages impacted by this PR

- @azure/template

### Issues associated with this PR


### Describe the problem that is addressed by this PR

Adds `supportsTracing` back to the template project.

### 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: Timo van Veenendaal <timov@microsoft.com>
2024-10-04 10:42:57 -04:00
Matthew Podwysocki e81c5a39b4
[playwright] Migrate creator to ESM/vitest (#31293)
### Packages impacted by this PR

- @azure/create-microsoft-playwright-testing

### Issues associated with this PR


### Describe the problem that is addressed by this PR


### 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)
2024-10-03 13:18:47 -04:00
Maor Leger dc16f1486b
[dev-tool] Update readme (#31295)
Just update the readme to specify dev-tool uses tsx instead of ts-node
now
2024-10-02 13:54:13 -07:00
Maor Leger 543f973590
[dev-tool] newline fix (#31294)
Fixes newline character to use `\n` in windows as well as other OS's
2024-10-02 20:28:04 +00:00
Maor Leger e99e5dd409
[dev-tool] Use tsx (#31284)
### Packages impacted by this PR

@azure/dev-tool

### Issues associated with this PR

Resolves #29672

### Describe the problem that is addressed by this PR

With the move to ESM and tshy we've been running into issues using the
existing TS_NODE
based loader in dev-tool. Specifically we're seeing errors such as:

```
[run-samples] Error: Must use import to load ES Module: D:\a_work\1\s\sdk\template\template\samples-dev\getConfigurationSetting.ts
require() of ES modules is not supported.
require() of D:\a_work\1\s\sdk\template\template\samples-dev\getConfigurationSetting.ts from D:\a_work\1\s\common\tools\dev-tool\src\commands\samples\run.ts is an ES module file as it is a .ts file whose nearest parent package.json contains "type": "module" which defines all .ts files in that package scope as ES modules.
Instead change the requiring code to use import(), or remove "type": "module" from D:\a_work\1\s\sdk\template\template\package.json.
```

ts-node is being phased out for tsx, a modern alternative that handles
hybrid scenarios like us in much simpler manner.

This PR switches our loader from ts-node to tsx. Now we can run samples
again for both cjs and esm!
2024-10-02 12:23:56 -07:00
Jeremy Meng 652a7ffa89 [eslint-plugin] fix ts-naming-options rule issue
***NO_CI***

Currently the rule only handles the case of

```ts
  clientMethod(options: MethodOptions) {}
```

but not when the options bag parameter has a default value

```ts
  clientMethod(options: MethodOptions = {}) {}
```

This change adds code to handle the latter.

- Add two unit tests

- Update the affected packages to suppress this rule as a warning

- react to the fix
2024-10-02 16:40:01 +00:00
Matthew Podwysocki f8e509681f
[instrumentation] Move to ESM/vitest (#31286)
### Packages impacted by this PR

- @azure/opentelemetry-instrumentation-azure-sdk

### Issues associated with this PR


### Describe the problem that is addressed by this PR

Migrates the package to ESM and vitest. Note one issue with mocking with
the propagator object.

### 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: Maor Leger <maorleger@users.noreply.github.com>
2024-10-01 18:08:34 -04:00
Maor Leger 47e5f01431
[dev-tool] migrate-package command improvements (#31269)
### Packages impacted by this PR

@azure/dev-tool

### Issues associated with this PR

N/A - part of migrations

### Describe the problem that is addressed by this PR

A few quality-of-life improvements as I migrate keyvault packages to
ESM:

1. Allow skipping browser test configuration
2. Ensure linters are happy by appending an empty line to updated json files
3. Replace `tsc -p .` with `dev-tool run build-package` in the scripts section
4. Replace `@azure-tools/test-utils` with `@azure-tools/test-utils-vitest`
5. Remove `tshy` from the dependencies to add since `build-package` uses
a vendored tshy version
2024-09-30 14:50:58 -07:00
Azure SDK Bot 3ddd97d9da
[EngSys] automatic rush update --full
<p dir="auto">This is an automatic PR generated weekly with changes from running the command rush update --full</p>
2024-09-30 11:00:39 -07:00
ZiWei Chen f732e630bb
[mgmt] trustedsigning release (#31255)
https://github.com/Azure/sdk-release-request/issues/5543
2024-09-30 12:58:39 +08:00
Maor Leger f0271b99c6
[dev-tool] Generate detailed report for ESM migrations (#31252)
### Packages impacted by this PR

dev-tool

### Issues associated with this PR

### Describe the problem that is addressed by this PR

A few QoL additions as we aim to complete the ESM migration:

- output markdown
- Add verbose mode to output individual package status

I only focused on ESM migration as we plan to migrate ESM + Vitest in a
single commit. This all builds off of the work Matt started and allows us to
keep an up-to-date wiki for folks to follow the migration status
2024-09-27 14:09:33 -07:00
ZiWei Chen 742d38b2f8
[mgmt] computeschedule release (#31172)
https://github.com/Azure/sdk-release-request/issues/5497
2024-09-27 19:36:20 +08:00
ZiWei Chen a4a78c97b5
[mgmt] fabric release (#31031)
https://github.com/Azure/sdk-release-request/issues/5418
swagger pr: https://github.com/Azure/azure-rest-api-specs/pull/30489
2024-09-27 16:55:27 +08:00
Matthew Podwysocki 61c8b9aae4 [EngSys] Add update-snippets to all package.json files ***NO_CI*** 2024-09-26 17:59:37 +00:00
ZiWei Chen 7e2c6bcf52
[mgmt] batch release (#31071)
https://github.com/Azure/sdk-release-request/issues/5438

breaking caused by
https://github.com/Azure/azure-rest-api-specs/pull/29957
2024-09-26 17:27:43 +08:00
ZiWei Chen 018b5ba3c5
[mgmt] containerservice release (#31223)
https://github.com/Azure/sdk-release-request/issues/5462
2024-09-26 17:19:46 +08:00
ZiWei Chen 9a5532b60f
[mgmt] standbypool release (#31087)
https://github.com/Azure/sdk-release-request/issues/5443
swagger pr: https://github.com/Azure/azure-rest-api-specs/pull/30550

---------

Co-authored-by: Jiao Di (MSFT) <80496810+v-jiaodi@users.noreply.github.com>
2024-09-26 15:57:42 +08:00
ZiWei Chen 86b305b49c
[mgmt] privatedns release (#31092)
https://github.com/Azure/sdk-release-request/issues/5466

---------

Co-authored-by: Mary Gao <yanmeigao1210@gmail.com>
Co-authored-by: Jiao Di (MSFT) <80496810+v-jiaodi@users.noreply.github.com>
2024-09-26 13:49:43 +08:00
ZiWei Chen fd0f48e819
[mgmt] sql release (#30693)
https://github.com/Azure/sdk-release-request/issues/5379
2024-09-26 11:41:22 +08:00
ZiWei Chen c0ff9baac8
[mgmt] elasticsan release (#30731)
https://github.com/Azure/sdk-release-request/issues/5402

---------

Co-authored-by: Jiao Di (MSFT) <80496810+v-jiaodi@users.noreply.github.com>
2024-09-26 11:35:45 +08:00
Jeremy Meng 0804cc1620
[dev-tool] only process api.json files if they are generated (#31213)
This PR fixes an error in extra-api command when a package doesn't
enable the docModel generation, we were trying to open an api.md file
that doesn't exists.
2024-09-25 13:30:53 -07:00
Maor Leger 64c72d5dda [engsys] Move all packages to build-package script ***NO_CI*** 2024-09-25 18:52:35 +00:00
howieleung 9acf1638a1
inference tracing (#31160)
### Packages impacted by this PR
ai-inference

### Issues associated with this PR


### Describe the problem that is addressed by this PR

- Added instrumentation to the inference client library.
- Created generated folder that contains the generated code after
linting and formatting.
- lint and format are missing in the package.json. Added it. And apply
lint and format fixed all errors within the inference library included
the code I didn't touch.


### 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?)_
Yes.  added the test and recording.

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

### 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)
2024-09-25 11:50:22 -07:00
Maor Leger bfefb073bb
[dev-tool] Add build-package command (#31197)
### Packages impacted by this PR

core

### Issues associated with this PR

N/A

### Describe the problem that is addressed by this PR

As we look to migrate all packages to tshy we want to make sure we are
able to move away from tshy if ever needed. This PR adds a build-package command
that can later be used to consolidate some of the commands and decouple a bit
from tshy

### 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?

We can just call tshy directly, but one of the "selling" points of tshy
is that we can always eject and do what it does manually if we run into issues.
This helps streamline that without having to update 400 package.json files if
needed.

Frankly, I am not 100% sure if this is needed and I admit that `tshy` is
still an entry in packages' json, but I think we can live with the `tshy`
entry longer than we can with a `tshy` tool migration.
2024-09-25 10:14:07 -07:00
Jeremy Meng 4abda1b83d
[eslint-plugin] remove legacy configs (#31203)
Now that we have moved to ESLint v9, classic configs are no longer used.
This PR removes them.
2024-09-25 09:35:27 -04:00
Matthew Podwysocki 9fcee80f1a
[core] Add README snippets for core (#31204)
### Packages impacted by this PR

- @azure/dev-tool
- @azure/abort-controller
- @azure/core-amqp
- @azure/core-auth
- @azure/core-client
- @azure-rest/core-client
- @azure/core-http-compat
- @azure/core-lro
- @azure/core-paging
- @azure/core-rest-pipeline
- @azure/core-tracing
- @azure/core-util
- @azure/core-xml
- @azure/logger
- @typespec/ts-http-runtime

### Issues associated with this PR


### Describe the problem that is addressed by this PR

Adds snippets extraction to core.

### 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)
2024-09-25 09:35:12 -04:00
ZiWei Chen 99409f1826
[mgmt] mongocluster preivew release (#31124)
https://github.com/Azure/sdk-release-request/issues/5488

---------

Co-authored-by: Mary Gao <yanmeigao1210@gmail.com>
2024-09-25 19:51:07 +08:00
Daniel Getu 3398d73db5
[Search] Regenerate with 2024-09-01-preview spec (#31133) 2024-09-24 23:34:03 -07:00
Jeremy Meng 13f5beb45a
[eslint-plugin] turning off some rules for test/snippets.spec.ts (#31205)
as its code is meant to be included in docs and not required to be
perfect.
2024-09-24 14:06:22 -07:00
Jeremy Meng 3b67e63dcd
[dev-tool] generate merged api.json from subpath exports (#31164)
This PR creates a merged api.json file from individual api.json files of
subpath exports. The `ApiModel` type from api-extractor-model already
allows multiple `EntryPoint` but so far only one is added.

With this new file, we can also attach more information to the
`metadata` properties. Even though they are not loaded by `ApiModel`, we
could still read that information from the file directly. This saves us
from having to upload multiple files to ApiView.
2024-09-23 16:52:22 -07:00
Minh-Anh Phan 2d2ccf5e42
[Dev-Tool] Update the build-test command to check for browser files (#31184)
### Describe the problem that is addressed by this PR
The current overrides for the browser specific files only replace them
if they are under "src" directory. This PR adds the "test" directory for
test coverage
2024-09-23 10:31:24 -07:00
Azure SDK Bot cf51ccd040
[EngSys] automatic rush update --full
<p dir="auto">This is an automatic PR generated weekly with changes from running the command rush update --full</p>
2024-09-23 09:41:08 -07:00
Matthew Podwysocki 8597959155
[dev-tool] Add snippets extraction (#31143)
### Packages impacted by this PR

- @azure/dev-tool

### Issues associated with this PR

### Describe the problem that is addressed by this PR

This PR implements a new dev-tool command: `dev-tool run
update-snippets`.

This command looks for code fences in markdown files and JSDoc comments,
and updates them with the contents of test methods in a file named
`snippets.spec.ts`.

For example, the following fence indicates that the contents of a test
named "new_configurationclient" should be used:

````
```js snippet:new_configurationclient
```
````

After running `dev-tool run update-snippets`, the contents of the
snippet will be populated:

````
```js snippet:new_configurationclient
import { ConfigurationClient } from "@azure/template";
import { DefaultAzureCredential } from "@azure/identity";

const client = new ConfigurationClient(
  "<app configuration endpoint>",
  new DefaultAzureCredential()
);
```
````

To accomplish this, the command uses the TypeScript compiler API to
extract and transpile snippets from `snippets.spec.ts`. Snippets are the
contents of calls to the `it` function. If syntax with the shape
`it(<literal string>, <function with block>)` appears in
`snippets.spec.ts`, it will be considered a snippet that is valid for
injection.

("Function with block" means either a `function () { ... }` expression
or an arrow function with a block on the arrow side (`() => { ... }`).
An arrow function that has an expression on the right hand side (`() =>
(...)`) will not be recognized.)

For example:

```ts
  it("new_configurationclient", function () {
    // @ts-ignore
    const client = new ConfigurationClient(
      process.env.ENDPOINT ?? "<app configuration endpoint>",
      new DefaultAzureCredential()
    );
  });
```

The transpiler automatically "cleans" and validates the snippet using
similar techniques as the sample transpiler. As a result, it enforces
the same syntactic rules that the sample transpiler does. In addition to
those, it removes references to `process.env` (if an alternative is
specified), removes compiler pragmas like `// @ts-ignore`, and
automatically inserts imports for symbols that the snippet uses. So in
the above snippet, imports for `ConfigurationClient` and
`DefaultAzureCredential` are required, automatically detected, and
injected into the resulting snippet.

Snippets without `snippet:${name}` tags are _errors_ when using this
command, so a package must be fully migrated to use it.

### 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)_

- #24536

### 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)
2024-09-20 19:05:10 -04:00
Jeremy Meng e11ec1aa1d
[eslint-plugin] ignore *.api.md files from linting (#31180)
### Packages impacted by this PR
`@eslint-plugin-azure-sdk`
2024-09-20 12:31:41 -07:00
Jeremy Meng a7b3656a05 [EngSys] upgrade dev dependency `typescript` version to `~5.6.2`
***NO_CI***

- replace version to "~5.6.2"
- rush update
- add `"skipLibCheck": true` for several packages that are affected by https://github.com/vitest-dev/vitest/issues/4688

- [formrecognizer] help TypeScript to understand with more explicit type

- [schema-registry-{avro,json}] skip lib check due to lru-cache issue with the Iterator TReturn change

https://github.com/isaacs/node-lru-cache/issues/348
2024-09-19 18:07:31 -07:00
Minh-Anh Phan fdac51005e
[Schema Registry] Post release update for Schema Registry (#31138)
Update package version after release
2024-09-17 12:17:57 -07:00
Azure SDK Bot 9e8083e3fd
[EngSys] automatic rush update --full
<p dir="auto">This is an automatic PR generated weekly with changes from running the command rush update --full</p>
2024-09-16 09:34:03 -07:00
Matthew Podwysocki ebb399928e
[notification-hubs] Separate broadcast from send operations (#31102)
### Packages impacted by this PR

- @azure/notification-hubs

### Issues associated with this PR

- #30140

### Describe the problem that is addressed by this PR

Separates the broadcast send and broadcast scheduled send into their own
methods instead of having in the base `sendNotification` and
`scheduleNotification` respectively. Creates two methods of
`sendBroadcastNotification` and `scheduleBroadcastNotification` for each
scenario.

cc @gunzip 

### 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?

This allows us to keep required fields for either devices or tag based
expressions for mainline sends and tag based for scheduled sends in the
main methods and only separate the broadcast parts which are less used.

### 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
- [x] 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)_
- [x] Added a changelog (if necessary)
2024-09-13 14:06:49 -04:00
Jeremy Meng e1e7eb4b9f
[eslint-plugin] exclude *.config.*ts from linting (#31050)
so we don't have errors in editors when opening vitest configs
2024-09-11 15:10:03 -07:00
Matthew Podwysocki 370f60bba5
[web pub sub] update protobuf to ESM (#31044)
### Packages impacted by this PR

- @azure/web-pubsub-client-protobuf

### Issues associated with this PR

Fixes build issues associated with #30988

### Describe the problem that is addressed by this PR


### 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)_

- #30988

### 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)
2024-09-11 12:50:57 -04:00
Azure SDK Bot 00d6be82ca
[EngSys] automatic rush update --full (#31063)
This is an automatic PR generated weekly with changes from running the
command rush update --full
2024-09-10 16:13:50 -07:00
Jeremy Meng bc7574ebf7
[Test] downgrade test-credential to v1 for Service Bus tests (#30913)
as it hasn't been migrated to vitest yet.
2024-09-10 13:41:49 -07:00
Azure SDK Bot f756c33c74
[EngSys] automatic rush update --full (#31034)
This is an automatic PR generated weekly with changes from running the
command rush update --full
2024-09-09 12:54:32 -07:00
ZiWei Chen 8ddc182af9
[mgmt] eventhub release (#30454)
https://github.com/Azure/sdk-release-request/issues/5318
2024-09-06 09:59:59 +08:00
ZiWei Chen 59d3d9a346
[mgmt] billing release (#30990)
https://github.com/Azure/sdk-release-request/issues/5457
2024-09-05 15:22:10 +08:00
Jackson Weber 14a1dc5156
[monitor] Update OTel Dependencies (#30980)
### Packages impacted by this PR
@azure/monitor-opentelemetry
@azure/monitor-opentelemetry-exporter
@azure/monitor-query
@azure/opentelemetry-instrumentation-azure-sdk

Updates OpenTelemetry dependencies.

### Checklists
- [x] 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)_
- [x] Added a changelog (if necessary)
2024-09-04 17:05:22 -04:00
Azure SDK Bot c1db23f973
[EngSys] automatic rush update --full (#30973)
This is an automatic PR generated weekly with changes from running the
command rush update --full
2024-09-03 07:09:09 -07:00
Jeremy Meng b7e1fd2698
[eslint-plugin] fix `ts-modules-only-named` invalid test cases (#30924)
There's some change in `typescript-eslint` that breaks some eslint test
cases.
Previously when running tests, in a rule's `create()` method, we get a
relative
path for `context.filename`, and when we pass a setting of `{ main:
"src/test.ts" }` in rule tester option, the two were considered the same
file,
therefore the following condition check failed, and we continue to
report error
for default exports.


6dd8b582e2/common/tools/eslint-plugin-azure-sdk/src/rules/ts-modules-only-named.ts (L31)

After the upgrade, `context.filename` now contains the full path, the
file
`src/test.ts` in our test fixture has a full path of
`/.../common/tools/eslint-plugin-azure-sdk/tests/fixture/src/test.ts`.
It is not
considered the same file as the passed settings implies
`/.../common/tools/eslint-plugin-azure-sdk/src/test.ts` when we run the
tests.
So we return earlier instead of going to check for default exports.

This is not an issue in real scenario where we have settings of `{ main:
"src/index.ts" }`, which would match the full path of the file we are
linting
when we run `eslint` from a package directory.

This PR fixes the test by passing `tests/fixture/src/test.ts` in the
settings so
that it matches the full path of `context.filename` we get in
`create()`.
2024-08-28 14:52:44 -07:00
ZiWei Chen f55aa15db3
[mgmt] hdinsight release (#30733)
https://github.com/Azure/sdk-release-request/issues/5414
2024-08-28 10:10:50 +08:00
Jeremy Meng 06529fc7d9
[EngSys] re-enable rush build cache (#30818)
Use AzureCLI@2 task to generate a SAS for cache write credential so we
don't need to store secrets anymore.
2024-08-27 14:24:31 -07:00
Jeremy Meng d5f6d677ec [EngSys] standardize OSS copyright header
- [eslint-plugin] github-source-headers rule should use "MIT License".

  This change updates our linter rule and the whole repository to have consistent
  license header.

***NO_CI***

- remove "All rights reserved." from header

- "MIT license" => "MIT License"
2024-08-27 13:01:38 -07:00
Jeremy Meng 7029a9a745
[dev-tool] add command to type check files not being built by tshy (#30810)
In our current setup, tshy won't build test and samples-dev folder even
they are
included in tsconfig.json. This PR adds a command that does the
type-checking
for all files included in tsconfig.json, and allow additional files to
be passed
in as well.
2024-08-27 09:23:48 -07:00
Jeremy Meng 6041bc3950
[eslint-plugin] add an override to turn off tsdoc/syntax for samples-dev (#30902)
because dev-tool is using @summary and @azsdk-weight when publishing
samples.
2024-08-26 16:18:45 -07:00
Jeremy Meng bf2a1f74bf
[EngSys] upgrade dev dependency `@eslint/js` to ^9.9.0 (#30901)
to be consistent with `eslint` version.

Also removed the eslint plugin helper. It was added to help
vscode-eslint
extension to resolve plugin packages. With flat config it is no longer
needed.
2024-08-26 16:08:52 -07:00
Azure SDK Bot 98c96590fc
[EngSys] automatic rush update --full
<p dir="auto">This is an automatic PR generated weekly with changes from running the command rush update --full</p>
2024-08-26 09:48:52 -07:00
Jeremy Meng 0a7913a414 remove temporary logging 2024-08-23 17:04:42 -07:00
Jeremy Meng c34a1c6482 skip linting for config files for the two packages that don't depend on dev-tool 2024-08-23 16:38:09 -07:00
Jeremy Meng 5adb09d715 [engsys] upgrade dev dependency eslint to v9
***NO_CI***

- bump eslint version to ^9.9.0
- [eslint-plugin] remove eslint-plugin-import for now. Will re-visit until it supports eslint v9
2024-08-23 16:37:40 -07:00
Jeremy Meng 038ed97cd2
[eslint-plugin] fix tests after upgrading typescript-eslint (#30889)
- adjust options passed `RuleTester` constructor
- remove `recommended` property that no longer exists
- use `output` of array to indicate multi-pass fixes with overlapping
range
- temporarily disable two test cases of ts-modules-only-named suite.
2024-08-23 16:13:22 -07:00
Jeremy Meng 2ce3c5feb4 [eslint-plugin] upgrade typescript-eslint to v8
***NO_CI***

- bump typescript-eslint dependencies version to ~8.2.0

- [eslint-plugin] suppress new linting errors appearing with v8

- [eslint-plugin] react to removed typescript-eslint property

  related PR https://github.com/typescript-eslint/typescript-eslint/pull/9025

- [eslint-plugin] turn off @typescript-eslint/no-unused-expressions

  as no-unused-expressions is already reported by eslint

- [dev-tool] fix new linting errors

- Remove @typescript-eslint/ban-types that no longer exists

  and replace with more specific rules wherever applicable.

- [cosmosdb] combine the test eslint config with the main one

- delete catch expression when caught error is not used

- fix or suppress linting errors in code files
2024-08-23 10:19:23 -07:00
ZiWei Chen 1c2e977494
[mgmt] hdinsightcontainers release (#30732)
https://github.com/Azure/sdk-release-request/issues/5410
2024-08-23 13:42:13 +08:00
ZiWei Chen e94e5614a1
[mgmt] azurestackhci release (#30873)
https://github.com/Azure/sdk-release-request/issues/5371
2024-08-23 13:40:29 +08:00
Maor Leger 58ac994ec5
[keyvault-certificates] Migrate to ESM and tshy (#30819)
### Packages impacted by this PR

@azure/keyvault-certificates

### Issues associated with this PR

N/A

### Describe the problem that is addressed by this PR

Migrates keyvault-certificates to ESM and tshy
2024-08-22 12:49:35 -07:00
Jeremy Meng 1764824335 add patterns for json and source blocks in markdown
because in flat config, the path is resolved relative to the config file.
The added patterns are for when the central flat config is used.
2024-08-22 11:41:29 -07:00
ZiWei Chen 73d5edced4
[mgmt] netapp release (#30714)
https://github.com/Azure/sdk-release-request/issues/5392
2024-08-22 15:54:45 +08:00
ZiWei Chen 09acdf0d1f
[mgmt] compute release (#30713)
https://github.com/Azure/sdk-release-request/issues/5383
2024-08-22 15:44:04 +08:00
Siddharth Singha Roy 21b1956dcc
Onboard @azure/microsoft-playwright-testing sdk (#30433)
### Packages impacted by this PR
@azure/microsoft-playwright-testing
@azure/create-microsoft-playwright-testing

### Issues associated with this PR


### Describe the problem that is addressed by this PR
- Onboard 2 packages
- @azure/microsoft-playwright-testing: Client library used to integrate
Playwright OSS with Microsoft Playwright Testing service
- @azure/create-microsoft-playwright-testing: Utility library to setup
@azure/microsoft-playwright-testing

#### What does the package do

- Playwright OSS client connects to the service-hosted browsers using
the API provided by Playwright OSS. The package provides an abstraction
over this interaction
- Test results are published from the machine running the tests to the
service.
 
To integrate the service with a Playwright project, users will need to
install a package along with Playwright OSS package. The package is
tightly coupled with Playwright OSS package and also is responsible for
entra id authentication.


### 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?)_
Yes


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


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

### Checklists
- [x] 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)_
- [x] Added a changelog (if necessary)

---------

Co-authored-by: Vansh Vardhan Singh <vanshvsingh@gmail.com>
2024-08-22 10:31:28 +05:30
Jeremy Meng a0f2553fce
[eslint-plugin] turn off `eslint-plugin-n` rules for browsers (#30795)
Add a configuration to turn off all `eslint-plugin-n` rules for browser
code files.

Also remove the redundant `plugins` block in `nCustomization` because
the
plugin's "flat/recommended" config already does that


e5e758ea0c/lib/index.js (L87)

-------

### Packages impacted by this PR

eslint-plugin-azure-sdk
2024-08-21 09:11:32 -07:00
Harsha Nalluru 5586f99081
[Document Intelligence] Generate 1.0.0-beta.3 version (#30373)
- Regeneration done
- Tests in sync with latest src
- Live tests 
- Migrated to ESM and vitest
- Added new tests for the new features
- Added custom code to support 
- the "completed" poller status for batch analysis
`/documentModels/{modelId}:analyzeBatch`
   - getting operation id from the poller
- and the new `/documentModels/{modelId}/analyzeResults/{resultId}/pdf`
and
`/documentModels/{modelId}/analyzeResults/{resultId}/figures/{figureId}`
methods
2024-08-20 22:21:29 +00:00
Maor Leger c6213d531f
[keyvault-common] Migrate to ESM and tshy (#30772)
### Packages impacted by this PR

@azure/keyvault-common

### Issues associated with this PR


### Describe the problem that is addressed by this PR

Migrates keyvault-common to ESM and tshy
2024-08-20 12:00:16 -07:00
Minh-Anh Phan c2459f58ba
[OpenAI] Add Live Test Pipeline (#30506)
### Packages impacted by this PR


### Issues associated with this PR


### Describe the problem that is addressed by this PR
Add live test pipeline for OpenAI service

### 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: Deyaaeldeen Almahallawi <dealmaha@microsoft.com>
2024-08-20 09:16:37 -07:00
Azure SDK Bot 013283a9d0
[EngSys] automatic rush update --full
<p dir="auto">This is an automatic PR generated weekly with changes from running the command rush update --full</p>
2024-08-19 08:58:35 -07:00
Jeremy Meng 0854e57c06
[linting] move monitor-opentelemetry-exporter to use flat config (#30792)
- add a config in eslint-plugin-azure-sdk for linting with type
information
- remove .eslint* and replace with eslint.config.mjs
- remove --ext from linting NPM scripts
- fix linting error: URL is now available in global scope on our
supported platforms.
2024-08-16 17:18:54 -07:00
Jeremy Meng 9c2c8eecc2
[eslint-plugin] upgrade `eslint-plugin-no-only-tests` to `^3.3.0` (#30775)
which allows up to remove the fixup as the new version supports flat
config.

-------

### Packages impacted by this PR
eslint-plugin-azure-sdk
2024-08-15 16:03:38 -07:00
Jeremy Meng d53f35436a
[eslint-plugin] enable eslint-plugin-promise for our flat config (#30774)
now that the plugin has flat config support

-------

### Packages impacted by this PR
eslint-plugin-azure-sdk
2024-08-15 16:03:06 -07:00