Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Azure OpenAI: TypeSpec updates for 2023-12-01-preview #26737

Merged
merged 48 commits into from
Dec 6, 2023

Conversation

trrwilson
Copy link
Member

@trrwilson trrwilson commented Nov 16, 2023

This PR transfers to TypeSpec the changes made in the 2023-12-01-preview service API version for Azure OpenAI:
https://github.com/Azure/azure-rest-api-specs/tree/main/specification/cognitiveservices/data-plane/AzureOpenAI/inference/preview/2023-12-01-preview

@trrwilson trrwilson added the OpenAI OpenAI service label Nov 16, 2023
Copy link

openapi-pipeline-app bot commented Nov 16, 2023

Next Steps to Merge

Next steps that must be taken to merge this PR:
  • ❌ Your PR has breaking changes (label: BreakingChangeReviewRequired). See the PR description for help.
  • ❌ The required check named Swagger ModelValidation has failed. Refer to the check in the PR's 'Checks' tab for details on how to fix it.

Copy link

openapi-pipeline-app bot commented Nov 16, 2023

Swagger Validation Report

️❌BreakingChange: 105 Errors, 0 Warnings failed [Detail]
compared swaggers (via Oad v0.10.4)] new version base version
generated.json 2023-06-01-preview(184593c) 2023-06-01-preview(main)
generated.json 2023-07-01-preview(184593c) 2023-07-01-preview(main)
generated.json 2023-08-01-preview(184593c) 2023-08-01-preview(main)
generated.json 2023-09-01-preview(184593c) 2023-09-01-preview(main)
generated.json 2023-05-15(184593c) 2023-05-15(main)
generated.json 2023-12-01-preview(184593c) 2023-12-01-preview(main)

Only 22 items are listed, please refer to log for more details.

Rule Message
1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: inference/preview/2023-06-01-preview/generated.json#L535:11
Old: inference/preview/2023-06-01-preview/generated.json#L535:11
1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: inference/preview/2023-06-01-preview/generated.json#L456:9
Old: inference/preview/2023-06-01-preview/generated.json#L456:9
1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: inference/preview/2023-06-01-preview/generated.json#L471:9
Old: inference/preview/2023-06-01-preview/generated.json#L471:9
1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: inference/preview/2023-06-01-preview/generated.json#L475:9
Old: inference/preview/2023-06-01-preview/generated.json#L475:9
1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: inference/preview/2023-06-01-preview/generated.json#L510:11
Old: inference/preview/2023-06-01-preview/generated.json#L510:11
1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: inference/preview/2023-06-01-preview/generated.json#L1109:9
Old: inference/preview/2023-06-01-preview/generated.json#L1231:9
1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: inference/preview/2023-06-01-preview/generated.json#L797:11
Old: inference/preview/2023-06-01-preview/generated.json#L714:11
1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: inference/preview/2023-06-01-preview/generated.json#L752:9
Old: inference/preview/2023-06-01-preview/generated.json#L669:9
1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: inference/preview/2023-07-01-preview/generated.json#L535:11
Old: inference/preview/2023-07-01-preview/generated.json#L535:11
1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: inference/preview/2023-07-01-preview/generated.json#L456:9
Old: inference/preview/2023-07-01-preview/generated.json#L456:9
1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: inference/preview/2023-07-01-preview/generated.json#L471:9
Old: inference/preview/2023-07-01-preview/generated.json#L471:9
1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: inference/preview/2023-07-01-preview/generated.json#L475:9
Old: inference/preview/2023-07-01-preview/generated.json#L475:9
1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: inference/preview/2023-07-01-preview/generated.json#L510:11
Old: inference/preview/2023-07-01-preview/generated.json#L510:11
1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: inference/preview/2023-07-01-preview/generated.json#L1168:9
Old: inference/preview/2023-07-01-preview/generated.json#L1302:9
1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: inference/preview/2023-07-01-preview/generated.json#L850:11
Old: inference/preview/2023-07-01-preview/generated.json#L741:11
1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: inference/preview/2023-07-01-preview/generated.json#L805:9
Old: inference/preview/2023-07-01-preview/generated.json#L696:9
1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: inference/preview/2023-08-01-preview/generated.json#L801:11
Old: inference/preview/2023-08-01-preview/generated.json#L635:11
1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: inference/preview/2023-08-01-preview/generated.json#L722:9
Old: inference/preview/2023-08-01-preview/generated.json#L556:9
1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: inference/preview/2023-08-01-preview/generated.json#L463:11
Old: inference/preview/2023-08-01-preview/generated.json#L466:11
1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: inference/preview/2023-08-01-preview/generated.json#L737:9
Old: inference/preview/2023-08-01-preview/generated.json#L571:9
1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: inference/preview/2023-08-01-preview/generated.json#L741:9
Old: inference/preview/2023-08-01-preview/generated.json#L575:9
1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: inference/preview/2023-08-01-preview/generated.json#L776:11
Old: inference/preview/2023-08-01-preview/generated.json#L610:11
️️✔️Breaking Change(Cross-Version) succeeded [Detail] [Expand]
There are no breaking changes.
️️✔️CredScan succeeded [Detail] [Expand]
There is no credential detected.
️⚠️LintDiff: 15 Warnings warning [Detail]
compared tags (via openapi-validator v2.1.6) new version base version
release_2023_05_15_autogen release_2023_05_15_autogen(184593c) release_2023_05_15_autogen(main)
release_2023_06_01_preview_autogen release_2023_06_01_preview_autogen(184593c) release_2023_06_01_preview_autogen(main)
release_2023_07_01_preview_autogen release_2023_07_01_preview_autogen(184593c) release_2023_07_01_preview_autogen(main)
release_2023_08_01_preview_autogen release_2023_08_01_preview_autogen(184593c) release_2023_08_01_preview_autogen(main)

