Skip to content

tsp - Use 'powershell' for @clientname decorator#1519

Merged
dolauli merged 1 commit into
Azure:mainfrom
dolauli:tsp-brown
Sep 16, 2025
Merged

tsp - Use 'powershell' for @clientname decorator#1519
dolauli merged 1 commit into
Azure:mainfrom
dolauli:tsp-brown

Conversation

@dolauli

@dolauli dolauli commented Sep 15, 2025

Copy link
Copy Markdown
Contributor

This pull request introduces targeted improvements to the PowerShell emitter and related utilities. The main changes focus on ensuring that the correct client name overrides are applied for PowerShell, and updating test logic to better handle file selection during compilation.

PowerShell-specific client name handling:

  • Updated both getClientName in convertor.ts and getSchemaForModel in modelUtils.ts to pass "powershell" as the override context, ensuring client name overrides are correctly applied for PowerShell scenarios. [1] [2]

Test robustness improvements:

  • Modified the GenerateModuleWithEmitter function in EmitterTest.ps1 to check for the existence of client.tsp and use it if available, otherwise defaulting to the current directory, making test compilation more flexible and robust.

@dolauli
dolauli merged commit b7d7288 into Azure:main Sep 16, 2025
103 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant