canopycms 0.0.0 → 0.0.2

package/src/comment-store.test.ts

@@ -1,442 +0,0 @@
-import { describe, expect, it, beforeEach, afterEach } from 'vitest'
-import fs from 'fs/promises'
-import path from 'path'
-import os from 'os'
-import { CommentStore } from './comment-store'
-
-describe('CommentStore', () => {
-  let tmpDir: string
-  let store: CommentStore
-
-  beforeEach(async () => {
-    tmpDir = await fs.mkdtemp(path.join(os.tmpdir(), 'comment-store-test-'))
-    store = new CommentStore(tmpDir)
-  })
-
-  afterEach(async () => {
-    await fs.rm(tmpDir, { recursive: true, force: true })
-  })
-
-  it('loads empty structure when file does not exist', async () => {
-    const data = await store.load()
-    expect(data.schemaVersion).toBe(1)
-    expect(data.threads).toEqual({})
-  })
-
-  it('creates new field comment thread', async () => {
-    const result = await store.addComment({
-      userId: 'user1',
-      text: 'First comment',
-      type: 'field',
-      entryPath: 'posts/hello',
-      canopyPath: 'title',
-    })
-
-    expect(result.threadId).toBeTruthy()
-    expect(result.commentId).toBeTruthy()
-
-    const threads = await store.listThreads()
-    expect(threads).toHaveLength(1)
-    expect(threads[0].comments).toHaveLength(1)
-    expect(threads[0].comments[0].text).toBe('First comment')
-    expect(threads[0].comments[0].userId).toBe('user1')
-    expect(threads[0].type).toBe('field')
-    expect(threads[0].entryPath).toBe('posts/hello')
-    expect(threads[0].canopyPath).toBe('title')
-    expect(threads[0].authorId).toBe('user1')
-    expect(threads[0].createdAt).toBeTruthy()
-  })
-
-  it('creates entry-level comment thread', async () => {
-    const result = await store.addComment({
-      userId: 'user1',
-      text: 'Entry comment',
-      type: 'entry',
-      entryPath: 'posts/hello',
-    })
-
-    const thread = await store.getThread(result.threadId)
-    expect(thread?.type).toBe('entry')
-    expect(thread?.entryPath).toBe('posts/hello')
-    expect(thread?.canopyPath).toBeUndefined()
-  })
-
-  it('creates branch-level comment thread', async () => {
-    const result = await store.addComment({
-      userId: 'user1',
-      text: 'Branch discussion',
-      type: 'branch',
-    })
-
-    const thread = await store.getThread(result.threadId)
-    expect(thread?.type).toBe('branch')
-    expect(thread?.entryPath).toBeUndefined()
-    expect(thread?.canopyPath).toBeUndefined()
-  })
-
-  it('adds comment to existing thread', async () => {
-    const first = await store.addComment({
-      userId: 'user1',
-      text: 'First comment',
-      type: 'field',
-      entryPath: 'posts/hello',
-      canopyPath: 'title',
-    })
-
-    const second = await store.addComment({
-      userId: 'user2',
-      text: 'Reply',
-      type: 'field',
-      threadId: first.threadId,
-      entryPath: 'posts/hello',
-      canopyPath: 'title',
-    })
-
-    expect(second.threadId).toBe(first.threadId)
-
-    const thread = await store.getThread(first.threadId)
-    expect(thread?.comments).toHaveLength(2)
-    expect(thread?.comments[1].text).toBe('Reply')
-    expect(thread?.comments[1].userId).toBe('user2')
-  })
-
-  it('resolves thread with userId', async () => {
-    const result = await store.addComment({
-      userId: 'user1',
-      text: 'Comment',
-      type: 'field',
-      entryPath: 'posts/hello',
-      canopyPath: 'title',
-    })
-
-    await store.resolveThread(result.threadId, 'reviewer1')
-
-    const thread = await store.getThread(result.threadId)
-    expect(thread?.resolved).toBe(true)
-    expect(thread?.resolvedBy).toBe('reviewer1')
-    expect(thread?.resolvedAt).toBeTruthy()
-  })
-
-  it('filters resolved threads when requested', async () => {
-    const unresolved = await store.addComment({
-      userId: 'user1',
-      text: 'Unresolved',
-      type: 'field',
-      entryPath: 'posts/hello',
-      canopyPath: 'title',
-    })
-
-    const resolved = await store.addComment({
-      userId: 'user2',
-      text: 'Resolved',
-      type: 'field',
-      entryPath: 'posts/hello',
-      canopyPath: 'description',
-    })
-
-    await store.resolveThread(resolved.threadId, 'user2')
-
-    const allThreads = await store.listThreads({ includeResolved: true })
-    expect(allThreads).toHaveLength(2)
-
-    const unresolvedOnly = await store.listThreads({ includeResolved: false })
-    expect(unresolvedOnly).toHaveLength(1)
-    expect(unresolvedOnly[0].id).toBe(unresolved.threadId)
-  })
-
-  it('gets threads for specific field', async () => {
-    await store.addComment({
-      userId: 'user1',
-      text: 'Title comment',
-      type: 'field',
-      entryPath: 'posts/hello',
-      canopyPath: 'title',
-    })
-
-    await store.addComment({
-      userId: 'user2',
-      text: 'Description comment',
-      type: 'field',
-      entryPath: 'posts/hello',
-      canopyPath: 'description',
-    })
-
-    const titleThreads = await store.getThreadsForField('posts/hello', 'title')
-    expect(titleThreads).toHaveLength(1)
-    expect(titleThreads[0].comments[0].text).toBe('Title comment')
-  })
-
-  it('gets threads for specific entry', async () => {
-    await store.addComment({
-      userId: 'user1',
-      text: 'Entry comment 1',
-      type: 'entry',
-      entryPath: 'posts/hello',
-    })
-
-    await store.addComment({
-      userId: 'user2',
-      text: 'Entry comment 2',
-      type: 'entry',
-      entryPath: 'posts/hello',
-    })
-
-    await store.addComment({
-      userId: 'user3',
-      text: 'Different entry',
-      type: 'entry',
-      entryPath: 'posts/world',
-    })
-
-    const entryThreads = await store.getThreadsForEntry('posts/hello')
-    expect(entryThreads).toHaveLength(2)
-  })
-
-  it('gets branch-level threads', async () => {
-    await store.addComment({
-      userId: 'user1',
-      text: 'Branch comment 1',
-      type: 'branch',
-    })
-
-    await store.addComment({
-      userId: 'user2',
-      text: 'Branch comment 2',
-      type: 'branch',
-    })
-
-    await store.addComment({
-      userId: 'user3',
-      text: 'Field comment',
-      type: 'field',
-      entryPath: 'posts/hello',
-      canopyPath: 'title',
-    })
-
-    const branchThreads = await store.getBranchThreads()
-    expect(branchThreads).toHaveLength(2)
-  })
-
-  it('sorts threads by createdAt timestamp', async () => {
-    // Add threads with slight delays to ensure different timestamps
-    await store.addComment({
-      userId: 'user1',
-      text: 'Second thread',
-      type: 'field',
-      entryPath: 'posts/hello',
-      canopyPath: 'title',
-    })
-
-    // Small delay
-    await new Promise((resolve) => setTimeout(resolve, 10))
-
-    await store.addComment({
-      userId: 'user2',
-      text: 'Third thread',
-      type: 'field',
-      entryPath: 'posts/hello',
-      canopyPath: 'title',
-    })
-
-    const threads = await store.getThreadsForField('posts/hello', 'title')
-    expect(threads).toHaveLength(2)
-    expect(threads[0].comments[0].text).toBe('Second thread')
-    expect(threads[1].comments[0].text).toBe('Third thread')
-  })
-
-  it('deletes thread', async () => {
-    const result = await store.addComment({
-      userId: 'user1',
-      text: 'To delete',
-      type: 'field',
-      entryPath: 'posts/hello',
-      canopyPath: 'title',
-    })
-
-    const deleted = await store.deleteThread(result.threadId)
-    expect(deleted).toBe(true)
-
-    const thread = await store.getThread(result.threadId)
-    expect(thread).toBeNull()
-  })
-
-  it('returns false when resolving non-existent thread', async () => {
-    const result = await store.resolveThread('non-existent', 'user1')
-    expect(result).toBe(false)
-  })
-
-  it('returns false when deleting non-existent thread', async () => {
-    const result = await store.deleteThread('non-existent')
-    expect(result).toBe(false)
-  })
-
-  it('persists data to file', async () => {
-    await store.addComment({
-      userId: 'user1',
-      text: 'Persisted',
-      type: 'field',
-      entryPath: 'posts/hello',
-      canopyPath: 'title',
-    })
-
-    // Create new store instance pointing to same directory
-    const newStore = new CommentStore(tmpDir)
-    const threads = await newStore.listThreads()
-
-    expect(threads).toHaveLength(1)
-    expect(threads[0].comments[0].text).toBe('Persisted')
-  })
-
-  it('includes version in saved file', async () => {
-    await store.addComment({
-      userId: 'user1',
-      text: 'Comment',
-      type: 'branch',
-    })
-
-    // Read raw file to check version
-    const filePath = path.join(tmpDir, '.canopy-meta', 'comments.json')
-    const raw = await fs.readFile(filePath, 'utf-8')
-    const data = JSON.parse(raw)
-    expect(data.version).toBe(1)
-
-    // Add another comment, version should increment
-    await store.addComment({
-      userId: 'user2',
-      text: 'Second',
-      type: 'branch',
-    })
-
-    const raw2 = await fs.readFile(filePath, 'utf-8')
-    const data2 = JSON.parse(raw2)
-    expect(data2.version).toBe(2)
-  })
-
-  describe('concurrency', () => {
-    it('handles concurrent addComment calls from different store instances', async () => {
-      const store1 = new CommentStore(tmpDir)
-      const store2 = new CommentStore(tmpDir)
-
-      // Both add comments concurrently
-      const results = await Promise.allSettled([
-        store1.addComment({
-          userId: 'user1',
-          text: 'Comment 1',
-          type: 'branch',
-        }),
-        store2.addComment({
-          userId: 'user2',
-          text: 'Comment 2',
-          type: 'branch',
-        }),
-      ])
-
-      // Check if any rejected
-      for (const r of results) {
-        if (r.status === 'rejected') {
-          console.error('Operation failed:', r.reason)
-        }
-      }
-
-      // Both should succeed (via retry)
-      const result1 = results[0].status === 'fulfilled' ? results[0].value : null
-      const result2 = results[1].status === 'fulfilled' ? results[1].value : null
-
-      expect(result1?.threadId).toBeDefined()
-      expect(result2?.threadId).toBeDefined()
-      expect(result1?.threadId).not.toBe(result2?.threadId)
-
-      // Both comments should be present
-      const threads = await store1.listThreads()
-      expect(threads.length).toBe(2)
-
-      const texts = threads.map((t) => t.comments[0].text).sort()
-      expect(texts).toEqual(['Comment 1', 'Comment 2'])
-    })
-
-    it('handles concurrent resolveThread calls', async () => {
-      // Create two threads first
-      const thread1 = await store.addComment({
-        userId: 'user1',
-        text: 'Thread 1',
-        type: 'branch',
-      })
-      const thread2 = await store.addComment({
-        userId: 'user2',
-        text: 'Thread 2',
-        type: 'branch',
-      })
-
-      const store1 = new CommentStore(tmpDir)
-      const store2 = new CommentStore(tmpDir)
-
-      // Resolve both threads concurrently
-      const [result1, result2] = await Promise.all([
-        store1.resolveThread(thread1.threadId, 'resolver1'),
-        store2.resolveThread(thread2.threadId, 'resolver2'),
-      ])
-
-      expect(result1).toBe(true)
-      expect(result2).toBe(true)
-
-      // Both threads should be resolved
-      const t1 = await store.getThread(thread1.threadId)
-      const t2 = await store.getThread(thread2.threadId)
-      expect(t1?.resolved).toBe(true)
-      expect(t2?.resolved).toBe(true)
-    })
-
-    it('handles concurrent deleteThread calls', async () => {
-      // Create two threads first
-      const thread1 = await store.addComment({
-        userId: 'user1',
-        text: 'Thread 1',
-        type: 'branch',
-      })
-      const thread2 = await store.addComment({
-        userId: 'user2',
-        text: 'Thread 2',
-        type: 'branch',
-      })
-
-      const store1 = new CommentStore(tmpDir)
-      const store2 = new CommentStore(tmpDir)
-
-      // Delete both threads concurrently
-      const [result1, result2] = await Promise.all([
-        store1.deleteThread(thread1.threadId),
-        store2.deleteThread(thread2.threadId),
-      ])
-
-      expect(result1).toBe(true)
-      expect(result2).toBe(true)
-
-      // Both threads should be gone
-      const threads = await store.listThreads()
-      expect(threads.length).toBe(0)
-    })
-
-    it('handles high contention with many concurrent writes', { retry: 1 }, async () => {
-      const numConcurrent = 10
-      const stores = Array.from({ length: numConcurrent }, () => new CommentStore(tmpDir))
-
-      // All add comments concurrently
-      const results = await Promise.all(
-        stores.map((s, i) =>
-          s.addComment({
-            userId: `user${i}`,
-            text: `Comment ${i}`,
-            type: 'branch',
-          }),
-        ),
-      )
-
-      // All should succeed
-      expect(results.every((r) => r.threadId && r.commentId)).toBe(true)
-
-      // All comments should be present
-      const threads = await store.listThreads()
-      expect(threads.length).toBe(numConcurrent)
-    })
-  })
-})