[must fix]The following errors/warnings are introduced by current PR:

Rule Message Related RPC [For API reviewers]
⚠️ Nullable Avoid the use of x-nullable.
Location: inference/preview/2023-06-01-preview/generated.json#L614
⚠️ PropertyType Property should have a defined type.
Location: inference/preview/2023-06-01-preview/generated.json#L672
⚠️ Nullable Avoid the use of x-nullable.
Location: inference/preview/2023-06-01-preview/generated.json#L701
⚠️ Nullable Avoid the use of x-nullable.
Location: inference/preview/2023-07-01-preview/generated.json#L626
⚠️ Nullable Avoid the use of x-nullable.
Location: inference/preview/2023-07-01-preview/generated.json#L659
⚠️ PropertyType Property should have a defined type.
Location: inference/preview/2023-07-01-preview/generated.json#L714
⚠️ Nullable Avoid the use of x-nullable.
Location: inference/preview/2023-07-01-preview/generated.json#L743
⚠️ EnumInsteadOfBoolean Booleans properties are not descriptive in all cases and can make them to use, evaluate whether is makes sense to keep the property as boolean or turn it into an enum.
Location: inference/preview/2023-08-01-preview/generated.json#L502
⚠️ Nullable Avoid the use of x-nullable.
Location: inference/preview/2023-08-01-preview/generated.json#L900
⚠️ Nullable Avoid the use of x-nullable.
Location: inference/preview/2023-08-01-preview/generated.json#L933
⚠️ PropertyType Property should have a defined type.
Location: inference/preview/2023-08-01-preview/generated.json#L988
⚠️ Nullable Avoid the use of x-nullable.
Location: inference/preview/2023-08-01-preview/generated.json#L1017
⚠️ Nullable Avoid the use of x-nullable.
Location: inference/stable/2023-05-15/generated.json#L426
⚠️ PropertyType Property should have a defined type.
Location: inference/stable/2023-05-15/generated.json#L484
⚠️ Nullable Avoid the use of x-nullable.
Location: inference/stable/2023-05-15/generated.json#L513


The following errors/warnings exist before current PR submission:

Only 22 items are listed, please refer to log for more details.

