Skip to content

docs: add Block Kit type instructions for slack-api-model - #1623

Merged
srtaalej merged 2 commits into
mainfrom
ale-add-block-kit-intructions
Jul 28, 2026
Merged

docs: add Block Kit type instructions for slack-api-model#1623
srtaalej merged 2 commits into
mainfrom
ale-add-block-kit-intructions

Conversation

@srtaalej

Copy link
Copy Markdown
Contributor

Add detailed agent instructions for adding new Block Kit types (layout blocks, elements, composition objects) to the slack-api-model module. Moves Block Kit guidance out of root AGENTS.md into a dedicated slack-api-model/AGENTS.md.

Extract Block Kit-specific agent instructions from root AGENTS.md into
a dedicated slack-api-model/AGENTS.md with detailed steps for adding
new layout blocks, elements, and composition objects.
@srtaalej
srtaalej requested a review from a team as a code owner July 21, 2026 21:38
@srtaalej srtaalej self-assigned this Jul 21, 2026
@srtaalej srtaalej added enhancement M-T: A feature request for new functionality docs M-T: Documentation work only semver:minor labels Jul 21, 2026
@codecov

codecov Bot commented Jul 21, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.37%. Comparing base (5998ea6) to head (ddc4373).

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #1623      +/-   ##
============================================
+ Coverage     73.32%   73.37%   +0.04%     
- Complexity     4520     4523       +3     
============================================
  Files           478      478              
  Lines         14300    14300              
  Branches       1490     1490              
============================================
+ Hits          10486    10492       +6     
+ Misses         2923     2918       -5     
+ Partials        891      890       -1     
Flag Coverage Δ
jdk-14 73.37% <ø> (+0.04%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

@WilliamBergamin WilliamBergamin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM not blocking comments 🚀

Comment thread slack-api-model/AGENTS.md

File: `slack-api-model-kotlin-extension/src/main/kotlin/com/slack/api/model/kotlin_extension/block/{Name}BlockBuilder.kt`

```kotlin

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Praise for kotlin

Comment thread slack-api-model/AGENTS.md
├── event/ # Slack event type definitions
├── view/ # Modal and Home tab view types
└── ... # Other model types (File, Message, User, etc.)
```

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not a biggest fan of these ASCII trees, I find they duplicate what the project structure already states 🤔

@srtaalej
srtaalej merged commit da4ba33 into main Jul 28, 2026
7 checks passed
@srtaalej
srtaalej deleted the ale-add-block-kit-intructions branch July 28, 2026 18:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs M-T: Documentation work only enhancement M-T: A feature request for new functionality semver:minor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants