@@ -40,7 +40,7 @@ Convert all Vortex bash scripts (`scripts/vortex/*.sh`) to PHP scripts as a stan
4040
4141### 📊 Test Statistics
4242
43- - ** Total Tests** : 383 tests, 1,740 assertions
43+ - ** Total Tests** : 429 tests, 1,861 assertions
4444- ** Test Groups** :
4545 - ` helpers ` - 55 tests for core helper functions
4646 - ` notify ` - 132 tests for notification system
@@ -753,6 +753,90 @@ function mockFileGetContents(string $path, string $content): void
753753
754754---
755755
756+ ## ✅ Phase 5.5: Sync PHP Scripts with Shell Updates - COMPLETE
757+
758+ After the initial conversion in Phases 1-5, the shell scripts received updates that
759+ were not reflected in the PHP versions. This phase synchronized all differences.
760+
761+ ### 5.5.1 Notification Scripts Sync
762+
763+ ** notify (router):**
764+ - Added ` VORTEX_NOTIFY_BRANCH ` (required) variable
765+ - Added ` VORTEX_NOTIFY_SHA ` (required) variable
766+ - Added ` VORTEX_NOTIFY_PR_NUMBER ` (optional) variable
767+ - Changed ` VORTEX_NOTIFY_ENVIRONMENT_URL ` from optional to required
768+ - Added ` putenv() ` exports for branch, SHA, PR number, label, environment URL, login URL
769+ - Added detailed summary output matching shell format
770+
771+ ** notify-slack:**
772+ - Fixed field order: Deployment, Time, Environment, Login (matching shell)
773+ - Added ` footer: 'Vortex Deployment' ` to Slack attachment
774+
775+ ** notify-github:**
776+ - Added ` VORTEX_NOTIFY_GITHUB_BRANCH ` (with fallback to ` VORTEX_NOTIFY_BRANCH ` ) as deployment ref
777+ - Replaced ` $notify_label ` with ` $notify_branch ` for all deployment API ` ref ` parameters
778+ - Fixed ` VORTEX_NOTIFY_GITHUB_ENVIRONMENT_TYPE ` default: falls back to ` VORTEX_NOTIFY_LABEL ` then ` 'PR' `
779+
780+ ** notify-jira:**
781+ - Added ` VORTEX_NOTIFY_JIRA_BRANCH ` (with fallback to ` VORTEX_NOTIFY_BRANCH ` ) for issue extraction
782+ - Changed issue extraction regex to match against branch (not label)
783+ - Replaced simple text-to-ADF with rich ` build_adf_comment() ` featuring clickable links, code marks, and hardBreaks
784+
785+ ** notify-newrelic:**
786+ - Reordered variables: enabled check first, then required variables
787+ - Renamed ` VORTEX_NOTIFY_NEWRELIC_USER_NAME ` to ` VORTEX_NOTIFY_NEWRELIC_USER ` (matching shell)
788+ - Added ` VORTEX_NOTIFY_NEWRELIC_SHA ` (with fallback to ` VORTEX_NOTIFY_SHA ` )
789+ - Added SHA-based revision fallback before auto-generated LABEL-DATE-TIME pattern
790+ - Added required variable validation after enabled check (user API key)
791+ - Added numeric validation for app ID regardless of source
792+
793+ ### 5.5.2 Deployment Scripts Sync
794+
795+ ** deploy (router):**
796+ - Fixed ` VORTEX_DEPLOY_ACTION ` default from ` 'deploy' ` to ` '' ` (matching shell)
797+
798+ ** deploy-artifact:**
799+ - Fixed SSH prefix from ` DEPLOY ` to ` DEPLOY_ARTIFACT ` (matching shell)
800+ - Added ` VORTEX_DEPLOY_ARTIFACT_SSH_FINGERPRINT ` and ` VORTEX_DEPLOY_ARTIFACT_SSH_FILE ` variable names with fallback chain
801+ - Updated git-artifact version from ` ~1.1 ` to ` ~1.2 ` (matching shell)
802+
803+ ** deploy-lagoon:**
804+ - Added tag deployment check: early exit for tag mode (` 'Lagoon does not support tag deployments. Skipping.' ` )
805+ - Added ` VORTEX_DEPLOY_MODE ` variable
806+ - Added ` VORTEX_DEPLOY_LAGOON_* ` prefixed variable support with fallbacks:
807+ - ` VORTEX_DEPLOY_LAGOON_ACTION ` (fallback: ` VORTEX_DEPLOY_ACTION ` , default: ` 'create' ` )
808+ - ` VORTEX_DEPLOY_LAGOON_PROJECT ` (fallback: ` LAGOON_PROJECT ` )
809+ - ` VORTEX_DEPLOY_LAGOON_BRANCH ` (fallback: ` VORTEX_DEPLOY_BRANCH ` )
810+ - ` VORTEX_DEPLOY_LAGOON_PR ` (fallback: ` VORTEX_DEPLOY_PR ` )
811+ - ` VORTEX_DEPLOY_LAGOON_PR_HEAD ` (fallback: ` VORTEX_DEPLOY_PR_HEAD ` )
812+ - ` VORTEX_DEPLOY_LAGOON_PR_BASE_BRANCH ` (fallback: ` VORTEX_DEPLOY_PR_BASE_BRANCH ` )
813+ - ` VORTEX_DEPLOY_LAGOON_SSH_FINGERPRINT ` (fallback chain)
814+ - ` VORTEX_DEPLOY_LAGOON_SSH_FILE ` (fallback chain)
815+ - ` VORTEX_DEPLOY_LAGOON_LAGOONCLI_PATH ` (fallback: ` VORTEX_LAGOONCLI_PATH ` )
816+ - ` VORTEX_DEPLOY_LAGOON_LAGOONCLI_FORCE_INSTALL ` (fallback: ` VORTEX_LAGOONCLI_FORCE_INSTALL ` )
817+ - ` VORTEX_DEPLOY_LAGOON_LAGOONCLI_VERSION ` (fallback: ` VORTEX_LAGOONCLI_VERSION ` )
818+ - Fixed SSH prefix from ` DEPLOY ` to ` DEPLOY_LAGOON ` (matching shell)
819+
820+ ** login-container-registry:**
821+ - Added ` VORTEX_LOGIN_CONTAINER_REGISTRY ` (fallback: ` VORTEX_CONTAINER_REGISTRY ` )
822+ - Added ` VORTEX_LOGIN_CONTAINER_REGISTRY_USER ` (fallback: ` VORTEX_CONTAINER_REGISTRY_USER ` )
823+ - Added ` VORTEX_LOGIN_CONTAINER_REGISTRY_PASS ` (fallback: ` VORTEX_CONTAINER_REGISTRY_PASS ` )
824+ - Added ` VORTEX_LOGIN_CONTAINER_REGISTRY_DOCKER_CONFIG ` (fallback: ` DOCKER_CONFIG ` )
825+
826+ ### 5.5.3 Test Updates
827+
828+ All test files updated to match script changes:
829+ - ` NotifyGithubTest.php ` - Updated for branch variable, assertions, and data providers
830+ - ` NotifyJiraTest.php ` - Updated for branch-based issue extraction
831+ - ` NotifyNewrelicTest.php ` - Updated for optional variables, custom error messages
832+ - ` NotifyRouterTest.php ` - Added required branch and SHA variables
833+ - ` DeployArtifactTest.php ` - Updated git-artifact version in mocks
834+ - ` DeployLagoonTest.php ` - Updated error message for new variable names
835+
836+ ** All 429 tests passing with 1,861 assertions** ✅
837+
838+ ---
839+
756840## Phase 6: Convert Remaining Scripts (TODO)
757841
758842### Database Scripts
@@ -762,9 +846,11 @@ function mockFileGetContents(string $path, string $content): void
7628464 . ⏳ ` download-db-container-registry.sh ` - Download from container registry
7638475 . ⏳ ` download-db-lagoon.sh ` - Download from Lagoon
7648486 . ⏳ ` download-db-acquia.sh ` - Download from Acquia
765- 7 . ⏳ ` export-db.sh ` - Export database router
766- 8 . ⏳ ` export-db-file.sh ` - Export to file
767- 9 . ⏳ ` export-db-image.sh ` - Export to container image
849+ 7 . ⏳ ` download-db-s3.sh ` - Download from S3
850+ 8 . ⏳ ` upload-db-s3.sh ` - Upload to S3
851+ 9 . ⏳ ` export-db.sh ` - Export database router
852+ 10 . ⏳ ` export-db-file.sh ` - Export to file
853+ 11 . ⏳ ` export-db-image.sh ` - Export to container image
768854
769855### Provisioning Scripts
77085610 . ⏳ ` provision.sh ` - Main provisioning
@@ -1125,6 +1211,6 @@ All deployment scripts converted as a single milestone:
11251211
11261212---
11271213
1128- **Last Updated**: 2025-12-15
1129- **Current Phase**: Phase 5 Complete ✅
1214+ **Last Updated**: 2026-02-20
1215+ **Current Phase**: Phase 5.5 Sync Complete ✅
11301216**Next Milestone**: Phase 6 - Database download/export scripts
0 commit comments