Skip to content

feat: add hosted credential request handoff - #45

Merged
mrgoonie merged 1 commit into
mainfrom
codex/feat-1-credential-handoff
Nov 8, 2026
Merged

mrgoonie merged 1 commit into
mainfrom
codex/feat-1-credential-handoff

Conversation

@mrgoonie

@mrgoonie mrgoonie commented Nov 8, 2026

Copy link
Copy Markdown
Contributor

Problem and behavior

Signed-in chat visitors need a safe way to respond when an agent requests credentials. With explicit CCP and identity opt-in, the widget now shows a typed request and opens the authenticated Dewee-hosted form. The visitor enters no credential value into the widget or chat. Status, cancellation, history restore and a value-free continuation stay bound to the verified user, agent and conversation by the Gateway.

The host pins the HTTPS form origin; only the fixed Dewee credential route and one-time token shape are accepted. Requests fail closed on an identity or conversation change, and the feature remains off by default. Direct Gateway transport does not support the hosted flow; its legacy stream and history now suppress credential-tool payloads and unmatched/malformed tool results to avoid exposing handoff data.

Part of #1. Gateway/CCP contract: nextlevelbuilder/dewee#2240, merged to dev at fd0cbd4. The issue remains open until an enabled Gateway/CCP deployment completes the real signed-in flow and recovery paths.

Validation

  • Independent security review, including agent-scoped capability, token/origin validation, popup lifecycle, duplicate/missing tool IDs, TS/PHP and legacy data redaction; findings fixed.
  • Windows: PHP_BIN=C:/tools/php/php.exe pnpm test — 404 passed, 5 skipped; pnpm typecheck, pnpm build, pnpm test:bundlers, php -l passed.
  • Clean Linux container: PHP syntax and 51 focused CCP, widget and legacy Gateway tests passed.
  • Gitleaks on changed lines: no leaks. A full diff context scan found an unchanged synthetic test fixture.

Surface parity: Node/Workers, PHP, widget, HTTP contract and package docs updated. Gateway/CCP authorization and trace/activity evidence are supplied by #2240; trace parity RGD/CCP/API-CLI follows that change. CLI/MCP N/A for visitor transport routes; no privileged management operation is added here. Public docs follow-up: nextlevelbuilder/dewee-web-v2#133. Live deployment verification remains outstanding.

@mrgoonie
mrgoonie merged commit 41b7cbb into main Nov 8, 2026
3 checks passed
@mrgoonie
mrgoonie deleted the codex/feat-1-credential-handoff branch November 8, 2026 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant