@vibecuting/component-project-helper 0.1.13 → 0.1.15

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vibecuting/component-project-helper",
-  "version": "0.1.13",
+  "version": "0.1.15",
   "type": "module",
   "license": "SEE LICENSE IN LICENSE",
   "main": "./src/index.ts",

package/scripts/discover-components.mjs

@@ -85,6 +85,10 @@ async function collectComponentSourceFiles(directory) {
       continue
     }
 
+    if (entry.name.includes('.test.') || entry.name.includes('.spec.')) {
+      continue
+    }
+
     if (entry.name.endsWith('.d.ts')) {
       continue
     }

package/scripts/postinstall.mjs

@@ -11,6 +11,16 @@ import {
 import { renderComponentProjectMarkdown } from './render-markdown.mjs'
 import { ComponentProjectGeneratedManifestSchema } from './schemas.mjs'
 
+const supportsColor =
+  Boolean(process.stdout.isTTY) && process.env.NO_COLOR !== '1' && process.env.CI !== 'true'
+
+const color = {
+  cyan: (value) => (supportsColor ? `\u001b[36m${value}\u001b[0m` : value),
+  green: (value) => (supportsColor ? `\u001b[32m${value}\u001b[0m` : value),
+  yellow: (value) => (supportsColor ? `\u001b[33m${value}\u001b[0m` : value),
+  dim: (value) => (supportsColor ? `\u001b[2m${value}\u001b[0m` : value),
+}
+
 async function ensureDirectory(dir) {
   await fs.mkdir(dir, { recursive: true })
 }
@@ -34,14 +44,16 @@ export async function runComponentProjectPostinstall(projectRoot = resolveCompon
   const manifestPath = resolveComponentProjectGeneratedManifestPath(projectRoot)
   const components = await discoverComponentProjectComponents(projectRoot)
 
-  console.log('[component-project-helper] postinstall start')
-  console.log(`[component-project-helper] project root: ${projectRoot}`)
-  console.log(`[component-project-helper] docs dir: ${docsDir}`)
-  console.log(`[component-project-helper] manifest path: ${manifestPath}`)
+  console.log(color.cyan('[component-project-helper] postinstall start'))
+  console.log(color.dim(`[component-project-helper] project root: ${projectRoot}`))
+  console.log(color.dim(`[component-project-helper] docs dir: ${docsDir}`))
+  console.log(color.dim(`[component-project-helper] manifest path: ${manifestPath}`))
   console.log(
-    `[component-project-helper] discovered ${components.length} component(s): ${
+    color.green(
+      `[component-project-helper] discovered ${components.length} component(s): ${
       components.map((component) => component.name).join(', ') || '(none)'
-    }`,
+      }`,
+    ),
   )
 
   await ensureDirectory(docsDir)
@@ -52,19 +64,23 @@ export async function runComponentProjectPostinstall(projectRoot = resolveCompon
     await fs.writeFile(filePath, renderComponentProjectMarkdown(component), 'utf8')
     files.push(filePath)
     console.log(
-      `[component-project-helper] wrote ${path.relative(projectRoot, filePath)} <- ${component.sourceFile}`,
+      color.green(
+        `[component-project-helper] wrote ${path.relative(projectRoot, filePath)} <- ${component.sourceFile}`,
+      ),
     )
   }
 
   console.log(
-    `[component-project-helper] cleaning stale docs relative to manifest ${path.relative(
+    color.yellow(
+      `[component-project-helper] cleaning stale docs relative to manifest ${path.relative(
       projectRoot,
       manifestPath,
-    )}`,
+      )}`,
+    ),
   )
   await removeStaleGeneratedFiles(manifestPath, files)
   await ensureComponentProjectSkillsScript(projectRoot)
-  console.log('[component-project-helper] ensured update-component-skills script')
+  console.log(color.green('[component-project-helper] ensured update-component-skills script'))
   console.log('[component-project-helper] ensured stale docs cleaned')
 
   const manifest = ComponentProjectGeneratedManifestSchema.parse({
@@ -75,12 +91,22 @@ export async function runComponentProjectPostinstall(projectRoot = resolveCompon
   })
   await fs.writeFile(manifestPath, `${JSON.stringify(manifest, null, 2)}\n`, 'utf8')
   console.log(
-    `[component-project-helper] postinstall complete, manifest files: ${files.length}`,
+    color.cyan(
+      `[component-project-helper] postinstall complete, manifest files: ${files.length}`,
+    ),
   )
 }
 
