Use proxy-port instead of port in the CLI (#1022)

I find that having `port` and `target-port` flags is confusing, because it's not clear what exactly the `port` flag is for. Add a `proxy-port` flag, but leave `port` around for backwards compatibility. This is a cosmetic CLI-only change - it does not change the structure of the run configs which are stored to disk, or the API for the UI.

Author: dmjb

cmd/thv/app/run.go

@@ -62,6 +62,7 @@ var (
 	runName              string
 	runHost              string
 	runPort              int
+	runProxyPort         int
 	runTargetPort        int
 	runTargetHost        string
 	runPermissionProfile string
@@ -94,7 +95,11 @@ func init() {
 	runCmd.Flags().StringVar(&runProxyMode, "proxy-mode", "sse", "Proxy mode for stdio transport (sse or streamable-http)")
 	runCmd.Flags().StringVar(&runName, "name", "", "Name of the MCP server (auto-generated from image if not provided)")
 	runCmd.Flags().StringVar(&runHost, "host", transport.LocalhostIPv4, "Host for the HTTP proxy to listen on (IP or hostname)")
+	runCmd.Flags().IntVar(&runProxyPort, "proxy-port", 0, "Port for the HTTP proxy to listen on (host port)")
 	runCmd.Flags().IntVar(&runPort, "port", 0, "Port for the HTTP proxy to listen on (host port)")
+	if err := runCmd.Flags().MarkDeprecated("port", "use --proxy-port instead"); err != nil {
+		logger.Warnf("Error marking port flag as deprecated: %v", err)
+	}
 	runCmd.Flags().IntVar(&runTargetPort, "target-port", 0,
 		"Port for the container to expose (only applicable to SSE or Streamable HTTP transport)")
 	runCmd.Flags().StringVar(
@@ -311,6 +316,12 @@ func runCmdFunc(cmd *cobra.Command, args []string) error {
 		}
 	}
 
+	// Determine effective port value - prefer --proxy-port over --port for backwards compatibility
+	effectivePort := runProxyPort
+	if effectivePort == 0 && runPort != 0 {
+		effectivePort = runPort
+	}
+
 	// Initialize a new RunConfig with values from command-line flags
 	// TODO: As noted elsewhere, we should use the builder pattern here to make it more readable.
 	runConfig, err := runner.NewRunConfigFromFlags(
@@ -330,7 +341,7 @@ func runCmdFunc(cmd *cobra.Command, args []string) error {
 		runPermissionProfile,
 		runTargetHost,
 		runTransport,
-		runPort,
+		effectivePort,
 		runTargetPort,
 		runEnv,
 		oidcIssuer,