Repository navigation
feat(logs): Export logs, click to copy, and a clearer log table - #567
Open
Tekunalogy wants to merge 3 commits into
Open
Tekunalogy wants to merge 3 commits into
Tekunalogy wants to merge 3 commits into
Conversation
|
Preview build: curl -s https://raw.githubusercontent.com/DeepWaterExploration/dweOS/main/install.sh | sudo bash -s preview-pr-567Rebuilt on every push. Closing this pull request deletes the preview. |
Tekunalogy
changed the title
Tekunalogy
marked this pull request as ready for review
October 15, 2026 04:13
Tekunalogy
force-pushed
the
feat/log-export
branch
from
October 15, 2026 04:13
b6b05e1 to
2c3ec3a
Compare
An Export button next to refresh downloads every log the backend holds as
dweos-logs-<time>.csv, with date, time, level, logger, file, line,
function and message columns, RFC 4180 quoting, and a BOM so Excel reads
it as UTF-8.
The shared helpers in lib/util/logs.ts also cover copying a log, which
falls back to execCommand("copy") because dweOS is usually opened over
plain HTTP, where navigator.clipboard isn't available.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Clicking anywhere on a row copies the whole log, with the message on its own line after the timestamp, level, logger and source. The row uses a pointer cursor and briefly highlights as feedback. The log detail dialog is removed: it was 30% opaque over the table and mostly repeated the row. The tour's logs step now explains the click to copy. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- Separate Date and Time (with milliseconds) columns. - The date, time and level columns are sized to their fixed-width contents, and level badges share one width, so the gaps line up on every row. - The message column takes the remaining width instead of being capped at 150px; a message cut off by the two-line clamp shows the full text in a callout on hover. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Tekunalogy
force-pushed
the
feat/log-export
branch
from
October 15, 2026 04:18
2c3ec3a to
f813390
Compare
Member
Long messages are truncated without means to view besides a zoom-out or copy-paste. This is why there was a dialog box before. |
Member
|
Looks like I missed the hover-over -> callout for viewing longer messages. Seems acceptable |
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.
What
dweos-logs-<local time>.csv, with columns Date, Time, Level, Logger, File, Line, Function, Message (RFC 4180 quoting, UTF-8 BOM for Excel)<timestamp> <LEVEL> [<logger>] <file>:<line> <function>():with the message on the next line, with a toast and a brief row highlight. dweOS is usually opened over plain HTTP, wherenavigator.clipboarddoesn't exist, so it falls back toexecCommand("copy")like the terminal does.No backend changes: the frontend already loads every log from
/api/logs. "All logs" covers logs since the last dweOS start, not the rotated log files on disk.Testing
tsc -b,npm run build🤖 Generated with Claude Code