Rule Message
IntegerTypeMustHaveFormat The integer type does not have a format, please add it.
Location: inference/preview/2023-06-01-preview/generated.json#L423
IntegerTypeMustHaveFormat The integer type does not have a format, please add it.
Location: inference/preview/2023-06-01-preview/generated.json#L494
IntegerTypeMustHaveFormat The integer type does not have a format, please add it.
Location: inference/preview/2023-06-01-preview/generated.json#L789
IntegerTypeMustHaveFormat The integer type does not have a format, please add it.
Location: inference/preview/2023-06-01-preview/generated.json#L1329
IntegerTypeMustHaveFormat The integer type does not have a format, please add it.
Location: inference/preview/2023-07-01-preview/generated.json#L423
IntegerTypeMustHaveFormat The integer type does not have a format, please add it.
Location: inference/preview/2023-07-01-preview/generated.json#L494
IntegerTypeMustHaveFormat The integer type does not have a format, please add it.
Location: inference/preview/2023-07-01-preview/generated.json#L842
IntegerTypeMustHaveFormat The integer type does not have a format, please add it.
Location: inference/preview/2023-07-01-preview/generated.json#L1426
IntegerTypeMustHaveFormat The integer type does not have a format, please add it.
Location: inference/preview/2023-08-01-preview/generated.json#L689
IntegerTypeMustHaveFormat The integer type does not have a format, please add it.
Location: inference/preview/2023-08-01-preview/generated.json#L760
IntegerTypeMustHaveFormat The integer type does not have a format, please add it.
Location: inference/preview/2023-08-01-preview/generated.json#L1126
IntegerTypeMustHaveFormat The integer type does not have a format, please add it.
Location: inference/preview/2023-08-01-preview/generated.json#L1710
IntegerTypeMustHaveFormat The integer type does not have a format, please add it.
Location: inference/stable/2023-05-15/generated.json#L315
IntegerTypeMustHaveFormat The integer type does not have a format, please add it.
Location: inference/stable/2023-05-15/generated.json#L596
⚠️ SecurityDefinitionDescription Security definition should have a description.
Location: inference/preview/2023-06-01-preview/generated.json#L46
⚠️ SecurityDefinitionDescription Security definition should have a description.
Location: inference/preview/2023-06-01-preview/generated.json#L51
⚠️ OperationId OperationId should be of the form 'Noun_Verb'
Location: inference/preview/2023-06-01-preview/generated.json#L64
⚠️ PathParameterSchema Path parameter should specify a maximum length (maxLength) and characters allowed (pattern).
Location: inference/preview/2023-06-01-preview/generated.json#L70
⚠️ ParameterDescription Parameter should have a description.
Location: inference/preview/2023-06-01-preview/generated.json#L77
⚠️ OperationId OperationId should be of the form 'Noun_Verb'
Location: inference/preview/2023-06-01-preview/generated.json#L115
⚠️ PathParameterSchema Path parameter should specify a maximum length (maxLength) and characters allowed (pattern).
Location: inference/preview/2023-06-01-preview/generated.json#L121
⚠️ ParameterDescription Parameter should have a description.
Location: inference/preview/2023-06-01-preview/generated.json#L128
️❌Avocado: 3 Errors, 0 Warnings failed [Detail]
Rule Message
UNREFERENCED_JSON_FILE The example JSON file is not referenced from the swagger file.
readme: data-plane/AzureOpenAI/inference/readme.md
json: preview/2023-12-01-preview/examples/get_image_operation_status.json
UNREFERENCED_JSON_FILE The example JSON file is not referenced from the swagger file.
readme: data-plane/AzureOpenAI/inference/readme.md
json: preview/2023-12-01-preview/examples/start_generate_image.json
UNREFERENCED_JSON_FILE The swagger JSON file is not referenced from the readme file.
readme: data-plane/AzureOpenAI/inference/readme.md
json: inference/preview/2023-12-01-preview/generated.json
️️✔️SwaggerAPIView succeeded [Detail] [Expand]
️️✔️TypeSpecAPIView succeeded [Detail] [Expand]
️❌ModelValidation: 29 Errors, 0 Warnings failed [Detail]

Only 22 items are listed, please refer to log for more details.

