* Updated artifactJson input to artifacts input
* Prioritized service directory from artifact object over the parameter input
* Returned error when service directory is not provided for the old usage
* Removed redudant line
* Initilized the variable in a loop
---------
Co-authored-by: ray chen <raychen@microsoft.com>
* Added optional artifact list to filter the package info to be returned
* Apply suggestions from code review
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* return full package info if the input artifact list is empty
* Fixed hashset issue
* Added artifacts parameter
---------
Co-authored-by: ray chen <raychen@microsoft.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* quotes
* generalize
* cant use trigger as a job
* Update eng/common/pipelines/templates/jobs/ai-eval-job.yml
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* target repo name fix
* compile time expression
* compile time expression
* show params for pipeline
* fix the pipeline variables
* another fix
* fix variables
---------
Co-authored-by: Juan Ospina <70209456+jeo02@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Fix property name for skipVerifyChangeLog
* Apply suggestion from @raych1
* Refactor ShouldVerifyChangeLog function condition
* Apply suggestion from @raych1
---------
Co-authored-by: Ray Chen <raychen@microsoft.com>
* Include groupId in the pkg name when create api view request
* Used the package info in the function signature
---------
Co-authored-by: ray chen <raychen@microsoft.com>
* Used another way to construct package info array
* Filtered out empty string for PackageInfoFiles
---------
Co-authored-by: ray chen <raychen@microsoft.com>
* add tool instructions prompt
* minor
* prompt revision
* prompt revision and fallback to core requirements
* rename langs to languages + revise prompt
* edit prompt and make checks concurrent
* fix prompt
* add venvPath argument
* minor prompt and log edit
---------
Co-authored-by: jennypng <63012604+JennyPng@users.noreply.github.com>
* Upgrade cspell
* Use stdin for cspell input
* Add pipeline support to Invoke-Cspell.ps1
* Quote parameters
* Only quote the needed
---------
Co-authored-by: Daniel Jurek <djurek@microsoft.com>
* Change logging functions to use Write-Host
Change logging helpers to always write to host and either set color or use devops/gh formatting.
We do not want to use Write-Error or Write-Warning directly because they can stop the script or not depending on preferences which makes it difficult to ensure local runs of scripts work the same as in the pipelines. So, we should never depend on these logging commands to cause a script to stop execution.
* Clean up error handling in CommandInvocation-Helpers
Removed legacy error handling for command failures.
* Fix error logging for command execution
* Add option to skip exiting
---------
Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com>
Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com>
Fails if you don't have `bicep` installed but do have `az bicep` and was unnecessary since the check below it did it anyway.
Co-authored-by: Heath Stewart <heaths@microsoft.com>
* Updated validate pkg template to use packageInfo
* Fixed typo
* Fixed the right variable to use
* output debug log
* Fixed errors in expression evaluation
* removed debug code
* Fixed an issue in pipeline
* Updated condition for variable setting step
* Join paths of the script path
* Use join-path
* return from the function rather than exit
---------
Co-authored-by: ray chen <raychen@microsoft.com>
* Mark TypeSpec upgrade PR as draft when generation fails
Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com>
* Add condition to run Create_PR job even when Generate fails
Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com>
* Include emitter name in PR title
Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com>
* Use emitterIdentifier variable for PR title instead of recalculating
Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com>
* Include scope in emitterIdentifier for branch and PR names
Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com>
* Read emitter name from package.json instead of extracting from path
Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com>
* Resolve emitterPackagePath to absolute path before reading
Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com>
* Append package.json to emitterPackagePath directory
Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com>
* Add log statement for fallback to package path
Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com>
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com>
* Update create-apireview template to use packageInfo
* Use new signature for FindArtifactForApiReviewFn
* Used new signature of FindArtifactForApiReviewFn
---------
Co-authored-by: ray chen <raychen@microsoft.com>
* Added 'ArtifactsJson' parameter
* Made script work for non-java
* Reduced duplicated parsing
* Fixed type error
---------
Co-authored-by: ray chen <raychen@microsoft.com>
* Revise tsp-client usage instructions in README
Updated usage instructions to use 'npm exec --prefix' for running tsp-client commands. The reason is `tsp-client` needs to resolve the input parameters based on the root folder of SDK repository.
* Update README.md to use brace expansion for variables
* Update README.md
---------
Co-authored-by: Ray Chen <raychen@microsoft.com>
