task: Make targets reuse JavaScript build

Ideally, the `makefile` serves as the de facto interface for project
scripts. This avoids multiple sources of truth and simplifies project
onboarding and documentation.

Previously, rerunning various `make` targets resulted in rerunning the
expensive and slow `make build` target. It's important to ensure the
build output exists for many targets, but is often unnecessary to rerun
the build when running targets multiple times--e.g., Swift test targets.

This introduces a caching mechanism. The `make build` target skips
running and relies upon the cache unless one of the following is true.

- dist doesn't exist
- REFRESH_JS_BUILD is set to true or 1
- build was invoked directly

This enables quicker reruns while also ensure the build is always
recreated when important to do so--e.g., direct invocations, CI runs,
releases.

See #253 (comment)

Author: dcalhoun

.buildkite/pipeline.yml

@@ -6,7 +6,7 @@ env:
 
 steps:
     - label: ':react: Build React App'
-      command: make build REFRESH_L10N=1
+      command: make build REFRESH_L10N=1 REFRESH_JS_BUILD=1
       plugins: &plugins
           - $CI_TOOLKIT_PLUGIN
           - $NVM_PLUGIN
@@ -21,7 +21,7 @@ steps:
 
     - label: ':android: Publish Android Library'
       command: |
-          make build REFRESH_L10N=1
+          make build REFRESH_L10N=1 REFRESH_JS_BUILD=1
           echo "--- :android: Publishing Android Library"
           ./android/gradlew -p ./android :gutenberg:prepareToPublishToS3 $(prepare_to_publish_to_s3_params) :gutenberg:publish
       agents:

CLAUDE.md

@@ -82,9 +82,9 @@ make build
 # - ios/Sources/GutenbergKit/Gutenberg/
 # - android/Gutenberg/src/main/assets/
 
-# By default, dependencies and translations are only installed if their directories don't exist
-# Force refresh of dependencies and translations
-make build REFRESH_DEPS=1 REFRESH_L10N=1
+# By default, the build is skipped if output directories already exist
+# Force refresh of dependencies, translations, and JS build
+make build REFRESH_DEPS=1 REFRESH_L10N=1 REFRESH_JS_BUILD=1
 
 # Clean build artifacts
 make clean            # Clean both dist and translations

Makefile

@@ -68,15 +68,20 @@ clean: ## Remove build artifacts and translation string files
 
 .PHONY: build
 build: npm-dependencies prep-translations ## Build the project for all platforms (iOS, Android, web)
-	@echo "--- :node: Building Gutenberg"
-
-	npm run build
-
-# Copy build products into place
-	@echo "--- :open_file_folder: Copying Build Products into place"
-	rm -rf ./ios/Sources/GutenbergKit/Gutenberg/ ./android/Gutenberg/src/main/assets/
-	cp -r ./dist/. ./ios/Sources/GutenbergKit/Gutenberg/
-	cp -r ./dist/. ./android/Gutenberg/src/main/assets
+# Skip unless...
+# - dist doesn't exist
+# - REFRESH_JS_BUILD is set to true or 1
+# - build was invoked directly
+	@if [ ! -d "dist" ] || [ "$(REFRESH_JS_BUILD)" = "true" ] || [ "$(REFRESH_JS_BUILD)" = "1" ] || echo "$(MAKECMDGOALS)" | grep -q "^build$$"; then \
+		echo "--- :node: Building Gutenberg"; \
+		npm run build; \
+		echo "--- :open_file_folder: Copying Build Products into place"; \
+		rm -rf ./ios/Sources/GutenbergKit/Gutenberg/ ./android/Gutenberg/src/main/assets/; \
+		cp -r ./dist/. ./ios/Sources/GutenbergKit/Gutenberg/; \
+		cp -r ./dist/. ./android/Gutenberg/src/main/assets; \
+	else \
+		echo "--- :white_check_mark: Skipping JS build (dist already exists). Use REFRESH_JS_BUILD=1 to force refresh."; \
+	fi
 
 .PHONY: build-swift-package
 build-swift-package: build ## Build the Swift package for iOS

bin/release.sh

@@ -243,7 +243,7 @@ build_project() {
         return
     fi
 
-    make build REFRESH_DEPS=1 REFRESH_L10N=1 STRICT_L10N=1
+    make build REFRESH_DEPS=1 REFRESH_L10N=1 REFRESH_JS_BUILD=1 STRICT_L10N=1
     print_success "Build completed successfully"
 }