feat(tui): show streaming output progress - #930
Open
huangzesen wants to merge 3 commits into
Open
Conversation
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
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.
Summary
lingtai.stream-progress/v1RAM snapshot over read-only loopback discoveryActive N s: exact chars below 1000, one-decimalk charsat or above 1000, plusdelay D.Dssnapshot freshnessView()reads cached RAM state onlymanifest.streamingfield from generated/example configs and/kanban; the kernel System resolver now owns the switchDependency
Requires the kernel API in Lingtai-AI/lingtai-kernel#1518.
Contract
Amendment — init no longer owns streaming
GenerateInitJSON, canonical/example templates, and the playground writer emit nomanifest.streamingkey/kanbanno longer shows an init-derived streaming row; the TUI adds no System-setting editorLINGTAI_STREAMING_ENABLED> optional booleansettings/system.json.streaming> onmax(now - updated_unix_ms, 0)freshness with no token conversionBehavior change / migration
Earlier TUI templates emitted
manifest.streaming: false. The paired kernel now retires and ignores that field with its existing unknown-field warning/Nudge path and performs no migration or init rewrite. Those agents stream by default after upgrade unless the operator setsLINGTAI_STREAMING_ENABLED=0(or another valid false form) or"streaming": falsein strictsettings/system.json; remove the stale init key to clear the warning.Validation
997aa3e4:go test ./i18n ./internal/streamprogress ./internal/tui -count=1— pass997aa3e4:go build ./...— pass997aa3e4:go vet ./...— passgofmt -l— cleangit diff --check— clean7c02b7cd(new basefce5c882+ reviewed head94c841bc) —go test ./i18n ./internal/streamprogress ./internal/tui,go build ./..., andgo vet ./...all passgo test -cforinternal/streamprogressandinternal/tui, plusgo build ./...— cross-compiles (not a native runtime claim)active=true,4000 chars; then the same generationactive=false,0 chars; worktree status unchangedgo testfori18n,internal/preset,internal/tui, andinternal/streamprogresspasses; matchinggo vet,gofmt, andgit diff --checkare cleantui.diffSHA-25675290125…) — PASS, no P0/P1 blockersEstimatedTokens/chars / 4/ old i18n-key scan,gofmt, andgit diff --check— clean/opt/homebrew/bin/lingtai-tuiand/opt/homebrew/bin/lingtaireportlingtai-tui dev-pr930-997aa3e4, binary SHA-2569af671c1…9f8e57d0: generation 93active=true,53 chars, snapshot freshness 7 ms (53 chars · delay 0.0s)Whole-suite disclosure:
go test ./...has one root-package architecture-document failure listing these three paths:tui/internal/preset/skills/swiss-knife/reference/academic-research/reference/api-semantic-scholar-batch.mdtui/internal/tui/app_cpr_test.gotui/internal/tui/lock_windows_test.goA corrected exact-base temporary Git-index control at
e8bf4df9reproduces the identical failing set. Other packages passed. This PR does not claim a fully green whole suite.Paired kernel disclosure: the focused kernel matrix is 170 passed / 2 failures reproduced on exact base. Full kernel pytest is not green evidence; it is baseline-noisy and aborts around 25% from the existing macOS
WindowsPathmonkeypatch contamination, with the fail-fast first failure reproduced on exact base.Remaining platform gates — do not merge yet
Telegram: @lingtaidev1bot | Nickname: 知微
Powered by LingTai AI: https://github.com/Lingtai-AI/lingtai