fix: simplify scripts following bolt-python and python-sdk patterns (#95)

Co-authored-by: Claude <svc-devxp-claude@slack-corp.com>

Author: WilliamBergamin

.github/maintainers_guide.md

@@ -84,6 +84,14 @@ This project uses [mypy](https://mypy.readthedocs.io/en/stable/index.html) to ch
 ./scripts/run_mypy.sh
 ```
 
+To clean your virtual environment (useful when testing scripts in isolation):
+
+```sh
+./scripts/uninstall_all.sh
+```
+
+> **Note**: Several scripts support a `--no-install` flag to prevent redundant dependency installation when scripts call other scripts internally.
+
 #### Develop Locally
 
 If you want to test the package locally you can.

.gitignore

@@ -122,3 +122,8 @@ cython_debug/
 #  and can be added to the global gitignore or merged into this file.  For a more nuclear
 #  option (not recommended) you can uncomment the following to ignore the entire idea folder.
 .idea/
+
+# LLM
+.claude/
+.cursor/
+

requirements/format.txt requirements/dev-tools.txtrequirements/format.txt renamed to requirements/dev-tools.txt

@@ -1,8 +1,14 @@
 #!/bin/bash
-source ./scripts/_utils.sh
+script_dir=$(dirname $0)
+cd ${script_dir}/..
 
-set_prj_as_cwd
+# Clean previous builds
+rm -rf dist/ build/ slack_cli_hooks.egg-info/
 
-clean_project
+# Install build dependencies unless --no-install is specified
+if [[ "$1" != "--no-install" ]]; then
+    pip install -r requirements/build.txt
+fi
 
-build
+# Build package
+python -m build && twine check dist/*

scripts/_utils.sh

@@ -1,10 +1,8 @@
 #!/bin/bash
-source ./scripts/_utils.sh
+script_dir=$(dirname $0)
+cd ${script_dir}/..
 
-set_prj_as_cwd
-
-clean_project
-
-build
+./scripts/build_pypi_package.sh
 
+# Upload to test PyPI
 twine upload --repository testpypi dist/*

scripts/build_pypi_package.sh

@@ -1,9 +1,11 @@
 #!/bin/bash
-source ./scripts/_utils.sh
+script_dir=$(dirname $0)
+cd ${script_dir}/..
 
-set_prj_as_cwd
+# Install dependencies unless --no-install is specified
+if [[ "$1" != "--no-install" ]]; then
+    pip install -U pip
+    pip install -r requirements/dev-tools.txt
+fi
 
-pip install -U pip
-pip install -r requirements/format.txt
-
-format
+black slack_cli_hooks/ tests/

scripts/deploy_to_test_pypi.sh

@@ -1,6 +1,8 @@
 #!/bin/bash
-source ./scripts/_utils.sh
+script_dir=$(dirname $0)
+cd ${script_dir}/..
 
-set_prj_as_cwd
-
-install_development_requirements
+pip install -U pip
+pip install -e .
+pip install -r requirements/testing.txt
+pip install -r requirements/dev-tools.txt

scripts/format.sh

@@ -2,19 +2,19 @@
 # all: ./scripts/install_and_run_tests.sh
 # single: ./scripts/install_and_run_tests.sh tests/scenario_tests/test_app.py
 
-test_target="$1"
-source ./scripts/_utils.sh
-
-set_prj_as_cwd
+script_dir=$(dirname $0)
+cd ${script_dir}/..
 
-install_development_requirements
+test_target="$1"
 
-format
+./scripts/install.sh
+./scripts/format.sh --no-install
+./scripts/lint.sh --no-install
 
-if [[ $test_target != "" ]]
-then
-    pytest -vv $1
+# Run tests
+if [[ $test_target != "" ]]; then
+  pytest -vv $test_target
 else
-    pytest && \
-    mypy --config-file pyproject.toml
+  pytest
+  ./scripts/run_mypy.sh --no-install
 fi

scripts/install.sh

@@ -1,10 +1,11 @@
 #!/bin/bash
-
-source ./scripts/_utils.sh
-
-set_prj_as_cwd
-
-pip install -U pip
-pip install -r requirements/format.txt
+script_dir=$(dirname $0)
+cd ${script_dir}/..
+
+# Install dependencies unless --no-install is specified
+if [[ "$1" != "--no-install" ]]; then
+    pip install -U pip
+    pip install -r requirements/dev-tools.txt
+fi
 
 flake8 slack_cli_hooks/ && flake8 tests/