@newlogic-digital/cli 1.5.0-next.3 → 1.5.0-next.5

package/index.mjs

@@ -3,10 +3,11 @@
 import init from './src/commands/init/index.mjs'
 import cms from './src/commands/cms/index.mjs'
 import blocks from './src/commands/blocks/index.mjs'
+import skills from './src/commands/skills/index.mjs'
 import { styleText } from 'node:util'
 import { version, name } from './src/utils.mjs'
 
-const knownCommands = ['init', 'cms', 'blocks']
+const knownCommands = ['init', 'cms', 'blocks', 'skills']
 
 function normalizeOptionName(name) {
   return name.replace(/-([a-z])/g, (_, letter) => letter.toUpperCase())
@@ -149,15 +150,19 @@ if (!command) {
   
     -- blocks --
     ${styleText('green', 'newlogic blocks list')} - Lists all available installable blocks with descriptions
-    ${styleText('green', 'newlogic blocks add')} ${styleText('yellow', '<name...>')} - Installs one or more blocks by kebab-case or PascalCase name
+    ${styleText('green', 'newlogic blocks add')} ${styleText('yellow', '<name[@variant]...>')} - Installs one or more blocks by kebab-case or PascalCase name
     ${styleText('green', 'newlogic blocks remove')} ${styleText('yellow', '<name...>')} - Removes one or more blocks and orphaned dependencies
     ${styleText('green', 'newlogic blocks update')} - Reinstalls all configured blocks from ${styleText('yellow', 'newlogic.config.json')}
     ${styleText('green', 'newlogic blocks add')} ${styleText('yellow', 'about-accordion')}
     ${styleText('green', 'newlogic blocks add')} ${styleText('yellow', 'AboutAccordion')}
+    ${styleText('green', 'newlogic blocks add')} ${styleText('yellow', 'header-nav-left@stimulus')}
     ${styleText('green', 'newlogic blocks add')} ${styleText('yellow', 'header-nav-left contact-info-card hero-floating-text')}
     ${styleText('green', 'newlogic blocks remove')} ${styleText('yellow', 'about-accordion')}
     ${styleText('green', 'newlogic blocks remove')} ${styleText('yellow', 'header-nav-left hero-floating-text')}
     ${styleText('green', 'newlogic blocks update')}
+
+    -- skills --
+    ${styleText('green', 'newlogic skills install')} - Installs the bundled ${styleText('yellow', 'newlogic-cli')} skill via ${styleText('blue', 'npx skills add')}
   `)
 
   process.exit(0)
@@ -187,6 +192,13 @@ if (command === 'blocks') {
   await blocks(action, names)
 }
 
+if (command === 'skills') {
+  const { positionals } = parseCommandArgs(rawArgs.slice(1))
+  const action = positionals[0]
+
+  await skills(action)
+}
+
 if (command && !knownCommands.includes(command)) {
   printUnknownCommand(command)
   process.exit(1)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@newlogic-digital/cli",
-  "version": "1.5.0-next.3",
+  "version": "1.5.0-next.5",
   "main": "index.mjs",
   "bin": {
     "newlogic-cli": "index.mjs",

package/skills/{newlogic → newlogic-cli}/SKILL.md

@@ -1,5 +1,5 @@
 ---
-name: newlogic
+name: newlogic-cli
 description: Use the installed Newlogic CLI when Codex needs to run `newlogic`, scaffold `@newlogic-digital/ui` or `@newlogic-digital/cms` projects, prepare CMS templates and components, or manage installable blocks through `newlogic.config.json`. Trigger this skill for tasks involving the commands `init`, `cms`, or `blocks`, especially when an agent should run the CLI safely and non-interactively.
 ---
 

package/skills/newlogic-cli/agents/openai.yaml

@@ -0,0 +1,6 @@
+interface:
+  display_name: "Newlogic CLI"
+  short_description: "Use installed Newlogic CLI for init, cms, and blocks."
+  icon_small: "./assets/favicon.svg"
+  icon_large: "./assets/maskable.png"
+  default_prompt: "Use $newlogic-cli to run Newlogic init, cms, or blocks commands safely and non-interactively."

package/skills/newlogic-cli/assets/favicon.svg

@@ -0,0 +1,3 @@
+<svg xmlns="http://www.w3.org/2000/svg" width="120" height="120" viewBox="0 0 120 120">
+  <path fill="#000" d="M89,0,62.68,26.36,36.32,0H0V120H31L57.32,93.64,83.68,120H120V0Zm0,8.53V52.72l-22.1-22.1Zm6,58.75V6H114V109.7L10.3,6H33.82ZM31,111.47V67.28l22.1,22.1Zm-6-58.75V114H6V10.29L109.7,114H86.18Z"/>
+</svg>

package/src/commands/blocks/index.mjs

@@ -134,13 +134,14 @@ function printBlocksUsage() {
     styleText(['white', 'bold'], 'Usage:'),
     '',
     `  ${styleText('green', 'newlogic blocks list')} - Lists all available blocks with descriptions`,
-    `  ${styleText('green', 'newlogic blocks add')} ${styleText('yellow', '<name...>')} - Adds one or more blocks by kebab-case or PascalCase name`,
+    `  ${styleText('green', 'newlogic blocks add')} ${styleText('yellow', '<name[@variant]...>')} - Adds one or more blocks by kebab-case or PascalCase name`,
     `  ${styleText('green', 'newlogic blocks remove')} ${styleText('yellow', '<name...>')} - Removes one or more blocks and orphaned dependencies`,
     `  ${styleText('green', 'newlogic blocks update')} - Reinstalls all configured blocks from ${styleText('yellow', 'newlogic.config.json')}`,
     '',
     styleText(['white', 'bold'], 'Examples:'),
     '',
     `  ${styleText('green', 'newlogic blocks add')} ${styleText('yellow', 'header-nav-left contact-info-card hero-floating-text')}`,
+    `  ${styleText('green', 'newlogic blocks add')} ${styleText('yellow', 'header-nav-left@stimulus')}`,
     `  ${styleText('green', 'newlogic blocks remove')} ${styleText('yellow', 'header-nav-left hero-floating-text')}`,
   ].join('\n'))
 }

package/src/commands/blocks/service.mjs

@@ -83,6 +83,51 @@ function assertBlockName(name, label = 'block name') {
   }
 }
 
+function parseBlockSpecifier(rawValue, {
+  label = 'block name',
+  allowObject = false,
+} = {}) {
+  if (allowObject && rawValue && typeof rawValue === 'object' && !Array.isArray(rawValue)) {
+    const name = normalizeBlockName(rawValue.name)
+    const variant = rawValue.variant == null ? '' : `${rawValue.variant}`.trim()
+
+    assertBlockName(name, 'configured block name')
+
+    if (rawValue.variant != null && !variant) {
+      throw new Error(`Configured block "${name}" has an empty variant`)
+    }
+
+    return {
+      name,
+      variant: variant || undefined,
+      variantExplicit: rawValue.variant != null,
+    }
+  }
+
+  const value = `${rawValue ?? ''}`.trim()
+
+  if (!value) {
+    throw new Error(`Missing ${label}`)
+  }
+
+  const variantSeparator = value.indexOf('@')
+  const rawName = variantSeparator === -1 ? value : value.slice(0, variantSeparator)
+  const rawVariant = variantSeparator === -1 ? '' : value.slice(variantSeparator + 1).trim()
+  const name = normalizeBlockName(rawName)
+
+  assertBlockName(name, label)
+
+  if (variantSeparator !== -1 && !rawVariant) {
+    throw new Error(`Invalid ${label}: "${value}"`)
+  }
+
+  return {
+    name,
+    variant: rawVariant || undefined,
+    variantExplicit: variantSeparator !== -1,
+  }
+}
+
 function toBlockKey(block) {
   return `${block.name}@${block.variant}`
 }
@@ -151,20 +196,14 @@ function loadProjectConfig(projectRoot) {
   }
 
   const blocks = blocksRaw.map((entry, index) => {
-    if (!entry || typeof entry !== 'object' || Array.isArray(entry)) {
+    if (typeof entry !== 'string' && (!entry || typeof entry !== 'object' || Array.isArray(entry))) {
       throw new Error(`Invalid block config entry at index ${index}`)
     }
 
-    const name = normalizeBlockName(entry.name)
-    const variant = `${entry.variant ?? ''}`.trim()
-
-    assertBlockName(name, 'configured block name')
-
-    if (!variant) {
-      throw new Error(`Configured block "${name}" is missing a variant`)
-    }
-
-    return { name, variant }
+    return parseBlockSpecifier(entry, {
+      label: `block config entry at index ${index}`,
+      allowObject: true,
+    })
   })
 
   return { configPath, config, blocks: dedupeExplicitBlocks(blocks) }
@@ -173,7 +212,7 @@ function loadProjectConfig(projectRoot) {
 function saveProjectConfig(configPath, config, blocks) {
   const nextConfig = {
     ...config,
-    blocks,
+    blocks: blocks.map(block => block.variantExplicit ? `${block.name}@${block.variant}` : block.name),
   }
 
   fs.writeFileSync(configPath, `${JSON.stringify(nextConfig, null, 2)}\n`)
@@ -194,15 +233,25 @@ function normalizeRequestedBlockNames(rawNames, label = 'block name') {
     throw new Error(`Missing ${label}`)
   }
 
-  const normalizedNames = rawNames.map((rawName) => {
-    const name = normalizeBlockName(rawName)
+  const normalizedNames = rawNames
+    .map(rawName => parseBlockSpecifier(rawName, { label }).name)
 
-    assertBlockName(name, label)
+  return [...new Set(normalizedNames)]
+}
 
-    return name
-  })
+function normalizeRequestedBlocks(rawNames, label = 'block name') {
+  if (!Array.isArray(rawNames) || rawNames.length === 0) {
+    throw new Error(`Missing ${label}`)
+  }
 
-  return [...new Set(normalizedNames)]
+  const blocks = rawNames.map(rawName => parseBlockSpecifier(rawName, { label }))
+  const byName = new Map()
+
+  for (const block of blocks) {
+    byName.set(block.name, block)
+  }
+
+  return [...byName.values()]
 }
 
 async function createMetaValidator(repository) {
@@ -488,16 +537,21 @@ export function createBlocksService({
   }
 
   async function addBlocks(rawNames) {
-    const names = normalizeRequestedBlockNames(rawNames)
+    const requestedBlocks = normalizeRequestedBlocks(rawNames)
     const rootBlocks = []
 
-    for (const name of names) {
-      const meta = await getMeta(name)
+    for (const requestedBlock of requestedBlocks) {
+      const meta = await getMeta(requestedBlock.name)
 
       rootBlocks.push({
-        name,
-        variant: meta.install.defaultVariant,
+        name: requestedBlock.name,
+        variant: requestedBlock.variant,
+        variantExplicit: requestedBlock.variantExplicit,
       })
+
+      if (requestedBlock.variant && !meta.install?.variants?.[requestedBlock.variant]) {
+        throw new Error(`Variant "${requestedBlock.variant}" is not available for block "${requestedBlock.name}"`)
+      }
     }
 
     const plan = await resolveInstallPlan(rootBlocks)
@@ -506,12 +560,12 @@ export function createBlocksService({
 
     const { configPath, config, blocks } = loadProjectConfig(projectRoot)
     const nextBlocks = dedupeExplicitBlocks([
-      ...blocks.filter(block => !names.includes(block.name)),
+      ...blocks.filter(block => !requestedBlocks.some(requestedBlock => requestedBlock.name === block.name)),
       ...rootBlocks,
     ])
 
     saveProjectConfig(configPath, config, nextBlocks)
-    logger.info(`added ${rootBlocks.map(block => `${block.name}@${block.variant}`).join(', ')}`)
+    logger.info(`added ${rootBlocks.map(block => block.variantExplicit ? `${block.name}@${block.variant}` : block.name).join(', ')}`)
 
     return {
       blocks: rootBlocks,

package/src/commands/skills/index.mjs

@@ -0,0 +1,62 @@
+import childProcess from 'node:child_process'
+import fs from 'node:fs'
+import { styleText } from 'node:util'
+import { packageRoot, resolveInside } from '../../utils.mjs'
+
+function label(color, text) {
+  return styleText([color, 'bold'], text)
+}
+
+function printSkillsUsage() {
+  console.log([
+    styleText(['blue', 'bold'], 'newlogic skills'),
+    '',
+    styleText(['white', 'bold'], 'Usage:'),
+    '',
+    `  ${styleText('green', 'newlogic skills install')} - Installs the bundled ${styleText('yellow', 'newlogic-cli')} skill`,
+    '',
+    styleText(['white', 'bold'], 'Notes:'),
+    '',
+    `  ${styleText('dim', 'Resolves the installed CLI location automatically and runs:')}`,
+    `  ${styleText('cyan', 'npx skills add <resolved-path>/skills/newlogic-cli')}`,
+  ].join('\n'))
+}
+
+function getBundledSkillPath(skillName = 'newlogic-cli') {
+  return resolveInside(packageRoot, 'skills', skillName)
+}
+
+function installBundledSkill(skillName = 'newlogic-cli', options = {}) {
+  const { execFileSync = childProcess.execFileSync } = options
+  const skillPath = getBundledSkillPath(skillName)
+
+  if (!fs.existsSync(skillPath)) {
+    throw new Error(`Bundled skill "${skillName}" not found at ${skillPath}`)
+  }
+
+  execFileSync('npx', ['skills', 'add', skillPath], { stdio: 'inherit' })
+
+  return skillPath
+}
+
+export { getBundledSkillPath, installBundledSkill }
+
+export default async function skills(action) {
+  if (!action || action === 'help' || action === '--help') {
+    printSkillsUsage()
+    return
+  }
+
+  try {
+    if (action === 'install') {
+      installBundledSkill()
+      return
+    }
+
+    throw new Error(`Unknown skills action "${action}"`)
+  }
+  catch (error) {
+    console.log(`${label('red', 'error')} ${error.message}`)
+    process.exit(1)
+  }
+}

package/src/utils.mjs

@@ -3,7 +3,8 @@ import fs from 'fs'
 import path from 'path'
 import { fileURLToPath } from 'url'
 
-const { version, name } = JSON.parse(fs.readFileSync(path.resolve(path.dirname((fileURLToPath(import.meta.url))), '../package.json')).toString())
+const packageRoot = path.resolve(path.dirname(fileURLToPath(import.meta.url)), '..')
+const { version, name } = JSON.parse(fs.readFileSync(path.resolve(packageRoot, 'package.json')).toString())
 
 const execSync = (cmd) => {
   try {
@@ -45,4 +46,4 @@ function resolveInside(rootDir, ...segments) {
   return nextPath
 }
 
-export { execSync, stripIndent, resolveInside, version, name }
+export { execSync, stripIndent, resolveInside, version, name, packageRoot }

package/skills/newlogic/agents/openai.yaml

@@ -1,4 +0,0 @@
-interface:
-  display_name: "Newlogic CLI"
-  short_description: "Use the local Newlogic CLI safely and non-interactively."
-  default_prompt: "Use this skill when you need to inspect or run the Newlogic CLI in this repository."