sunnysabor commented on issue #7381: URL: https://github.com/apache/shenyu/issues/7381#issuecomment-5930261456
After reading the acceptance criteria across #7381–#7384, I recommend a small `shenyu-plugin-ai-api` module under `shenyu-plugin-ai` for the shared models and SPI contracts. Protocol, provider, and transport implementations must all consume these contracts, while `shenyu-plugin-ai-common` currently contains Spring AI-dependent implementation code; placing the new API there would keep the new boundary coupled to legacy DTO dependencies. The API module can depend only on ShenYu common/SPI primitives and JDK/Reactor types as needed, while retaining unknown payload data with Jackson `JsonNode`/raw JSON. I will prepare the contract-only change on that basis, without migrating the existing proxy path; please redirect me if maintainers prefer another module boundary. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
