From a7e33ca32fbca402e34ec325939cdc2eb1ae9efc Mon Sep 17 00:00:00 2001 From: Eden Zimbelman Date: Tue, 1 Sep 2026 00:39:49 -0700 Subject: [PATCH] refactor(block-kit): inline BlockCompositions calls in actions example MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The actions block example mixed a static import of plainText with fully-qualified BlockCompositions.plainText calls in the same file. Because these examples are surfaced in docs.slack.dev as focused snippets (only the exampleNN() body is shown — the imports are not visible to the reader), a bare plainText(...) reads as an unattributed free function. Drop the static import and qualify the remaining calls so every helper in the surfaced range names its class (BlockCompositions.plainText, BlockElements.button, Blocks.actions) — self-contained and consistent. Co-Authored-By: Claude --- block-kit/src/main/java/blocks/Actions.java | 20 +++++++++----------- 1 file changed, 9 insertions(+), 11 deletions(-) diff --git a/block-kit/src/main/java/blocks/Actions.java b/block-kit/src/main/java/blocks/Actions.java index 7b08368..5fc7323 100644 --- a/block-kit/src/main/java/blocks/Actions.java +++ b/block-kit/src/main/java/blocks/Actions.java @@ -1,7 +1,5 @@ package blocks; -import static com.slack.api.model.block.composition.BlockCompositions.plainText; - import com.slack.api.model.block.ActionsBlock; import com.slack.api.model.block.Blocks; import com.slack.api.model.block.composition.BlockCompositions; @@ -24,19 +22,19 @@ public static ActionsBlock example01() { .placeholder(BlockCompositions.plainText("Which witch is the witchiest witch?")) .options(List.of( OptionObject.builder() - .text(plainText("Matilda")) + .text(BlockCompositions.plainText("Matilda")) .value("matilda") .build(), OptionObject.builder() - .text(plainText("Glinda")) + .text(BlockCompositions.plainText("Glinda")) .value("glinda") .build(), OptionObject.builder() - .text(plainText("Granny Weatherwax")) + .text(BlockCompositions.plainText("Granny Weatherwax")) .value("grannyWeatherwax") .build(), OptionObject.builder() - .text(plainText("Hermione")) + .text(BlockCompositions.plainText("Hermione")) .value("hermione") .build()))), BlockElements.button(b -> b.text(BlockCompositions.plainText("Cancel")) @@ -57,23 +55,23 @@ public static ActionsBlock example02() { BlockElements.overflowMenu(o -> o.actionId("overflow") .options(List.of( OptionObject.builder() - .text(plainText("*this is plain_text text*")) + .text(BlockCompositions.plainText("*this is plain_text text*")) .value("value-0") .build(), OptionObject.builder() - .text(plainText("*this is plain_text text*")) + .text(BlockCompositions.plainText("*this is plain_text text*")) .value("value-1") .build(), OptionObject.builder() - .text(plainText("*this is plain_text text*")) + .text(BlockCompositions.plainText("*this is plain_text text*")) .value("value-2") .build(), OptionObject.builder() - .text(plainText("*this is plain_text text*")) + .text(BlockCompositions.plainText("*this is plain_text text*")) .value("value-3") .build(), OptionObject.builder() - .text(plainText("*this is plain_text text*")) + .text(BlockCompositions.plainText("*this is plain_text text*")) .value("value-4") .build()))), BlockElements.button(b -> b.text(BlockCompositions.plainText("Click Me"))