format code for tests --format flag

Author: Chrilleweb

CHANGELOG.md

@@ -15,15 +15,18 @@ Thanks for your interest in contributing to dotenv-diff! Contributions of all ki
 ## Guidelines
 
 **Code Quality**
+
 - Keep changes small and focused
 - Follow the existing code style and project structure
 - Write clear, descriptive commit messages
 
 **Testing & Documentation**
+
 - Add or update tests when introducing new behavior
 - Update the README or documentation when relevant
 
 **Communication**
+
 - For larger changes or new features, open an issue first to discuss
 - Avoid large refactors or breaking changes unless discussed beforehand
 
@@ -39,4 +42,4 @@ Feel free to open an issue for discussion — all feedback is welcome.
 
 ---
 
-Thanks for contributing! 💚
+Thanks for contributing! 💚

CONTRIBUTING.md

@@ -1,11 +1,6 @@
 {
   "strict": false,
   "example": ".env.example",
-  "ignore": [
-    "NODE_ENV",
-    "VITE_MODE"
-  ],
-  "ignoreUrls": [
-    "https://example.com"
-  ]
-}
+  "ignore": ["NODE_ENV", "VITE_MODE"],
+  "ignoreUrls": ["https://example.com"]
+}

dotenv-diff.config.json

@@ -18,11 +18,10 @@ export default [
       ...tseslint.configs.recommended.rules,
       'no-unused-vars': 'off',
       '@typescript-eslint/no-unused-vars': ['warn'],
-      'quotes': ['warn', 'single', { avoidEscape: true }],
+      quotes: ['warn', 'single', { avoidEscape: true }],
     },
   },
   {
     ignores: ['dist/**'],
   },
 ];
-

eslint.config.js

@@ -29,7 +29,7 @@
     "test:unit": "vitest run test/unit",
     "test:e2e": "vitest run test/e2e",
     "lint": "eslint ./src --ext .ts",
-    "format": "prettier --write \"src/**/*.ts\" \"src/*.ts\"",
+    "format": "prettier --write \"**/*.{ts,js,json,md,yml}\"",
     "start": "node dist/bin/dotenv-diff.js",
     "dotenv-diff": "dotenv-diff",
     "prepublishOnly": "npm run lint && npm test && npm run build"

package.json

@@ -29,24 +29,24 @@ function tmpDir() {
 
 describe('added .env to gitignore with --compare and --fix', () => {
   it('will warn about .env not ignored by .gitignore', () => {
-      const cwd = tmpDir();
-  
-      fs.mkdirSync(path.join(cwd, '.git'));
-      fs.writeFileSync(path.join(cwd, '.env'), 'API_KEY=test\n');
-      fs.writeFileSync(path.join(cwd, '.env.example'), 'API_KEY=test\n');
-
-      fs.writeFileSync(path.join(cwd, '.gitignore'), 'node_modules\n');
-  
-      fs.mkdirSync(path.join(cwd, 'src'), { recursive: true });
-      fs.writeFileSync(
-        path.join(cwd, 'src', 'index.ts'),
-        `const apiKey = process.env.API_KEY;`.trimStart(),
-      );
-  
-      const res = runCli(cwd, ['--compare', '--fix']);
-  
-      expect(res.status).toBe(0);
-      expect(res.stdout).toContain('Auto-fix applied:');
-      expect(res.stdout).toContain('Added .env to .gitignore');
-    });
+    const cwd = tmpDir();
+
+    fs.mkdirSync(path.join(cwd, '.git'));
+    fs.writeFileSync(path.join(cwd, '.env'), 'API_KEY=test\n');
+    fs.writeFileSync(path.join(cwd, '.env.example'), 'API_KEY=test\n');
+
+    fs.writeFileSync(path.join(cwd, '.gitignore'), 'node_modules\n');
+
+    fs.mkdirSync(path.join(cwd, 'src'), { recursive: true });
+    fs.writeFileSync(
+      path.join(cwd, 'src', 'index.ts'),
+      `const apiKey = process.env.API_KEY;`.trimStart(),
+    );
+
+    const res = runCli(cwd, ['--compare', '--fix']);
+
+    expect(res.status).toBe(0);
+    expect(res.stdout).toContain('Auto-fix applied:');
+    expect(res.stdout).toContain('Added .env to .gitignore');
+  });
 });

