pulse-js-framework 1.5.1 → 1.5.3

package/cli/index.js

@@ -87,6 +87,7 @@ Release Options:
   --no-push        Create commit and tag but don't push
   --title <text>   Release title for changelog
   --skip-prompt    Use empty changelog (for automation)
+  --from-commits   Auto-extract changelog from git commits since last tag
 
 Examples:
   pulse create my-app
@@ -107,6 +108,7 @@ Examples:
   pulse release patch
   pulse release minor --title "New Features"
   pulse release major --dry-run
+  pulse release patch --from-commits
 
 Documentation: https://github.com/vincenthirtz/pulse-js-framework
   `);

package/cli/release.js

@@ -113,6 +113,85 @@ function getCurrentMonthYear() {
   return `${months[now.getMonth()]} ${now.getFullYear()}`;
 }
 
+/**
+ * Get the last git tag
+ */
+function getLastTag() {
+  try {
+    return execSync('git describe --tags --abbrev=0', { cwd: root, encoding: 'utf-8' }).trim();
+  } catch {
+    return null;
+  }
+}
+
+/**
+ * Get commits since the last tag (or all commits if no tag exists)
+ */
+function getCommitsSinceLastTag() {
+  const lastTag = getLastTag();
+  const range = lastTag ? `${lastTag}..HEAD` : 'HEAD';
+
+  try {
+    const output = execSync(`git log ${range} --pretty=format:"%s"`, {
+      cwd: root,
+      encoding: 'utf-8'
+    });
+    return output.split('\n').filter(line => line.trim());
+  } catch {
+    return [];
+  }
+}
+
+/**
+ * Parse commit messages into changelog categories
+ * Supports conventional commits: feat:, fix:, docs:, chore:, refactor:, perf:, test:, style:
+ */
+function parseCommitMessages(commits) {
+  const changes = { added: [], changed: [], fixed: [], removed: [] };
+
+  for (const commit of commits) {
+    const lowerCommit = commit.toLowerCase();
+
+    // Skip version commits (e.g., "v1.5.0")
+    if (/^v?\d+\.\d+\.\d+/.test(commit)) continue;
+
+    // Skip merge commits
+    if (lowerCommit.startsWith('merge ')) continue;
+
+    // Parse conventional commits
+    if (lowerCommit.startsWith('feat:') || lowerCommit.startsWith('feat(')) {
+      changes.added.push(cleanCommitMessage(commit, 'feat'));
+    } else if (lowerCommit.startsWith('fix:') || lowerCommit.startsWith('fix(')) {
+      changes.fixed.push(cleanCommitMessage(commit, 'fix'));
+    } else if (lowerCommit.startsWith('remove') || lowerCommit.startsWith('deprecate')) {
+      changes.removed.push(commit);
+    } else if (
+      lowerCommit.startsWith('refactor:') ||
+      lowerCommit.startsWith('perf:') ||
+      lowerCommit.startsWith('chore:') ||
+      lowerCommit.startsWith('docs:') ||
+      lowerCommit.startsWith('style:') ||
+      lowerCommit.startsWith('test:')
+    ) {
+      changes.changed.push(cleanCommitMessage(commit, commit.split(':')[0]));
+    } else {
+      // Default: treat as a change
+      changes.changed.push(commit);
+    }
+  }
+
+  return changes;
+}
+
+/**
+ * Clean commit message by removing the conventional commit prefix
+ */
+function cleanCommitMessage(message, prefix) {
+  // Remove "prefix:" or "prefix(scope):"
+  const regex = new RegExp(`^${prefix}(\\([^)]+\\))?:\\s*`, 'i');
+  return message.replace(regex, '').trim();
+}
+
 /**
  * Update package.json version
  */
@@ -296,30 +375,76 @@ function updateReadme(newVersion) {
 }
 
 /**
- * Execute git commands
+ * Build commit message from version, title, and changes
  */
-function gitCommitTagPush(newVersion, dryRun = false) {
-  const commands = [
-    'git add -A',
-    `git commit -m "$(cat <<'EOF'\nv${newVersion}\n\nCo-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>\nEOF\n)"`,
-    `git tag -a v${newVersion} -m "Release v${newVersion}"`,
-    'git push',
-    'git push --tags'
+function buildCommitMessage(newVersion, title, changes) {
+  // Build header: "v1.5.2 - Title" or just "v1.5.2"
+  let message = `v${newVersion}`;
+  if (title) {
+    message += ` - ${title}`;
+  }
+  message += '\n\n';
+
+  // Add change items as bullet points
+  const allChanges = [
+    ...(changes.added || []),
+    ...(changes.changed || []),
+    ...(changes.fixed || []),
+    ...(changes.removed || [])
   ];
 
-  for (const cmd of commands) {
-    if (dryRun) {
-      log.info(`  [dry-run] ${cmd}`);
-    } else {
-      log.info(`  Running: ${cmd.split('\n')[0]}...`);
-      try {
-        execSync(cmd, { cwd: root, stdio: 'inherit', shell: '/bin/bash' });
-      } catch (error) {
-        log.error(`  Failed: ${error.message}`);
-        throw error;
-      }
+  if (allChanges.length > 0) {
+    for (const change of allChanges) {
+      message += `- ${change}\n`;
     }
+    message += '\n';
+  }
+
+  // Add co-author
+  message += 'Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>';
+
+  return message;
+}
+
+/**
+ * Execute git commands
+ */
+function gitCommitTagPush(newVersion, title, changes, dryRun = false) {
+  const commitMessage = buildCommitMessage(newVersion, title, changes);
+
+  if (dryRun) {
+    log.info('  [dry-run] git add -A');
+    log.info('  [dry-run] git commit with message:');
+    log.info('  ' + commitMessage.split('\n').join('\n  '));
+    log.info(`  [dry-run] git tag -a v${newVersion} -m "Release v${newVersion}"`);
+    log.info('  [dry-run] git push');
+    log.info('  [dry-run] git push --tags');
+    return;
   }
+
+  // git add
+  log.info('  Running: git add -A...');
+  execSync('git add -A', { cwd: root, stdio: 'inherit' });
+
+  // git commit with heredoc for proper message formatting
+  log.info('  Running: git commit...');
+  const escapedMessage = commitMessage.replace(/'/g, "'\\''");
+  execSync(`git commit -m $'${escapedMessage.replace(/\n/g, '\\n')}'`, {
+    cwd: root,
+    stdio: 'inherit',
+    shell: '/bin/bash'
+  });
+
+  // git tag
+  log.info(`  Running: git tag v${newVersion}...`);
+  execSync(`git tag -a v${newVersion} -m "Release v${newVersion}"`, { cwd: root, stdio: 'inherit' });
+
+  // git push
+  log.info('  Running: git push...');
+  execSync('git push', { cwd: root, stdio: 'inherit' });
+
+  log.info('  Running: git push --tags...');
+  execSync('git push --tags', { cwd: root, stdio: 'inherit' });
 }
 
 /**
@@ -335,15 +460,24 @@ Types:
   major     Bump major version (1.0.0 -> 2.0.0)
 
 Options:
-  --dry-run       Show what would be done without making changes
-  --no-push       Create commit and tag but don't push
-  --title <text>  Release title (e.g., "Performance Improvements")
-  --skip-prompt   Use empty changelog (for automated releases)
+  --dry-run        Show what would be done without making changes
+  --no-push        Create commit and tag but don't push
+  --title <text>   Release title (e.g., "Performance Improvements")
+  --skip-prompt    Use empty changelog (for automated releases)
+  --from-commits   Auto-extract changelog from git commits since last tag
+  --yes, -y        Auto-confirm all prompts
+  --changes <json> Pass changelog as JSON (e.g., '{"added":["Feature 1"],"fixed":["Bug 1"]}')
+  --added <items>  Comma-separated list of added features
+  --changed <items> Comma-separated list of changes
+  --fixed <items>  Comma-separated list of fixes
 
 Examples:
   pulse release patch
-  pulse release minor --title "New Features"
+  pulse release minor --title "New Features" -y
   pulse release major --dry-run
+  pulse release patch --from-commits --title "Bug Fixes" -y
+  pulse release patch --title "Security" --fixed "XSS vulnerability,SQL injection" -y
+  pulse release patch --title "New API" --added "Feature A,Feature B" --fixed "Bug X" -y
   `);
 }
 
