fix(supervisor): distinguish invalid response outcome and drop unneeded core changeset

Author: nicktrn

.changeset/supervisor-http-client-request-metrics.md

@@ -3,4 +3,4 @@ area: supervisor
 type: improvement
 ---
 
-The supervisor now reports a Prometheus metric for its outbound HTTP requests, so failed calls to upstream services are visible for monitoring.
+Improved supervisor observability: it now reports metrics for its outbound requests, making failed calls to upstream services easier to monitor.

.server-changes/supervisor-outbound-request-metrics.md

@@ -75,6 +75,8 @@ export class SupervisorHttpClient {
 
       if (result.success) {
         this.onHttpRequestComplete({ name, method, status: "2xx", outcome: "ok" });
+      } else if (result.statusCode === 200) {
+        this.onHttpRequestComplete({ name, method, status: "200", outcome: "invalid_response" });
       } else if (typeof result.statusCode === "number") {
         this.onHttpRequestComplete({
           name,

packages/core/src/v3/runEngineWorker/supervisor/http.ts

@@ -4,7 +4,7 @@ export type SupervisorHttpRequestMetric = {
   name: string;
   method: string;
   status: string;
-  outcome: "ok" | "http_error" | "network_error";
+  outcome: "ok" | "http_error" | "invalid_response" | "network_error";
 };
 
 export type SupervisorClientCommonOptions = {