Skip to content

[ T3 Code ] Chillzyg: Fix ACP auth refresh retry#6002

Closed
Chillzyg wants to merge 1 commit into
UnsafeLabs:mainfrom
Chillzyg:codex/acp-auth-refresh-829
Closed

[ T3 Code ] Chillzyg: Fix ACP auth refresh retry#6002
Chillzyg wants to merge 1 commit into
UnsafeLabs:mainfrom
Chillzyg:codex/acp-auth-refresh-829

Conversation

@Chillzyg
Copy link
Copy Markdown

@Chillzyg Chillzyg commented Jun 2, 2026

Summary

  • remember successful ACP authenticate payloads and refresh-token metadata
  • add a one-shot auth-expiration recovery wrapper for session requests that calls onSessionExpired, runs logout cleanup, re-authenticates, and replays the failed request once
  • cover the expired-session prompt path with an in-memory protocol regression test

Verification

  • npx -y bun@1.3.11 run test --filter=effect-acp
  • npx -y bun@1.3.11 run typecheck from packages/effect-acp
  • npx -y bun@1.3.11 run fmt:check
  • npx -y bun@1.3.11 run lint packages/effect-acp/src/client.ts packages/effect-acp/src/client.test.ts
  • npx -y bun@1.3.11 run typecheck from repo root

/claim #829

@Chillzyg Chillzyg changed the title Fix ACP auth refresh retry [ T3 Code ] Chillzyg: Fix ACP auth refresh retry Jun 2, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 2, 2026

Unfortunately the changes in this PR didn't fully resolve the issue. Please rework your solution and submit a new pull request.

Make sure to review the acceptance criteria in the linked issue and verify all conditions are met before resubmitting. See CONTRIBUTING.md for guidelines.

@github-actions github-actions Bot closed this Jun 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant