chore: add build, chore, security labels and align release notes categories by mwbrooks · Pull Request #73 · slackapi/slack-mcp-plugin

mwbrooks · 2026-07-02T22:45:38Z

Summary

This pull request introduces 3 new labels (chore, build, and security) and fixes the .github/release.yml to not use non-existent labels.

Adds a chore label for maintainer task PRs.
- e.g. repo structure, required files, release scaffolding, general maintenance work - matching the conventional-commit chore: prefix.
Adds a build label for changes to compilation and CI processes
- e.g. build, ci/cd, scripts, etc
Adds a security label so security fixes surface distinctly in release notes.
Aligns .github/release.yml with the labels that actually exist on the repo.
- The prior config referenced four non-existent labels (build, code health, docs, security), so those changelog sections silently fell through to "Other changes."
- The documentation label was renamed to docs to match other slackapi projects.

Preview

  What's Changed

  🚀 Enhancements

  - feat: add /slack:channel-digest command by @WilliamBergamin in #55
  - feat: support --dry-run flag on draft-announcement by @mwbrooks in #82

  🐛 Bug Fixes

  - fix: handle empty channel IDs in summarize-channel gracefully by @mwbrooks in #78
  - fix: correct block-kit validation for section blocks by @WilliamBergamin in #85

  📚 Documentation

  - docs: clarify Ollama setup in AGENTS.md by @WilliamBergamin in #58
  - docs: add screenshots to draft-announcement walkthrough by @mwbrooks in #91

  🤖 Build

  - ci: automate versioning and releases with changesets by @WilliamBergamin in #60
  - ci: publish GitHub Releases from changesets release step by @WilliamBergamin in #71

  🧪 Tests

  - test: add DeepEval coverage for find-discussions by @mwbrooks in #62
  - test: raise ToolCorrectnessMetric threshold to 0.85 by @WilliamBergamin in #88

  🧹 Chores

  - chore: add slack-skills-plugin-team as default CODEOWNERS team by @mwbrooks in #63
  - chore: add pull request template by @mwbrooks in #69
  - chore: add chore + security labels and align release notes categories by @mwbrooks in #73 ← this PR

  ⬆️ Dependencies

  - chore(deps): bump actions/checkout from 6.0.3 to 7.0.0 by @dependabot in #64
  - chore(deps): bump actions/setup-python from 6.2.0 to 6.3.0 by @dependabot in #67

  🔒 Security

  - fix: patch prompt-injection in canvas rendering pipeline by @mwbrooks in #98
  - chore(deps): bump requests from 2.31.0 to 2.32.4 (CVE-2024-XXXXX) by @dependabot in #99

  📦 Other changes

  (empty when every merged PR carries at least one recognized label — which is the goal)

  Full Changelog: https://github.com/slackapi/slack-mcp-plugin/compare/v0.4.0...v0.5.0

Testing

This PR itself uses the chore label, serving as the first live test of the new label + release-notes category.

Notes

No changeset needed - updates to CI/repo metadata don't affect plugin consumers (per maintainers_guide.md).

Requirements

I've read and understood the Contributing Guidelines and have done my best effort to follow them.
I've read and agree to the Code of Conduct.
I've run make test and the tests pass.

Introduce a `chore` label for maintainer PRs (CI, build tooling, release scaffolding, repo files) matching the conventional-commit `chore:` prefix already in use, plus a `security` label for vulnerability fixes and hardening. Rework `.github/release.yml` to match the labels that actually exist on the repo — the prior config referenced non-existent `build`, `code health`, `docs`, and `security` labels, so those changelog sections silently fell through to "Other changes". Also splits `dependencies` into its own section so Dependabot PRs don't drown out real chores.

mwbrooks · 2026-07-02T23:10:57Z

      labels:
-        - docs
-    - title: 🤖 Build
+        - documentation


fixed: This was pointing to the wrong label name.

📚 note: IIRC docs is matched across most projects. I'm tempted to suggest it's used here too?

https://github.com/slackapi/node-slack-sdk/labels

https://github.com/slackapi/python-slack-sdk/labels

https://github.com/slackapi/slack-cli/labels

Sure, we can fix this label if we've been using docs consistently!

@zimeg Commit 8b6f2ed updates to docs and I've updated the label itself to label:docs as well.

