Skip to content

feat(messaging): add text interaction mode#352

Open
huntharo wants to merge 11 commits into
mainfrom
feat/messaging-text-mode-agent-threads
Open

feat(messaging): add text interaction mode#352
huntharo wants to merge 11 commits into
mainfrom
feat/messaging-text-mode-agent-threads

Conversation

@huntharo
Copy link
Copy Markdown
Contributor

@huntharo huntharo commented May 12, 2026

Summary

  • I added global and per-platform Messaging interaction mode settings with buttons and text options, including TOML/env parsing and Settings UI controls.
  • I made text mode render interactive messaging surfaces as voice-friendly text while preserving the original semantic actions for pending-intent routing.
  • I added a structured ephemeral model-backed mapper for ambiguous text replies, with a repo-owned prompt, safe action-id validation, pass-through handling, and short clarification prompts.
  • I made the mapper prefer the Codex helper-object path used by thread naming, which starts ephemeral gpt-5.4-mini helper threads, and kept Grok object calls as a fallback option.
  • I kept deterministic matching first so exact replies do not pay model latency.

Verification

  • pnpm --filter @pwragent/desktop typecheck
  • pnpm test apps/desktop/src/main/__tests__/codex-client.test.ts apps/desktop/src/main/__tests__/messaging-interaction-mapper.test.ts apps/desktop/src/main/__tests__/messaging-controller.test.ts
  • pnpm test apps/desktop/src/main/__tests__/codex-client.test.ts apps/desktop/src/main/__tests__/messaging-interaction-mapper.test.ts apps/desktop/src/main/__tests__/messaging-controller.test.ts apps/desktop/src/main/__tests__/messaging-command-catalog.test.ts apps/desktop/src/main/__tests__/messaging-config.test.ts apps/desktop/src/main/__tests__/desktop-settings-service.test.ts apps/desktop/src/main/__tests__/desktop-settings-patch-translator.test.ts apps/desktop/src/main/__tests__/telegram-adapter.test.ts apps/desktop/src/renderer/src/features/settings/__tests__/settings-patch-delta.test.ts apps/desktop/src/renderer/src/features/settings/__tests__/settings-screen.test.tsx
  • pnpm lint:boundaries
  • git diff --check

Notes

  • This now covers the CarPlay-friendly “say what you mean” path for offered actions. Full hidden multi-turn helper sessions, SOUL.md-style personality files, memory, and visible agent threads remain follow-up plan units.
  • I restarted the dev-profile app from this checkout and verified the messaging lease is owned by the new instance.

@huntharo huntharo marked this pull request as ready for review May 12, 2026 21:56
@huntharo huntharo force-pushed the feat/messaging-text-mode-agent-threads branch 7 times, most recently from f0a1858 to c4dce6a Compare May 15, 2026 22:24
@huntharo huntharo force-pushed the feat/messaging-text-mode-agent-threads branch from c4dce6a to 34d198c Compare May 19, 2026 21:10
@huntharo huntharo added the invalid This doesn't seem right label May 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

invalid This doesn't seem right

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant