Skip to content

File edit differences to the Server Activity Log - #105

Merged
macery12 merged 2 commits into
macery12:developfrom
bimbab189:develop
Feb 3, 2026
Merged

macery12 merged 2 commits into
macery12:developfrom
bimbab189:develop

Conversation

@bimbab189

Copy link
Copy Markdown
Collaborator

This pull request improves the Server Activity Log by displaying detailed, review-friendly differences whenever a server owner or subuser edits a text-based file through the web editor. Each activity entry now records the exact file path and presents a structured summary of added and removed lines, with an expandable pull request style view of the changes.

Key Changes

Introduced a dedicated file write endpoint with change tracking ([/files/write-with-diff] to capture sufficient context for calculating differences between the original and updated content.
Implemented server-side calculation of file content differences for text files without relying on development-only dependencies, ensuring production compatibility.
Updated the client file editor save flow to send both the original content and the updated content, enabling accurate calculation of differences.
Enhanced the Activity Log interface to display line change statistics and a collapsible, hunk-based view of the differences for [server:file.write] events.
Added performance safeguards for large files by recording only summary information when detailed calculation is not practical.

Compatibility Notes

The existing [/files/write] endpoint remains unchanged and continues to be used as a fallback when original content is not provided.
Change tracking is currently intended for edits performed through the web editor; other write paths continue to operate normally without generating detailed differences.

Please review and make changes. I would love for this idea to be fully integrated.

@macery12

macery12 commented Feb 3, 2026

Copy link
Copy Markdown
Owner

very well done, will impliment into v1.1.5

@macery12
macery12 changed the base branch from develop to release/v4.0.0-rc4 February 3, 2026 02:18
@macery12
macery12 changed the base branch from release/v4.0.0-rc4 to develop February 3, 2026 02:19
@macery12
macery12 merged commit 8724a4f into macery12:develop Feb 3, 2026
3 of 15 checks passed
macery12 added a commit that referenced this pull request Feb 3, 2026
File edit differences to the Server Activity Log
macery12 added a commit that referenced this pull request Feb 3, 2026
File edit differences to the Server Activity Log
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants