feat(launcher): add console and web chat launchers - #19
Merged
Merged
Conversation
uterflec
added
enhancement
uterflec
marked this pull request as ready for review
September 22, 2026 06:45
ktsoator
reviewed
Sep 22, 2026
Add session management and streaming agent execution. Expose model names and define idempotent session deletion.
Add session navigation, streamed responses, and tool output. Embed frontend assets and exclude web files from language statistics.
Add shared configuration and console and web entry points. Include an assistant example using the bundled interface.
Allow runs without prechecking tool history completeness. Remove matching UI restrictions and tests, and rebuild web assets.
…getSession Use explicit field names in sessionSummary and sessionSnapshot literals for clarity and backward compatibility. Move session store lookup outside s.mu in getSession so storage I/O does not block concurrent server operations.
Drop the else branch and return early to follow Go guard clause conventions.
Print a notice instead of returning an error when generation ends with a non-standard stop reason, preventing the interactive console from abruptly terminating.
ktsoator
force-pushed
the
feat/web-launcher
branch
from
September 22, 2026 07:37
b8a4952 to
4649917
Compare
Extract hardcoded request limits, write timeouts, error codes, and SSE event names into centralized constants in http.go. Define a typed RunStatus for agent run terminal outcomes to avoid raw string comparisons.
Collaborator
|
Consolidated the scattered protocol constants, limits ( |
Check candidate session IDs against the listed sessions returned by the backend before attempting to sync. If the remembered session is stale (such as when an in-memory backend is restarted), silently clear the remembered ID and reset to a clean new session instead of triggering a false-positive 404 error banner. Also rebuild frontend distribution assets.
Collaborator
|
Fixed a frontend startup issue where restarting the in-memory server left a stale session ID in |
ktsoator
self-requested a review
September 23, 2026 01:25
ktsoator
requested changes
Sep 23, 2026
Collaborator
There was a problem hiding this comment.
Hi, @uterflec :)
Two correctness issues were reproduced in the session snapshot and SSE write-deadline handling. Please fix these and add regression coverage before merging; the existing passing Go checks do not cover these cases.
uterflec
marked this pull request as draft
September 23, 2026 07:31
Read history and running state under the same lock. Add regression coverage for concurrent run completion.
Clear deadlines after successful flushes to allow idle streams. Add HTTP/2 coverage for idle connections and event gaps.
uterflec
marked this pull request as ready for review
September 23, 2026 07:48
ktsoator
approved these changes
Sep 23, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add console and web launchers for configured agents, backed by HTTP session management and SSE streaming. Bundle a browser interface for text, thinking, tool calls, and conversation history, with an assistant example.
Validation:
go test ./...;npm run buildinweb.