Skip to content

feat(storage): operator 2026-10-24 UI batch — sunburst, move-apply, zero margins - #153

Open
ywatanabe1989 wants to merge 14 commits into
developfrom
feature/sac-scitex-storage-5c2a6e59
Open

ywatanabe1989 wants to merge 14 commits into
developfrom
feature/sac-scitex-storage-5c2a6e59

Conversation

@ywatanabe1989

Copy link
Copy Markdown
Member

Operator 2026-10-24 feedback batch (storage-ui-feedback-20261023): picker-aware no-project notice; zero margins on largest div (screenshot-verified); interactive usage sunburst replaces static doughnuts + project-scoped disk display (new usage-sunburst route); duplicates as an app (would-scan scope); browser move-apply with explicit confirm + can_edit gate (fake-transport verified-delete test). Full suite 1396 passed/9 skipped incl. audit-all. Screenshots on the card.

…23 feedback)

Standalone renders no header picker, so directing users to pick
a project in the header is a dead-end instruction there. The notice
now names the hub-mounted requirement without a provider, keeping
the pick-in-header direction where the picker can exist.

Tests: test_views 59 passed.
…sert, PS-233 guarded django import)

Pristine 82104de audits clean; the slice added 3 findings (verified
against a scratch worktree at 82104de, since removed). No new behavior.
Content fills the pane edge to edge; live dock clearance kept.
Screenshot-verified standalone (dark): header, tabs, cards flush
to viewport edges.
…ator 2026-10-24)

Usage tab shows a sunburst link + project-only bytes when scoped;
per-folder breakdown lives in the zoomable sunburst page (new
usage-sunburst route, depth-capped, top-N + Other rollup, fd-missing
and timeout render friendly notes). Volume cards hidden under project
scope. Screenshot-verified standalone (dark).
…operator 2026-10-24)

POST move-apply: explicit confirm checkbox, optional content
re-hash, project scope requires edit rights, transport pre-flight,
bounded plan+apply. Plan tab keeps read-only planner; the disabled
notice is replaced by the apply form. 7 new view tests incl. a
fake-transport verified-delete success path.
…6-10-24)

Safe move = rsync + checksum validate, then stage the source aside
(.stx-restorable sibling, atomic rename) instead of deleting; sweeper
deletes after 7-day retention (sweep-restorable CLI verb, dry-run by
default). Manifest records staged_path. Browser apply retires; CLI
keeps direct delete. 19 new tests (staging/sweep guards/marker
checks, retire paths, CLI verb).
…2026-10-24)

apply_archive retire_source stages verified sources restorable
(atomic sibling rename, registry, guarded sweeper) instead of
deleting; manifest records staged_path; report names the staging.
CLI: sweep-restorable verb (dry-run default, --yes deletes expired)
+ archive --retire-source. Browser apply retires. Docs/skills/README
updated. 22 new tests.

This branch has not been deployed

No deployments
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.

1 participant