test(logs): route /logs/search stubs by content + document hand-marshal bypass (issue #21)

internal/api/client_test.go

@@ -421,11 +421,11 @@ func TestFunctionMethodsUseGeneratedRoutes(t *testing.T) {
 			var body map[string]any
 			require.NoError(t, json.NewDecoder(r.Body).Decode(&body))
 			logSearchBodies = append(logSearchBodies, body)
-			if len(logSearchBodies) == 1 {
-				writeAPIJSON(t, w, http.StatusOK, logsResponse("function runtime"))
+			if logSearchIsBuild(body) {
+				writeAPIJSON(t, w, http.StatusOK, logsResponse("deployment build"))
 				return
 			}
-			writeAPIJSON(t, w, http.StatusOK, logsResponse("deployment build"))
+			writeAPIJSON(t, w, http.StatusOK, logsResponse("function runtime"))
 		case r.Method == http.MethodGet && r.URL.Path == "/projects/"+projectIDText+"/functions/"+functionIDText+"/schedulers":
 			writeAPIJSON(t, w, http.StatusOK, map[string]any{
 				"data":     []any{functionSchedulerResponse(schedulerIDText, functionIDText, projectIDText, "hello scheduler", true)},
@@ -720,6 +720,18 @@ func functionDeploymentResponse(id, projectID, functionID string) map[string]any
 	}
 }
 
+// logSearchIsBuild reports whether a captured /logs/search request body targets
+// build logs (carries a deployment selector) rather than runtime logs. Routing
+// stub responses on request content is more robust than relying on call order.
+func logSearchIsBuild(body map[string]any) bool {
+	resource, ok := body["resource"].(map[string]any)
+	if !ok {
+		return false
+	}
+	_, hasDeployments := resource["deployments"]
+	return hasDeployments
+}
+
 func logsResponse(message string) map[string]any {
 	return map[string]any{
 		"data": []any{

internal/api/frontends_test.go

@@ -96,11 +96,11 @@ func TestFrontendDomainAndLogsMethodsUseGeneratedRoutes(t *testing.T) {
 			var body map[string]any
 			require.NoError(t, json.NewDecoder(r.Body).Decode(&body))
 			logSearchBodies = append(logSearchBodies, body)
-			if len(logSearchBodies) == 1 {
-				writeAPIJSON(t, w, http.StatusOK, logsResponse("frontend runtime"))
+			if logSearchIsBuild(body) {
+				writeAPIJSON(t, w, http.StatusOK, logsResponse("frontend build"))
 				return
 			}
-			writeAPIJSON(t, w, http.StatusOK, logsResponse("frontend build"))
+			writeAPIJSON(t, w, http.StatusOK, logsResponse("frontend runtime"))
 		case r.Method == http.MethodPost && r.URL.Path == "/projects/"+projectIDText+"/frontends/"+frontendIDText+"/domain":
 			require.NoError(t, json.NewDecoder(r.Body).Decode(&createBody))
 			writeAPIJSON(t, w, http.StatusCreated, frontendCustomDomainResponse("app.example.com", "provisioning"))

internal/api/logs.go

@@ -17,6 +17,13 @@ const (
 	logResourceTypeFunction = "function"
 )
 
+// logSearchRequest is a hand-marshalled body for POST /projects/{id}/logs/search,
+// sent via SearchProjectLogsWithBodyWithResponse rather than the typed
+// SearchProjectLogsWithResponse. The generated apiclient models the request's
+// `resource` selector as an oapi-codegen oneOf union (apiclient.LogRequestResource),
+// built through From.../As... accessors; this flat struct produces the identical
+// wire format while keeping the call sites readable. Intentional — do not
+// "simplify" it back to the typed union call.
 type logSearchRequest struct {
 	Resource logRequestResource `json:"resource"`
 	Limit    *int               `json:"limit,omitempty"`