Rule Message
INVALID_TYPE Expected type object but found type string
Url: inference/preview/2023-12-01-preview/generated.json#L671:34
ExampleUrl: preview/2023-12-01-preview/examples/audio_transcription_text.json#L8:13
RESPONSE_BODY_NOT_IN_EXAMPLE Response statusCode 200 for operation GetAudioTranscriptionAsPlainText has no response body provided in the example, however the response does have a "schema" defined in the swagger spec.
Url: inference/preview/2023-12-01-preview/generated.json#L86:22
ExampleUrl: preview/2023-12-01-preview/examples/audio_transcription_text.json#L10:16
INVALID_TYPE Expected type object but found type string
Url: inference/preview/2023-12-01-preview/generated.json#L859:32
ExampleUrl: preview/2023-12-01-preview/examples/audio_translation_text.json#L8:13
RESPONSE_BODY_NOT_IN_EXAMPLE Response statusCode 200 for operation GetAudioTranslationAsPlainText has no response body provided in the example, however the response does have a "schema" defined in the swagger spec.
Url: inference/preview/2023-12-01-preview/generated.json#L137:22
ExampleUrl: preview/2023-12-01-preview/examples/audio_translation_text.json#L10:16
OBJECT_MISSING_REQUIRED_PROPERTY Missing required property: system_fingerprint
Url: inference/preview/2023-12-01-preview/generated.json#L1613:24
ExampleUrl: preview/2023-12-01-preview/examples/chat_completions.json
INVALID_FORMAT Object didn't pass validation for format uri: {search endpoint}
Url: inference/preview/2023-12-01-preview/generated.json#L1196:21
ExampleUrl: preview/2023-12-01-preview/examples/extensions_chat_completions_azure_search_minimum.json#L8:13
OBJECT_MISSING_REQUIRED_PROPERTY Missing required property: system_fingerprint
Url: inference/preview/2023-12-01-preview/generated.json#L1613:24
ExampleUrl: preview/2023-12-01-preview/examples/extensions_chat_completions_azure_search_minimum.json
INVALID_FORMAT Object didn't pass validation for format uri: {search endpoint}
Url: inference/preview/2023-12-01-preview/generated.json#L1196:21
ExampleUrl: preview/2023-12-01-preview/examples/extensions_chat_completions_azure_search_image_vector.json#L8:13
OBJECT_MISSING_REQUIRED_PROPERTY Missing required property: system_fingerprint
Url: inference/preview/2023-12-01-preview/generated.json#L1613:24
ExampleUrl: preview/2023-12-01-preview/examples/extensions_chat_completions_azure_search_image_vector.json
INVALID_FORMAT Object didn't pass validation for format uri: {search endpoint}
Url: inference/preview/2023-12-01-preview/generated.json#L1196:21
ExampleUrl: preview/2023-12-01-preview/examples/extensions_chat_completions_azure_search_advanced.json#L8:13
OBJECT_MISSING_REQUIRED_PROPERTY Missing required property: system_fingerprint
Url: inference/preview/2023-12-01-preview/generated.json#L1613:24
ExampleUrl: preview/2023-12-01-preview/examples/extensions_chat_completions_azure_search_advanced.json
OBJECT_MISSING_REQUIRED_PROPERTY Missing required property: system_fingerprint
Url: inference/preview/2023-12-01-preview/generated.json#L1613:24
ExampleUrl: preview/2023-12-01-preview/examples/extensions_chat_completions_aml_index.json
OBJECT_MISSING_REQUIRED_PROPERTY Missing required property: system_fingerprint
Url: inference/preview/2023-12-01-preview/generated.json#L1613:24
ExampleUrl: preview/2023-12-01-preview/examples/extensions_chat_completions_cosmos_db.json
INVALID_FORMAT Object didn't pass validation for format uri: {search endpoint}
Url: inference/preview/2023-12-01-preview/generated.json#L2657:21
ExampleUrl: preview/2023-12-01-preview/examples/extensions_chat_completions_elasticsearch.json#L8:13
OBJECT_MISSING_REQUIRED_PROPERTY Missing required property: system_fingerprint
Url: inference/preview/2023-12-01-preview/generated.json#L1613:24
ExampleUrl: preview/2023-12-01-preview/examples/extensions_chat_completions_elasticsearch.json
OBJECT_MISSING_REQUIRED_PROPERTY Missing required property: key
Url: inference/preview/2023-12-01-preview/generated.json#L3089:46
ExampleUrl: preview/2023-12-01-preview/examples/extensions_chat_completions_pinecone.json#L8:13
OBJECT_ADDITIONAL_PROPERTIES Additional properties not allowed: apiKey
Url: inference/preview/2023-12-01-preview/generated.json#L3089:46
ExampleUrl: preview/2023-12-01-preview/examples/extensions_chat_completions_pinecone.json#L8:13
OBJECT_MISSING_REQUIRED_PROPERTY Missing required property: system_fingerprint
Url: inference/preview/2023-12-01-preview/generated.json#L1613:24
ExampleUrl: preview/2023-12-01-preview/examples/extensions_chat_completions_pinecone.json
OBJECT_ADDITIONAL_PROPERTIES Additional properties not allowed: revised_prompt
Url: inference/preview/2023-12-01-preview/generated.json#L3014:25
ExampleUrl: preview/2023-12-01-preview/examples/image_generation.json#L27:27
REQUIRED_PARAMETER_EXAMPLE_NOT_FOUND In operation GetAudioTranscriptionAsResponseObject, parameter file is required in the swagger spec but is not present in the provided example parameter values.
Url: inference/preview/2023-12-01-preview/generated.json#L440:15
ExampleUrl: preview/2023-12-01-preview/examples/audio_transcription_object.json
REQUIRED_PARAMETER_EXAMPLE_NOT_FOUND In operation GetAudioTranslationAsResponseObject, parameter file is required in the swagger spec but is not present in the provided example parameter values.
Url: inference/preview/2023-12-01-preview/generated.json#L507:15
ExampleUrl: preview/2023-12-01-preview/examples/audio_translation_object.json
INVALID_FORMAT Object didn't pass validation for format uri: {search endpoint}
Url: inference/preview/2023-08-01-preview/generated.json#L517:21
ExampleUrl: preview/2023-08-01-preview/examples/extensions_chat_completions.json#L8:13
️️✔️SemanticValidation succeeded [Detail] [Expand]
Validation passes for SemanticValidation.
️️✔️PoliCheck succeeded [Detail] [Expand]
Validation passed for PoliCheck.
️️✔️SpellCheck succeeded [Detail] [Expand]
Validation passes for SpellCheck.
️️✔️Lint(RPaaS) succeeded [Detail] [Expand]
Validation passes for Lint(RPaaS).
️️✔️PR Summary succeeded [Detail] [Expand]
Validation passes for Summary.
️️✔️Automated merging requirements met succeeded [Detail] [Expand]
Posted by Swagger Pipeline | How to fix these errors?