-const isDirectRun = process.argv[1] && path.resolve(process.argv[1]) === path.resolve(new URL(import.meta.url).pathname)
+async function isDirectRun() {
+  if (!process.argv[1]) {
+    return false
+  }
+
+  const entryPath = await fs.realpath(process.argv[1])
+  const currentPath = await fs.realpath(new URL(import.meta.url).pathname)
+  return entryPath === currentPath
+}
 
-if (isDirectRun) {
+if (await isDirectRun()) {
   await runComponentProjectPostinstall()
 }

package/scripts/preuninstall.mjs

@@ -1,5 +1,5 @@
 import fs from 'node:fs/promises'
-import path from 'node:path'
+import { fileURLToPath } from 'node:url'
 
 import { resolveComponentProjectGeneratedManifestPath } from './runtime-root.mjs'
 import { ComponentProjectGeneratedManifestSchema } from './schemas.mjs'
@@ -30,8 +30,16 @@ export async function runComponentProjectPreuninstall(
   await removeComponentProjectSkillsScript(projectRoot)
 }
 
-const isDirectRun = process.argv[1] && path.resolve(process.argv[1]) === path.resolve(new URL(import.meta.url).pathname)
+async function isDirectRun() {
+  if (!process.argv[1]) {
+    return false
+  }
+
+  const entryPath = await fs.realpath(process.argv[1])
+  const currentPath = await fs.realpath(fileURLToPath(import.meta.url))
+  return entryPath === currentPath
+}
 
-if (isDirectRun) {
+if (await isDirectRun()) {
   await runComponentProjectPreuninstall()
 }

package/src/discovery/index.test.ts

@@ -151,3 +151,36 @@ test('discovers decorated components from installed video-project-core', async (
     },
   ])
 })
+
+test('ignores test files from installed packages', async () => {
+  const projectRoot = await fs.mkdtemp(path.join(tmpdir(), 'component-project-helper-tests-'))
+  const packageTestDir = path.join(
+    projectRoot,
+    'node_modules',
+    '@vibecuting',
+    'video-project-helper',
+    'src',
+  )
+  await fs.mkdir(packageTestDir, { recursive: true })
+
+  await fs.writeFile(
+    path.join(packageTestDir, 'index.test.ts'),
+    `
+      @VideoComponent({
+        name: 'ShouldBeIgnored',
+        description: 'This fixture lives in a test file and must not be discovered',
+        sourceFile: 'node_modules/@vibecuting/video-project-helper/src/index.test.ts',
+        aspectRatio: '16:9',
+        sceneType: 'scene',
+        tags: ['test'],
+        propsTypeName: 'ShouldBeIgnoredProps',
+      })
+      export const ShouldBeIgnored = () => null
+    `,
+    'utf8',
+  )
+
+  const discovered = await discoverComponentProjectComponents(projectRoot)
+
+  expect(discovered).toEqual([])
+})

package/src/discovery/index.ts

@@ -32,6 +32,10 @@ async function collectComponentSourceFiles(directory: string): Promise<string[]>
       continue
     }
 
+    if (entry.name.includes('.test.') || entry.name.includes('.spec.')) {
+      continue
+    }
+
     if (entry.name.endsWith('.d.ts')) {
       continue
     }

package/src/lifecycle/postinstall.ts

@@ -12,6 +12,16 @@ import {
 } from '../runtime'
 import { ensureComponentProjectSkillsScript } from './package-json'
 
+const supportsColor =
+  Boolean(process.stdout.isTTY) && process.env.NO_COLOR !== '1' && process.env.CI !== 'true'
+
+const color = {
+  cyan: (value: string): string => (supportsColor ? `\u001b[36m${value}\u001b[0m` : value),
+  green: (value: string): string => (supportsColor ? `\u001b[32m${value}\u001b[0m` : value),
+  yellow: (value: string): string => (supportsColor ? `\u001b[33m${value}\u001b[0m` : value),
+  dim: (value: string): string => (supportsColor ? `\u001b[2m${value}\u001b[0m` : value),
+}
+
 async function ensureDirectory(dir: string): Promise<void> {
   await fs.mkdir(dir, { recursive: true })
 }
