Fix streaming drafts, UTC timestamps and permission review - #20
Merged
Merged
Conversation
This was referenced Oct 2, 2026
rcarmo
marked this pull request as ready for review
October 2, 2026 18:22
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.
Fix streaming drafts, UTC timestamps and permission review
PR: #20
Base:
mainHead:
review/ui-streaming-permissionsFirst PR of the published stack: #20 UI → #21 Windows → #22 FFI. Neither child is required to review this UI diff. CI/review follow-up is handed to the maintainer; no automatic merge is enabled.
Changes
CURRENT_TIMESTAMPstrings as UTC. Keep explicit timezone offsets, periodically update ages and show absolute local time on hover.docs/BROWSER_REVIEW.mdand link it from the README.No Copilot SDK dependency, native adapter, token handling, upload-ownership change or LAN deployment wrapper is included. Copilot-shaped display normalization is passive; it can render any matching request payload without starting an adapter.
Verification
f382691.fcntlstartup limitation. An attempted system-metrics app smoke failed at that unchanged POSIX import; it is addressed by the separate Windows PR. This PR is not labelled full-Windows-suite green.Review focus
draft-stream.js,timestamps.js,agent-request-details.js,permission-clipboard.js,status.jsand the scoped style overrides instyles.csscontain the maintained code. Generated JS/CSS/maps and the HTML asset hash were rebuilt for this branch. Approval callbacks and IDs are unchanged; tests never approve real user requests.