mirror of
https://github.com/paperclipai/paperclip.git
synced 2026-10-15 03:51:22 +02:00
Merge branch 'codex/runner-copilot-acp' into codex/runner-acp-production
* codex/runner-copilot-acp: Preserve Copilot policy rejection codes across sidecar transport
This commit is contained in:
commit
b13d375d46
3 files changed
+15
No files matched your search
@@ -697,6 +697,8 @@ fn response_error_classification(error: &ResponseError) -> &'static str {
|
||||
"AGENT_DISCONNECTED" => return "agent_disconnected",
|
||||
"AUTH_REQUIRED" => return "authentication_required",
|
||||
"COPILOT_AUTH_REQUIRED" => return "authentication_required",
|
||||
"COPILOT_POLICY_VIOLATION" => return "copilot_policy_violation",
|
||||
"COPILOT_DETACHED_WORK_UNSUPPORTED" => return "copilot_detached_work_unsupported",
|
||||
"COPILOT_ENTITLEMENT_DENIED" => return "provider_entitlement_denied",
|
||||
"COPILOT_MODEL_UNAVAILABLE" => return "requested_model_unsupported",
|
||||
"SESSION_RESUME_REQUIRED" => return "session_resume_required",
|
||||
@@ -888,6 +890,11 @@ mod tests {
|
||||
"provider_lifetime_owned"
|
||||
);
|
||||
let admission_failures = [
|
||||
("COPILOT_POLICY_VIOLATION", "copilot_policy_violation"),
|
||||
(
|
||||
"COPILOT_DETACHED_WORK_UNSUPPORTED",
|
||||
"copilot_detached_work_unsupported",
|
||||
),
|
||||
(
|
||||
"ACPX_RUNTIME_ADMISSION_VERIFICATION_TIMEOUT",
|
||||
"runtime_admission_verification_timeout",
|
||||
|
||||
@@ -114,6 +114,12 @@ describe("ACPX sidecar input sequencing", () => {
|
||||
expect(acpxBootstrapBlockedError(null, "turn.start")).toBeNull();
|
||||
});
|
||||
|
||||
it.each(["COPILOT_POLICY_VIOLATION", "COPILOT_DETACHED_WORK_UNSUPPORTED"])("preserves Copilot policy identity %s", code => {
|
||||
const error = Object.assign(new Error("safe fixed policy message"), { code });
|
||||
expect(acpxSidecarErrorCode(error)).toBe(code);
|
||||
expect(acpxSidecarErrorCode(new Error("wrapped", { cause: error }))).toBe(code);
|
||||
});
|
||||
|
||||
it("preserves stable ACPX error identities without copying startup stderr", () => {
|
||||
const missingModule = Object.assign(new Error("provider exited"), {
|
||||
detailCode: "AGENT_STARTUP_FAILED",
|
||||
|
||||
@@ -130,6 +130,8 @@ const GENERIC_ACPX_OUTPUT_CODES = new Set([
|
||||
]);
|
||||
|
||||
const STABLE_ACPX_SIDECAR_CODES = new Set([
|
||||
"COPILOT_POLICY_VIOLATION",
|
||||
"COPILOT_DETACHED_WORK_UNSUPPORTED",
|
||||
"ACP_MODEL_UNSUPPORTED",
|
||||
"ACP_SESSION_INIT_FAILED",
|
||||
"AGENT_DISCONNECTED",
|
||||
|
||||
Reference in new issue
Block a user