slicejs-cli 3.3.0 → 3.4.1

package/tests/types-generator.test.js

@@ -1,8 +1,8 @@
 import { test } from 'node:test';
 import assert from 'node:assert/strict';
 import fs from 'node:fs';
-import os from 'node:os';
 import path from 'node:path';
+import { createTestProject, cleanupTestProject } from './helpers/setup.js';
 
 import {
   ensureEditorConfigForTypes,
@@ -151,206 +151,194 @@ test('generateDeclarationContent creates build typing map', () => {
 });
 
 test('generateTypesFile creates declaration file from local components', async () => {
-  const tmpRoot = fs.mkdtempSync(path.join(os.tmpdir(), 'slice-types-'));
+  const tmpRoot = await createTestProject();
   const srcDir = path.join(tmpRoot, 'src');
-  const visualDir = path.join(srcDir, 'Components', 'Visual', 'Button');
-  const noStaticDir = path.join(srcDir, 'Components', 'Visual', 'Tabs');
-  const serviceDir = path.join(srcDir, 'Components', 'Service', 'FetchManager');
-  const outputFile = path.join(srcDir, 'slice-build.generated.d.ts');
 
-  fs.mkdirSync(visualDir, { recursive: true });
-  fs.mkdirSync(noStaticDir, { recursive: true });
-  fs.mkdirSync(serviceDir, { recursive: true });
-
-  fs.writeFileSync(
-    path.join(srcDir, 'sliceConfig.json'),
-    JSON.stringify(
-      {
-        paths: {
-          components: {
-            Visual: { path: '/Components/Visual', type: 'Visual' },
-            Service: { path: '/Components/Service', type: 'Service' }
-          }
+  try {
+    const visualDir = path.join(srcDir, 'Components', 'Visual', 'Button');
+    const noStaticDir = path.join(srcDir, 'Components', 'Visual', 'Tabs');
+    const serviceDir = path.join(srcDir, 'Components', 'Service', 'FetchManager');
+    const outputFile = path.join(srcDir, 'slice-build.generated.d.ts');
+
+    fs.mkdirSync(noStaticDir, { recursive: true });
+
+    fs.writeFileSync(
+      path.join(visualDir, 'Button.js'),
+      `
+      export default class Button extends HTMLElement {
+        static props = {
+          value: { type: 'string', default: 'Button', allowedValues: ['Button', 'Submit'] },
+          size: { type: 'number', allowedValues: [12, 16] },
+          disabled: { type: 'boolean', default: false }
+        };
+      }
+      `,
+      'utf8'
+    );
+
+    fs.writeFileSync(
+      path.join(noStaticDir, 'Tabs.js'),
+      `
+      export default class Tabs extends HTMLElement {
+        constructor() {
+          super();
         }
-      },
-      null,
-      2
-    ),
-    'utf8'
-  );
-
-  fs.writeFileSync(
-    path.join(srcDir, 'Components', 'components.js'),
-    `const components = {"Button": "Visual", "Tabs": "Visual", "FetchManager": "Service"};\n\nexport default components;\n`,
-    'utf8'
-  );
-
-  fs.writeFileSync(
-    path.join(visualDir, 'Button.js'),
-    `
-    export default class Button extends HTMLElement {
-      static props = {
-        value: { type: 'string', default: 'Button', allowedValues: ['Button', 'Submit'] },
-        size: { type: 'number', allowedValues: [12, 16] },
-        disabled: { type: 'boolean', default: false }
-      };
-    }
-    `,
-    'utf8'
-  );
-
-  fs.writeFileSync(
-    path.join(noStaticDir, 'Tabs.js'),
-    `
-    export default class Tabs extends HTMLElement {
-      constructor() {
-        super();
       }
-    }
-    `,
-    'utf8'
-  );
-
-  fs.writeFileSync(
-    path.join(serviceDir, 'FetchManager.js'),
-    `
-    export default class FetchManager extends HTMLElement {
-      static props = {
-        baseUrl: { type: 'string', required: true }
-      };
-    }
-    `,
-    'utf8'
-  );
-
-  const result = await generateTypesFile({
-    projectRoot: tmpRoot,
-    outputPath: outputFile
-  });
-
-  assert.equal(result.componentsProcessed, 3);
-  assert.equal(fs.existsSync(result.outputPath), true);
-
-  const declaration = fs.readFileSync(result.outputPath, 'utf8');
-  assert.match(declaration, /export interface ButtonProps/);
-  assert.match(declaration, /value\?: 'Button' \| 'Submit';/);
-  assert.match(declaration, /size\?: 12 \| 16;/);
-  assert.match(declaration, /export interface TabsProps/);
-  assert.match(declaration, /\[key: string\]: unknown;/);
-  assert.match(declaration, /export interface FetchManagerProps/);
-  assert.match(declaration, /Tabs: TabsProps;/);
-  assert.match(declaration, /build<K extends SliceComponentName>/);
+      `,
+      'utf8'
+    );
+
+    fs.writeFileSync(
+      path.join(serviceDir, 'FetchManager.js'),
+      `
+      export default class FetchManager extends HTMLElement {
+        static props = {
+          baseUrl: { type: 'string', required: true }
+        };
+      }
+      `,
+      'utf8'
+    );
+
+    fs.writeFileSync(
+      path.join(srcDir, 'Components', 'components.js'),
+      'const components = {"Button": "Visual", "Tabs": "Visual", "FetchManager": "Service"};\n\nexport default components;\n',
+      'utf8'
+    );
+
+    const result = await generateTypesFile({
+      projectRoot: tmpRoot,
+      outputPath: outputFile
+    });
+
+    assert.equal(result.componentsProcessed, 3);
+    assert.equal(fs.existsSync(result.outputPath), true);
+
+    const declaration = fs.readFileSync(result.outputPath, 'utf8');
+    assert.match(declaration, /export interface ButtonProps/);
+    assert.match(declaration, /value\?: 'Button' \| 'Submit';/);
+    assert.match(declaration, /size\?: 12 \| 16;/);
+    assert.match(declaration, /export interface TabsProps/);
+    assert.match(declaration, /\[key: string\]: unknown;/);
+    assert.match(declaration, /export interface FetchManagerProps/);
+    assert.match(declaration, /Tabs: TabsProps;/);
+    assert.match(declaration, /build<K extends SliceComponentName>/);
+  } finally {
+    await cleanupTestProject(tmpRoot);
+  }
 });
 
 test('ensureEditorConfigForTypes creates jsconfig when missing', async () => {
-  const tmpRoot = fs.mkdtempSync(path.join(os.tmpdir(), 'slice-types-jsconfig-create-'));
+  const tmpRoot = await createTestProject();
   const outputFile = path.join(tmpRoot, 'src', 'slice-build.generated.d.ts');
 
-  fs.mkdirSync(path.dirname(outputFile), { recursive: true });
-  fs.writeFileSync(outputFile, 'export {};\n', 'utf8');
-
-  const result = await ensureEditorConfigForTypes({
-    projectRoot: tmpRoot,
-    outputPath: outputFile
-  });
-
-  assert.equal(result.mode, 'created_jsconfig');
-
-  const jsconfigPath = path.join(tmpRoot, 'jsconfig.json');
-  assert.equal(fs.existsSync(jsconfigPath), true);
-
-  const jsconfig = JSON.parse(fs.readFileSync(jsconfigPath, 'utf8'));
-  assert.equal(Array.isArray(jsconfig.include), true);
-  assert.equal(jsconfig.include.includes('src/Components/**/*.js'), true);
-  assert.equal(jsconfig.include.includes('src/**/*.d.ts'), true);
-  assert.equal(jsconfig.include.includes('api/**/*.js'), false);
-  assert.equal(jsconfig.include.includes('tests/**/*.js'), false);
-  assert.equal(jsconfig.include.includes('src/**/*.js'), false);
-  assert.equal(jsconfig.compilerOptions.checkJs, true);
-  assert.equal(jsconfig.compilerOptions.strictNullChecks, false);
-  assert.equal(jsconfig.compilerOptions.noImplicitAny, false);
-  assert.equal(jsconfig.compilerOptions.strict, false);
-  assert.equal(Array.isArray(jsconfig.exclude), true);
-  assert.equal(jsconfig.exclude.includes('src/libs/**'), true);
-  assert.equal(jsconfig.exclude.includes('tests/**'), true);
+  try {
+    fs.writeFileSync(outputFile, 'export {};\n', 'utf8');
+
+    const result = await ensureEditorConfigForTypes({
+      projectRoot: tmpRoot,
+      outputPath: outputFile
+    });
+
+    assert.equal(result.mode, 'created_jsconfig');
+
+    const jsconfigPath = path.join(tmpRoot, 'jsconfig.json');
+    assert.equal(fs.existsSync(jsconfigPath), true);
+
+    const jsconfig = JSON.parse(fs.readFileSync(jsconfigPath, 'utf8'));
+    assert.equal(Array.isArray(jsconfig.include), true);
+    assert.equal(jsconfig.include.includes('src/Components/**/*.js'), true);
+    assert.equal(jsconfig.include.includes('src/**/*.d.ts'), true);
+    assert.equal(jsconfig.include.includes('api/**/*.js'), false);
+    assert.equal(jsconfig.include.includes('tests/**/*.js'), false);
+    assert.equal(jsconfig.include.includes('src/**/*.js'), false);
+    assert.equal(jsconfig.compilerOptions.checkJs, true);
+    assert.equal(jsconfig.compilerOptions.strictNullChecks, false);
+    assert.equal(jsconfig.compilerOptions.noImplicitAny, false);
+    assert.equal(jsconfig.compilerOptions.strict, false);
+    assert.equal(Array.isArray(jsconfig.exclude), true);
+    assert.equal(jsconfig.exclude.includes('src/libs/**'), true);
+    assert.equal(jsconfig.exclude.includes('tests/**'), true);
+  } finally {
+    await cleanupTestProject(tmpRoot);
+  }
 });
 
-test('ensureNoCheckInPublicVendorFiles adds ts-nocheck to js files in publicFolders', async () => {
-  const tmpRoot = fs.mkdtempSync(path.join(os.tmpdir(), 'slice-types-publicfolders-'));
+test('types functions use PathHelper with explicit projectRoot', async () => {
+  const tmpRoot = await createTestProject({ visualComponents: ['Button'] });
   const srcDir = path.join(tmpRoot, 'src');
-  const libsDir = path.join(srcDir, 'libs', 'vendor');
-  const assetsDir = path.join(srcDir, 'assets', 'scripts');
-
-  fs.mkdirSync(libsDir, { recursive: true });
-  fs.mkdirSync(assetsDir, { recursive: true });
-
-  fs.writeFileSync(
-    path.join(srcDir, 'sliceConfig.json'),
-    JSON.stringify(
-      {
-        publicFolders: ['/libs', '/assets'],
-        paths: { components: {} }
-      },
-      null,
-      2
-    ),
-    'utf8'
-  );
-
-  const vendorA = path.join(libsDir, 'a.js');
-  const vendorB = path.join(assetsDir, 'b.js');
-  fs.writeFileSync(vendorA, 'const a = 1;\n', 'utf8');
-  fs.writeFileSync(vendorB, '// @ts-nocheck\nconst b = 2;\n', 'utf8');
-
-  const result = await ensureNoCheckInPublicVendorFiles(tmpRoot);
-
-  assert.equal(result.scannedFiles, 2);
-  assert.equal(result.updatedFiles, 1);
-  assert.equal(fs.readFileSync(vendorA, 'utf8').startsWith('// @ts-nocheck\n'), true);
-  assert.equal(fs.readFileSync(vendorB, 'utf8').startsWith('// @ts-nocheck\n'), true);
+  const outputFile = path.join(srcDir, 'slice-build.generated.d.ts');
+
+  try {
+    const visualDir = path.join(srcDir, 'Components', 'Visual', 'Button');
+    fs.writeFileSync(
+      path.join(visualDir, 'Button.js'),
+      `export default class Button extends HTMLElement {
+        static props = { value: { type: 'string' } };
+      }`,
+      'utf8'
+    );
+
+    const result = await generateTypesFile({
+      projectRoot: tmpRoot,
+      outputPath: outputFile
+    });
+
+    assert.equal(result.componentsProcessed, 1);
+    assert.equal(fs.existsSync(result.outputPath), true);
+
+    const declaration = fs.readFileSync(result.outputPath, 'utf8');
+    assert.match(declaration, /export interface ButtonProps/);
+    assert.match(declaration, /build<K extends SliceComponentName>/);
+  } finally {
+    await cleanupTestProject(tmpRoot);
+  }
 });
 
 test('ensureEditorConfigForTypes augments existing jsconfig include list', async () => {
-  const tmpRoot = fs.mkdtempSync(path.join(os.tmpdir(), 'slice-types-jsconfig-update-'));
+  const tmpRoot = await createTestProject();
   const outputFile = path.join(tmpRoot, 'src', 'slice-build.generated.d.ts');
   const jsconfigPath = path.join(tmpRoot, 'jsconfig.json');
 
-  fs.mkdirSync(path.dirname(outputFile), { recursive: true });
-  fs.writeFileSync(outputFile, 'export {};\n', 'utf8');
-  fs.writeFileSync(
-    jsconfigPath,
-    JSON.stringify(
-      {
-        compilerOptions: {
-          allowJs: true
+  try {
+    fs.writeFileSync(outputFile, 'export {};\n', 'utf8');
+    fs.writeFileSync(
+      jsconfigPath,
+      JSON.stringify(
+        {
+          compilerOptions: {
+            allowJs: true
+          },
+          include: ['src/Components/**/*.js', 'src/**/*.js', 'api/**/*.js', 'tests/**/*.js']
         },
-        include: ['src/Components/**/*.js', 'src/**/*.js', 'api/**/*.js', 'tests/**/*.js']
-      },
-      null,
-      2
-    ),
-    'utf8'
-  );
-
-  const result = await ensureEditorConfigForTypes({
-    projectRoot: tmpRoot,
-    outputPath: outputFile
-  });
-
-  assert.equal(result.mode, 'updated_jsconfig');
-
-  const jsconfig = JSON.parse(fs.readFileSync(jsconfigPath, 'utf8'));
-  assert.equal(jsconfig.include.includes('src/Components/**/*.js'), true);
-  assert.equal(jsconfig.include.includes('src/**/*.d.ts'), true);
-  assert.equal(jsconfig.include.includes('src/**/*.js'), false);
-  assert.equal(jsconfig.include.includes('api/**/*.js'), false);
-  assert.equal(jsconfig.include.includes('tests/**/*.js'), false);
-  assert.equal(jsconfig.compilerOptions.allowJs, true);
-  assert.equal(jsconfig.compilerOptions.checkJs, true);
-  assert.equal(jsconfig.compilerOptions.noImplicitAny, false);
-  assert.equal(jsconfig.compilerOptions.strictNullChecks, false);
-  assert.equal(jsconfig.compilerOptions.strict, false);
-  assert.equal(Array.isArray(jsconfig.exclude), true);
-  assert.equal(jsconfig.exclude.includes('src/libs/**'), true);
-  assert.equal(jsconfig.exclude.includes('tests/**'), true);
+        null,
+        2
+      ),
+      'utf8'
+    );
+
+    const result = await ensureEditorConfigForTypes({
+      projectRoot: tmpRoot,
+      outputPath: outputFile
+    });
+
+    assert.equal(result.mode, 'updated_jsconfig');
+
+    const jsconfig = JSON.parse(fs.readFileSync(jsconfigPath, 'utf8'));
+    assert.equal(jsconfig.include.includes('src/Components/**/*.js'), true);
+    assert.equal(jsconfig.include.includes('src/**/*.d.ts'), true);
+    assert.equal(jsconfig.include.includes('src/**/*.js'), false);
+    assert.equal(jsconfig.include.includes('api/**/*.js'), false);
+    assert.equal(jsconfig.include.includes('tests/**/*.js'), false);
+    assert.equal(jsconfig.compilerOptions.allowJs, true);
+    assert.equal(jsconfig.compilerOptions.checkJs, true);
+    assert.equal(jsconfig.compilerOptions.noImplicitAny, false);
+    assert.equal(jsconfig.compilerOptions.strictNullChecks, false);
+    assert.equal(jsconfig.compilerOptions.strict, false);
+    assert.equal(Array.isArray(jsconfig.exclude), true);
+    assert.equal(jsconfig.exclude.includes('src/libs/**'), true);
+    assert.equal(jsconfig.exclude.includes('tests/**'), true);
+  } finally {
+    await cleanupTestProject(tmpRoot);
+  }
 });

package/tests/update-manager-notifications.test.js

@@ -1,88 +1,88 @@
-import { test } from 'node:test';
-import assert from 'node:assert/strict';
-import Print from '../commands/Print.js';
-import { UpdateManager } from '../commands/utils/updateManager.js';
-
-test('notifyAvailableUpdates shows advisory output and never invokes interactive update flow', async () => {
-  const manager = new UpdateManager();
-  const calls = {
-    display: 0,
-    prompted: 0,
-    info: [],
-    error: []
-  };
-
-  manager.checkForUpdates = async () => ({
-    hasUpdates: true,
-    updates: [
-      {
-        name: 'slicejs-web-framework',
-        displayName: 'Slice.js Framework',
-        current: '2.4.3',
-        latest: '2.5.0',
-        type: 'framework'
-      }
-    ],
-    allCurrent: false
-  });
-
-  manager.displayUpdates = () => {
-    calls.display += 1;
-  };
-
-  manager.checkAndPromptUpdates = async () => {
-    calls.prompted += 1;
-    return true;
-  };
-
-  const originalInfo = Print.info;
-  const originalError = Print.error;
-
-  Print.info = (message) => calls.info.push(message);
-  Print.error = (message) => calls.error.push(message);
-
-  try {
-    const result = await manager.notifyAvailableUpdates();
-
-    assert.equal(result, true);
-    assert.equal(calls.display, 1);
-    assert.equal(calls.prompted, 0);
-    assert.equal(calls.error.length, 0);
-    assert.equal(calls.info.length, 1);
-    assert.match(calls.info[0], /slice update/);
-  } finally {
-    Print.info = originalInfo;
-    Print.error = originalError;
-  }
-});
-
-test('notifyAvailableUpdates returns false and prints nothing when no updates are available', async () => {
-  const manager = new UpdateManager();
-  const calls = {
-    display: 0,
-    info: []
-  };
-
-  manager.checkForUpdates = async () => ({
-    hasUpdates: false,
-    updates: [],
-    allCurrent: true
-  });
-
-  manager.displayUpdates = () => {
-    calls.display += 1;
-  };
-
-  const originalInfo = Print.info;
-  Print.info = (message) => calls.info.push(message);
-
-  try {
-    const result = await manager.notifyAvailableUpdates();
-
-    assert.equal(result, false);
-    assert.equal(calls.display, 0);
-    assert.equal(calls.info.length, 0);
-  } finally {
-    Print.info = originalInfo;
-  }
-});
+import { test } from 'node:test';
+import assert from 'node:assert/strict';
+import Print from '../commands/Print.js';
+import { UpdateManager } from '../commands/utils/updateManager.js';
+
+test('notifyAvailableUpdates shows advisory output and never invokes interactive update flow', async () => {
+  const manager = new UpdateManager();
+  const calls = {
+    display: 0,
+    prompted: 0,
+    info: [],
+    error: []
+  };
+
+  manager.checkForUpdates = async () => ({
+    hasUpdates: true,
+    updates: [
+      {
+        name: 'slicejs-web-framework',
+        displayName: 'Slice.js Framework',
+        current: '2.4.3',
+        latest: '2.5.0',
+        type: 'framework'
+      }
+    ],
+    allCurrent: false
+  });
+
+  manager.displayUpdates = () => {
+    calls.display += 1;
+  };
+
+  manager.checkAndPromptUpdates = async () => {
+    calls.prompted += 1;
+    return true;
+  };
+
+  const originalInfo = Print.info;
+  const originalError = Print.error;
+
+  Print.info = (message) => calls.info.push(message);
+  Print.error = (message) => calls.error.push(message);
+
+  try {
+    const result = await manager.notifyAvailableUpdates();
+
+    assert.equal(result, true);
+    assert.equal(calls.display, 1);
+    assert.equal(calls.prompted, 0);
+    assert.equal(calls.error.length, 0);
+    assert.equal(calls.info.length, 1);
+    assert.match(calls.info[0], /slice update/);
+  } finally {
+    Print.info = originalInfo;
+    Print.error = originalError;
+  }
+});
+
+test('notifyAvailableUpdates returns false and prints nothing when no updates are available', async () => {
+  const manager = new UpdateManager();
+  const calls = {
+    display: 0,
+    info: []
+  };
+
+  manager.checkForUpdates = async () => ({
+    hasUpdates: false,
+    updates: [],
+    allCurrent: true
+  });
+
+  manager.displayUpdates = () => {
+    calls.display += 1;
+  };
+
+  const originalInfo = Print.info;
+  Print.info = (message) => calls.info.push(message);
+
+  try {
+    const result = await manager.notifyAvailableUpdates();
+
+    assert.equal(result, false);
+    assert.equal(calls.display, 0);
+    assert.equal(calls.info.length, 0);
+  } finally {
+    Print.info = originalInfo;
+  }
+});

package/.github/workflows/docs-render-cicd.yml

@@ -1,65 +0,0 @@
-name: Docs Parse + Render CD
-
-on:
-  pull_request:
-    branches: [main]
-  push:
-    branches: [main]
-  workflow_dispatch:
-
-permissions:
-  contents: write
-
-jobs:
-  docs-pipeline:
-    runs-on: ubuntu-latest
-
-    steps:
-      - name: Checkout
-        uses: actions/checkout@v4
-        with:
-          fetch-depth: 0
-
-      - name: Setup Node
-        uses: actions/setup-node@v4
-        with:
-          node-version: '20'
-          cache: 'npm'
-
-      - name: Install dependencies
-        run: npm ci
-
-      - name: Validate markdown docs
-        run: npm run docs:lint-md
-
-      - name: Generate documentation artifacts
-        run: npm run docs:generate
-
-      - name: Check generated changes (PR)
-        if: github.event_name == 'pull_request'
-        run: |
-          if ! git diff --quiet; then
-            echo "Generated files are out of date. Run 'npm run docs:generate' and commit changes."
-            git status --short
-            exit 1
-          fi
-
-      - name: Commit generated docs (main)
-        if: github.event_name == 'push' && github.ref == 'refs/heads/main'
-        run: |
-          if ! git diff --quiet; then
-            git config user.name "github-actions[bot]"
-            git config user.email "41898282+github-actions[bot]@users.noreply.github.com"
-            git add src/Components src/Styles src/routes.js
-            git commit -m "chore(docs): auto-generate docs artifacts from markdown"
-            git push
-          else
-            echo "No generated changes to commit."
-          fi
-
-      - name: Trigger Render deploy
-        if: github.event_name == 'push' && github.ref == 'refs/heads/main' && secrets.RENDER_DEPLOY_HOOK_URL != ''
-        run: |
-          curl --fail --silent --show-error --request POST "$RENDER_DEPLOY_HOOK_URL"
-        env:
-          RENDER_DEPLOY_HOOK_URL: ${{ secrets.RENDER_DEPLOY_HOOK_URL }}