Skip to content

[Eng] Improve typespec devexp with New-DevTSPModule #28954

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.

Sign up for GitHub

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

Open
VeryEarly wants to merge 27 commits into
base: main
Choose a base branch
from
Open

[Eng] Improve typespec devexp with New-DevTSPModule #28954

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
27 commits
Select commit Hold shift + click to select a range
598f2c4
init update tsp
VeryEarly Nov 4, 2025
b62eb3c
basic features
VeryEarly Nov 5, 2025
d563459
finish local test and add readme.md
VeryEarly Dec 4, 2025
d214897
update readme.md
VeryEarly Dec 4, 2025
31dc9b1
update readme.md
VeryEarly Dec 5, 2025
7bc1936
resolve AI comments and fix typo
VeryEarly Dec 5, 2025
8483e3b
improve build script
VeryEarly Dec 17, 2025
0bb2931
rename update-devtspmodule to new-devtspmodule
VeryEarly Dec 17, 2025
511db29
update build script to use New-DevTSPModule instead of tsp-client
VeryEarly Dec 17, 2025
69b896a
pipeline test
VeryEarly Dec 17, 2025
eda8d96
Merge remote-tracking branch 'Azure/main' into yabo/test-devtsp
VeryEarly Dec 17, 2025
5602159
Merge remote-tracking branch 'Azure/main' into yabo/tsp-client
VeryEarly Dec 17, 2025
01c0f89
correct comments
VeryEarly Dec 17, 2025
eeb0ba6
skip build/import azdev if imported
VeryEarly Dec 17, 2025
472a27b
upgrade node version to 20.19.6
VeryEarly Dec 18, 2025
4f37b14
add debug log for findnomcommand
VeryEarly Dec 18, 2025
d67ab9c
typo
VeryEarly Dec 18, 2025
115af9b
typo
VeryEarly Dec 18, 2025
58a939b
fix temp location
VeryEarly Dec 18, 2025
20bff40
use local tsp
VeryEarly Dec 19, 2025
ad85937
list components under npm\prefix
VeryEarly Dec 19, 2025
2b81f60
try fixing find command from path
VeryEarly Dec 19, 2025
90f10d1
try fix
VeryEarly Dec 19, 2025
90defe8
upgrade node versions for all
VeryEarly Dec 19, 2025
2e4a42a
revert test changes
VeryEarly Dec 19, 2025
9ec4193
Merge branch 'yabo/test-devtsp' into yabo/tsp-client
VeryEarly Dec 19, 2025
2b5e93c
add changelog
VeryEarly Dec 19, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 2 additions & 7 deletions .azure-pipelines/azure-powershell-ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,7 @@ jobs:
- task: NodeTool@0
displayName: Install Autorest
inputs:
versionSpec: '18.20.8'
versionSpec: '20.19.6'
command: custom
verbose: false
customCommand: install autorest@latest
Expand All @@ -107,11 +107,6 @@ jobs:
targetType: inline
script: "$env:NODE_OPTIONS=\"--max-old-space-size=65536\""
pwsh: true
- task: Npm@1
displayName: Install tsp client
inputs:
command: custom
customCommand: install -g @azure-tools/[email protected]
- task: PowerShell@2
displayName: 'Build'
inputs:
Expand Down Expand Up @@ -212,7 +207,7 @@ jobs:
- task: NodeTool@0
displayName: Install autorest
inputs:
versionSpec: '18.20.8'
versionSpec: '20.19.6'
command: custom
verbose: false
customCommand: install autorest@latest
Expand Down
9 changes: 2 additions & 7 deletions .azure-pipelines/powershell-core.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ jobs:
- task: NodeTool@0
displayName: Install Autorest
inputs:
versionSpec: '18.20.8'
versionSpec: '20.19.6'
command: custom
verbose: false
customCommand: install autorest@latest
Expand All @@ -57,11 +57,6 @@ jobs:
targetType: inline
script: "$env:NODE_OPTIONS=\"--max-old-space-size=65536\""
pwsh: true
- task: Npm@1
displayName: Install tsp client
inputs:
command: custom
customCommand: install -g @azure-tools/[email protected]
- task: PowerShell@2
displayName: 'Build'
inputs:
Expand Down Expand Up @@ -107,7 +102,7 @@ jobs:
- task: NodeTool@0
displayName: Install autorest
inputs:
versionSpec: '18.20.8'
versionSpec: '20.19.6'
command: custom
verbose: false
customCommand: install autorest@latest
Expand Down
7 changes: 1 addition & 6 deletions .azure-pipelines/security-tools.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ jobs:
- task: NodeTool@0
displayName: Install Autorest
inputs:
versionSpec: '18.20.7'
versionSpec: '20.19.6'
command: custom
verbose: false
customCommand: install autorest@latest
Expand All @@ -37,11 +37,6 @@ jobs:
targetType: inline
script: "$env:NODE_OPTIONS=\"--max-old-space-size=65536\""
pwsh: true
- task: Npm@1
displayName: Install tsp client
inputs:
command: custom
customCommand: install -g @azure-tools/[email protected]
- task: PowerShell@2
displayName: 'Build'
inputs:
Expand Down
7 changes: 1 addition & 6 deletions .azure-pipelines/util/analyze-steps.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ steps:
- task: NodeTool@0
displayName: Install autorest
inputs:
versionSpec: '18.20.7'
versionSpec: '20.19.6'
command: custom
verbose: false
customCommand: install autorest@latest
Expand All @@ -21,11 +21,6 @@ steps:
targetType: inline
script: "$env:NODE_OPTIONS=\"--max-old-space-size=65536\""
pwsh: true
- task: Npm@1
displayName: Install tsp client
inputs:
command: custom
customCommand: install -g @azure-tools/[email protected]
- task: UseDotNet@2
displayName: 'Use .NET SDK'
inputs:
Expand Down
9 changes: 2 additions & 7 deletions .azure-pipelines/windows-powershell.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ jobs:
- task: NodeTool@0
displayName: Install Autorest
inputs:
versionSpec: '18.20.8'
versionSpec: '20.19.6'
command: custom
verbose: false
customCommand: install autorest@latest
Expand All @@ -53,11 +53,6 @@ jobs:
targetType: inline
script: "$env:NODE_OPTIONS=\"--max-old-space-size=65536\""
pwsh: true
- task: Npm@1
displayName: Install tsp client
inputs:
command: custom
customCommand: install -g @azure-tools/[email protected]
- task: PowerShell@2
displayName: 'Build'
inputs:
Expand Down Expand Up @@ -103,7 +98,7 @@ jobs:
- task: NodeTool@0
displayName: Install autorest
inputs:
versionSpec: '18.20.8'
versionSpec: '20.19.6'
command: custom
verbose: false
customCommand: install autorest@latest
Expand Down
2 changes: 1 addition & 1 deletion tools/AzDev/AzDev/AzDev.psd1
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ FunctionsToExport = 'Connect-DevCommonRepo', 'Disconnect-DevCommonRepo'
CmdletsToExport = 'Get-DevContext', 'Set-DevContext',
'Get-DevModule', 'Get-DevProject',
'Update-DevAssembly',
'Open-DevSwagger'
'Open-DevSwagger', 'New-DevTSPModule'