test/e2e/cli.compare.e2e.test.ts

@@ -30,22 +30,22 @@ function tmpDir() {
 describe('Expiration Warnings', () => {
   it('warns about expired environment variables', () => {
     const cwd = tmpDir();
-    
+
     // Create .env with an expired variable (yesterday)
     const yesterday = new Date();
     yesterday.setDate(yesterday.getDate() - 1);
     const expiredDate = yesterday.toISOString().split('T')[0]; // YYYY-MM-DD format
-    
+
     fs.writeFileSync(
       path.join(cwd, '.env'),
-      `# @expire ${expiredDate}\nEXPIRED_API_KEY=secret123\n\nVALID_KEY=value456\n`
+      `# @expire ${expiredDate}\nEXPIRED_API_KEY=secret123\n\nVALID_KEY=value456\n`,
     );
 
     // Create a file that uses these env vars
     fs.mkdirSync(path.join(cwd, 'src'), { recursive: true });
     fs.writeFileSync(
       path.join(cwd, 'src', 'index.js'),
-      'console.log(process.env.EXPIRED_API_KEY, process.env.VALID_KEY);'
+      'console.log(process.env.EXPIRED_API_KEY, process.env.VALID_KEY);',
     );
 
     const res = runCli(cwd, []);
@@ -58,22 +58,22 @@ describe('Expiration Warnings', () => {
 
   it('warns about soon-to-expire environment variables', () => {
     const cwd = tmpDir();
-    
+
     // Create .env with a variable expiring in 3 days
     const futureDate = new Date();
     futureDate.setDate(futureDate.getDate() + 3);
     const expiringDate = futureDate.toISOString().split('T')[0];
-    
+
     fs.writeFileSync(
       path.join(cwd, '.env'),
-      `# @expire ${expiringDate}\nSOON_EXPIRED_KEY=secret789\n\nVALID_KEY=value123\n`
+      `# @expire ${expiringDate}\nSOON_EXPIRED_KEY=secret789\n\nVALID_KEY=value123\n`,
     );
 
     // Create a file that uses these env vars
     fs.mkdirSync(path.join(cwd, 'src'), { recursive: true });
     fs.writeFileSync(
       path.join(cwd, 'src', 'index.js'),
-      'console.log(process.env.SOON_EXPIRED_KEY, process.env.VALID_KEY);'
+      'console.log(process.env.SOON_EXPIRED_KEY, process.env.VALID_KEY);',
     );
 
     const res = runCli(cwd, []);
@@ -86,47 +86,49 @@ describe('Expiration Warnings', () => {
 
   it('does not warn when no expiration comments are present', () => {
     const cwd = tmpDir();
-    
+
     fs.writeFileSync(
       path.join(cwd, '.env'),
-      `API_KEY=secret123\nVALID_KEY=value456\n`
+      `API_KEY=secret123\nVALID_KEY=value456\n`,
     );
 
     // Create a file that uses these env vars
     fs.mkdirSync(path.join(cwd, 'src'), { recursive: true });
     fs.writeFileSync(
       path.join(cwd, 'src', 'index.js'),
-      'console.log(process.env.API_KEY, process.env.VALID_KEY);'
+      'console.log(process.env.API_KEY, process.env.VALID_KEY);',
     );
 
     const res = runCli(cwd, []);
 
     expect(res.status).toBe(0);
-    expect(res.stdout).not.toContain('Environment variables with expiration dates');
+    expect(res.stdout).not.toContain(
+      'Environment variables with expiration dates',
+    );
     expect(res.stdout).not.toContain('expired');
   });
 
   it('handles multiple expiration patterns correctly', () => {
     const cwd = tmpDir();
-    
+
     const yesterday = new Date();
     yesterday.setDate(yesterday.getDate() - 1);
     const expiredDate = yesterday.toISOString().split('T')[0];
-    
+
     const futureDate = new Date();
     futureDate.setDate(futureDate.getDate() + 7);
     const validDate = futureDate.toISOString().split('T')[0];
-    
+
     fs.writeFileSync(
       path.join(cwd, '.env'),
-      `# @expire ${expiredDate}\nOLD_TOKEN=expired123\n\n# This one is still valid\n# @expire ${validDate}\nNEW_TOKEN=valid456\n\nNO_EXPIRY=permanent\n`
+      `# @expire ${expiredDate}\nOLD_TOKEN=expired123\n\n# This one is still valid\n# @expire ${validDate}\nNEW_TOKEN=valid456\n\nNO_EXPIRY=permanent\n`,
     );
 
     // Create a file that uses these env vars
     fs.mkdirSync(path.join(cwd, 'src'), { recursive: true });
     fs.writeFileSync(
       path.join(cwd, 'src', 'index.js'),
-      'console.log(process.env.OLD_TOKEN, process.env.NEW_TOKEN, process.env.NO_EXPIRY);'
+      'console.log(process.env.OLD_TOKEN, process.env.NEW_TOKEN, process.env.NO_EXPIRY);',
     );
 
     const res = runCli(cwd, []);
@@ -141,21 +143,21 @@ describe('Expiration Warnings', () => {
 
   it('does not warn about expiration when --no-expire-warnings is used', () => {
     const cwd = tmpDir();
-    
+
     const yesterday = new Date();
     yesterday.setDate(yesterday.getDate() - 1);
     const expiredDate = yesterday.toISOString().split('T')[0];
-    
+
     fs.writeFileSync(
       path.join(cwd, '.env'),
-      `# @expire ${expiredDate}\nEXPIRED_KEY=secret123\n\nVALID_KEY=value456\n`
+      `# @expire ${expiredDate}\nEXPIRED_KEY=secret123\n\nVALID_KEY=value456\n`,
     );
 
     // Create a file that uses these env vars
     fs.mkdirSync(path.join(cwd, 'src'), { recursive: true });
     fs.writeFileSync(
       path.join(cwd, 'src', 'index.js'),
-      'console.log(process.env.EXPIRED_KEY, process.env.VALID_KEY);'
+      'console.log(process.env.EXPIRED_KEY, process.env.VALID_KEY);',
     );
 
     const res = runCli(cwd, ['--no-expire-warnings']);
@@ -166,21 +168,21 @@ describe('Expiration Warnings', () => {
 
   it('supports different comment styles for expiration', () => {
     const cwd = tmpDir();
-    
+
     const yesterday = new Date();
     yesterday.setDate(yesterday.getDate() - 1);
     const expiredDate = yesterday.toISOString().split('T')[0];
-    
+
     fs.writeFileSync(
       path.join(cwd, '.env'),
-      `// @expire ${expiredDate}\nJS_STYLE_KEY=value1\n\n# @expire ${expiredDate}\nSHELL_STYLE_KEY=value2\n\n@expire ${expiredDate}\nNO_COMMENT_KEY=value3\n`
+      `// @expire ${expiredDate}\nJS_STYLE_KEY=value1\n\n# @expire ${expiredDate}\nSHELL_STYLE_KEY=value2\n\n@expire ${expiredDate}\nNO_COMMENT_KEY=value3\n`,
     );
 
     // Create a file that uses these env vars
     fs.mkdirSync(path.join(cwd, 'src'), { recursive: true });
     fs.writeFileSync(
       path.join(cwd, 'src', 'index.js'),
-      'console.log(process.env.JS_STYLE_KEY, process.env.SHELL_STYLE_KEY, process.env.NO_COMMENT_KEY);'
+      'console.log(process.env.JS_STYLE_KEY, process.env.SHELL_STYLE_KEY, process.env.NO_COMMENT_KEY);',
     );
 
     const res = runCli(cwd, []);
@@ -194,21 +196,21 @@ describe('Expiration Warnings', () => {
 
   it('exits with error in strict mode when expired variables exist', () => {
     const cwd = tmpDir();
-    
+
     const yesterday = new Date();
     yesterday.setDate(yesterday.getDate() - 1);
     const expiredDate = yesterday.toISOString().split('T')[0];
-    
+
     fs.writeFileSync(
       path.join(cwd, '.env'),
-      `# @expire ${expiredDate}\nEXPIRED_KEY=secret123\n`
+      `# @expire ${expiredDate}\nEXPIRED_KEY=secret123\n`,
     );
 
     // Create a file that uses the env var
     fs.mkdirSync(path.join(cwd, 'src'), { recursive: true });
     fs.writeFileSync(
       path.join(cwd, 'src', 'index.js'),
-      'console.log(process.env.EXPIRED_KEY);'
+      'console.log(process.env.EXPIRED_KEY);',
     );
 
     const res = runCli(cwd, ['--strict']);
@@ -218,4 +220,4 @@ describe('Expiration Warnings', () => {
     expect(res.stdout).toContain('EXPIRED_KEY');
     expect(res.stdout).toContain('EXPIRED');
   });
-});
+});

test/e2e/cli.detectExpired.e2e.test.ts

@@ -445,45 +445,44 @@ describe('duplicate detection', () => {
     expect(res.stdout).toContain('NEW_PHP_VAR');
   });
   it('should exclude file when using --exclude-files', () => {
-  const cwd = tmpDir();
-  fs.writeFileSync(path.join(cwd, '.env'), 'EXISTING=value\n');
-  fs.writeFileSync(
-    path.join(cwd, 'index.php'),
-    `
+    const cwd = tmpDir();
+    fs.writeFileSync(path.join(cwd, '.env'), 'EXISTING=value\n');
+    fs.writeFileSync(
+      path.join(cwd, 'index.php'),
+      `
       <?php
       $var = getenv('NEW_PHP_VAR');
     `,
-  );
+    );
 
-  // Exclude .php files → NEW_PHP_VAR må ikke blive fundet
-  const res = runCli(cwd, ['--scan-usage', '--exclude-files', '*.php']);
-  expect(res.status).toBe(0); // exit clean
-  expect(res.stdout).not.toContain('NEW_PHP_VAR');
-});
+    // Exclude .php files → NEW_PHP_VAR må ikke blive fundet
+    const res = runCli(cwd, ['--scan-usage', '--exclude-files', '*.php']);
+    expect(res.status).toBe(0); // exit clean
+    expect(res.stdout).not.toContain('NEW_PHP_VAR');
+  });
 
-it('should only scan files provided with --files', () => {
-  const cwd = tmpDir();
-  fs.writeFileSync(path.join(cwd, '.env'), 'EXISTING=value\n');
+  it('should only scan files provided with --files', () => {
+    const cwd = tmpDir();
+    fs.writeFileSync(path.join(cwd, '.env'), 'EXISTING=value\n');
 
-  // To filer: en php med NEW_PHP_VAR, en js med NEW_JS_VAR
-  fs.writeFileSync(
-    path.join(cwd, 'index.php'),
-    `
+    // To filer: en php med NEW_PHP_VAR, en js med NEW_JS_VAR
+    fs.writeFileSync(
+      path.join(cwd, 'index.php'),
+      `
       <?php
       $var = getenv('NEW_PHP_VAR');
     `,
-  );
-  fs.writeFileSync(
-    path.join(cwd, 'index.js'),
-    `
+    );
+    fs.writeFileSync(
+      path.join(cwd, 'index.js'),
+      `
       console.log(process.env.NEW_JS_VAR);
     `,
-  );
-
-  const res = runCli(cwd, ['--scan-usage', '--files', 'index.js']);
-  expect(res.status).toBe(1);
-  expect(res.stdout).toContain('NEW_JS_VAR');
-  expect(res.stdout).not.toContain('NEW_PHP_VAR');
-});
+    );
 
+    const res = runCli(cwd, ['--scan-usage', '--files', 'index.js']);
+    expect(res.status).toBe(1);
+    expect(res.stdout).toContain('NEW_JS_VAR');
+    expect(res.stdout).not.toContain('NEW_PHP_VAR');
+  });
 });