refactor(sidecar): drop redundant commands list from PackageLinkedResponse

The response carried both commands: list<str> and projectedCommands; no client
reads the former anymore (projectedCommands supersedes it). Remove it from the
wire + generated types; the sidecar keeps its internal commands vector to derive
projected_commands.

Author: NathanFlurry

crates/sidecar-core/src/frames.rs

@@ -168,13 +168,11 @@ pub fn vm_configured_response(
 
 pub fn package_linked_response(
     request: &RequestFrame,
-    commands: Vec<String>,
     projected_commands: Vec<ProjectedCommand>,
 ) -> ResponseFrame {
     respond(
         request,
         ResponsePayload::PackageLinked(PackageLinkedResponse {
-            commands,
             projected_commands,
         }),
     )

crates/sidecar-protocol/protocol/secure_exec_sidecar_v1.bare

@@ -227,7 +227,6 @@ type ProjectedCommand struct {
 }
 
 type PackageLinkedResponse struct {
-  commands: list<str>
   projectedCommands: list<ProjectedCommand>
 }
 

crates/sidecar/src/vm.rs

@@ -613,7 +613,7 @@ where
             .collect();
 
         Ok(DispatchResult {
-            response: package_linked_response(request, commands, projected_commands),
+            response: package_linked_response(request, projected_commands),
             events: Vec::new(),
         })
     }

packages/core/src/generated-protocol.ts

@@ -95,7 +95,6 @@ export type LiveResponsePayload =
 	  }
 	| {
 			type: "package_linked";
-			commands: string[];
 			projected_commands: LiveProjectedCommand[];
 	  }
 	| {
@@ -271,7 +270,6 @@ export function fromGeneratedResponsePayload(
 		case "PackageLinkedResponse":
 			return {
 				type: "package_linked",
-				commands: [...payload.val.commands],
 				projected_commands: payload.val.projectedCommands.map(
 					(command) => ({
 						name: command.name,