Fix .NET SDK package; ensure no auto update (#392)

* Fix .NET SDK package missing props file

The _GenerateVersionProps target was running BeforeTargets=BeforeBuild,
but dotnet pack evaluates ItemGroup wildcards before targets run.
This meant the generated .props file wasn't being included in the package.

Fixed by:
1. Adding Pack to BeforeTargets so the target runs before packing
2. Explicitly adding the props file to pack content inside the target
3. Changing the static ItemGroup to only include the .targets file

* Pass --no-auto-update to CLI across all SDKs

Prevents the CLI from auto-updating itself, ensuring the version bundled
with the SDK is the version that runs.

Author: SteveSandersonMS

dotnet/src/Client.cs

@@ -883,7 +883,7 @@ private async Task VerifyProtocolVersionAsync(Connection connection, Cancellatio
             args.AddRange(options.CliArgs);
         }
 
-        args.AddRange(["--headless", "--log-level", options.LogLevel]);
+        args.AddRange(["--headless", "--no-auto-update", "--log-level", options.LogLevel]);
 
         if (options.UseStdio)
         {

dotnet/src/GitHub.Copilot.SDK.csproj

@@ -29,7 +29,7 @@
     </ItemGroup>
 
     <!-- Generate version props file at build time (gitignored) -->
-    <Target Name="_GenerateVersionProps" BeforeTargets="BeforeBuild">
+    <Target Name="_GenerateVersionProps" BeforeTargets="BeforeBuild;Pack">
         <Exec Command="node -e &quot;console.log(require('./nodejs/package-lock.json').packages['node_modules/@github/copilot'].version)&quot;" WorkingDirectory="$(MSBuildThisFileDirectory)../.." ConsoleToMSBuild="true" StandardOutputImportance="low">
             <Output TaskParameter="ConsoleOutput" PropertyName="CopilotCliVersion" />
         </Exec>
@@ -44,12 +44,16 @@
             </_VersionPropsContent>
         </PropertyGroup>
         <WriteLinesToFile File="$(MSBuildThisFileDirectory)build\GitHub.Copilot.SDK.props" Lines="$(_VersionPropsContent)" Overwrite="true" WriteOnlyWhenDifferent="true" />
+        <!-- Explicitly add props file to package content after generation -->
+        <ItemGroup>
+            <None Include="build\GitHub.Copilot.SDK.props" Pack="true" PackagePath="build\" />
+        </ItemGroup>
     </Target>
 
-    <!-- Include .targets and .props files in package -->
+    <!-- Include .targets file in package (props is added dynamically by _GenerateVersionProps) -->
     <!-- Also import the .targets for local dev (same logic consumers get) -->
     <ItemGroup>
-        <None Include="build\GitHub.Copilot.SDK.*" Pack="true" PackagePath="build\" CopyToOutputDirectory="Never" />
+        <None Include="build\GitHub.Copilot.SDK.targets" Pack="true" PackagePath="build\" CopyToOutputDirectory="Never" />
     </ItemGroup>
     <Import Project="build\GitHub.Copilot.SDK.targets" />
 

go/client.go

@@ -1313,7 +1313,7 @@ func (c *Client) verifyProtocolVersion(ctx context.Context) error {
 // This spawns the CLI server as a subprocess using the configured transport
 // mode (stdio or TCP).
 func (c *Client) startCLIServer(ctx context.Context) error {
-	args := []string{"--headless", "--log-level", c.options.LogLevel}
+	args := []string{"--headless", "--no-auto-update", "--log-level", c.options.LogLevel}
 
 	// Choose transport mode
 	if c.useStdio {

nodejs/src/client.ts

@@ -986,6 +986,7 @@ export class CopilotClient {
             const args = [
                 ...this.options.cliArgs,
                 "--headless",
+                "--no-auto-update",
                 "--log-level",
                 this.options.logLevel,
             ];

python/copilot/client.py

@@ -1116,7 +1116,7 @@ async def _start_cli_server(self) -> None:
         if not os.path.exists(cli_path):
             raise RuntimeError(f"Copilot CLI not found at {cli_path}")
 
-        args = ["--headless", "--log-level", self.options["log_level"]]
+        args = ["--headless", "--no-auto-update", "--log-level", self.options["log_level"]]
 
         # Add auth-related flags
         if self.options.get("github_token"):