@nextsparkjs/core 0.1.0-beta.62 → 0.1.0-beta.64

package/dist/styles/classes.json

@@ -1,5 +1,5 @@
 {
-  "generated": "2026-01-22T00:37:48.187Z",
+  "generated": "2026-01-22T01:03:14.201Z",
   "totalClasses": 1047,
   "classes": [
     "!text-2xl",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nextsparkjs/core",
-  "version": "0.1.0-beta.62",
+  "version": "0.1.0-beta.64",
   "description": "NextSpark - The complete SaaS framework for Next.js",
   "license": "MIT",
   "author": "NextSpark <hello@nextspark.dev>",

package/scripts/build/registry/discovery/permissions.mjs

@@ -108,12 +108,14 @@ function parseEntitiesFromConfig(content) {
   //   - Simple word: customers
   //   - Quoted with hyphens: "ai-agents" or 'ai-agents'
   //
-  // Lookahead handles:
-  //   - Next entity (quoted or unquoted)
-  //   - End of content ($)
-  //   - Trailing comments (// ...)
-  //   - Closing brace of entities object (})
-  const entityBlockRegex = /["']?([\w-]+)["']?:\s*\[([\s\S]*?)\],?(?=\s*(?:\/\/[^\n]*\n)?\s*(?:["'][\w-]+["']:|[\w-]+:|\}|$))/g
+  // IMPORTANT: The closing bracket must be at the start of a line (after newline + whitespace)
+  // This prevents matching `]` inside nested arrays like `roles: ['owner', 'admin']`
+  // which appear in the MIDDLE of a line, not at the start.
+  //
+  // The regex requires:
+  //   - Entity key at start of line (newline + whitespace)
+  //   - Closing bracket at start of line (newline + whitespace)
+  const entityBlockRegex = /\n\s*['"]?([\w-]+)['"]?:\s*\[([\s\S]*?)\n\s*\]/g
   let entityMatch
 
   while ((entityMatch = entityBlockRegex.exec(entitiesContent)) !== null) {

package/scripts/build/registry/discovery/permissions.test.mjs

@@ -14,8 +14,10 @@ import assert from 'node:assert'
  * Test the entity block regex pattern
  */
 describe('Entity Block Regex', () => {
-  // The regex pattern from permissions.mjs
-  const entityBlockRegex = /["']?([\w-]+)["']?:\s*\[([\s\S]*?)\],?(?=\s*(?:\/\/[^\n]*\n)?\s*(?:["'][\w-]+["']:|[\w-]+:|\}|$))/g
+  // The regex pattern from permissions.mjs (MUST match the actual implementation!)
+  // IMPORTANT: The closing bracket must be at the start of a line (after newline + whitespace)
+  // This prevents matching `]` inside nested arrays like `roles: ['owner', 'admin']`
+  const entityBlockRegex = /\n\s*['"]?([\w-]+)['"]?:\s*\[([\s\S]*?)\n\s*\]/g
 
   it('should match simple entity keys', () => {
     const content = `
@@ -141,9 +143,14 @@ describe('Entity Block Regex', () => {
     assert.strictEqual(matches[0][1], 'ai-landing-page-templates')
   })
 
-  it('should handle empty actions array', () => {
+  it('should handle empty actions array (multiline format)', () => {
+    // Note: Empty arrays on a single line (e.g., "customers: [],") won't be matched
+    // because the regex requires the closing ] to be on its own line.
+    // In practice, if you define permissions for an entity, you'd have at least one.
+    // This test covers the multiline empty array case.
     const content = `
-    customers: [],
+    customers: [
+    ],
     tasks: [
       { action: 'read', roles: ['member'] }
     ]
@@ -154,6 +161,65 @@ describe('Entity Block Regex', () => {
     assert.strictEqual(matches[0][1], 'customers')
     assert.strictEqual(matches[0][2].trim(), '')
   })
+
+  /**
+   * CRITICAL TEST: This is the exact bug that broke beta.62
+   * The regex was matching `roles: ['owner', 'admin']` inside action objects
+   * as if it were an entity key, causing "Invalid entity slug: roles" errors.
+   */
+  it('should NOT match roles inside action objects as entity keys', () => {
+    // This is the exact structure from a real permissions.config.ts
+    const content = `
+    patterns: [
+      { action: 'create', label: 'Create Patterns', description: 'Can create patterns', roles: ['owner', 'admin'] },
+      { action: 'read', label: 'View Patterns', description: 'Can view patterns', roles: ['owner', 'admin', 'member'] },
+      { action: 'delete', label: 'Delete Patterns', description: 'Can delete patterns', roles: ['owner', 'admin'], dangerous: true },
+    ],
+
+    'ai-agents': [
+      { action: 'create', label: 'Create AI Agents', description: 'Can create agents', roles: ['owner', 'admin'] },
+      { action: 'read', label: 'View AI Agents', description: 'Can view agents', roles: ['owner', 'admin', 'member'] },
+    ],
+
+    analysis: [
+      { action: 'create', label: 'Create Analysis', description: 'Can create analyses', roles: ['owner'] },
+      { action: 'read', label: 'View Analysis', description: 'Can view analyses', roles: ['owner', 'admin', 'member'] },
+    ],
+  }`
+
+    const matches = [...content.matchAll(entityBlockRegex)]
+
+    // Should find exactly 3 entity keys: patterns, ai-agents, analysis
+    // Should NOT find 'roles' as an entity key
+    assert.strictEqual(matches.length, 3, `Expected 3 entities but found ${matches.length}: ${matches.map(m => m[1]).join(', ')}`)
+
+    const entityNames = matches.map(m => m[1])
+    assert.ok(entityNames.includes('patterns'), 'Should include patterns')
+    assert.ok(entityNames.includes('ai-agents'), 'Should include ai-agents')
+    assert.ok(entityNames.includes('analysis'), 'Should include analysis')
+    assert.ok(!entityNames.includes('roles'), 'Should NOT include roles (that\'s inside action objects)')
+  })
+
+  it('should handle real-world permissions config structure', () => {
+    // Mimics the exact indentation and structure of a real config file
+    const content = `
+    // PATTERNS ENTITY
+    patterns: [
+      { action: 'create', label: 'Create', roles: ['owner', 'admin'] },
+      { action: 'read', label: 'Read', roles: ['owner', 'admin', 'member'] },
+    ],
+
+    // AI-AGENTS ENTITY (hyphenated, quoted)
+    'ai-agents': [
+      { action: 'create', label: 'Create', roles: ['owner'] },
+    ],
+  }`
+
+    const matches = [...content.matchAll(entityBlockRegex)]
+    assert.strictEqual(matches.length, 2)
+    assert.strictEqual(matches[0][1], 'patterns')
+    assert.strictEqual(matches[1][1], 'ai-agents')
+  })
 })
 
 /**

package/scripts/build/registry/index.mjs

@@ -12,7 +12,6 @@ export { CONFIG, CONTENT_TYPES, rootDir } from './config.mjs'
 
 // Discovery functions
 export {
-  discoverAll,
   discoverPlugins,
   discoverRouteFiles,
   discoverEntities,