fix(setup-env): trim verbose comments per review

Address review feedback on PR #87 (shayan-bstack): reduce the two
explanatory comment blocks to the essential rationale, matching the
file's existing comment density. Rebuilt dist. No behaviour change.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: rounak610

setup-env/dist/index.js

@@ -34600,9 +34600,7 @@ class ActionInput {
       this.buildName = core.getInput(INPUT.BUILD_NAME);
       this.projectName = core.getInput(INPUT.PROJECT_NAME);
 
-      // Whether the workflow actually asked us for a name. _validateInput() replaces
-      // both fields with generated defaults when they are blank, so the only place
-      // this can be observed is here, before validation runs.
+      // Capture before _validateInput() replaces blanks with generated defaults.
       this.buildNameProvided = Boolean(this.buildName && this.buildName.trim());
       this.projectNameProvided = Boolean(this.projectName && this.projectName.trim());
       this.githubApp = core.getInput(INPUT.GITHUB_APP);
@@ -34638,14 +34636,8 @@ class ActionInput {
     core.exportVariable(ENV_VARS.BROWSERSTACK_ACCESS_KEY, this.accessKey);
     core.info(`Use ${ENV_VARS.BROWSERSTACK_ACCESS_KEY} environment variable for your access key in your tests\n`);
 
-    // Only export the name variables when the workflow actually supplied them.
-    //
-    // Every BrowserStack SDK resolves names as: CLI args > env vars > config file.
-    // Exporting a generated default here therefore does not "fill a gap" -- it
-    // OUTRANKS whatever the user configured in browserstack.json / browserstack.yml
-    // and silently replaces it. Users who want the generated values still get them
-    // by opting in with the `BUILD_INFO` and `REPO_NAME` tokens, which
-    // InputValidator already understands.
+    // Export only when supplied: an env var outranks the user's browserstack.json,
+    // so a generated default would silently replace it. BUILD_INFO / REPO_NAME opt in.
     if (this.projectNameProvided) {
       core.exportVariable(ENV_VARS.BROWSERSTACK_PROJECT_NAME, this.projectName);
       core.info(`${ENV_VARS.BROWSERSTACK_PROJECT_NAME} environment variable set as: ${this.projectName}`);

setup-env/src/actionInput/index.js

@@ -33,9 +33,7 @@ class ActionInput {
       this.buildName = core.getInput(INPUT.BUILD_NAME);
       this.projectName = core.getInput(INPUT.PROJECT_NAME);
 
-      // Whether the workflow actually asked us for a name. _validateInput() replaces
-      // both fields with generated defaults when they are blank, so the only place
-      // this can be observed is here, before validation runs.
+      // Capture before _validateInput() replaces blanks with generated defaults.
       this.buildNameProvided = Boolean(this.buildName && this.buildName.trim());
       this.projectNameProvided = Boolean(this.projectName && this.projectName.trim());
       this.githubApp = core.getInput(INPUT.GITHUB_APP);
@@ -71,14 +69,8 @@ class ActionInput {
     core.exportVariable(ENV_VARS.BROWSERSTACK_ACCESS_KEY, this.accessKey);
     core.info(`Use ${ENV_VARS.BROWSERSTACK_ACCESS_KEY} environment variable for your access key in your tests\n`);
 
-    // Only export the name variables when the workflow actually supplied them.
-    //
-    // Every BrowserStack SDK resolves names as: CLI args > env vars > config file.
-    // Exporting a generated default here therefore does not "fill a gap" -- it
-    // OUTRANKS whatever the user configured in browserstack.json / browserstack.yml
-    // and silently replaces it. Users who want the generated values still get them
-    // by opting in with the `BUILD_INFO` and `REPO_NAME` tokens, which
-    // InputValidator already understands.
+    // Export only when supplied: an env var outranks the user's browserstack.json,
+    // so a generated default would silently replace it. BUILD_INFO / REPO_NAME opt in.
     if (this.projectNameProvided) {
       core.exportVariable(ENV_VARS.BROWSERSTACK_PROJECT_NAME, this.projectName);
       core.info(`${ENV_VARS.BROWSERSTACK_PROJECT_NAME} environment variable set as: ${this.projectName}`);