Skip to content

Commit 1e1bb9a

Browse files
committed
SimpleChatTC: update readme wrt promise related trapping
1 parent c155bd3 commit 1e1bb9a

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

tools/server/public_simplechat/readme.md

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -385,16 +385,22 @@ structures also.
385385
DONE: rather both tool_calls structure wrt assistant messages and tool role based tool call
386386
result messages are generated as needed.
387387

388+
#### Related stuff
389+
390+
Promise as well as users of promise (for now fetch) have been trapped wrt their then and catch flow,
391+
so that any scheduled asynchronous code or related async error handling using promise mechanism also
392+
gets executed, before tool calling returns and thus data / error generated by those async code also
393+
get incorporated in result sent to ai engine on the server side.
388394

389395
#### ToDo
390396

391397
WebFetch and Local web proxy/caching server
392398

393-
Try and trap promises based flows to ensure all generated results or errors if any are caught
394-
before responding back to the ai model.
399+
Is the promise land trap deep enough, need to think through and explore around this once later.
395400

396401
Trap error responses.
397402

403+
398404
### Debuging the handshake
399405

400406
When working with llama.cpp server based GenAi/LLM running locally

0 commit comments

Comments
 (0)