refactor: put this file into bin instead of src

Author: mfranzke

.npmignore

@@ -1,5 +1,5 @@
 /*
 /*/
 !/build/
-!/src/cli.js
+!/bin/cli.js
 CONTRIBUTION.md

src/cli.js renamed to bin/cli.js

@@ -8,14 +8,14 @@
 import { readFile, writeFile } from 'node:fs/promises';
 import path from 'node:path';
 import process from 'node:process';
-import { buildTimeTransform } from './index.js';
+import { buildTimeTransform } from '../src/index.js';
 
 const help = `
 CSS if() Build-time Transformation CLI
 
 Usage:
   npx css-if-transform <input.css> [output.css] [options]
-  node src/cli.js <input.css> [output.css] [options]
+  node bin/cli.js <input.css> [output.css] [options]
 
 Options:
   --minify         Minify the output CSS

css-if-polyfill-0.0.1.tgz

@@ -14,7 +14,7 @@
   "author": "Maximilian Franzke",
   "license": "MIT",
   "bin": {
-    "css-if-transform": "./src/cli.js"
+    "css-if-transform": "./bin/cli.js"
   },
   "main": "dist/index.cjs",
   "types": "dist/index.d.ts",
@@ -32,8 +32,8 @@
     }
   },
   "files": [
+    "bin/cli.js",
     "dist/",
-    "src/cli.js",
     "README.md",
     "LICENSE"
   ],

package-lock.json

@@ -33,7 +33,7 @@ describe('CLI Tool', () => {
 
 	test('transforms CSS file with output to file', async () => {
 		const { stdout } = await execAsync(
-			`node src/cli.js ${testInputFile} ${testOutputFile} --stats`
+			`node bin/cli.js ${testInputFile} ${testOutputFile} --stats`
 		);
 
 		expect(stdout).toContain('Reading CSS from');
@@ -49,7 +49,7 @@ describe('CLI Tool', () => {
 
 	test('outputs to stdout when no output file specified', async () => {
 		const { stdout } = await execAsync(
-			`node src/cli.js ${testInputFile} --stats`
+			`node bin/cli.js ${testInputFile} --stats`
 		);
 
 		expect(stdout).toContain('Transformation Statistics');
@@ -58,7 +58,7 @@ describe('CLI Tool', () => {
 	});
 
 	test('shows help when no arguments provided', async () => {
-		const { stdout } = await execAsync('node src/cli.js --help');
+		const { stdout } = await execAsync('node bin/cli.js --help');
 
 		expect(stdout).toContain('CSS if() Build-time Transformation CLI');
 		expect(stdout).toContain('Usage:');
@@ -67,7 +67,7 @@ describe('CLI Tool', () => {
 
 	test('handles minification option', async () => {
 		const { stdout } = await execAsync(
-			`node src/cli.js ${testInputFile} ${testOutputFile} --minify --stats`
+			`node bin/cli.js ${testInputFile} ${testOutputFile} --minify --stats`
 		);
 
 		expect(stdout).toContain('CSS was minified');
@@ -79,7 +79,7 @@ describe('CLI Tool', () => {
 
 	test('handles non-existent input file gracefully', async () => {
 		try {
-			await execAsync('node src/cli.js non-existent.css');
+			await execAsync('node bin/cli.js non-existent.css');
 			expect.fail('Should have thrown an error');
 		} catch (error) {
 			expect(error.stdout || error.stderr).toContain('Error');

package.json

@@ -11,6 +11,6 @@
 		"outDir": "./dist",
 		"rootDir": "./src"
 	},
-	"include": ["src/**/*"],
+	"include": ["src/**/*", "bin/cli.js"],
 	"exclude": ["node_modules", "dist", "test"]
 }

test/cli.test.js

@@ -5,6 +5,14 @@ const xoConfig = [
 		rules: {
 			'import-x/order': 0 // We use a prettier plugin to organize imports
 		}
+	},
+	// Provide an overwrite for bin/cli.js to not check for n/no-unpublished-bin rule
+	// This is needed because the bin/cli.js file is actually published to npm, so it seems to be a false positive
+	{
+		files: ['bin/cli.js'],
+		rules: {
+			'n/no-unpublished-bin': 0
+		}
 	}
 ];