@fishawack/lab-env 4.39.0-beta.6 → 4.39.0-beta.7

package/CHANGELOG.md

@@ -1,5 +1,20 @@
 ## Changelog
 
+### 4.39.0-beta.7 (2025-05-21)
+
+#### Features
+
+* added eslint configuration and global deps ([2edd07a](https://bitbucket.org/fishawackdigital/lab-env/commits/2edd07a602c601dcb1ff7e668f002a1f5ec675e3))
+* added positional flag to lint command ([a9d1090](https://bitbucket.org/fishawackdigital/lab-env/commits/a9d109079721765a462c7fbfefecd4ae7177a4dd))
+* copy eslint.config.js to project repo ([900f2e7](https://bitbucket.org/fishawackdigital/lab-env/commits/900f2e765322633a1f6e82bc4389c2ea2e8ff635))
+* show the installing npm module in guide ([d5b4113](https://bitbucket.org/fishawackdigital/lab-env/commits/d5b41136c15f7e5ce74e05240742b817fb999474))
+
+#### Bug Fixes
+
+* remove airbnb dep as not needed ([494b6af](https://bitbucket.org/fishawackdigital/lab-env/commits/494b6afd2a635a0e817599cf5ae8ef1f0315d746))
+* use global prettier instead of npx ([912e598](https://bitbucket.org/fishawackdigital/lab-env/commits/912e598703705586a5aeb923100b82ea19335a4f))
+* use var as let not supported in cloudfront functions v5 ([6656ff3](https://bitbucket.org/fishawackdigital/lab-env/commits/6656ff319e812273c20b9b14425a90be194c4035))
+
 ### 4.39.0-beta.6 (2025-05-15)
 
 #### Features

package/_Test/provision.js

@@ -9,15 +9,24 @@ describe("provision", () => {
     let config;
     let repo = "lab-env-test-suite";
     let account = "fishawack";
+    let branch = "test";
+    let name = aws.slug(repo, account, branch);
 
     describe("static", () => {
         before(async () => {
             fetch = (await import("node-fetch")).default;
 
-            config = await aws.static(repo, account);
+            config = await aws.static(
+                name,
+                account,
+                undefined,
+                undefined,
+                repo,
+                branch,
+            );
 
             await aws.s3.addFileToS3Bucket(
-                repo,
+                name,
                 account,
                 "index.html",
                 new TextEncoder().encode("test"),
@@ -29,10 +38,10 @@ describe("provision", () => {
         });
 
         after(async () => {
-            await aws.s3.removeFileToS3Bucket(repo, account, "index.html");
+            await aws.s3.removeFileToS3Bucket(name, account, "index.html");
 
             await aws.staticTerminate(
-                repo,
+                name,
                 account,
                 repo,
                 "development",

package/_Test/s3.js

@@ -38,7 +38,9 @@ describe("s3", () => {
                 await s3.removeFileToS3Bucket(repo, account, "index.html");
 
                 await s3.removeS3Bucket(repo, account);
-            } catch (e) {}
+            } catch {
+                /* empty */
+            }
         });
     });
 });

package/_Test/utilities.js

@@ -22,6 +22,7 @@ describe("utilities", async () => {
         });
 
         it("Should convert _ to -", async () => {
+            // eslint-disable-next-line no-useless-escape
             expect(s3Safe("___").match(/\_/g)).to.be.null;
         });
 
@@ -40,6 +41,7 @@ describe("utilities", async () => {
         it("Should remove any special chars", async () => {
             expect(
                 s3Safe("!@#$%^&*()=_+;,./<>?~`|").match(
+                    // eslint-disable-next-line no-useless-escape
                     /[\!\@\#\$\%\^\&\*\(\)\=\_\+\;\,\.\/\<\>\?\~\`\|]/g,
                 ),
             ).to.be.null;
@@ -64,6 +66,7 @@ describe("utilities", async () => {
         });
 
         it("Should convert _ to -", async () => {
+            // eslint-disable-next-line no-useless-escape
             expect(ebSafe("___").match(/\_/g)).to.be.null;
         });
 
@@ -82,6 +85,7 @@ describe("utilities", async () => {
         it("Should remove any special chars", async () => {
             expect(
                 ebSafe("!@#$%^&*()=_+;,./<>?~`|").match(
+                    // eslint-disable-next-line no-useless-escape
                     /[\!\@\#\$\%\^\&\*\(\)\=\_\+\;\,\.\/\<\>\?\~\`\|]/g,
                 ),
             ).to.be.null;

package/cli.js

@@ -144,9 +144,8 @@ const args = hideBin(process.argv);
 // Remove network leftover by docker-compose run
 function shutdown() {
     try {
-        execSync(
-            `docker network rm ${_.repo_safe}_default &>/dev/null`,
-            _.opts,
-        );
-    } catch (e) {}
+        execSync(`docker network rm ${_.repoSafe}_default &>/dev/null`, _.opts);
+    } catch {
+        /* empty */
+    }
 }

package/commands/check.js

@@ -25,6 +25,7 @@ module.exports = [
         ) {
             _.command(
                 "php",
+                // eslint-disable-next-line no-useless-escape
                 `${argv.v ? `composer install --dry-run 2>&1 | grep '\\- Installing' | while read -r line; do echo \\$\{line/'Installing'\}; done && ` : ""}composer install --dry-run 2>&1 | grep -q 'Nothing to install, update or remove' && echo -e '\\033[0;32mComposer deps OK\\033[0m' || echo -e '\\033[0;31mComposer deps missing or outdated\\033[0m'`,
             );
         }

package/commands/clean.js

@@ -3,8 +3,8 @@ const _ = require("../globals.js");
 module.exports = [
     "clean",
     "cleans out dependencies",
-    (yargs) => {},
-    (argv) => {
+    () => {},
+    () => {
         _.command("core", `git clean -xfd node_modules/ 2>/dev/null || true`);
 
         if (

package/commands/content.js

@@ -17,8 +17,8 @@ module.exports = [
             const stringify = JSON.stringify(
                 {
                     "aws-s3": "fishawack",
-                    location: `fw-auto-content/${_.repo_safe}`,
-                    key: `fw-s3-${_.repo_safe}`,
+                    location: `fw-auto-content/${_.repoSafe}`,
+                    key: `fw-s3-${_.repoSafe}`,
                     sync: true,
                 },
                 null,

package/commands/create/cmds/dekey.js

@@ -1,13 +1,12 @@
 const _ = require("../../../globals.js");
-const utilities = require("../libs/utilities");
 const inquirer = require("inquirer");
 const aws = require("../services/aws/index.js");
 
 module.exports = [
     "dekey",
     false,
-    (yargs) => {},
-    async (argv) => {
+    () => {},
+    async () => {
         let users = [];
         let clients = [];
 

package/commands/create/cmds/deprovision.js

@@ -48,7 +48,7 @@ module.exports = [
         }
 
         const slug = aws.slug(
-            _.repo_safe,
+            _.repoSafe,
             answers.client,
             branch,
             answers.stack === "static" ? "s3" : "eb",
@@ -70,7 +70,7 @@ module.exports = [
         await aws[`${answers.stack}Terminate`](
             slug,
             answers.client,
-            _.repo_safe,
+            _.repoSafe,
             branch,
             answers.id,
         );

package/commands/create/cmds/diagnose.js

@@ -42,6 +42,7 @@ module.exports = [
             let preset = await guide.preset(argv);
 
             if (argv.l == null && preset === "permanent") {
+                // eslint-disable-next-line no-param-reassign
                 argv.legacy = argv.l = true;
             }
 
@@ -73,8 +74,8 @@ module.exports = [
                 await guide.strict(argv);
             }
 
-            while (!(await test.global_modules())) {
-                await guide.global_modules(argv);
+            while (!(await test.globalModules())) {
+                await guide.globalModules(argv);
             }
 
             if (preset === "permanent" || preset === "freelancer") {

package/commands/create/cmds/key.js

@@ -11,8 +11,8 @@ const htmlEmail = require("fs").readFileSync(
 module.exports = [
     "key",
     false,
-    (yargs) => {},
-    async (argv) => {
+    () => {},
+    async () => {
         let users = [];
         let clients = [];
         let sendEmail = false;

package/commands/create/cmds/provision.js

@@ -84,7 +84,7 @@ module.exports = [
                         type: "input",
                         name: "username",
                         message: "Username",
-                        default: `${_.repo_safe}User${credentials.length ? `-${credentials.length}` : ""}`,
+                        default: `${_.repoSafe}User${credentials.length ? `-${credentials.length}` : ""}`,
                         validate: (input) => !!input.length,
                     },
                     {
@@ -115,7 +115,7 @@ module.exports = [
         let config = {};
         let infastructure;
         const slug = aws.slug(
-            _.repo_safe,
+            _.repoSafe,
             answers.client,
             branch,
             answers.stack === "static" ? "s3" : "eb",
@@ -131,7 +131,7 @@ module.exports = [
                     { Key: "automated", Value: true },
                 ],
                 credentials,
-                _.repo_safe,
+                _.repoSafe,
                 branch,
                 answers.framework,
                 answers.availability,

package/commands/create/libs/aws-cloudfront-auth.js

@@ -1,15 +1,18 @@
+// eslint-disable-next-line no-unused-vars
 function handler(event) {
+    var query, index, key, response;
+
     // Redirect if www to non-www
     if (event.request.headers.host.value.includes("www.")) {
-        var query = "";
-        var index = 0;
+        query = "";
+        index = 0;
 
-        for (var key in event.request.querystring) {
+        for (key in event.request.querystring) {
             query += `${index ? "&" : "?"}${key}=${event.request.querystring[key].value}`;
             index++;
         }
 
-        var response = {
+        response = {
             statusCode: 301,
             statusDescription: "Moved Permanently",
             headers: {
@@ -27,10 +30,10 @@ function handler(event) {
 
     // Redirect if no trailing slash
     if (!event.request.uri.includes(".") && !event.request.uri.endsWith("/")) {
-        var query = "";
-        var index = 0;
+        query = "";
+        index = 0;
 
-        for (var key in event.request.querystring) {
+        for (key in event.request.querystring) {
             query += `${index ? "&" : "?"}${key}=${event.request.querystring[key].value}`;
             index++;
         }
@@ -70,7 +73,7 @@ function handler(event) {
     // But if we get here, we must either be missing the auth header or the
     // credentials failed to match what we expected.
     // Request the browser present the Basic Auth dialog.
-    var response = {
+    response = {
         statusCode: 401,
         statusDescription: "Unauthorized",
         headers: {

package/commands/create/libs/aws-cloudfront-response.js

@@ -1,3 +1,4 @@
+// eslint-disable-next-line no-unused-vars
 function handler(event) {
     // Add security headers
     var response = event.response;

package/commands/create/libs/aws-cloudfront-simple.js

@@ -1,15 +1,18 @@
+// eslint-disable-next-line no-unused-vars
 function handler(event) {
+    var query, index, key, response;
+
     // Redirect if www to non-www
     if (event.request.headers.host.value.includes("www.")) {
-        var query = "";
-        var index = 0;
+        query = "";
+        index = 0;
 
-        for (var key in event.request.querystring) {
+        for (key in event.request.querystring) {
             query += `${index ? "&" : "?"}${key}=${event.request.querystring[key].value}`;
             index++;
         }
 
-        var response = {
+        response = {
             statusCode: 301,
             statusDescription: "Moved Permanently",
             headers: {
@@ -27,10 +30,10 @@ function handler(event) {
 
     // Redirect if no trailing slash
     if (!event.request.uri.includes(".") && !event.request.uri.endsWith("/")) {
-        var query = "";
-        var index = 0;
+        query = "";
+        index = 0;
 
-        for (var key in event.request.querystring) {
+        for (key in event.request.querystring) {
             query += `${index ? "&" : "?"}${key}=${event.request.querystring[key].value}`;
             index++;
         }

package/commands/create/libs/utilities.js

@@ -47,6 +47,7 @@ module.exports.Spinner = class Spinner {
     }
 
     action(message, action, success, failure) {
+        // eslint-disable-next-line no-async-promise-executor
         return new Promise(async (resolve, reject) => {
             let instance = new this.constructor(message);
             let res;
@@ -65,6 +66,7 @@ module.exports.Spinner = class Spinner {
     }
 
     simple(message, action) {
+        // eslint-disable-next-line no-async-promise-executor
         return new Promise(async (resolve, reject) => {
             let instance = new this.constructor(message, true);
             let res;
@@ -127,6 +129,7 @@ module.exports.nameSafe = (name, service = "s3") => {
     safe = `${prefix}${safe}`;
     safe = `${safe}${suffix}`;
     safe = safe.replace(/\./g, "-");
+    // eslint-disable-next-line no-useless-escape
     safe = safe.replace(/\_/g, "-");
 
     return safe;

package/commands/create/libs/vars.js

@@ -11,7 +11,7 @@ try {
     miscFile = JSON.parse(
         fs.readFileSync(path.join(os.homedir(), "targets", "misc.json")),
     );
-} catch (e) {
+} catch {
     miscFile = {
         bitbucket: {},
         gitlab: {},
@@ -29,7 +29,7 @@ try {
             encoding: "utf8",
         }),
     );
-} catch (e) {
+} catch {
     ftppassFile = {};
 }
 
@@ -45,7 +45,7 @@ try {
             { encoding: "utf8" },
         ),
     );
-} catch (e) {
+} catch {
     creds.egnyte = {};
 }
 
@@ -71,27 +71,23 @@ module.exports.headers = {
 };
 
 // Required global node modules
-module.exports.global_modules = [
-    {
-        package: "@commitlint/cli",
-        version: "19",
-    },
-    {
-        package: "@commitlint/config-conventional",
-        version: "19",
-    },
-    {
-        package: "husky",
-        version: "9",
-    },
-    {
-        package: "prettier",
-        version: "3",
-    },
-    {
-        package: "@prettier/plugin-php",
-        version: "0",
-    },
+module.exports.globalModules = [
+    // HUSKY
+    { package: "husky", version: "9" },
+    // COMMITLINT
+    { package: "@commitlint/cli", version: "19" },
+    { package: "@commitlint/config-conventional", version: "19" },
+    // PRETTIER
+    { package: "prettier", version: "3" },
+    { package: "@prettier/plugin-php", version: "0" },
+    // ESLINT
+    { package: "@eslint/js", version: "9.27.0" },
+    { package: "eslint-config-prettier", version: "10.1.5" },
+    { package: "eslint-plugin-import", version: "2.31.0" },
+    { package: "eslint-plugin-prettier", version: "5.4.0" },
+    { package: "eslint-plugin-vue", version: "10.1.0" },
+    { package: "eslint", version: "9.27.0" },
+    { package: "globals", version: "16.1.0" },
 ];
 
 // User presets

package/commands/create/services/aws/cloudfront.js

@@ -212,7 +212,7 @@ module.exports.createCloudFrontFunction = async (name, account, fn, config) => {
                 );
             },
         );
-    } catch (e) {
+    } catch {
         res = await Spinner.prototype.simple(
             `Retrieving the already existing CloudFront function`,
             () => {

package/commands/create/services/aws/iam.js

@@ -36,7 +36,7 @@ module.exports.createIAMUser = async (UserName, account, tags = []) => {
                 );
             },
         );
-    } catch (e) {
+    } catch {
         res = await Spinner.prototype.simple(
             `Retrieving the already existing IAM user ${UserName}`,
             () => {
@@ -81,7 +81,9 @@ module.exports.removeIAMUser = async (UserName, account) => {
                 return client.send(new DeleteUserCommand({ UserName }));
             },
         );
-    } catch (e) {}
+    } catch {
+        /* empty */
+    }
 
     return res;
 };
@@ -347,7 +349,7 @@ module.exports.ensureEBInstanceProfileExists = async (account) => {
 
     try {
         await module.exports.getRole(role, account);
-    } catch (e) {
+    } catch {
         await module.exports.createRole(
             role,
             account,

package/commands/create/services/aws/index.js

@@ -116,29 +116,39 @@ module.exports.static = async (
 module.exports.staticTerminate = async (name, account, repo, branch, id) => {
     try {
         await module.exports.s3.emptyS3Bucket(name, account);
-    } catch (e) {}
+    } catch {
+        /* empty */
+    }
 
     try {
         await module.exports.s3.removeS3Bucket(name, account);
-    } catch (e) {}
+    } catch {
+        /* empty */
+    }
 
     try {
         await module.exports.cloudfront.removeCloudFrontDistribution(
             id,
             account,
         );
-    } catch (e) {}
+    } catch {
+        /* empty */
+    }
 
     try {
         await module.exports.cloudfront.removeCloudFrontFunction(name, account);
-    } catch (e) {}
+    } catch {
+        /* empty */
+    }
 
     try {
         await module.exports.cloudfront.removeCloudFrontFunction(
             module.exports.slug(`${repo}-response`, account, branch),
             account,
         );
-    } catch (e) {}
+    } catch {
+        /* empty */
+    }
 };
 
 module.exports.fullstack = async (
@@ -181,14 +191,18 @@ module.exports.fullstack = async (
             repo,
             account,
         );
-    } catch (e) {}
+    } catch {
+        /* empty */
+    }
 
     let hasKeyPair = false;
 
     try {
         await module.exports.ec2.getKeyPair("id_rsa_prev", account);
         hasKeyPair = true;
-    } catch (e) {}
+    } catch {
+        /* empty */
+    }
 
     // LetsEncrypt CN records are limited to 64 chars so use only the hash from the generated name
     const CNAMEPrefix = `fw-auto-${name.split("-")[name.split("-").length - 1]}`;
@@ -270,27 +284,37 @@ module.exports.fullstackTerminate = async (name, account, repo, branch) => {
 
     try {
         await module.exports.iam.removeIAMUser(s3Slug, account);
-    } catch (e) {}
+    } catch {
+        /* empty */
+    }
 
     try {
         await module.exports.s3.emptyS3Bucket(s3Slug, account);
-    } catch (e) {}
+    } catch {
+        /* empty */
+    }
 
     try {
         await module.exports.s3.removeS3Bucket(s3Slug, account);
-    } catch (e) {}
+    } catch {
+        /* empty */
+    }
 
     try {
         await module.exports.elasticbeanstalk.removeElasticBeanstalkEnvironment(
             name,
             account,
         );
-    } catch (e) {}
+    } catch {
+        /* empty */
+    }
 
     try {
         await module.exports.elasticbeanstalk.removeElasticBeanstalkApplication(
             repo,
             account,
         );
-    } catch (e) {}
+    } catch {
+        /* empty */
+    }
 };

package/commands/create/services/aws/s3.js

@@ -154,6 +154,7 @@ module.exports.emptyS3Bucket = async (bucket, account) => {
     const client = createClient(S3Client, account);
 
     await Spinner.prototype.simple(`Emptying s3 bucket`, () => {
+        // eslint-disable-next-line no-async-promise-executor
         return new Promise(async (resolve, reject) => {
             try {
                 let ContinuationToken = null;

package/commands/create/services/bitbucket.js

@@ -18,7 +18,7 @@ module.exports.check = async () => {
         }
 
         return true;
-    } catch (error) {
+    } catch {
         spinner.update(
             `Failed to communicate with ${misc.bitbucket.workspace} on Bitbucket.`,
             "fail",
@@ -41,7 +41,7 @@ module.exports.remove = async (slug) => {
         } else {
             throw "";
         }
-    } catch (error) {
+    } catch {
         spinner.update(`Failed to delete ${slug} on Bitbucket.`, "fail");
     }
 };
@@ -98,14 +98,14 @@ module.exports.fork = async (slug, template) => {
         }
 
         return true;
-    } catch (error) {
+    } catch {
         spinner.update(`Failed to create ${slug} on Bitbucket.`, "fail");
 
         return false;
     }
 };
 
-module.exports.enablePipelines = async (slug, config) => {
+module.exports.enablePipelines = async (slug) => {
     let spinner = new Spinner(`Enabling pipelines for ${slug}`);
 
     try {

package/commands/create/services/egnyte.js

@@ -25,7 +25,7 @@ module.exports.request = async (username, password, host) => {
             !(await new Promise((resolve, reject) => {
                 exec(
                     `curl -I -k --ftp-ssl --ftp-pasv -u "${username.replace("$", "\\$")}:${password}" ftp://${host}`,
-                    (error, stdout, stderr) => {
+                    (error, stdout) => {
                         if (error) {
                             reject(error);
                             return;
@@ -42,7 +42,7 @@ module.exports.request = async (username, password, host) => {
         }
 
         return true;
-    } catch (error) {
+    } catch {
         spinner.update(`Failed to communicate with Egnyte.`, "fail");
 
         return false;

package/commands/create/services/gitlab.js

@@ -13,7 +13,7 @@ module.exports.check = async () => {
                 await new Promise((resolve, reject) => {
                     exec(
                         `curl -H "Content-Type:application/json" http://${misc.gitlab.workspace}/api/v4/users?private_token=${misc.gitlab.token}`,
-                        (error, stdout, stderr) => {
+                        (error, stdout) => {
                             if (error) {
                                 reject(error);
                                 return;
@@ -33,7 +33,7 @@ module.exports.check = async () => {
         }
 
         return true;
-    } catch (error) {
+    } catch {
         spinner.update(
             `Failed to communicate with ${misc.gitlab.workspace} on Gitlab.`,
             "fail",