Skip to content

Commit

Permalink
✨ Mess around with SSE sizes and tool result sizes
Browse files Browse the repository at this point in the history
  • Loading branch information
wesen committed Jan 27, 2025
1 parent 7fb4a0d commit de017d3
Show file tree
Hide file tree
Showing 5 changed files with 59 additions and 4 deletions.
40 changes: 40 additions & 0 deletions cmd/tools/simplify-html/examples/filter-pubmed.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,40 @@
selectors:
- type: css
mode: filter
selector: "div[role='banner']"
- type: css
mode: filter
selector: ".usa-banner"
- type: css
mode: filter
selector: ".ncbi-header"
- type: css
mode: filter
selector: "[role='navigation']"
- type: css
mode: filter
selector: ".search-form"
- type: css
mode: filter
selector: ".multiple-results-actions"
- type: css
mode: filter
selector: ".action-panel"
- type: css
mode: filter
selector: ".search-options"
- type: css
mode: filter
selector: ".sort-dropdown"
- type: css
mode: filter
selector: ".display-options"
- type: css
mode: filter
selector: ".inner-wrap .static-filters"
- type: css
mode: filter
selector: "footer"
- type: css
mode: filter
selector: ".results-amount-container"
6 changes: 5 additions & 1 deletion pkg/client/sse.go
Original file line number Diff line number Diff line change
Expand Up @@ -106,6 +106,7 @@ func (t *SSETransport) Send(ctx context.Context, request *protocol.Request) (*pr
return nil, fmt.Errorf("server error: %s", string(event.Data))
}

t.logger.Debug().Int("dataLength", len(event.Data)).Msg("FOOBAR")
t.logger.Debug().
Str("event", string(event.Event)).
RawJSON("data", event.Data).
Expand Down Expand Up @@ -142,7 +143,10 @@ func (t *SSETransport) initializeSSE(ctx context.Context) error {
err := t.sseClient.SubscribeWithContext(subCtx, "", func(msg *sse.Event) {
t.logger.Debug().
Str("event", string(msg.Event)).
RawJSON("data", msg.Data).
Str("data", string(msg.Data)).
Str("retry", string(msg.Retry)).
Str("id", string(msg.ID)).
Str("comment", string(msg.Comment)).
Msg("Received SSE event")

// Handle endpoint event
Expand Down
7 changes: 6 additions & 1 deletion pkg/client/stdio.go
Original file line number Diff line number Diff line change
Expand Up @@ -26,8 +26,13 @@ type StdioTransport struct {

// NewStdioTransport creates a new stdio transport
func NewStdioTransport(logger zerolog.Logger) *StdioTransport {
scanner := bufio.NewScanner(os.Stdin)
// Set 1MB buffer size to avoid "token too long" errors
buf := make([]byte, 1024*1024)
scanner.Buffer(buf, len(buf))

return &StdioTransport{
scanner: bufio.NewScanner(os.Stdin),
scanner: scanner,
writer: json.NewEncoder(os.Stdout),
logger: logger,
}
Expand Down
8 changes: 7 additions & 1 deletion pkg/cmds/shell-tool-provider.go
Original file line number Diff line number Diff line change
Expand Up @@ -150,7 +150,13 @@ func (p *ShellToolProvider) CallTool(ctx context.Context, name string, arguments
return protocol.NewErrorToolResult(protocol.NewTextContent(err.Error())), nil
}

result := protocol.NewToolResult(protocol.WithText(buf.String()))
text := buf.String()
l := 62 * 1024
if len(text) > l {
text = text[:l]
}

result := protocol.NewToolResult(protocol.WithText(text))

if p.tracingDir != "" {
timestamp := time.Now().Format("2006-01-02T15-04-05.000")
Expand Down
2 changes: 1 addition & 1 deletion pkg/server/transports/stdio/sse_bridge.go
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ func NewSSEBridgeServer(logger zerolog.Logger, sseURL string) *SSEBridgeServer {
}

// Create a new logger that writes to the tagged stderr
taggedLogger := logger.Output(consoleWriter)
taggedLogger := logger.Output(consoleWriter).With().Caller().Logger()

// Strip trailing slashes from the SSE URL
sseURL = strings.TrimRight(sseURL, "/")
Expand Down

0 comments on commit de017d3

Please sign in to comment.