Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Regression: Smart reply stopped working after version stable3.7 #10274

Open
Tobias4Real opened this issue Oct 17, 2024 · 0 comments
Open

Regression: Smart reply stopped working after version stable3.7 #10274

Tobias4Real opened this issue Oct 17, 2024 · 0 comments

Comments

@Tobias4Real
Copy link

Tobias4Real commented Oct 17, 2024

Steps to reproduce

  1. Install the latest mail app and the "OpenAI and LocalAI integrations" app from the app store
  2. Set OpenAI or an AI provider of you own choosing as the AI provider in the settings, enter the API provider details and select a model like 'gpt-4o-mini'.
    Set the the text completion endpoint to 'Chat completions'.

Enable text processing through LLMs in groupware administrator settings.
3. Open a mail containing text

Expected behavior

Smart replies will be generated and shown

Actual behavior

No smart replies are shown, but thread summaries do work.

Mail app version

4.0.1

Mailserver or service

private

Operating system

Fedora Linux 40 (Workstation Edition)

PHP engine version

PHP 8.3

Web server

Nginx

Database

PostgreSQL

Additional info

I get the following error in the javascript console:

TypeError: right-hand side of 'in' should be an object, got undefined convertAxiosError convert.js:30 smartReply AiIntergrationsService.js:37 fetchMessage ThreadEnvelope.vue:394 mounted ThreadEnvelope.vue:315 VueJS 18 addEnvelopes mutations.js:301 addEnvelopes mutations.js:295 wrappedMutationHandler vuex.esm.js:844 commitIterator vuex.esm.js:466 commit vuex.esm.js:465 _withCommit vuex.esm.js:624 commit vuex.esm.js:464 boundCommit vuex.esm.js:409 fetchEnvelopes actions.js:646 vue.runtime.esm.js:3065

And the following error in the nextcloud log:
Smart reply failed: Failed to decode smart replies JSON output

The reason that this happens is that the LLM wraps the json it responds with in a code fence:

"```json


The function AIIntegrationsService::getSmartReply should check for it / prune it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant