musora-content-services 2.104.0 → 2.104.2

package/.claude/settings.local.json

@@ -1,13 +1,16 @@
 {
   "permissions": {
     "allow": [
-      "Read(//app/musora-platform-backend/**)",
-      "Read(//app/musora-platform-frontend/**)",
       "Bash(find:*)",
-      "Bash(sed:*)",
-      "Read(//app/**)",
+      "Bash(docker exec:*)",
+      "Bash(npm test:*)",
+      "WebSearch",
+      "WebFetch(domain:watermelondb.dev)",
+      "WebFetch(domain:github.com)",
+      "Bash(git checkout:*)",
+      "Bash(npm run doc:*)",
       "Bash(cat:*)",
-      "Bash(docker exec:*)"
+      "Bash(tr:*)"
     ],
     "deny": [],
     "ask": []

package/CHANGELOG.md

@@ -2,6 +2,10 @@
 
 All notable changes to this project will be documented in this file. See [standard-version](https://github.com/conventional-changelog/standard-version) for commit guidelines.
 
+### [2.104.2](https://github.com/railroadmedia/musora-content-services/compare/v2.104.1...v2.104.2) (2025-12-12)
+
+### [2.104.1](https://github.com/railroadmedia/musora-content-services/compare/v2.104.0...v2.104.1) (2025-12-12)
+
 ## [2.104.0](https://github.com/railroadmedia/musora-content-services/compare/v2.100.3...v2.104.0) (2025-12-12)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "musora-content-services",
-  "version": "2.104.0",
+  "version": "2.104.2",
   "description": "A package for Musoras content services ",
   "main": "src/index.js",
   "type": "module",

package/src/contentTypeConfig.js

@@ -862,7 +862,7 @@ export function getChildFieldsForContentType(contentType, asQueryString = true)
 export function getFieldsForContentType(contentType, asQueryString = true) {
   const fields = contentType
     ? DEFAULT_FIELDS.concat(contentTypeConfig?.[contentType]?.fields ?? [])
-    : DEFAULT_FIELDS
+    : DEFAULT_FIELDS.slice() // ensure copy, not original reference
   return asQueryString ? fields.toString() + ',' : fields
 }
 

package/src/index.d.ts

@@ -9,7 +9,8 @@ import {
 	getAwardStatistics,
 	getCompletedAwards,
 	getContentAwards,
-	getInProgressAwards
+	getInProgressAwards,
+	resetAllAwards
 } from './services/awards/award-query.js';
 
 import {
@@ -683,6 +684,7 @@ declare module 'musora-content-services' {
 		reportPlaylist,
 		requestEmailChange,
 		reset,
+		resetAllAwards,
 		resetAllLearningPaths,
 		resetPassword,
 		restoreComment,

package/src/index.js

@@ -13,7 +13,8 @@ import {
 	getAwardStatistics,
 	getCompletedAwards,
 	getContentAwards,
-	getInProgressAwards
+	getInProgressAwards,
+	resetAllAwards
 } from './services/awards/award-query.js';
 
 import {
@@ -682,6 +683,7 @@ export {
 	reportPlaylist,
 	requestEmailChange,
 	reset,
+	resetAllAwards,
 	resetAllLearningPaths,
 	resetPassword,
 	restoreComment,

package/src/services/awards/award-query.js

@@ -493,3 +493,16 @@ export async function getAwardStatistics(brand = null) {
     }
   }
 }
+
+/**
+ * @returns {Promise<{ deletedCount: number }>}
+ */
+export async function resetAllAwards() {
+  try {
+    const result = await db.userAwardProgress.deleteAllAwards()
+    return result
+  } catch (error) {
+    console.error('Failed to reset awards:', error)
+    return { deletedCount: 0 }
+  }
+}

package/src/services/awards/internal/award-manager.js

@@ -129,6 +129,11 @@ export class AwardManager {
       const completedCount = completedLessonIds.length
       const progressPercentage = Math.round((completedCount / childIds.length) * 100)
 
+      if (progressPercentage === 100) {
+        await this.grantAward(award, collection)
+        return
+      }
+
       const progressData = {
         completedLessonIds,
         totalLessons: childIds.length,

package/src/services/sync/repositories/user-award-progress.ts

@@ -130,4 +130,30 @@ export default class UserAwardProgressRepository extends SyncRepository<UserAwar
 
     return { definitions, progress: progressMap }
   }
+
+  async deleteAllAwards() {
+    const allProgress = await this.getAll()
+    const ids = allProgress.data.map(p => p.id)
+
+    if (ids.length === 0) {
+      return { deletedCount: 0 }
+    }
+
+    await this.deleteSome(ids)
+
+    const response = await this.store.pushRecordIdsImpatiently(ids)
+
+    if (response && response.ok) {
+      await this.store.db.write(async () => {
+        const collection = this.store.db.get<UserAwardProgress>(UserAwardProgress.table)
+        const deletedRecords = await collection
+          .query(Q.where('id', Q.oneOf(ids)))
+          .fetch()
+
+        await Promise.all(deletedRecords.map(record => record.destroyPermanently()))
+      })
+    }
+
+    return { deletedCount: ids.length }
+  }
 }

package/test/awards/award-completion-flow.test.js

@@ -210,4 +210,94 @@ describe('Award Completion Flow - E2E Scenarios', () => {
       expect(db.userAwardProgress.recordAwardProgress).not.toHaveBeenCalled()
     })
   })
+
+  describe('Scenario: Race condition - eligibility check fails but progress reaches 100%', () => {
+    const multiLessonAward = getAwardByContentId(417049)
+    const parentCourseId = 417049
+
+    test('grants award with completedAt when updateAwardProgress calculates 100%', async () => {
+      let callCount = 0
+
+      db.contentProgress.getSomeProgressByContentIds.mockImplementation((contentIds) => {
+        callCount++
+
+        if (callCount === 1) {
+          const partialRecords = contentIds
+            .filter(id => [417045, 417046, 417047].includes(id))
+            .map(id => ({
+              content_id: id,
+              state: 'completed',
+              created_at: Math.floor(Date.now() / 1000)
+            }))
+          return Promise.resolve({ data: partialRecords })
+        }
+
+        const allRecords = contentIds.map(id => ({
+          content_id: id,
+          state: 'completed',
+          created_at: Math.floor(Date.now() / 1000)
+        }))
+        return Promise.resolve({ data: allRecords })
+      })
+
+      await awardManager.onContentCompleted(parentCourseId)
+
+      expect(db.userAwardProgress.recordAwardProgress).toHaveBeenCalledWith(
+        multiLessonAward._id,
+        100,
+        expect.objectContaining({
+          completedAt: expect.any(Number),
+          completionData: expect.objectContaining({
+            completed_at: expect.any(String)
+          }),
+          immediate: true
+        })
+      )
+
+      expect(listeners.granted).toHaveBeenCalledTimes(1)
+      expect(listeners.progress).not.toHaveBeenCalled()
+    })
+
+    test('emits awardGranted event when race condition triggers completion', async () => {
+      let callCount = 0
+
+      db.contentProgress.getSomeProgressByContentIds.mockImplementation((contentIds) => {
+        callCount++
+
+        if (callCount === 1) {
+          const partialRecords = contentIds
+            .filter(id => [417045, 417046].includes(id))
+            .map(id => ({
+              content_id: id,
+              state: 'completed',
+              created_at: Math.floor(Date.now() / 1000)
+            }))
+          return Promise.resolve({ data: partialRecords })
+        }
+
+        const allRecords = contentIds.map(id => ({
+          content_id: id,
+          state: 'completed',
+          created_at: Math.floor(Date.now() / 1000)
+        }))
+        return Promise.resolve({ data: allRecords })
+      })
+
+      await awardManager.onContentCompleted(parentCourseId)
+
+      expect(listeners.granted).toHaveBeenCalledTimes(1)
+      expect(listeners.granted).toHaveBeenCalledWith(
+        expect.objectContaining({
+          awardId: multiLessonAward._id,
+          definition: expect.objectContaining({
+            name: multiLessonAward.name
+          }),
+          completionData: expect.objectContaining({
+            completed_at: expect.any(String)
+          }),
+          timestamp: expect.any(Number)
+        })
+      )
+    })
+  })
 })

package/.yarnrc.yml

@@ -1 +0,0 @@
-nodeLinker: node-modules

package/test/reporting.test.js

@@ -1,132 +0,0 @@
-/**
- * Tests for Reporting Service
- *
- * Note: These tests use mocked HTTP calls. For integration tests,
- * ensure the backend API is running at the configured baseUrl.
- */
-
-import { initializeTestService } from './initializeTests.js'
-import {
-  reportContent,
-  reportForumPost,
-  submitReport
-} from '../src/services/reporting/reporting.ts'
-
-// Mock HttpClient to avoid actual API calls in tests
-jest.mock('../src/infrastructure/http/HttpClient', () => {
-  return {
-    HttpClient: jest.fn().mockImplementation(() => ({
-      post: jest.fn().mockResolvedValue({
-        report_id: 12345,
-        message: 'Report submitted successfully',
-        is_duplicate: false
-      })
-    }))
-  }
-})
-
-describe('Reporting Service', () => {
-  beforeEach(() => {
-    initializeTestService()
-  })
-
-  describe('reportContent', () => {
-    test('should submit a content report with video_not_working category', async () => {
-      const result = await reportContent({
-        contentId: 123,
-        category: 'video_not_working',
-        description: 'Video freezes at 2:30'
-      })
-
-      expect(result).toBeDefined()
-      expect(result.report_id).toBe(12345)
-      expect(result.message).toBe('Report submitted successfully')
-      expect(result.is_duplicate).toBe(false)
-    })
-
-    test('should submit a content report without description', async () => {
-      const result = await reportContent({
-        contentId: 456,
-        category: 'incorrect_content'
-      })
-
-      expect(result).toBeDefined()
-      expect(result.report_id).toBeDefined()
-    })
-
-    test('should submit a content report with other category and description', async () => {
-      const result = await reportContent({
-        contentId: 789,
-        category: 'other',
-        description: 'Audio quality is very poor'
-      })
-
-      expect(result).toBeDefined()
-      expect(result.report_id).toBeDefined()
-      expect(result.message).toBeDefined()
-    })
-  })
-
-  describe('reportForumPost', () => {
-    test('should submit a forum post report with required brand', async () => {
-      const result = await reportForumPost({
-        postId: 555,
-        brand: 'drumeo',
-        category: 'spam'
-      })
-
-      expect(result).toBeDefined()
-      expect(result.report_id).toBeDefined()
-    })
-
-    test('should submit a forum post report with description', async () => {
-      const result = await reportForumPost({
-        postId: 666,
-        brand: 'pianote',
-        category: 'harassment',
-        description: 'User is harassing other members'
-      })
-
-      expect(result).toBeDefined()
-      expect(result.report_id).toBeDefined()
-      expect(result.message).toBeDefined()
-    })
-  })
-
-  describe('submitReport', () => {
-    test('should submit a generic report for content', async () => {
-      const result = await submitReport({
-        reportableType: 'content',
-        reportableId: 999,
-        category: 'technical_issue',
-        description: 'Page not loading'
-      })
-
-      expect(result).toBeDefined()
-      expect(result.report_id).toBeDefined()
-    })
-
-    test('should submit a generic report for playlist', async () => {
-      const result = await submitReport({
-        reportableType: 'playlist',
-        reportableId: 777,
-        category: 'incorrect_content'
-      })
-
-      expect(result).toBeDefined()
-      expect(result.report_id).toBeDefined()
-    })
-
-    test('should submit a generic report for comment', async () => {
-      const result = await submitReport({
-        reportableType: 'comment',
-        reportableId: 888,
-        category: 'inappropriate',
-        description: 'Comment contains offensive language'
-      })
-
-      expect(result).toBeDefined()
-      expect(result.report_id).toBeDefined()
-    })
-  })
-})