Copy link

openapi-pipeline-app bot commented Nov 16, 2023

Swagger Generation Artifacts

️️✔️ApiDocPreview succeeded [Detail] [Expand]
 Please click here to preview with your @microsoft account. 
️️✔️SDK Breaking Change Tracking succeeded [Detail] [Expand]

Breaking Changes Tracking

️❌ azure-sdk-for-net-track2 failed [Detail]
  • Failed [Logs] Generate from 35a74c328a2aa1d6d3fb17e4084489a379ed5261. SDK Automation 14.0.0
    command	pwsh ./eng/scripts/Automation-Sdk-Init.ps1 ../azure-sdk-for-net_tmp/initInput.json ../azure-sdk-for-net_tmp/initOutput.json
    warn		specification/cognitiveservices/data-plane/AzureOpenAI/inference/readme.md skipped due to azure-sdk-for-net-track2 not found in swagger-to-sdk
    command	pwsh ./eng/scripts/Invoke-GenerateAndBuildV2.ps1 ../azure-sdk-for-net_tmp/generateInput.json ../azure-sdk-for-net_tmp/generateOutput.json
    cmderr	[Invoke-GenerateAndBuildV2.ps1] �[31;1mGeneratePackage: �[0m/mnt/vss/_work/1/s/azure-sdk-for-net/eng/scripts/Invoke-GenerateAndBuildV2.ps1:131
    cmderr	[Invoke-GenerateAndBuildV2.ps1] �[36;1mLine |
    cmderr	[Invoke-GenerateAndBuildV2.ps1] �[36;1m 131 | �[0m               �[36;1mGeneratePackage `�[0m
    cmderr	[Invoke-GenerateAndBuildV2.ps1] �[36;1m     | �[31;1m               ~~~~~~~~~~~~~~~~~
    cmderr	[Invoke-GenerateAndBuildV2.ps1] �[31;1m�[36;1m     | �[31;1mFailed to generate sdk. exit code: False
    cmderr	[Invoke-GenerateAndBuildV2.ps1] �[0m
    cmderr	[Invoke-GenerateAndBuildV2.ps1] �[31;1mGet-ChildItem: �[0m/mnt/vss/_work/1/s/azure-sdk-for-net/eng/scripts/automation/GenerateAndBuildLib.ps1:807
    cmderr	[Invoke-GenerateAndBuildV2.ps1] �[36;1mLine |
    cmderr	[Invoke-GenerateAndBuildV2.ps1] �[36;1m 807 | �[0m … rtifacts += �[36;1mGet-ChildItem $artifactsPath -Filter *.nupkg -exclude *.s�[0m …
    cmderr	[Invoke-GenerateAndBuildV2.ps1] �[36;1m     | �[31;1m               ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    cmderr	[Invoke-GenerateAndBuildV2.ps1] �[31;1m�[36;1m     | �[31;1mCannot find path
    cmderr	[Invoke-GenerateAndBuildV2.ps1] �[36;1m     | �[31;1m'/mnt/vss/_work/1/s/azure-sdk-for-net/artifacts/packages/Debug/' because
    cmderr	[Invoke-GenerateAndBuildV2.ps1] �[36;1m     | �[31;1mit does not exist.
    cmderr	[Invoke-GenerateAndBuildV2.ps1] �[0m
    cmderr	[Invoke-GenerateAndBuildV2.ps1] �[31;1mGeneratePackage: �[0m/mnt/vss/_work/1/s/azure-sdk-for-net/eng/scripts/Invoke-GenerateAndBuildV2.ps1:131
    cmderr	[Invoke-GenerateAndBuildV2.ps1] �[36;1mLine |
    cmderr	[Invoke-GenerateAndBuildV2.ps1] �[36;1m 131 | �[0m               �[36;1mGeneratePackage `�[0m
    cmderr	[Invoke-GenerateAndBuildV2.ps1] �[36;1m     | �[31;1m               ~~~~~~~~~~~~~~~~~
    cmderr	[Invoke-GenerateAndBuildV2.ps1] �[31;1m�[36;1m     | �[31;1mFailed to generate sdk artifact
    cmderr	[Invoke-GenerateAndBuildV2.ps1] �[0m
  • Azure.AI.OpenAI [View full logs]  [Preview SDK Changes]
    info	[Changelog]
️❌ azure-sdk-for-java failed [Detail]
  • Pipeline Framework Failed [Logs] Generate from 35a74c328a2aa1d6d3fb17e4084489a379ed5261. SDK Automation 14.0.0
    command	./eng/mgmt/automation/init.sh ../azure-sdk-for-java_tmp/initInput.json ../azure-sdk-for-java_tmp/initOutput.json
    cmderr	[init.sh] [notice] A new release of pip is available: 23.0.1 -> 23.3.1
    cmderr	[init.sh] [notice] To update, run: pip install --upgrade pip
    cmderr	[init.sh] [notice] A new release of pip is available: 23.0.1 -> 23.3.1
    cmderr	[init.sh] [notice] To update, run: pip install --upgrade pip
    warn		specification/cognitiveservices/data-plane/AzureOpenAI/inference/readme.md skipped due to azure-sdk-for-java not found in swagger-to-sdk
    command	./eng/mgmt/automation/generate.py ../azure-sdk-for-java_tmp/generateInput.json ../azure-sdk-for-java_tmp/generateOutput.json
    cmderr	[generate.py]  WARN EBADENGINE Unsupported engine {
    cmderr	[generate.py] npm WARN EBADENGINE   package: '@typespec/[email protected]',
    cmderr	[generate.py] npm WARN EBADENGINE   required: { node: '>=18.0.0' },
    cmderr	[generate.py] npm WARN EBADENGINE   current: { node: 'v16.20.2', npm: '8.19.4' }
    cmderr	[generate.py] npm WARN EBADENGINE }
    cmderr	[generate.py] npm WARN EBADENGINE Unsupported engine {
    cmderr	[generate.py] npm WARN EBADENGINE   package: '@typespec/[email protected]',
    cmderr	[generate.py] npm WARN EBADENGINE   required: { node: '>=18.0.0' },
    cmderr	[generate.py] npm WARN EBADENGINE   current: { node: 'v16.20.2', npm: '8.19.4' }
    cmderr	[generate.py] npm WARN EBADENGINE }
    cmderr	[generate.py] npm WARN EBADENGINE Unsupported engine {
    cmderr	[generate.py] npm WARN EBADENGINE   package: '@typespec/[email protected]',
    cmderr	[generate.py] npm WARN EBADENGINE   required: { node: '>=18.0.0' },
    cmderr	[generate.py] npm WARN EBADENGINE   current: { node: 'v16.20.2', npm: '8.19.4' }
    cmderr	[generate.py] npm WARN EBADENGINE }
    cmderr	[generate.py] npm WARN EBADENGINE Unsupported engine {
    cmderr	[generate.py] npm WARN EBADENGINE   package: '@typespec/[email protected]',
    cmderr	[generate.py] npm WARN EBADENGINE   required: { node: '>=18.0.0' },
    cmderr	[generate.py] npm WARN EBADENGINE   current: { node: 'v16.20.2', npm: '8.19.4' }
    cmderr	[generate.py] npm WARN EBADENGINE }
    cmderr	[generate.py] npm WARN EBADENGINE Unsupported engine {
    cmderr	[generate.py] npm WARN EBADENGINE   package: '@typespec/[email protected]',
    cmderr	[generate.py] npm WARN EBADENGINE   required: { node: '>=18.0.0' },
    cmderr	[generate.py] npm WARN EBADENGINE   current: { node: 'v16.20.2', npm: '8.19.4' }
    cmderr	[generate.py] npm WARN EBADENGINE }
    cmderr	[generate.py] npm WARN EBADENGINE Unsupported engine {
    cmderr	[generate.py] npm WARN EBADENGINE   package: '@azure-tools/[email protected]',
    cmderr	[generate.py] npm WARN EBADENGINE   required: { node: '>=18.0.0' },
    cmderr	[generate.py] npm WARN EBADENGINE   current: { node: 'v16.20.2', npm: '8.19.4' }
    cmderr	[generate.py] npm WARN EBADENGINE }
    cmderr	[generate.py] npm WARN EBADENGINE Unsupported engine {
    cmderr	[generate.py] npm WARN EBADENGINE   package: '@azure-tools/[email protected]',
    cmderr	[generate.py] npm WARN EBADENGINE   required: { node: '>=18.0.0' },
    cmderr	[generate.py] npm WARN EBADENGINE   current: { node: 'v16.20.2', npm: '8.19.4' }
    cmderr	[generate.py] npm WARN EBADENGINE }
    cmderr	[generate.py]
    cmderr	[generate.py] npm notice New major version of npm available! 8.19.4 -> 10.2.4
    cmderr	[generate.py] npm notice Changelog: <https://github.com/npm/cli/releases/tag/v10.2.4>
    cmderr	[generate.py] npm notice Run `npm install -g [email protected]` to update!
    cmderr	[generate.py] npm notice
    cmderr	[generate.py] java.lang.RuntimeException: java.lang.NullPointerException
    cmderr	[generate.py] 	at com.azure.autorest.TypeSpecPlugin.customizeGeneratedCode(TypeSpecPlugin.java:154)
    cmderr	[generate.py] 	at com.azure.typespec.Main.handleDPG(Main.java:137)
    cmderr	[generate.py] 	at com.azure.typespec.Main.main(Main.java:93)
    cmderr	[generate.py] Caused by: java.lang.NullPointerException
    cmderr	[generate.py] 	at com.azure.autorest.TypeSpecPlugin.customizeGeneratedCode(TypeSpecPlugin.java:148)
    cmderr	[generate.py] 	... 2 more
    cmderr	[generate.py] Emitter "@azure-tools/typespec-java" crashed! This is a bug.
    cmderr	[generate.py] Please file an issue at https://github.com/Azure/autorest.java/issues
    cmderr	[generate.py] Error: JAR ended with code '1'.
    cmderr	[generate.py]     at ChildProcess.<anonymous> (file:///mnt/vss/_work/1/s/azure-sdk-for-java/sdk/openai/azure-ai-openai/TempTypeSpecFiles/OpenAI.Inference/node_modules/@azure-tools/typespec-java/dist/src/emitter.js:106:37)
    cmderr	[generate.py]     at ChildProcess.emit (node:events:513:28)
    cmderr	[generate.py]     at Process.ChildProcess._handle.onexit (node:internal/child_process:293:12)
    cmderr	[generate.py] --------------------------------------------------
    cmderr	[generate.py] Library Version                0.11.2
    cmderr	[generate.py] TypeSpec Compiler Version      0.50.0
    cmderr	[generate.py] --------------------------------------------------
    cmderr	[generate.py] �[31;1mWrite-Error: �[31;1mFailed to generate sdk project at /mnt/vss/_work/1/s/azure-sdk-for-java/sdk/openai/azure-ai-openai�[0m
    error	Fatal error: The "path" argument must be of type string. Received undefined
    The "path" argument must be of type string. Received undefined
    TypeError [ERR_INVALID_ARG_TYPE]: The "path" argument must be of type string. Received undefined
        at new NodeError (node:internal/errors:387:5)
        at validateString (node:internal/validators:162:11)
        at Object.basename (node:path:1309:5)
        at Object.exports.getPackageData (/mnt/vss/_work/1/a/unified-pipeline-runtime/private/openapi-sdk-automation/lib/types/PackageData.js:38:82)
        at /mnt/vss/_work/1/a/unified-pipeline-runtime/private/openapi-sdk-automation/lib/automation/workflow.js:210:194
        at Array.map (<anonymous>)
        at workflowHandleReadmeMdOrTypeSpecProject (/mnt/vss/_work/1/a/unified-pipeline-runtime/private/openapi-sdk-automation/lib/automation/workflow.js:210:164)
        at processTicksAndRejections (node:internal/process/task_queues:96:5)
        at async Object.exports.workflowMain (/mnt/vss/_work/1/a/unified-pipeline-runtime/private/openapi-sdk-automation/lib/automation/workflow.js:98:13)
        at async Object.exports.sdkAutoMain (/mnt/vss/_work/1/a/unified-pipeline-runtime/private/openapi-sdk-automation/lib/automation/entrypoint.js:211:13)
        at async /mnt/vss/_work/1/a/unified-pipeline-runtime/private/openapi-sdk-automation/lib/cli/cli.js:20:18
️🔄 azure-sdk-for-js inProgress [Detail]
Posted by Swagger Pipeline | How to fix these errors?

Copy link

openapi-pipeline-app bot commented Nov 16, 2023

Generated ApiView

Language Package Name ApiView Link
TypeSpec OpenAI.Inference https://apiview.dev/Assemblies/Review/ecbe771b2aa441cc90ecfe39ca2d0d8b

Copy link

Swagger pipeline started successfully. If there is ApiView generated, it will be updated in this comment.

@AzureRestAPISpecReview AzureRestAPISpecReview added the BreakingChangeReviewRequired <valid label in PR review process>add this label when breaking change review is required label Nov 16, 2023
Copy link

Swagger pipeline started successfully. If there is ApiView generated, it will be updated in this comment.

Copy link

Swagger pipeline started successfully. If there is ApiView generated, it will be updated in this comment.

Copy link

Swagger pipeline started successfully. If there is ApiView generated, it will be updated in this comment.

Copy link

Swagger pipeline started successfully. If there is ApiView generated, it will be updated in this comment.

Copy link

Swagger pipeline started successfully. If there is ApiView generated, it will be updated in this comment.

Copy link

Swagger pipeline started successfully. If there is ApiView generated, it will be updated in this comment.

Copy link

Swagger pipeline started successfully. If there is ApiView generated, it will be updated in this comment.

jpalvarezl and others added 7 commits November 30, 2023 08:05
* Update spell checking instructions (#26874)

Having one custom-words.txt for the entire repo has been causing a number of merging issues so we are trying to move away from it and instead have teams use override in the cspell configuration file directly.

* Update readme.python.md (#26894)

* Update readme.python.md

* Update readme.python.md

* Update readme.python.md

* Update readme.python.md

---------

Co-authored-by: Yuchao Yan <[email protected]>

* [Cognitive Services - Azure AI Content Safety] Update package print name. (#26911)

* Update package print name.

* update.

* update name.

* Added TypeSpec customization support for Java

---------

Co-authored-by: Wes Haggard <[email protected]>
Co-authored-by: ChenxiJiang333 <[email protected]>
Co-authored-by: Yuchao Yan <[email protected]>
Co-authored-by: jiezhou_microsoft <[email protected]>
@trrwilson trrwilson marked this pull request as ready for review December 1, 2023 18:12
@AzureRestAPISpecReview AzureRestAPISpecReview added the TypeSpec Authored with TypeSpec label Dec 5, 2023
@lmazuel lmazuel merged commit d402f68 into main Dec 6, 2023
23 of 30 checks passed
@lmazuel lmazuel deleted the user/travisw/aoai-2023-12-01-preview-typespec branch December 6, 2023 02:31
type: "function";

@doc("The name of the function that should be called.")
name: "string";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The type should be string and not the literal "string", I think.

Suggested change
name: "string";
name: string;

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I opened #26960.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BreakingChangeReviewRequired <valid label in PR review process>add this label when breaking change review is required data-plane OpenAI OpenAI service TypeSpec Authored with TypeSpec
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

9 participants