SEZ9 commented on issue #12999: URL: https://github.com/apache/seatunnel/issues/12999#issuecomment-6433032144
Thanks for the detailed write-up — the three defects and the way they chain together are clearly laid out, and the scope note about this being the CLI's agent orchestration rather than the data engine is helpful. The plan of three stacked PRs, one per defect, sounds reasonable given that the third depends on state introduced by the first. A couple of things that would make review easier: - For the first defect, a regression test that replays a question/answer turn and asserts the original source, sink and table details still appear in the `## User Request` section the generator receives. - For the second defect, a minimal transcript or benchmark preset that shows a request reaching generation with an empty `Connector Option Rules` block and ending in `Failed to generate config.`, so we can confirm the fix ask-for-source-and-destination behaviour against it. Looking forward to the PRs. <!-- streview-comment:2078 --> -- 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]
