Skip to content

refactor(tui): Forward block_storage to gen ResourceBehaviour - #1977

Merged
gtema merged 1 commit into
mainfrom
tui-thin-block-storage
Sep 3, 2026
Merged

refactor(tui): Forward block_storage to gen ResourceBehaviour#1977
gtema merged 1 commit into
mainfrom
tui-thin-block-storage

Conversation

@gtema

@gtema gtema commented Sep 3, 2026

Copy link
Copy Markdown
Owner

PR #1975 added the generated mechanical tier for the block_storage
resources. Thin the hand-written ResourceBehaviour impls to one-line
forwarders onto super::generated::<r>::Generated::<method>, keeping
only custom logic:

  • volumes.rs: confirm_request delete prompt plus its
    TryFrom<&serde_json::Value> for the delete request builder.
  • backups.rs / snapshots.rs: pure forwarders; their view_key()
    now resolves through crate::mode::BLOCK_STORAGE_* via the companion
    instead of a bare string literal.

All three resources use the list_detailed operation, so the generated
tier resolves the ListDetailed request variant, matching the prior
hand code.

Assisted-By: Claude Sonnet 5 noreply@anthropic.com
Signed-off-by: Artem Goncharov artem.goncharov@gmail.com

PR #1975 added the generated mechanical tier for the block_storage
resources. Thin the hand-written `ResourceBehaviour` impls to one-line
forwarders onto `super::generated::<r>::Generated::<method>`, keeping
only custom logic:

- `volumes.rs`: `confirm_request` delete prompt plus its
  `TryFrom<&serde_json::Value>` for the delete request builder.
- `backups.rs` / `snapshots.rs`: pure forwarders; their `view_key()`
  now resolves through `crate::mode::BLOCK_STORAGE_*` via the companion
  instead of a bare string literal.

All three resources use the `list_detailed` operation, so the generated
tier resolves the `ListDetailed` request variant, matching the prior
hand code.

Assisted-By: Claude Sonnet 5 <noreply@anthropic.com>
Signed-off-by: Artem Goncharov <artem.goncharov@gmail.com>
@gtema
gtema force-pushed the tui-thin-block-storage branch from 8bfc33d to 9f8aaa1 Compare September 3, 2026 16:16
@gtema
gtema enabled auto-merge (squash) September 3, 2026 16:16
@gtema
gtema merged commit f81179a into main Sep 3, 2026
15 checks passed
@gtema-release-plz gtema-release-plz Bot mentioned this pull request Sep 3, 2026
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