slicejs-cli 3.3.0 → 3.4.0

package/tests/local-cli-delegation.test.js

@@ -1,79 +1,79 @@
-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 {
-  findNearestLocalCliEntry,
-  resolveLocalCliCandidate,
-  shouldDelegateToLocalCli,
-  isLocalDelegationDisabled
-} from '../commands/utils/LocalCliDelegation.js';
-
-test('isLocalDelegationDisabled returns true when env flag is set', () => {
-  const env = { SLICE_NO_LOCAL_DELEGATION: '1' };
-  assert.equal(isLocalDelegationDisabled(env), true);
-});
-
-test('isLocalDelegationDisabled returns false when env flag is missing', () => {
-  const env = {};
-  assert.equal(isLocalDelegationDisabled(env), false);
-});
-
-test('shouldDelegateToLocalCli is false when candidate is null', () => {
-  assert.equal(shouldDelegateToLocalCli('/tmp/current/client.js', null), false);
-});
-
-test('shouldDelegateToLocalCli is false when candidate realpath equals current realpath', () => {
-  const same = '/tmp/current/client.js';
-  assert.equal(shouldDelegateToLocalCli(same, same), false);
-});
-
-test('shouldDelegateToLocalCli is true when candidate differs from current', () => {
-  const current = '/tmp/global/client.js';
-  const local = '/tmp/project/node_modules/slicejs-cli/client.js';
-  assert.equal(shouldDelegateToLocalCli(current, local), true);
-});
-
-test('findNearestLocalCliEntry returns null when no candidate resolver hits', () => {
-  const cwd = path.join('/tmp', 'slice-nonexistent-project');
-  const resolver = () => null;
-  const result = findNearestLocalCliEntry(cwd, resolver);
-  assert.equal(result, null);
-});
-
-test('findNearestLocalCliEntry returns first match while traversing upward', () => {
-  const cwd = '/repo/apps/web/src';
-  const calls = [];
-  const resolver = (dir) => {
-    calls.push(dir);
-    if (dir === '/repo/apps/web') {
-      return '/repo/apps/web/node_modules/slicejs-cli/client.js';
-    }
-    return null;
-  };
-
-  const result = findNearestLocalCliEntry(cwd, resolver);
-
-  assert.equal(result, '/repo/apps/web/node_modules/slicejs-cli/client.js');
-  assert.deepEqual(calls, ['/repo/apps/web/src', '/repo/apps/web']);
-});
-
-test('findNearestLocalCliEntry returns null when resolver is not a function', () => {
-  const result = findNearestLocalCliEntry('/repo/apps/web/src', null);
-  assert.equal(result, null);
-});
-
-test('resolveLocalCliCandidate returns null when candidate path is a directory', () => {
-  const tempRoot = fs.mkdtempSync(path.join(os.tmpdir(), 'slice-cli-delegation-'));
-  const candidateDirectory = path.join(tempRoot, 'node_modules', 'slicejs-cli', 'client.js');
-
-  fs.mkdirSync(candidateDirectory, { recursive: true });
-
-  try {
-    const result = resolveLocalCliCandidate(tempRoot);
-    assert.equal(result, null);
-  } finally {
-    fs.rmSync(tempRoot, { recursive: true, force: true });
-  }
-});
+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 {
+  findNearestLocalCliEntry,
+  resolveLocalCliCandidate,
+  shouldDelegateToLocalCli,
+  isLocalDelegationDisabled
+} from '../commands/utils/LocalCliDelegation.js';
+
+test('isLocalDelegationDisabled returns true when env flag is set', () => {
+  const env = { SLICE_NO_LOCAL_DELEGATION: '1' };
+  assert.equal(isLocalDelegationDisabled(env), true);
+});
+
+test('isLocalDelegationDisabled returns false when env flag is missing', () => {
+  const env = {};
+  assert.equal(isLocalDelegationDisabled(env), false);
+});
+
+test('shouldDelegateToLocalCli is false when candidate is null', () => {
+  assert.equal(shouldDelegateToLocalCli('/tmp/current/client.js', null), false);
+});
+
+test('shouldDelegateToLocalCli is false when candidate realpath equals current realpath', () => {
+  const same = '/tmp/current/client.js';
+  assert.equal(shouldDelegateToLocalCli(same, same), false);
+});
+
+test('shouldDelegateToLocalCli is true when candidate differs from current', () => {
+  const current = '/tmp/global/client.js';
+  const local = '/tmp/project/node_modules/slicejs-cli/client.js';
+  assert.equal(shouldDelegateToLocalCli(current, local), true);
+});
+
+test('findNearestLocalCliEntry returns null when no candidate resolver hits', () => {
+  const cwd = path.join('/tmp', 'slice-nonexistent-project');
+  const resolver = () => null;
+  const result = findNearestLocalCliEntry(cwd, resolver);
+  assert.equal(result, null);
+});
+
+test('findNearestLocalCliEntry returns first match while traversing upward', () => {
+  const cwd = '/repo/apps/web/src';
+  const calls = [];
+  const resolver = (dir) => {
+    calls.push(dir);
+    if (dir === '/repo/apps/web') {
+      return '/repo/apps/web/node_modules/slicejs-cli/client.js';
+    }
+    return null;
+  };
+
+  const result = findNearestLocalCliEntry(cwd, resolver);
+
+  assert.equal(result, '/repo/apps/web/node_modules/slicejs-cli/client.js');
+  assert.deepEqual(calls, ['/repo/apps/web/src', '/repo/apps/web']);
+});
+
+test('findNearestLocalCliEntry returns null when resolver is not a function', () => {
+  const result = findNearestLocalCliEntry('/repo/apps/web/src', null);
+  assert.equal(result, null);
+});
+
+test('resolveLocalCliCandidate returns null when candidate path is a directory', () => {
+  const tempRoot = fs.mkdtempSync(path.join(os.tmpdir(), 'slice-cli-delegation-'));
+  const candidateDirectory = path.join(tempRoot, 'node_modules', 'slicejs-cli', 'client.js');
+
+  fs.mkdirSync(candidateDirectory, { recursive: true });
+
+  try {
+    const result = resolveLocalCliCandidate(tempRoot);
+    assert.equal(result, null);
+  } finally {
+    fs.rmSync(tempRoot, { recursive: true, force: true });
+  }
+});

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 }}