# Variables to export from this module
VariablesToExport = '*'
Expand Down
3 changes: 3 additions & 0 deletions tools/AzDev/CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,9 @@
- Quick start templates
- Versioning and publishing AzDev module

## 2025/12/22
- Feature: Added cmdlet `New-DevTSPModule` for TypeSpec development.

## 2025/8/26
- Feature: Recognize AutoRest.PowerShell version (v3/v4) for AutoRest-based projects and show as `SubType` in `Get-DevProject` output.

Expand Down
8 changes: 5 additions & 3 deletions tools/AzDev/build.ps1
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
$module = 'AzDev'
$artifacts = "$PSScriptRoot/../../artifacts"
$artifacts = Join-Path $PSScriptRoot ".." ".." "artifacts"
$moduleOut = Join-Path $artifacts $module

dotnet publish $PSScriptRoot/src --sc -o "$artifacts/$module/bin"
Copy-Item "$PSScriptRoot/$module/*" "$artifacts/$module" -Recurse -Force
if (Test-Path $moduleOut) { Remove-Item $moduleOut -Recurse -Force }
Copy link

Copilot AI Dec 19, 2025

Choose a reason for hiding this comment

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

The build script uses Remove-Item with -Recurse -Force which can have different behaviors on different platforms, particularly with symbolic links. Consider using a cross-platform compatible approach or adding error handling to gracefully handle platform-specific failures. Also ensure this works correctly on case-sensitive filesystems (Linux/macOS).

Copilot generated this review using guidance from repository custom instructions.
dotnet publish (Join-Path $PSScriptRoot "src") --sc -o (Join-Path $moduleOut "bin")
Copy-Item (Join-Path $PSScriptRoot $module "*") $moduleOut -Recurse -Force
12 changes: 11 additions & 1 deletion tools/AzDev/src/Services/YamlHelper.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,13 +20,23 @@ internal static class YamlHelper
private static IDeserializer Deserializer => _lazyDeserializer.Value;
private static Lazy<IDeserializer> _lazyDeserializer = new Lazy<IDeserializer>(BuildDeserializer);

private static ISerializer Serializer => _lazySerializer.Value;
private static Lazy<ISerializer> _lazySerializer = new Lazy<ISerializer>(BuildSerializer);

private static IDeserializer BuildDeserializer()
{
return new DeserializerBuilder()
.IgnoreUnmatchedProperties()
.Build();
}

private static ISerializer BuildSerializer()
{
return new SerializerBuilder()
.WithNamingConvention(YamlDotNet.Serialization.NamingConventions.CamelCaseNamingConvention.Instance)
.Build();
}

public static T Deserialize<T>(string yaml)
{
return Deserializer.Deserialize<T>(yaml);
Expand All @@ -48,6 +58,6 @@ public static bool TryDeserialize<T>(string yaml, out T obj)

public static string Serialize<T>(T obj)
{
throw new NotImplementedException();
return Serializer.Serialize(obj);
}
}
Loading