Restores the Security category to its prior slot near the bottom of the changelog category list.

mwbrooks · 2026-07-02T23:13:01Z

-        - build
-    - title: 🧪 Testing/Code Health
+        - test
+    - title: 🧹 Chores


note: Moved belong Tests because externally developers are impacted by this less.

zimeg

🔏 Quick comment but I like these changes @mwbrooks!

zimeg · 2026-07-02T23:34:23Z

      labels:
-        - docs
-    - title: 🤖 Build
+        - documentation


📚 note: IIRC docs is matched across most projects. I'm tempted to suggest it's used here too?

https://github.com/slackapi/node-slack-sdk/labels

https://github.com/slackapi/python-slack-sdk/labels

https://github.com/slackapi/slack-cli/labels

Match the shorter `docs` form used across other slackapi projects. The GitHub label itself was renamed in place (preserves existing PR/issue associations); this updates the `.github/release.yml` reference to match.

Match slack-cli's `build` label ("Changes to compilation and CI processes", color #615196) for CI and build-tooling PRs. Narrows the `chore` scope to repo files, release scaffolding, and general maintenance.

zimeg

@mwbrooks Great positioning for maintenance ongoing! Please feel free to merge this. I'm looking forward to sharing this change as needed with adjacent projects.

zimeg · 2026-07-03T00:11:58Z

    - title: 🔒 Security
      labels:
        - security


🔏 thought(non-blocking): Following bugs or enhancements with this section might be easiest to follow in updates but I'm most excited for correct labels at this time. Let's save these reorderings for a wide spread follow up?

zimeg · 2026-07-03T00:13:12Z

 - Feature requests for new skills should be discussed in the issue before
  implementation begins.
 - Labels: `bug` for confirmed issues, `enhancement` for feature requests,
-  `build` for CI/tooling changes.


👁️‍🗨️ note: I might continue to use build for some of these changes. Please do feel free to correct me in ongoing changes if something seems odd!

mwbrooks added chore Maintainer work: repo files, release scaffolding, general maintenance semver:patch Changes trigger a patch version bump labels Jul 2, 2026

mwbrooks self-assigned this Jul 2, 2026

mwbrooks commented Jul 2, 2026

View reviewed changes

chore: move Security back to original position in release.yml

ea06d05

Restores the Security category to its prior slot near the bottom of the changelog category list.

mwbrooks commented Jul 2, 2026

View reviewed changes

mwbrooks marked this pull request as ready for review July 2, 2026 23:14

mwbrooks requested a review from a team as a code owner July 2, 2026 23:14

zimeg reviewed Jul 2, 2026

View reviewed changes

mwbrooks added 2 commits July 2, 2026 16:37

chore: rename documentation label to docs

8b6f2ed

Match the shorter `docs` form used across other slackapi projects. The GitHub label itself was renamed in place (preserves existing PR/issue associations); this updates the `.github/release.yml` reference to match.

chore: add Build category and build label

cf285eb

Match slack-cli's `build` label ("Changes to compilation and CI processes", color #615196) for CI and build-tooling PRs. Narrows the `chore` scope to repo files, release scaffolding, and general maintenance.

mwbrooks changed the title ~~chore: add chore + security labels and align release notes categories~~ chore: add build, chore, security labels and align release notes categories Jul 3, 2026

zimeg approved these changes Jul 3, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

chore: add build, chore, security labels and align release notes categories#73

chore: add build, chore, security labels and align release notes categories#73
mwbrooks wants to merge 4 commits into
mainfrom
mwbrooks-add-chore-label

mwbrooks commented Jul 2, 2026 •

edited

Loading

Uh oh!

mwbrooks Jul 2, 2026

Uh oh!

zimeg Jul 2, 2026

Uh oh!

mwbrooks Jul 2, 2026

Uh oh!

mwbrooks Jul 2, 2026

Uh oh!

mwbrooks Jul 2, 2026

Uh oh!

zimeg left a comment

Uh oh!

zimeg Jul 2, 2026

Uh oh!

zimeg left a comment

Uh oh!

zimeg Jul 3, 2026

Uh oh!

zimeg Jul 3, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

mwbrooks commented Jul 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Preview

Testing

Notes

Requirements

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

zimeg left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

zimeg left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

mwbrooks commented Jul 2, 2026 •

edited

Loading