@friggframework/admin-scripts 2.0.0--canary.517.7d448e0.0 → 2.0.0--canary.517.8d90337.0

package/index.js

@@ -9,7 +9,6 @@
 const {
     ScriptFactory,
     getScriptFactory,
-    createScriptFactory,
 } = require('./src/application/script-factory');
 const { AdminScriptBase } = require('./src/application/admin-script-base');
 const {
@@ -49,7 +48,6 @@ module.exports = {
     AdminScriptBase,
     ScriptFactory,
     getScriptFactory,
-    createScriptFactory,
     AdminScriptContext,
     createAdminScriptContext,
     ScriptRunner,

package/package.json

@@ -1,19 +1,19 @@
 {
     "name": "@friggframework/admin-scripts",
     "prettier": "@friggframework/prettier-config",
-    "version": "2.0.0--canary.517.7d448e0.0",
+    "version": "2.0.0--canary.517.8d90337.0",
     "description": "Admin Script Runner for Frigg - Execute maintenance and operational scripts in hosted environments",
     "dependencies": {
         "@aws-sdk/client-scheduler": "^3.588.0",
-        "@friggframework/core": "2.0.0--canary.517.7d448e0.0",
+        "@friggframework/core": "2.0.0--canary.517.8d90337.0",
         "@hapi/boom": "^10.0.1",
         "express": "^4.18.2",
         "serverless-http": "^3.2.0"
     },
     "devDependencies": {
-        "@friggframework/eslint-config": "2.0.0--canary.517.7d448e0.0",
-        "@friggframework/prettier-config": "2.0.0--canary.517.7d448e0.0",
-        "@friggframework/test": "2.0.0--canary.517.7d448e0.0",
+        "@friggframework/eslint-config": "2.0.0--canary.517.8d90337.0",
+        "@friggframework/prettier-config": "2.0.0--canary.517.8d90337.0",
+        "@friggframework/test": "2.0.0--canary.517.8d90337.0",
         "eslint": "^8.22.0",
         "jest": "^29.7.0",
         "prettier": "^2.7.1",
@@ -44,5 +44,5 @@
         "maintenance",
         "operations"
     ],
-    "gitHead": "7d448e09858f5a215e43b3a210fe15efe4d76732"
+    "gitHead": "8d903378eed651256cbda1a3e1a6aab54fa46475"
 }

package/src/application/__tests__/script-factory.test.js

@@ -1,8 +1,4 @@
-const {
-    ScriptFactory,
-    createScriptFactory,
-    getScriptFactory,
-} = require('../script-factory');
+const { ScriptFactory, getScriptFactory } = require('../script-factory');
 const { AdminScriptBase } = require('../admin-script-base');
 
 describe('ScriptFactory', () => {
@@ -346,9 +342,9 @@ describe('ScriptFactory', () => {
             expect(factory1).toBeInstanceOf(ScriptFactory);
         });
 
-        it('createScriptFactory() should create new instance', () => {
-            const factory1 = createScriptFactory();
-            const factory2 = createScriptFactory();
+        it('new ScriptFactory() creates independent instances', () => {
+            const factory1 = new ScriptFactory();
+            const factory2 = new ScriptFactory();
 
             expect(factory1).not.toBe(factory2);
             expect(factory1).toBeInstanceOf(ScriptFactory);
@@ -364,7 +360,7 @@ describe('ScriptFactory', () => {
                 };
             }
 
-            const customFactory = createScriptFactory();
+            const customFactory = new ScriptFactory();
             customFactory.register(TestScript);
 
             const globalFactory = getScriptFactory();

package/src/application/script-factory.js

@@ -151,13 +151,4 @@ function getScriptFactory() {
     return globalFactory;
 }
 
-/**
- * Create a new script factory instance
- * @param {Array} scripts - Initial scripts to register
- * @returns {ScriptFactory} New factory
- */
-function createScriptFactory(scripts = []) {
-    return new ScriptFactory(scripts);
-}
-
-module.exports = { ScriptFactory, getScriptFactory, createScriptFactory };
+module.exports = { ScriptFactory, getScriptFactory };