@@ -40,14 +50,16 @@ export async function runComponentProjectPostinstall(
   const manifestPath = resolveComponentProjectGeneratedManifestPath(projectRoot)
   const components = await discoverComponentProjectComponents(projectRoot)
 
-  console.log('[component-project-helper] postinstall start')
-  console.log(`[component-project-helper] project root: ${projectRoot}`)
-  console.log(`[component-project-helper] docs dir: ${docsDir}`)
-  console.log(`[component-project-helper] manifest path: ${manifestPath}`)
+  console.log(color.cyan('[component-project-helper] postinstall start'))
+  console.log(color.dim(`[component-project-helper] project root: ${projectRoot}`))
+  console.log(color.dim(`[component-project-helper] docs dir: ${docsDir}`))
+  console.log(color.dim(`[component-project-helper] manifest path: ${manifestPath}`))
   console.log(
-    `[component-project-helper] discovered ${components.length} component(s): ${
+    color.green(
+      `[component-project-helper] discovered ${components.length} component(s): ${
       components.map((component) => component.name).join(', ') || '(none)'
-    }`,
+      }`,
+    ),
   )
 
   await ensureDirectory(docsDir)
@@ -60,19 +72,23 @@ export async function runComponentProjectPostinstall(
     await fs.writeFile(filePath, markdown, 'utf8')
     files.push(filePath)
     console.log(
-      `[component-project-helper] wrote ${path.relative(projectRoot, filePath)} <- ${component.sourceFile}`,
+      color.green(
+        `[component-project-helper] wrote ${path.relative(projectRoot, filePath)} <- ${component.sourceFile}`,
+      ),
     )
   }
 
   console.log(
-    `[component-project-helper] cleaning stale docs relative to manifest ${path.relative(
+    color.yellow(
+      `[component-project-helper] cleaning stale docs relative to manifest ${path.relative(
       projectRoot,
       manifestPath,
-    )}`,
+      )}`,
+    ),
   )
   await removeStaleGeneratedFiles(manifestPath, files)
   await ensureComponentProjectSkillsScript(projectRoot)
-  console.log('[component-project-helper] ensured update-component-skills script')
+  console.log(color.green('[component-project-helper] ensured update-component-skills script'))
 
   const manifest = ComponentProjectGeneratedManifestSchema.parse({
     projectRoot,
@@ -83,14 +99,29 @@ export async function runComponentProjectPostinstall(
 
   await fs.writeFile(manifestPath, `${JSON.stringify(manifest, null, 2)}\n`, 'utf8')
   console.log(
-    `[component-project-helper] postinstall complete, manifest files: ${files.length}`,
+    color.cyan(
+      `[component-project-helper] postinstall complete, manifest files: ${files.length}`,
+    ),
   )
 }
 
+async function isDirectRun(): Promise<boolean> {
+  if (!process.argv[1]) {
+    return false
+  }
+
+  const [entryPath, currentPath] = await Promise.all([
+    fs.realpath(process.argv[1]),
+    fs.realpath(fileURLToPath(import.meta.url)),
+  ])
+
+  return entryPath === currentPath
+}
+
 async function main(): Promise<void> {
   await runComponentProjectPostinstall()
 }
 
-if (process.argv[1] && fileURLToPath(import.meta.url) === path.resolve(process.argv[1])) {
+if (await isDirectRun()) {
   await main()
 }

package/src/lifecycle/preuninstall.ts

@@ -1,5 +1,4 @@
 import fs from 'node:fs/promises'
-import path from 'node:path'
 import { fileURLToPath } from 'node:url'
 
 import { ComponentProjectGeneratedManifestSchema } from '../schemas'
@@ -37,6 +36,19 @@ async function main(): Promise<void> {
   await runComponentProjectPreuninstall()
 }
 
-if (process.argv[1] && fileURLToPath(import.meta.url) === path.resolve(process.argv[1])) {
+async function isDirectRun(): Promise<boolean> {
+  if (!process.argv[1]) {
+    return false
+  }
+
+  const [entryPath, currentPath] = await Promise.all([
+    fs.realpath(process.argv[1]),
+    fs.realpath(fileURLToPath(import.meta.url)),
+  ])
+
+  return entryPath === currentPath
+}
+
+if (await isDirectRun()) {
   await main()
 }