Cordy/Cairn
2
1
Fork 0
Code Issues 20 Pull requests Projects Releases Packages Wiki Activity Actions

QA batch 5b - the activity feed finally sees activity (v0.6.254) #703

Merged
Cordy merged 9 commits from fix-qa5b into main 2026-09-23 20:52:07 +00:00
Conversation 0 Commits 9 Files changed 6 +310 -4
Cordy commented 2026-09-23 20:49:10 +00:00
Owner
Copy link

Fifth batch, second slice: #673.

  • New internal/storage/actrec decorator wraps the scoped store in main — the single point REST, WebDAV, TUS finalize and WOPI saves all flow through with the virtual paths the feed is queried with. Records upload/update (Stat-before-write distinguishes), rename/move (via activity.Kind), copy. Skips internal bookkeeping and userless contexts.
  • The API layer's four own recordings removed (no double counting); share-*/restore/admin-transfer recordings stay (they're API/unscoped-level concepts).
  • The recorder is late-bound in main (the activity store opens after the storage stack is assembled).
  • Named the local verb, with a comment, so the #433 audit scanner's action-assignment carrier doesn't claim activity verbs for the audit registry.

Red-green: internal/storage/actrec/actrec_test.go (upload→update→rename→copy sequence, manifest + userless writes skipped).

Known limit noted in the changelog: per-file queries only, folder rollups out of scope.

Closes #673

Fifth batch, second slice: #673. - New `internal/storage/actrec` decorator wraps the scoped store in main — the single point REST, WebDAV, TUS finalize and WOPI saves all flow through with the virtual paths the feed is queried with. Records upload/update (Stat-before-write distinguishes), rename/move (via `activity.Kind`), copy. Skips internal bookkeeping and userless contexts. - The API layer's four own recordings removed (no double counting); share-*/restore/admin-transfer recordings stay (they're API/unscoped-level concepts). - The recorder is late-bound in main (the activity store opens after the storage stack is assembled). - Named the local `verb`, with a comment, so the #433 audit scanner's action-assignment carrier doesn't claim activity verbs for the audit registry. Red-green: internal/storage/actrec/actrec_test.go (upload→update→rename→copy sequence, manifest + userless writes skipped). Known limit noted in the changelog: per-file queries only, folder rollups out of scope. Closes #673
New actrec storage decorator at the top of the scoped stack records upload/update/rename/move/copy for every surface - REST, WebDAV, TUS finalize and WOPI editor saves - with the same virtual paths the feed is queried with; internal bookkeeping and userless contexts are skipped; the API layer's own four recordings are removed so nothing double-counts. Known limit: the feed answers per-file queries; folder rollups stay out of scope.
changelog: v0.6.254
All checks were successful
ci / test-and-build (pull_request) Successful in 1m8s
Details
e9f9ac2a80
Cordy merged commit 5d5ccec404 into main 2026-09-23 20:52:07 +00:00
Cordy deleted branch fix-qa5b 2026-09-23 20:52:10 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Cordy/Cairn#703
No description provided.
Delete branch "fix-qa5b"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?