[PB-6281]: add DocumentsProvider write/upload pipeline#469
Conversation
terrerox
commented
May 28, 2026
terrerox
commented
- enable write mode in openDocument using pending upload tokens
- implement createDocument flow with unique-name resolution and stale token cleanup
- add pipe-to-temp encryption + upload orchestration (single and multipart)
- finish bridge upload and create drive file entry with metadata timestamps
- refactor decrypt callback handling via awaitCryptoService and shared auth error constant
terrerox
force-pushed
the
feature/PB-6281-upload-file
branch
from
5b81068 to
19346e5
Compare
terrerox
force-pushed
the
feature/PB-6281-upload-file
branch
2 times, most recently
from
5b81068 to
abd0357
Compare
terrerox
force-pushed
the
feature/PB-6281-documents-provider-write
branch
from
68bdcde to
90eaa8a
Compare
- enable write mode in openDocument using pending upload tokens - implement createDocument flow with unique-name resolution and stale token cleanup - add pipe-to-temp encryption + upload orchestration (single and multipart) - finish bridge upload and create drive file entry with metadata timestamps - refactor decrypt callback handling via awaitCryptoService and shared auth error constant
terrerox
force-pushed
the
feature/PB-6281-documents-provider-write
branch
from
90eaa8a to
a227a68
Compare
|
CandelR
left a comment
CandelR
left a comment
There was a problem hiding this comment.
File is at 858 lines now. Might be worth splitting before it gets harder to navigate:
-
FolderListingCache:FolderLoad,folderLoads, and the cache/notification helpers (startBackgroundLoad,appendRows,finishLoad,patchRowDisplayName,removeRow,invalidateChildren,notifyChildren). -
upload/DocumentUploadOrchestrator: the full write side: pending upload registry, name resolution, and the pipeline (openForWrite->runUpload->encryptInputToTemp->uploadEncryptedFile->finalizeAndRecordFile). Provider just callscreateDocument,openForWrite,queryPendingUpload. -
download/DocumentDownloadOrchestrator:openDocumentBlocking,materializeIntoCache,openCached,decryptBlocking. Nothing fancy, already pretty self-contained.
Provider ends up around 200 lines, just the SAF contract and wiring. Shared utils (apiClient, streamPages, mutate, rawUuid) can stay, not worth the extra hop.
The proposal no blocks this PR at all, just flagging it for a follow-up.
2 participants
