Merge pull request #51 from Sharmaz/chore/internal-improvements

Chore/internal improvements

Author: Sharmaz

.github/workflows/e2e_tests.yml

@@ -8,13 +8,14 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        node: [20]
+        node: [24]
     steps:
       - uses: actions/checkout@v4
-      - name: Setup Node 20
+      - name: Setup Node ${{ matrix.node }}
         uses: actions/setup-node@v4
         with:
-          node-version: 20
-      - run: npm install
+          node-version: ${{ matrix.node }}
+          cache: 'npm'
+      - run: npm ci
       - run: npm run build
       - run: npm run test:e2e

.github/workflows/linter.yml

@@ -8,12 +8,13 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        node: [20]
+        node: [24]
     steps:
       - uses: actions/checkout@v4
-      - name: Setup Node 20
+      - name: Setup Node ${{ matrix.node }}
         uses: actions/setup-node@v4
         with:
-          node-version: 20
-      - run: npm install
+          node-version: ${{ matrix.node }}
+          cache: 'npm'
+      - run: npm ci
       - run: npm run lint

.github/workflows/publish.yml

@@ -6,18 +6,17 @@ on:
 jobs:
   build:
     runs-on: ubuntu-latest
-    strategy:
-      matrix:
-        node: [20]
+    permissions:
+      contents: read
+      id-token: write
     steps:
       - uses: actions/checkout@v4
-      - name: Setup Node 20
+      - name: Setup Node
         uses: actions/setup-node@v4
         with:
-          node-version: 20
+          node-version: 22
           registry-url: 'https://registry.npmjs.org'
+      - run: npm install -g npm@latest
       - run: npm ci
       - run: npm run build
-      - run: npm publish
-        env:
-          NODE_AUTH_TOKEN: ${{secrets.NPM_TOKEN}}
+      - run: npm publish --provenance

.github/workflows/unit_tests.yml

@@ -8,13 +8,14 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        node: [20]
+        node: [24]
     steps:
       - uses: actions/checkout@v4
-      - name: Setup Node 20
+      - name: Setup Node ${{ matrix.node }}
         uses: actions/setup-node@v4
         with:
-          node-version: 20
-      - run: npm install
+          node-version: ${{ matrix.node }}
+          cache: 'npm'
+      - run: npm ci
       - run: npm run build
       - run: npm run test:unit

.gitignore

@@ -89,3 +89,6 @@ lerna-debug.log
 # System Files
 .DS_Store
 Thumbs.db
+
+# AI generated files
+CLAUDE.md

.prettierrc.json

@@ -1,6 +1,6 @@
 {
   "tabWidth": 2,
-  "useTabs": true,
+  "useTabs": false,
   "printWidth": 120,
   "semi": true,
   "trailingComma": "es5",

__tests__/__mocks__/dataMock.js

@@ -3,7 +3,6 @@ import { join } from 'node:path';
 
 import templateCategories from '../../src/templateList';
 
-
 const dirname = url.fileURLToPath(new URL('.', import.meta.url));
 const mainPath = join(dirname, '../..');
 const appNameMock = 'test-app';

__tests__/e2e/react_rspack.e2e.test.js

@@ -8,7 +8,6 @@ beforeAll(() => rm(genPath, { recursive: true, force: true }));
 afterEach(() => rm(genPath, { recursive: true, force: true }));
 
 describe('react-rspack template', () => {
-
   test('prompt shows React Rust', async () => {
     const { exitCode, stdout } = await runTest([mainPath], [ENTER, DOWN, DOWN, DOWN]);
 
@@ -18,7 +17,7 @@ describe('react-rspack template', () => {
 
   test('implemented react-rspack template', async () => {
     await execa`node index.js --template react-rspack ${appNameMock}`;
-    
+
     const packageJson = readFileSync(`${genPath}/package.json`, 'utf8');
     expect(packageJson).toContain('rspack');
   });

__tests__/unit/appName.unit.test.js

@@ -11,7 +11,7 @@ describe('App name fuctions', () => {
     const formattedNameFromSpaces = formatAppName(wrongFormatSpaces);
     const formattedNameFromLowDashes = formatAppName(wrongFormatLowDash);
     const formattedNameFromUpperCases = formatAppName(wrongFormatUpperCase);
-  
+
     expect(formattedNameFromSpaces).toBe(appNameMock);
     expect(formattedNameFromLowDashes).toBe(appNameMock);
     expect(formattedNameFromUpperCases).toBe(appNameMock);

__tests__/unit/colors.unit.test.js

@@ -0,0 +1,80 @@
+import { describe, test, expect } from '@jest/globals';
+import { separator, setColor, setUnderlineOff } from '../../src/utils/colors';
+
+describe('separator', () => {
+  test('returns a string with correct padding', () => {
+    const result = separator('React');
+    expect(typeof result).toBe('string');
+    expect(result).toContain('|');
+  });
+
+  test('shorter titles produce more padding', () => {
+    const short = separator('React');
+    const long = separator('React Dev Webpack');
+    expect(short.length).toBeGreaterThan(long.length);
+  });
+});
+
+describe('setColor', () => {
+  test('returns colored string for Frontend', () => {
+    const result = setColor('Frontend');
+    expect(result).toContain('Frontend');
+  });
+
+  test('returns colored string for Backend', () => {
+    const result = setColor('Backend');
+    expect(result).toContain('Backend');
+  });
+
+  test('returns colored string for VanillaJS', () => {
+    const result = setColor('VanillaJS');
+    expect(result).toContain('VanillaJS');
+  });
+
+  test('returns colored string for NodeJS', () => {
+    const result = setColor('NodeJS');
+    expect(result).toContain('NodeJS');
+  });
+
+  test('returns colored string for values containing Node', () => {
+    const result = setColor('Node Express');
+    expect(result).toContain('Node Express');
+  });
+
+  test('returns colored string for values containing JavaScript', () => {
+    const result = setColor('React JavaScript');
+    expect(result).toContain('React JavaScript');
+  });
+
+  test('returns colored string for values containing Rspack', () => {
+    const result = setColor('React Rspack');
+    expect(result).toContain('React Rspack');
+  });
+
+  test('returns colored string for values containing Rust', () => {
+    const result = setColor('React Rust');
+    expect(result).toContain('React Rust');
+  });
+
+  test('returns colored string for values containing TypeScript', () => {
+    const result = setColor('React TypeScript');
+    expect(result).toContain('React TypeScript');
+  });
+
+  test('returns colored string for values containing React', () => {
+    const result = setColor('React');
+    expect(result).toContain('React');
+  });
+
+  test('returns uncolored string for unknown values', () => {
+    const result = setColor('Unknown');
+    expect(result).toBe('Unknown');
+  });
+});
+
+describe('setUnderlineOff', () => {
+  test('returns a reset string', () => {
+    const result = setUnderlineOff('some text');
+    expect(result).toContain('some text');
+  });
+});