fix: Create no longer drops appPorts. Exec ignores -f if a running container is found.

Author: vercel-eddie

pkg/commands/create.go

@@ -419,10 +419,17 @@ func generateDevcontainerConfig(p interact.Printer, baseConfigPath, featureRef s
 	}
 	cfg.SetFeature(featureRef, featureOpts)
 	interceptAddr := fmt.Sprintf(":%d", interceptPort)
-	cfg.AppPort = devcontainer.PortMappings{
-		{HostPort: appPort, ContainerPort: appPort},
-		{HostPort: interceptPort, ContainerPort: interceptPort},
+	// Remove any existing entry for the app port to avoid duplicates.
+	filtered := cfg.AppPort[:0]
+	for _, m := range cfg.AppPort {
+		if m.ContainerPort != appPort {
+			filtered = append(filtered, m)
+		}
 	}
+	cfg.AppPort = append(filtered,
+		devcontainer.PortMapping{HostPort: appPort, ContainerPort: appPort},
+		devcontainer.PortMapping{HostPort: interceptPort, ContainerPort: interceptPort},
+	)
 	cfg.EnsureContainerEnv(meta.EnvInterceptorAddr, interceptAddr)
 	cfg.EnsureCapAdd("NET_ADMIN")
 	cfg.EnsureRunArgs("-l", labelBridgeDeployment+"="+dcName)

pkg/commands/exec.go

@@ -76,13 +76,11 @@ func runExec(ctx context.Context, c *cli.Command) error {
 			return fmt.Errorf("interceptor is not healthy: %w", err)
 		}
 
-		dcConfigPath := c.String("devcontainer-config")
-		if dcConfigPath == "" {
-			label, err := ct.InspectLabel(ctx, containerID, meta.LabelDevcontainerConfigFile)
-			if err != nil || label == "" {
-				return fmt.Errorf("could not determine config path for running bridge %q", deploymentName)
-			}
-			dcConfigPath = label
+		// Always read the config from the container — this is the config it
+		// was actually created with, regardless of what -f points to now.
+		dcConfigPath, err := ct.InspectLabel(ctx, containerID, meta.LabelDevcontainerConfigFile)
+		if err != nil || dcConfigPath == "" {
+			return fmt.Errorf("could not determine config path for running bridge %q", deploymentName)
 		}
 		workspaceFolder, _ := filepath.Abs(filepath.Dir(filepath.Dir(filepath.Dir(dcConfigPath))))
 		return execInDevcontainer(ctx, workspaceFolder, dcConfigPath, nil, cmdArgs)