New Constructor for PromptChatMemoryAdvisor

[KushParsaniya]

\Overloaded Constructor for PromptChatMemoryAdvisor with Default systemTextAdvise

Summary

Currently, the PromptChatMemoryAdvisor requires passing a systemTextAdvise string when a custom ConversationId is used. To enhance usability, this proposal suggests adding an overloaded constructor to allow users to pass only the ChatMemory, ConversationId, and chatHistoryWindowSize, while the systemTextAdvise defaults to DEFAULT_SYSTEM_TEXT_ADVISE.

Problem

When using PromptChatMemoryAdvisor, developers are required to pass the systemTextAdvise explicitly, even when a default value would suffice. This is inconvenient, especially in cases where the developer is already passing a custom ConversationId and doesn't need to customize the systemTextAdvise.

For example:

@GetMapping("/{prompt}")
public String prompt(@PathVariable String prompt, HttpServletRequest request) {
    return chatClient
            .prompt()
            .advisors(new PromptChatMemoryAdvisor(
                    jdbcChatMemory,
                    request.getHeader("id"),
                    20,
                    """
                    Use the conversation memory from the MEMORY section to provide accurate answers.

                    ---------------------
                    MEMORY:
                    {memory}
                    ---------------------

                    """))
            .user(prompt)
            .call().content();
}