Conversation
ADD-SP
left a comment
ADD-SP
left a comment
There was a problem hiding this comment.
Emm, it looks like OpenClaw will remove the antigravity oauth support soon.
Just hide the antigravity oauth, or do you want to remove it? |
|
remove antigravity support |
ADD-SP
left a comment
ADD-SP
left a comment
There was a problem hiding this comment.
Sorry for the late reply, I have wrote down some review comments.
| /// ``` | ||
| /// | ||
| /// Returns `Some(line)` for data events, `None` for events to suppress. | ||
| pub fn translate_gemini_sse_line( |
There was a problem hiding this comment.
This document appears to be intended solely for coding agent implementing this feature, as it contains extensive details and becomes less relevant after merging this PR. Please delete it. Otherwise, we must maintain consistency between the documentation and the code.
There was a problem hiding this comment.
This document appears to be intended solely for coding agent implementing this feature, as it contains extensive details and becomes less relevant after merging this PR. Please delete it. Otherwise, we must maintain consistency between the documentation and the code.
| state.dlp_scanner.clone(), | ||
| ); | ||
| Response::from_parts(parts, dlp_body) | ||
| } |
There was a problem hiding this comment.
This enables DLP for streaming response, just double-check, is this intentional?
There was a problem hiding this comment.
Emm, there is a highly complicated docker setup guide in the official OpenClaw docs, is this simple Dockerfile good enough?
|
close this PR and part of the contribution has been moved to #86. Thanks! @u20024804 @ADD-SP |
3 participants
The new architecture and flow docs in the docs folder