Sorting env

Author: owjs3901

dist/index.js

@@ -1,6 +1,6 @@
 {
 	"name": "collect-env",
-	"version": "1.0.2",
+	"version": "1.0.3",
 	"description": "Collect env values and write to a file",
 	"author": "JeongMin Oh",
 	"license": "Apache-2.0",

package.json

@@ -129,8 +129,8 @@ test('should filter environment variables by prefix', async () => {
 
   const expectedContent = [
     'API_KEY=secret123',
-    'API_URL=https://api.example.com',
     'API_TIMEOUT=5000',
+    'API_URL=https://api.example.com',
   ].join('\n')
 
   expect(mockWriteFile).toHaveBeenCalledWith(testOutput, expectedContent)
@@ -162,8 +162,8 @@ test('should remove prefix from environment variables', async () => {
 
   const expectedContent = [
     'KEY=secret123',
-    'URL=https://api.example.com',
     'TIMEOUT=5000',
+    'URL=https://api.example.com',
   ].join('\n')
 
   expect(mockWriteFile).toHaveBeenCalledWith(testOutput, expectedContent)

src/__tests__/run.test.ts

@@ -13,17 +13,16 @@ export async function run() {
   debug(`process.env: ${JSON.stringify(process.env)}`)
   debug(`secrets: ${JSON.stringify(secrets)}`)
 
-  try {
-    await writeFile(
-      output,
-      Object.entries({ ...secrets, ...process.env })
-        .filter(([key]) => key.startsWith(prefix))
-        .map(
-          ([key, value]) =>
-            `${removePrefix ? key.substring(prefix.length) : key}=${value}`,
-        )
-        .join('\n'),
+  const result = Object.entries({ ...secrets, ...process.env })
+    .filter(([key]) => key.startsWith(prefix))
+    .map(
+      ([key, value]) =>
+        `${removePrefix ? key.substring(prefix.length) : key}=${value}`,
     )
+  result.sort()
+
+  try {
+    await writeFile(output, result.join('\n'))
   } catch (err: unknown) {
     setFailed(err as Error)
   }