fix #11026 model config return

[kenwoodjw]

Summary

fix #11026 model config return

Tip

Close issue syntax: Fixes #<issue number> or Resolves #<issue number>, see documentation for more details.

Screenshots

Before:	After:
...	...

Checklist

Important

Please review the checklist below before submitting your pull request.

• This change requires a documentation update, included: Dify Document
• I understand that this PR may be closed in case there was no previous discussion or issues. (This doesn't apply to typos!)
• I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change.
• I've updated the documentation accordingly.
• I ran dev/reformat(backend) and cd web && npx lint-staged(frontend) to appease the lint gods

[crazywoola]

Please run dev/reformat to pass the lint.

[laipz8200]

The property is already present in the 'model' object when we call the 'to_dict()' method. It appears that this issue cannot be resolved.

[kenwoodjw]

I understand the issue now, as model_config corresponds to model_config_fields, not simple_model_config_fields. Perhaps this approach would work:

simple_model_config_fields = {
    "model": fields.Raw(attribute="model"),
    "pre_prompt": fields.String,
}

What do you think？

[laipz8200]

In fact, we are currently losing the parameters in the completion_params. As you can see, this field only contains a 'stop' now, but the model parameters such as 'temperature' and 'top_p' should be contained in here.

[kenwoodjw]

@laipz8200 So the mode_config should return like that

"model_config": {
                "model": {
                    "provider": "deepseek",
                    "name": "deepseek-chat",
                    "mode": "chat",
                    "completion_params": {
                        "stop": [],
                        "top_p": [],
                        "temperature": []
                    }
                },
                "pre_prompt": ""
            }

or simple ?

[laipz8200]

@laipz8200 So the mode_config should return like that

"model_config": {
                "model": {
                    "provider": "deepseek",
                    "name": "deepseek-chat",
                    "mode": "chat",
                    "completion_params": {
                        "stop": [],
                        "top_p": [],
                        "temperature": []
                    }
                },
                "pre_prompt": ""
            }

or simple ?

Yes, here is the correct structure of completion_params(from the front-end):

[kenwoodjw]

After my test, I opened the parameter options.

Returning the results we desire.

[laipz8200]

We found that is better to fix it at the frond-end, would you mind checking #11298?

[kenwoodjw]

We found that is better to fix it at the frond-end, would you mind checking #11298?

Ok, I will close the PR