Skip to content

Conversation

@zimeg
Copy link
Member

@zimeg zimeg commented Jan 27, 2026

Summary

This PR adds a default app and manifest option to file watching for server restarts and app reinstalls.

Requirements

@zimeg zimeg self-assigned this Jan 27, 2026
@zimeg zimeg added semver:minor enhancement M-T: A feature request for new functionality pkg:cli-hooks applies to `@slack/cli-hooks` labels Jan 27, 2026
@codecov
Copy link

codecov bot commented Jan 27, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.11%. Comparing base (bfa23c8) to head (5fcd109).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@                    Coverage Diff                     @@
##           zimeg-fix-hooks-daemon    #2480      +/-   ##
==========================================================
+ Coverage                   93.10%   93.11%   +0.01%     
==========================================================
  Files                          40       40              
  Lines                       11250    11269      +19     
  Branches                      713      713              
==========================================================
+ Hits                        10474    10493      +19     
  Misses                        764      764              
  Partials                       12       12              
Flag Coverage Δ
cli-hooks 95.42% <100.00%> (+0.12%) ⬆️
cli-test 94.79% <ø> (ø)
oauth 77.39% <ø> (ø)
socket-mode 61.87% <ø> (ø)
web-api 98.11% <ø> (ø)
webhook 96.66% <ø> (ø)

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

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Member

@mwbrooks mwbrooks left a comment

Choose a reason for hiding this comment

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

Exciting times!

paths: ['.'],
app: {
paths: ['.'],
'filter-regex': '.js$',
Copy link
Member

Choose a reason for hiding this comment

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

suggestion: I think we should also support .ts for TypeScript developers using Bolt JS?

Copy link
Member Author

Choose a reason for hiding this comment

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

@mwbrooks We might use the built changes from tsc for now, though these changes might be good to document as well!

A note on this is mentioned now in the CHANGELOG and also updates to the sample app:

TypeScript developers should run `tsc --watch` in a separate terminal during development. This compiles `.ts` files to `.js` on changes, and the default watch configuration will detect changes to the compiled `dist/*.js` files and restart the app server. This approach works best with the default settings.

👾 slack-samples/bolt-ts-starter-template#178

@changeset-bot
Copy link

changeset-bot bot commented Jan 30, 2026

🦋 Changeset detected

Latest commit: 5fcd109

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@slack/cli-hooks Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@zimeg zimeg marked this pull request as ready for review January 30, 2026 06:43
@zimeg zimeg requested a review from a team as a code owner January 30, 2026 06:43
@zimeg
Copy link
Member Author

zimeg commented Jan 30, 2026

🏷️ Follows: slackapi/slack-cli#310

@zimeg zimeg requested a review from mwbrooks January 30, 2026 07:30
@zimeg zimeg added this to the cli-hooks@1.3.0 milestone Jan 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement M-T: A feature request for new functionality pkg:cli-hooks applies to `@slack/cli-hooks` semver:minor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants