feat: env remove command supports the ".env" file for Bolt Framework apps

[zimeg]

Changelog

The slack env remove command will remove environment variables from the .env file for apps that aren't built or run on Slack infrastructure.

Summary

This PR brings support for Bolt frameworks removing variables from the .env file with the env remove command! 🧾

$ slack env remove

Follows #437 and #451 and #456 recent.

Preview

Reviewers

Use the various env commands to interact with this file:

$ cat .env
# Tests follow
EXAMPLE_VARIABLE=12
PLACEHOLDER="false"           # Leftover comments
DATABASE_SECRET="---start---
EOF
---end---"
DETAILS=
$ slack env list
$ slack env remove
$ slack env list
$ cat .env

Notes

• ROSI continue to call the API. For local development the .env file is now supported.
• Inlined comments for a removed variable are also removed. This is different from env add but might make sense?

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.

[codecov]

Codecov Report

❌ Patch coverage is 85.84071% with 16 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.02%. Comparing base (24c1d2c) to head (d6e4207).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
cmd/env/remove.go	87.17%	5 Missing and 5 partials ⚠️
internal/slackdotenv/slackdotenv.go	82.85%	4 Missing and 2 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #457      +/-   ##
==========================================
+ Coverage   70.96%   71.02%   +0.05%     
==========================================
  Files         220      220              
  Lines       18466    18531      +65     
==========================================
+ Hits        13105    13162      +57     
- Misses       4183     4188       +5     
- Partials     1178     1181       +3     

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[mwbrooks]

✅ Great work @zimeg! The design of this command is starting to showcase we how want all of our commands to be structure - linear flow in cmd/ with clean calls into internal/.

🧪 Manual test works well! I noticed removing an non-existent key will output success, which isn't wrong and reasonable for our first release!

[mwbrooks]

praise: Nice and clean 👌🏻

[zimeg]

@mwbrooks Thanks so much for the review and thoughts to these changes!

I'm also a fan of these command patterns and do hope it brings organization to other packages that we might want to untangle 🎁

Let's get this merged to complete Bolt support for these commands - I agree that follow up remains but I'm feeling alright with these changes overall 🌲