Moved @tryghost/staff-service to Ghost core

fix https://linear.app/ghost/issue/ENG-2067/staff-service

- this commit migrates the code and test for the staff service
  into Ghost core so we can keep all related code together and reduce
  overhead
- as a result of this change, we can remove the package and associated
  references

Author: daniellockyer

compose.yml

@@ -104,7 +104,6 @@ services:
       - node_modules_ghost_session-service:/home/ghost/ghost/session-service/node_modules:delegated
       - node_modules_ghost_settings-path-manager:/home/ghost/ghost/settings-path-manager/node_modules:delegated
       - node_modules_ghost_slack-notifications:/home/ghost/ghost/slack-notifications/node_modules:delegated
-      - node_modules_ghost_staff-service:/home/ghost/ghost/staff-service/node_modules:delegated
       - node_modules_ghost_stats-service:/home/ghost/ghost/stats-service/node_modules:delegated
       - node_modules_ghost_stripe:/home/ghost/ghost/stripe/node_modules:delegated
       - node_modules_ghost_tiers:/home/ghost/ghost/tiers/node_modules:delegated
@@ -275,7 +274,6 @@ volumes:
   node_modules_ghost_session-service: {}
   node_modules_ghost_settings-path-manager: {}
   node_modules_ghost_slack-notifications: {}
-  node_modules_ghost_staff-service: {}
   node_modules_ghost_stats-service: {}
   node_modules_ghost_stripe: {}
   node_modules_ghost_tiers: {}

ghost/staff-service/lib/StaffService.js renamed to ghost/core/core/server/services/staff/StaffService.js

@@ -8,7 +8,7 @@ class StaffServiceWrapper {
             return;
         }
 
-        const StaffService = require('@tryghost/staff-service');
+        const StaffService = require('./StaffService');
 
         const logging = require('@tryghost/logging');
         const models = require('../../models');

ghost/staff-service/lib/StaffServiceEmails.js renamed to ghost/core/core/server/services/staff/StaffServiceEmails.js

@@ -40,7 +40,7 @@
     "test:browser:setup": "npx playwright install",
     "test:ci:e2e": "c8 -c ./.c8rc.e2e.json -o coverage-e2e yarn test:e2e -b --retries=2 --reporter=./test/utils/mocha-retry-reporter.js",
     "test:ci:regression": "yarn test:regression -b --retries=2 --reporter=./test/utils/mocha-retry-reporter.js",
-    "test:ci:integration": "c8 -c ./.c8rc.e2e.json -o coverage-integration --lines 57 --functions 47 --branches 77 --statements 57 yarn test:integration -b --retries=2 --reporter=./test/utils/mocha-retry-reporter.js",
+    "test:ci:integration": "c8 -c ./.c8rc.e2e.json -o coverage-integration --lines 57 --functions 47 --branches 76 --statements 57 yarn test:integration -b --retries=2 --reporter=./test/utils/mocha-retry-reporter.js",
     "test:unit:slow": "yarn test:unit --reporter=mocha-slow-test-reporter",
     "test:int:slow": "yarn test:integration --reporter=mocha-slow-test-reporter",
     "test:e2e:slow": "yarn test:e2e --reporter=mocha-slow-test-reporter",
@@ -84,6 +84,7 @@
     "@tryghost/debug": "0.1.32",
     "@tryghost/domain-events": "0.0.0",
     "@tryghost/donations": "0.0.0",
+    "@tryghost/email-addresses": "0.0.0",
     "@tryghost/email-analytics-provider-mailgun": "0.0.0",
     "@tryghost/email-analytics-service": "0.0.0",
     "@tryghost/email-content-generator": "0.0.0",
@@ -150,7 +151,6 @@
     "@tryghost/settings-path-manager": "0.0.0",
     "@tryghost/slack-notifications": "0.0.0",
     "@tryghost/social-urls": "0.1.43",
-    "@tryghost/staff-service": "0.0.0",
     "@tryghost/stats-service": "0.0.0",
     "@tryghost/string": "0.2.12",
     "@tryghost/tiers": "0.0.0",
@@ -189,6 +189,7 @@
     "glob": "8.1.0",
     "got": "11.8.6",
     "gscan": "4.47.0",
+    "handlebars": "4.7.8",
     "human-number": "2.0.4",
     "iconv-lite": "0.6.3",
     "image-size": "1.2.0",
@@ -198,6 +199,7 @@
     "json-stable-stringify": "1.2.0",
     "jsonpath": "1.1.1",
     "jsonwebtoken": "8.5.1",
+    "juice": "9.1.0",
     "keypair": "1.0.4",
     "knex": "2.4.2",
     "knex-migrator": "5.3.0",

ghost/staff-service/lib/email-templates/donation.hbs renamed to ghost/core/core/server/services/staff/email-templates/donation.hbs

@@ -8,7 +8,7 @@ const {MilestoneCreatedEvent} = require('@tryghost/milestones');
 const DomainEvents = require('@tryghost/domain-events');
 
 require('should');
-const StaffService = require('../index');
+const StaffService = require('../../../../../core/server/services/staff/StaffService');
 
 function testCommonMailData({mailStub, getEmailAlertUsersStub}) {
     getEmailAlertUsersStub.calledWith(