Merge branch 'main' into feat/managed-agent-api-commands

Author: heimanba

packages/sdk/src/internal/providers/qoder/adapter.ts

@@ -451,7 +451,9 @@ export class QoderAdapter implements ProviderAdapter {
 		const currentMetadata = (current.metadata ?? {}) as Record<string, unknown>;
 		const metadata = { ...((body.metadata ?? {}) as Record<string, string | null>) };
 		for (const key of Object.keys(currentMetadata)) {
-			if (!key.startsWith("agents.") && !(key in metadata)) metadata[key] = null;
+			// Qoder injects created_by into environment responses but rejects it on writes,
+			// including deletion tombstones such as { created_by: null }.
+			if (key !== "created_by" && !key.startsWith("agents.") && !(key in metadata)) metadata[key] = null;
 		}
 		body.metadata = metadata;
 		const res = (await client.post(`/environments/${id}`, body)) as Record<string, unknown>;

packages/sdk/tests/e2e/qoder-adapter-pagination.test.ts

@@ -166,7 +166,9 @@ describe("QoderAdapter environment contract", () => {
 		const { calls, restore } = mockFetch([
 			{
 				status: 200,
-				body: { metadata: { keep: "old", remove: "stale", "agents.project": "test-project" } },
+				body: {
+					metadata: { keep: "old", remove: "stale", created_by: "forward", "agents.project": "test-project" },
+				},
 			},
 			{ status: 200, body: { id: "env_1", type: "environment" } },
 		]);
@@ -190,6 +192,7 @@ describe("QoderAdapter environment contract", () => {
 				"agents.resource": "dev",
 			},
 		});
+		expect(calls[1]?.body).not.toHaveProperty("metadata.created_by");
 	});
 
 	test("exports normalized setup scripts and omits response-only package defaults", async () => {