|
| 1 | +#Requires -Modules @{ ModuleName='Pester'; ModuleVersion='5.0.0' } |
| 2 | +# PR #481: a profile can pin its agent pane to a supported ACP agent installed |
| 3 | +# inside that profile's WSL distro. The routing assertion is deterministic once |
| 4 | +# a native WSL agent is present; the chat assertion additionally requires auth. |
| 5 | + |
| 6 | +BeforeDiscovery { |
| 7 | + $devPkg = Get-AppxPackage | Where-Object { $_.PackageFamilyName -like 'IntelligentTerminal_*' } |
| 8 | + $script:Ready = [bool]( |
| 9 | + $devPkg -and |
| 10 | + (Get-Command winapp -ErrorAction SilentlyContinue) -and |
| 11 | + (Get-Command wsl.exe -ErrorAction SilentlyContinue) |
| 12 | + ) |
| 13 | +} |
| 14 | + |
| 15 | +Describe 'Feature profile-scoped WSL agent backend' -Tag 'Feature' -Skip:(-not $script:Ready) { |
| 16 | + BeforeAll { |
| 17 | + Import-Module (Join-Path $PSScriptRoot '..\ItE2E\ItE2E.psd1') -Force |
| 18 | + |
| 19 | + $script:app = $null |
| 20 | + $script:skipReason = $null |
| 21 | + $script:oldAgentPaneId = $null |
| 22 | + $script:profileGuid = '{e2e48100-4810-4810-9810-000000000001}' |
| 23 | + $script:profileName = 'ITE2E PR481 WSL Agent' |
| 24 | + |
| 25 | + $distroProbe = Invoke-Native -FilePath 'wsl.exe' -Arguments @( |
| 26 | + '-e', 'sh', '-lc', 'printf "%s" "${WSL_DISTRO_NAME:-}"' |
| 27 | + ) -TimeoutSec 45 |
| 28 | + $script:distro = $distroProbe.StdOut.Trim() |
| 29 | + if ($distroProbe.ExitCode -ne 0 -or -not $script:distro) { |
| 30 | + $script:skipReason = 'no runnable default WSL distro is available' |
| 31 | + return |
| 32 | + } |
| 33 | + if ($script:distro -match '["\r\n]') { |
| 34 | + $script:skipReason = 'the default WSL distro name cannot be represented safely in a test profile' |
| 35 | + return |
| 36 | + } |
| 37 | + |
| 38 | + # Match the product's source probe: reject Windows executables leaked |
| 39 | + # through WSL interop, and require native npx for adapter-backed agents. |
| 40 | + # Probe one literal command at a time; wsl.exe's command-line expansion |
| 41 | + # can consume loop variables in a compound bash script before bash runs. |
| 42 | + $knownAgents = @('copilot', 'gemini', 'opencode', 'claude', 'codex') |
| 43 | + $script:agent = $null |
| 44 | + foreach ($candidate in $knownAgents) { |
| 45 | + $pathProbe = Invoke-Native -FilePath 'wsl.exe' -Arguments @( |
| 46 | + '-d', $script:distro, '--', 'sh', '-lc', "command -v $candidate 2>/dev/null" |
| 47 | + ) -TimeoutSec 15 |
| 48 | + $path = $pathProbe.StdOut.Trim() |
| 49 | + if ($pathProbe.ExitCode -ne 0 -or |
| 50 | + -not $path -or |
| 51 | + $path -match '(?i)^/mnt/[a-z]/|\.(exe|cmd|bat)$') { |
| 52 | + continue |
| 53 | + } |
| 54 | + |
| 55 | + if ($candidate -in @('claude', 'codex')) { |
| 56 | + $npxProbe = Invoke-Native -FilePath 'wsl.exe' -Arguments @( |
| 57 | + '-d', $script:distro, '--', 'sh', '-lc', 'command -v npx 2>/dev/null' |
| 58 | + ) -TimeoutSec 15 |
| 59 | + $npxPath = $npxProbe.StdOut.Trim() |
| 60 | + if ($npxProbe.ExitCode -ne 0 -or |
| 61 | + -not $npxPath -or |
| 62 | + $npxPath -match '(?i)^/mnt/[a-z]/|\.(exe|cmd|bat)$') { |
| 63 | + continue |
| 64 | + } |
| 65 | + } |
| 66 | + |
| 67 | + $script:agent = $candidate |
| 68 | + break |
| 69 | + } |
| 70 | + if (-not $script:agent) { |
| 71 | + $script:skipReason = "$($script:distro) has no supported native Linux ACP agent" |
| 72 | + return |
| 73 | + } |
| 74 | + |
| 75 | + $unconfiguredProfile = [pscustomobject][ordered]@{ |
| 76 | + guid = $script:profileGuid |
| 77 | + name = $script:profileName |
| 78 | + commandline = "wsl.exe -d `"$($script:distro)`"" |
| 79 | + } |
| 80 | + $unconfiguredProfiles = [pscustomobject][ordered]@{ |
| 81 | + defaults = [pscustomobject]@{} |
| 82 | + list = @($unconfiguredProfile) |
| 83 | + } |
| 84 | + $script:backend = "wsl:$($script:distro):$($script:agent)" |
| 85 | + $script:app = Start-Terminal -Package Dev -PassFre $true -Settings @{ |
| 86 | + acpAgent = $script:agent |
| 87 | + defaultProfile = $script:profileGuid |
| 88 | + profiles = $unconfiguredProfiles |
| 89 | + } |
| 90 | + $oldAgentPane = Get-AgentPaneSession -App $script:app |
| 91 | + if ($oldAgentPane) { |
| 92 | + $script:oldAgentPaneId = $oldAgentPane.PaneSessionId |
| 93 | + } |
| 94 | + |
| 95 | + # Apply the profile backend only after Start-Terminal has captured log |
| 96 | + # offsets. This exercises settings hot reload and keeps source-routing |
| 97 | + # assertions scoped to this run. |
| 98 | + $configuredProfile = [pscustomobject][ordered]@{ |
| 99 | + guid = $script:profileGuid |
| 100 | + name = $script:profileName |
| 101 | + commandline = "wsl.exe -d `"$($script:distro)`"" |
| 102 | + agentPaneBackend = $script:backend |
| 103 | + } |
| 104 | + $configuredProfiles = [pscustomobject][ordered]@{ |
| 105 | + defaults = [pscustomobject]@{} |
| 106 | + list = @($configuredProfile) |
| 107 | + } |
| 108 | + Set-WtSetting -App $script:app -Key 'profiles' -Value $configuredProfiles | Out-Null |
| 109 | + Open-AgentPane -App $script:app | Out-Null |
| 110 | + } |
| 111 | + AfterAll { |
| 112 | + if ($script:app) { |
| 113 | + Stop-Terminal -App $script:app |
| 114 | + } |
| 115 | + } |
| 116 | + |
| 117 | + It 'Hot reload routes the profile agent through its WSL distro without host fallback' { |
| 118 | + if ($script:skipReason) { |
| 119 | + Set-ItResult -Skipped -Because $script:skipReason |
| 120 | + return |
| 121 | + } |
| 122 | + |
| 123 | + $settings = Get-WtSettingsObject -App $script:app |
| 124 | + $profile = @($settings.profiles.list) | |
| 125 | + Where-Object { $_.guid -eq $script:profileGuid } | |
| 126 | + Select-Object -First 1 |
| 127 | + $profile.agentPaneBackend | Should -Be $script:backend |
| 128 | + |
| 129 | + if ($script:oldAgentPaneId) { |
| 130 | + (Test-Until -TimeoutSec 30 -IntervalSec 0.5 -Condition { |
| 131 | + -not (Get-AgentPaneSession -App $script:app -PaneSessionId $script:oldAgentPaneId) |
| 132 | + }) | Should -BeTrue -Because 'changing this profile backend must replace its old host helper' |
| 133 | + } |
| 134 | + |
| 135 | + $agentPattern = [regex]::Escape($script:agent) |
| 136 | + $sourcePattern = [regex]::Escape("wsl:$($script:distro)") |
| 137 | + Assert-Log -App $script:app -Name 'wta-main_master.log' -Pattern ( |
| 138 | + 'resolving agent CLI for helper.*requested_agent_id=Some\("' + $agentPattern + |
| 139 | + '"\).*resolved_agent_source=' + $sourcePattern |
| 140 | + ) -TimeoutSec 60 |
| 141 | + Assert-Log -App $script:app -Name 'wta-main_master.log' -Pattern ( |
| 142 | + 'agent CLI spawned.*agent_source=' + $sourcePattern |
| 143 | + ) -TimeoutSec 60 |
| 144 | + } |
| 145 | + |
| 146 | + It 'The profile-selected WSL agent connects and answers a chat round trip' { |
| 147 | + if ($script:skipReason) { |
| 148 | + Set-ItResult -Skipped -Because $script:skipReason |
| 149 | + return |
| 150 | + } |
| 151 | + if (-not (Wait-AgentReady -App $script:app -TimeoutSec 150)) { |
| 152 | + Set-ItResult -Skipped -Because "$($script:agent) is installed in $($script:distro) but is not authenticated or its ACP server did not connect" |
| 153 | + return |
| 154 | + } |
| 155 | + |
| 156 | + $oldAgentPaneIds = @($script:oldAgentPaneId) | Where-Object { $_ } |
| 157 | + $agentPane = Wait-NewAgentPaneSession -App $script:app ` |
| 158 | + -ExcludePaneSessionId $oldAgentPaneIds -TimeoutSec 30 |
| 159 | + Send-AgentPrompt -App $script:app -PaneSessionId $agentPane.PaneSessionId ` |
| 160 | + -Text 'What is 480 plus 1? Reply with only the number.' | Out-Null |
| 161 | + Assert-AgentPaneText -App $script:app -PaneSessionId $agentPane.PaneSessionId ` |
| 162 | + -Pattern '\b481\b' -TimeoutSec 150 |
| 163 | + } |
| 164 | +} |
0 commit comments