[3473232] allow altering via hook, provide model choice per language, add default prompt (with small improvement)

added 1 commit

787999ff - 3473232 and 3466972: Provide support for not choosing a model and taking the default one

Hi @wouters_frederik . Thank you, before I start the MR, could you merge with 1.0.x again. The changes from merging this seems to have caused some merge conflicts: https://www.drupal.org/project/ai/issues/3464024

added 30 commits

787999ff...f1ea5d3e - 20 earlier commits
cda5c012 - Issue #3474233 by scott_euser: Leaving maximum chunk size blank results in fatal errors
93cf91e6 - Fix phpcs/phpstan issues in ai_search and vdb_providers
a89ff375 - Issue #3474256 by marcus_johansson: Fix PHPStan issues
962e897a - Adding basic README and beginning of structure for docs
5aa8ab11 - Issue #3474622 by marcus_johansson: Make AI Assistant Form more user friendly
1aa54981 - Issue #3474662 by marcus_johansson: AI Chatbot is hidden behind Gin Sidebar
4fdcb76c - Issue #3474442 by marcus_johansson: Latest dev build gives haystack error
4c11a735 - Issue #3474981: isZilliz check in MilvusV2 is not correct
530d5a6d - [3473232] added settings form for per language model selection and prompt
516a6aba - 3473232 and 3466972: Provide support for not choosing a model and taking the default one

Compare with previous version

Merged and resolved conflicts.

Thanks @wouters_frederik for the merge, there are some code issues and spelling issues reported, could you fix those, see:

If you need help with this or help setting up testing locally, let me know. For cspell, its based on american english, so sometimes you might have words you don't agree with, just add those to the list in the file .gitlab.ci.yml under _CSPELL_WORDS.

Since we are getting close to a beta release we will run the ship a little bit tighter and make sure all code linters pass for all MR (and I will stop pushing directly to 1.0.x ).

added 1 commit

af28c598 - [3473232] code cleanup

Compare with previous version

added 1 commit

eb491ec2 - 3473232 spelling extravaganza

Compare with previous version

Looks great, getting merged into dev!

approved this merge request

Ah, no - there are merge conflicts? Could you have a look at those?

Should node_modules be git-ignored? if they are listed in package.json, that should be enough (also, I am not sure if it's legally correct to store node_modules in another repository - potential license conflict)

I personally think that there could be much more (by the orders of magnitude, in fact) altering factors than can be handled in a contrib module. If we have single alter hook/event, there would be no need to store all the possible prompt variations in ai_translate module itself, and all the specifics will be done in third party/custom module space

closed

[3473232] allow altering via hook, provide model choice per language, add default prompt (with small improvement)

Merge request reports

Activity