Repository navigation
feat: keep server bytes, save given bytes as-is - #803
Merged
Merged
Conversation
A checkout without a generated caldav/_version.py (any fresh git worktree) also emits the "install the build package" UserWarning, so test_import_warns saw two warnings and failed. Prompt: (...) We should fix test breakages and continue [merging prs] [the jmap test failed in fresh worktrees used for the merge train] Assisted-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Reviewed-by: Tobias Brox <tobias@redpill-linpro.com>
A calendar object now keeps the calendar data the server sent, before vcal.fix(), in the read-only raw_data attribute: the GET body byte for byte after load(), the calendar-data text after a REPORT. It is a snapshot of the last read, cleared by save(). save(raw_data=...) PUTs the given UTF-8 bytes unchanged, with the usual If-Match / Schedule-Tag preconditions, no_overwrite / no_create wildcards and no SEQUENCE bump; the bytes win over local edits. Sync tools need this to compare and copy what servers store. Fixes #763 Prompt: yes, but check that no other session is working on #763 [answer to whether a subagent should implement #763, keeping the server's bytes and sending given bytes unchanged, for caldav 3.6] Followup-Prompt: Also continue with the subagents [resume this work after an interruption] Assisted-By: Claude Opus 5.5 (1M context) Reviewed-by: Tobias Brox <tobias@redpill-linpro.com>
tobixen
force-pushed
the
feat/raw-bytes
branch
from
October 19, 2026 23:37
71ecadb to
6ebba9d
Compare
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
1 participant
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.
Stacked on #802 - only the last commit belongs to this PR; the two before it are 802's.
Commit message: