fix(cli,manifests): address CodeRabbit review feedback

- send.go: RUN_ERROR now returns an error (exit non-zero) instead of nil;
  RUN_FINISHED and RUN_ERROR both flush trailing newline before returning
- ambient-api-server.yaml: add resource requests/limits to migration
  initContainer (cpu: 50m-500m, memory: 128Mi-512Mi)

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: Ambient Code Bot

components/ambient-cli/cmd/acpctl/session/send.go

@@ -136,8 +136,16 @@ func runSend(cmd *cobra.Command, args []string) error {
 				}
 				fmt.Fprintf(out, "→ %s\n", preview)
 			}
-		case "RUN_FINISHED", "RUN_ERROR":
+		case "RUN_FINISHED":
+			if inText {
+				fmt.Fprintln(out)
+			}
 			return nil
+		case "RUN_ERROR":
+			if inText {
+				fmt.Fprintln(out)
+			}
+			return fmt.Errorf("run failed")
 		}
 	}
 

components/manifests/overlays/mpp-openshift/ambient-api-server.yaml

@@ -68,6 +68,13 @@ spec:
           volumeMounts:
             - name: db-secrets
               mountPath: /secrets/db
+          resources:
+            requests:
+              cpu: 50m
+              memory: 128Mi
+            limits:
+              cpu: 500m
+              memory: 512Mi
           securityContext:
             allowPrivilegeEscalation: false
             readOnlyRootFilesystem: false