@@ -362,6 +496,8 @@ export async function runRelease(args) {
   const dryRun = args.includes('--dry-run');
   const noPush = args.includes('--no-push');
   const skipPrompt = args.includes('--skip-prompt');
+  const fromCommits = args.includes('--from-commits');
+  const autoConfirm = args.includes('--yes') || args.includes('-y');
 
   let title = '';
   const titleIndex = args.indexOf('--title');
@@ -369,6 +505,33 @@ export async function runRelease(args) {
     title = args[titleIndex + 1];
   }
 
+  // Parse --changes JSON option
+  let changesFromArgs = null;
+  const changesIndex = args.indexOf('--changes');
+  if (changesIndex !== -1 && args[changesIndex + 1]) {
+    try {
+      changesFromArgs = JSON.parse(args[changesIndex + 1]);
+    } catch (e) {
+      log.error('Invalid JSON for --changes option');
+      process.exit(1);
+    }
+  }
+
+  // Parse individual change type options
+  const addedIndex = args.indexOf('--added');
+  const changedIndex = args.indexOf('--changed');
+  const fixedIndex = args.indexOf('--fixed');
+  const removedIndex = args.indexOf('--removed');
+
+  if (!changesFromArgs && (addedIndex !== -1 || changedIndex !== -1 || fixedIndex !== -1 || removedIndex !== -1)) {
+    changesFromArgs = {
+      added: addedIndex !== -1 && args[addedIndex + 1] ? args[addedIndex + 1].split(',').map(s => s.trim()) : [],
+      changed: changedIndex !== -1 && args[changedIndex + 1] ? args[changedIndex + 1].split(',').map(s => s.trim()) : [],
+      fixed: fixedIndex !== -1 && args[fixedIndex + 1] ? args[fixedIndex + 1].split(',').map(s => s.trim()) : [],
+      removed: removedIndex !== -1 && args[removedIndex + 1] ? args[removedIndex + 1].split(',').map(s => s.trim()) : []
+    };
+  }
+
   // Read current version
   const pkg = JSON.parse(readFileSync(join(root, 'package.json'), 'utf-8'));
   const currentVersion = pkg.version;
@@ -389,10 +552,14 @@ export async function runRelease(args) {
     if (status.trim()) {
       log.warn('You have uncommitted changes:');
       log.info(status);
-      const proceed = await prompt('Continue anyway? (y/N) ');
-      if (proceed.toLowerCase() !== 'y') {
-        log.info('Aborted.');
-        process.exit(0);
+      if (!autoConfirm) {
+        const proceed = await prompt('Continue anyway? (y/N) ');
+        if (proceed.toLowerCase() !== 'y') {
+          log.info('Aborted.');
+          process.exit(0);
+        }
+      } else {
+        log.info('Auto-confirming with --yes flag');
       }
     }
   } catch (error) {
@@ -403,7 +570,50 @@ export async function runRelease(args) {
   // Collect changelog entries
   let changes = { added: [], changed: [], fixed: [], removed: [] };
 
-  if (!skipPrompt) {
+  // Use changes from command-line arguments if provided
+  if (changesFromArgs) {
+    changes = {
+      added: changesFromArgs.added || [],
+      changed: changesFromArgs.changed || [],
+      fixed: changesFromArgs.fixed || [],
+      removed: changesFromArgs.removed || []
+    };
+    const totalChanges = Object.values(changes).flat().length;
+    if (totalChanges > 0) {
+      log.info('');
+      log.info('Changelog entries from arguments:');
+      if (changes.added.length) log.info(`  Added: ${changes.added.length} items`);
+      if (changes.changed.length) log.info(`  Changed: ${changes.changed.length} items`);
+      if (changes.fixed.length) log.info(`  Fixed: ${changes.fixed.length} items`);
+      if (changes.removed.length) log.info(`  Removed: ${changes.removed.length} items`);
+    }
+  } else if (fromCommits) {
+    // Auto-extract from git commits since last tag
+    const lastTag = getLastTag();
+    const commits = getCommitsSinceLastTag();
+
+    if (commits.length === 0) {
+      log.warn('No commits found since last tag');
+    } else {
+      log.info(`Found ${commits.length} commits since ${lastTag || 'beginning'}`);
+      changes = parseCommitMessages(commits);
+
+      // Show extracted changes
+      const totalChanges = Object.values(changes).flat().length;
+      if (totalChanges > 0) {
+        log.info('');
+        log.info('Extracted changelog entries:');
+        if (changes.added.length) log.info(`  Added: ${changes.added.length} items`);
+        if (changes.changed.length) log.info(`  Changed: ${changes.changed.length} items`);
+        if (changes.fixed.length) log.info(`  Fixed: ${changes.fixed.length} items`);
+        if (changes.removed.length) log.info(`  Removed: ${changes.removed.length} items`);
+      }
+    }
+
+    if (!title && !autoConfirm) {
+      title = await prompt('Release title (e.g., "Performance Improvements"): ');
+    }
+  } else if (!skipPrompt && !autoConfirm) {
     if (!title) {
       title = await prompt('Release title (e.g., "Performance Improvements"): ');
     }
@@ -420,7 +630,7 @@ export async function runRelease(args) {
 
   const hasChanges = Object.values(changes).some(arr => arr.length > 0);
 
-  if (!hasChanges && !skipPrompt) {
+  if (!hasChanges && !skipPrompt && !autoConfirm) {
     const proceed = await prompt('No changelog entries. Continue? (y/N) ');
     if (proceed.toLowerCase() !== 'y') {
       log.info('Aborted.');
@@ -466,8 +676,10 @@ export async function runRelease(args) {
   if (!dryRun) {
     if (noPush) {
       // Only commit and tag, no push
+      const commitMessage = buildCommitMessage(newVersion, title, changes);
       execSync('git add -A', { cwd: root, stdio: 'inherit' });
-      execSync(`git commit -m "v${newVersion}\n\nCo-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>"`, {
+      const escapedMessage = commitMessage.replace(/'/g, "'\\''");
+      execSync(`git commit -m $'${escapedMessage.replace(/\n/g, '\\n')}'`, {
         cwd: root,
         stdio: 'inherit',
         shell: '/bin/bash'
@@ -475,10 +687,10 @@ export async function runRelease(args) {
       execSync(`git tag -a v${newVersion} -m "Release v${newVersion}"`, { cwd: root, stdio: 'inherit' });
       log.info('  Created commit and tag (--no-push specified)');
     } else {
-      gitCommitTagPush(newVersion, false);
+      gitCommitTagPush(newVersion, title, changes, false);
     }
   } else {
-    gitCommitTagPush(newVersion, true);
+    gitCommitTagPush(newVersion, title, changes, true);
   }
 
   log.info('');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pulse-js-framework",
-  "version": "1.5.1",
+  "version": "1.5.3",
   "description": "A declarative DOM framework with CSS selector-based structure and reactive pulsations",
   "type": "module",
   "main": "index.js",

package/runtime/dom.js

@@ -28,6 +28,38 @@ const log = loggers.dom;
 // Cache hit returns a shallow copy to prevent mutation of cached config
 const selectorCache = new LRUCache(500);
 
+/**
+ * Safely insert a node before a reference node
+ * Returns false if the parent is detached (no parentNode)
+ * @private
+ * @param {Node} newNode - Node to insert
+ * @param {Node} refNode - Reference node (insert before this)
+ * @returns {boolean} True if insertion succeeded
+ */
+function safeInsertBefore(newNode, refNode) {
+  if (!refNode.parentNode) {
+    log.warn('Cannot insert node: reference node has no parent (may be detached)');
+    return false;
+  }
+  refNode.parentNode.insertBefore(newNode, refNode);
+  return true;
+}
+
+/**
+ * Resolve a selector string or element to a DOM element
+ * @private
+ * @param {string|HTMLElement} target - CSS selector or DOM element
+ * @param {string} context - Context name for error messages
+ * @returns {{element: HTMLElement|null, selector: string}} Resolved element and original selector
+ */
+function resolveSelector(target, context = 'target') {
+  if (typeof target === 'string') {
+    const element = document.querySelector(target);
+    return { element, selector: target };
+  }
+  return { element: target, selector: '(element)' };
+}
+
 // Lifecycle tracking
 let mountCallbacks = [];
 let unmountCallbacks = [];
@@ -58,8 +90,13 @@ export function onUnmount(fn) {
 
 /**
  * Parse a CSS selector-like string into element configuration
- * Supports: tag, #id, .class, [attr=value]
- * Results are cached for performance.
+ * Results are cached for performance using LRU cache.
+ *
+ * Supported syntax:
+ * - Tag: `div`, `span`, `custom-element`
+ * - ID: `#app`, `#my-id`, `#_private`
+ * - Classes: `.class`, `.my-class`, `.-modifier`
+ * - Attributes: `[attr]`, `[attr=value]`, `[attr="quoted value"]`, `[attr='single quoted']`
  *
  * Examples:
  *   "div" -> { tag: "div" }
@@ -67,6 +104,10 @@ export function onUnmount(fn) {
  *   ".container" -> { tag: "div", classes: ["container"] }
  *   "button.primary.large" -> { tag: "button", classes: ["primary", "large"] }
  *   "input[type=text][placeholder=Name]" -> { tag: "input", attrs: { type: "text", placeholder: "Name" } }
+ *   "div[data-id=\"complex-123\"]" -> { tag: "div", attrs: { "data-id": "complex-123" } }
+ *
+ * @param {string} selector - CSS selector-like string
+ * @returns {{tag: string, id: string|null, classes: string[], attrs: Object}} Parsed configuration
  */
 export function parseSelector(selector) {
   if (!selector || selector === '') {
@@ -101,32 +142,43 @@ export function parseSelector(selector) {
     remaining = remaining.slice(tagMatch[0].length);
   }
 
-  // Match ID
-  const idMatch = remaining.match(/#([a-zA-Z][a-zA-Z0-9-_]*)/);
+  // Match ID (supports starting with letter, underscore, or hyphen followed by valid chars)
+  const idMatch = remaining.match(/#([a-zA-Z_-][a-zA-Z0-9-_]*)/);
   if (idMatch) {
     config.id = idMatch[1];
     remaining = remaining.replace(idMatch[0], '');
   }
 
-  // Match classes
-  const classMatches = remaining.matchAll(/\.([a-zA-Z][a-zA-Z0-9-_]*)/g);
+  // Match classes (supports starting with letter, underscore, or hyphen)
+  const classMatches = remaining.matchAll(/\.([a-zA-Z_-][a-zA-Z0-9-_]*)/g);
   for (const match of classMatches) {
     config.classes.push(match[1]);
   }
 
-  // Match attributes
-  const attrMatches = remaining.matchAll(/\[([a-zA-Z][a-zA-Z0-9-_]*)(?:=([^\]]+))?\]/g);
+  // Match attributes - improved regex handles quoted values with special characters
+  // Matches: [attr], [attr=value], [attr="quoted value"], [attr='quoted value']
+  const attrRegex = /\[([a-zA-Z_][a-zA-Z0-9-_]*)(?:=(?:"([^"]*)"|'([^']*)'|([^\]]*)))?\]/g;
+  const attrMatches = remaining.matchAll(attrRegex);
   for (const match of attrMatches) {
     const key = match[1];
-    let value = match[2] || '';
-    // Remove quotes if present
-    if ((value.startsWith('"') && value.endsWith('"')) ||
-        (value.startsWith("'") && value.endsWith("'"))) {
-      value = value.slice(1, -1);
-    }
+    // Value can be in match[2] (double-quoted), match[3] (single-quoted), or match[4] (unquoted)
+    const value = match[2] ?? match[3] ?? match[4] ?? '';
     config.attrs[key] = value;
   }
 
+  // Validate: check for unparsed content (malformed selector parts)
+  // Remove all parsed parts to see if anything remains
+  let unparsed = remaining
+    .replace(/#[a-zA-Z_-][a-zA-Z0-9-_]*/g, '')  // Remove IDs
+    .replace(/\.[a-zA-Z_-][a-zA-Z0-9-_]*/g, '') // Remove classes
+    .replace(/\[([a-zA-Z_][a-zA-Z0-9-_]*)(?:=(?:"[^"]*"|'[^']*'|[^\]]*))?\]/g, '') // Remove attrs
+    .trim();
+
+  if (unparsed) {
+    log.warn(`Selector "${selector}" contains unrecognized parts: "${unparsed}". ` +
+      'Supported syntax: tag#id.class[attr=value]');
+  }
+
   // Cache the result (LRU cache handles eviction automatically)
   selectorCache.set(selector, config);
 
@@ -217,7 +269,11 @@ function appendChild(parent, child) {
             }
           }
         }
-        placeholder.parentNode.insertBefore(fragment, placeholder.nextSibling);
+        if (placeholder.parentNode) {
+          placeholder.parentNode.insertBefore(fragment, placeholder.nextSibling);
+        } else {
+          log.warn('Cannot insert reactive children: placeholder has no parent node');
+        }
       }
     });
   }
@@ -552,12 +608,8 @@ export function model(element, pulseValue) {
  * @throws {Error} If target element is not found
  */
 export function mount(target, element) {
-  const originalTarget = target;
-  if (typeof target === 'string') {
-    target = document.querySelector(target);
-  }
-  if (!target) {
-    const selector = typeof originalTarget === 'string' ? originalTarget : '(element)';
+  const { element: resolved, selector } = resolveSelector(target, 'mount');
+  if (!resolved) {
     throw new Error(
       `[Pulse] Mount target not found: "${selector}". ` +
       `Ensure the element exists in the DOM before mounting. ` +
@@ -565,7 +617,7 @@ export function mount(target, element) {
       `or place your script at the end of <body>.`
     );
   }
-  target.appendChild(element);
+  resolved.appendChild(element);
   return () => {
     element.remove();
   };
@@ -649,12 +701,10 @@ export function show(condition, element) {
  * Portal - render children into a different DOM location
  */
 export function portal(children, target) {
-  const resolvedTarget = typeof target === 'string'
-    ? document.querySelector(target)
-    : target;
+  const { element: resolvedTarget, selector } = resolveSelector(target, 'portal');
 
   if (!resolvedTarget) {
-    log.warn('Portal target not found:', target);
+    log.warn(`Portal target not found: "${selector}"`);
     return document.createComment('portal-target-not-found');
   }
 

package/runtime/logger.js

@@ -39,6 +39,13 @@ let globalLevel = LogLevel.INFO;
 /** @type {LogFormatter|null} */
 let globalFormatter = null;
 
+/**
+ * Namespace formatting helpers
+ * @private
+ */
+const NAMESPACE_SEPARATOR = ':';
+const formatNamespace = (ns) => `[${ns}]`;
+
 /**
  * @callback LogFormatter
  * @param {'error'|'warn'|'info'|'debug'} level - The log level
@@ -97,13 +104,14 @@ export function setFormatter(formatter) {
 function formatArgs(namespace, args) {
   if (!namespace) return args;
 
+  const prefix = formatNamespace(namespace);
   // If first arg is a string, prepend namespace
   if (typeof args[0] === 'string') {
-    return [`[${namespace}] ${args[0]}`, ...args.slice(1)];
+    return [`${prefix} ${args[0]}`, ...args.slice(1)];
   }
 
   // Otherwise, add namespace as first arg
-  return [`[${namespace}]`, ...args];
+  return [prefix, ...args];
 }
 
 /**
@@ -217,7 +225,7 @@ export function createLogger(namespace = null, options = {}) {
      */
     group(label) {
       if (shouldLog(LogLevel.DEBUG)) {
-        console.group(namespace ? `[${namespace}] ${label}` : label);
+        console.group(namespace ? `${formatNamespace(namespace)} ${label}` : label);
       }
     },
 
@@ -264,7 +272,7 @@ export function createLogger(namespace = null, options = {}) {
      */
     child(childNamespace) {
       const combined = namespace
-        ? `${namespace}:${childNamespace}`
+        ? `${namespace}${NAMESPACE_SEPARATOR}${childNamespace}`
         : childNamespace;
       return createLogger(combined, options);
     }