Improve loadSecrets function

Return early if no env vars with valid secret references are found

dist/index.js

@@ -4179,6 +4179,9 @@ const loadSecrets = async (shouldExportEnv) => {
     // Iterate over them to find 1Password references, extract the secret values,
     // and make them available in the next steps either as step outputs or as environment variables.
     const res = await exec.getExecOutput(`sh -c "op env ls"`);
+    if (res.stdout === "") {
+        return;
+    }
     const envs = res.stdout.replace(/\n+$/g, "").split(/\r?\n/);
     for (const envName of envs) {
         extractSecret(envName, shouldExportEnv);

src/utils.test.ts

@@ -140,6 +140,14 @@ describe("loadSecrets", () => {
 		);
 	});
 
+	it("return early if no env vars with secrets found", async () => {
+		(exec.getExecOutput as jest.Mock).mockReturnValueOnce({ stdout: "" });
+		await loadSecrets(true);
+
+		expect(exec.getExecOutput).toHaveBeenCalledWith('sh -c "op env ls"');
+		expect(core.exportVariable).not.toHaveBeenCalled();
+	});
+
 	describe("core.exportVariable", () => {
 		it("is called when shouldExportEnv is true", async () => {
 			await loadSecrets(true);

src/utils.ts

@@ -77,6 +77,11 @@ export const loadSecrets = async (shouldExportEnv: boolean): Promise<void> => {
 	// Iterate over them to find 1Password references, extract the secret values,
 	// and make them available in the next steps either as step outputs or as environment variables.
 	const res = await exec.getExecOutput(`sh -c "op env ls"`);
+
+	if (res.stdout === "") {
+		return;
+	}
+
 	const envs = res.stdout.replace(/\n+$/g, "").split(/\r?\n/);
 	for (const envName of envs) {
 		extractSecret(envName, shouldExportEnv);