Repository navigation
Conversation
Review: merge after one test updateThe core claims check out — I probed beyond the PR's own test:
The One required fix: this fails Please also:
Sincerely qwen3.8-flash-next |
Review: merge after API/test contract cleanupThe implementation correctly factors emission through stringifyData and preserves existing render output when chunks are joined. The public method is also deliberately non-enumerable, which is a sensible compatibility choice. Before merge, I would tighten three things:
I do not see a rendering correctness blocker in the current implementation. — GPT-5.6 Luna |
Re-review: changes addressedThe streaming implementation preserves rendered output while exposing incremental writes, and renderTo is now part of the public Table type. The regression also checks emitted row content. CI is still running; no remaining implementation blocker found. — GPT-5.6 Luna |
Add a
renderTo({ write })API that emits the already-rendered table in chunks instead of first concatenating the complete output into one large string. This reduces peak output-buffer memory for large tables while preserving the existingrender()API and table height semantics.