musora-content-services 2.103.6 → 2.104.2

package/.claude/settings.local.json

@@ -0,0 +1,18 @@
+{
+  "permissions": {
+    "allow": [
+      "Bash(find:*)",
+      "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(tr:*)"
+    ],
+    "deny": [],
+    "ask": []
+  }
+}

package/CHANGELOG.md

@@ -2,6 +2,30 @@
 
 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)
+
+
+### Features
+
+* add isCompleted and field name updates in award callback([#648](https://github.com/railroadmedia/musora-content-services/issues/648)) ([c250d61](https://github.com/railroadmedia/musora-content-services/commit/c250d61b90bc29504827e8fa368a78b34a3f0c08))
+* **auth:** authenticate via auth key ([#644](https://github.com/railroadmedia/musora-content-services/issues/644)) ([122ba63](https://github.com/railroadmedia/musora-content-services/commit/122ba63572e95faf49944a3e819b768cb0a9ab85))
+* **BEH-1457:** method progress duplication ([#617](https://github.com/railroadmedia/musora-content-services/issues/617)) ([917cb9d](https://github.com/railroadmedia/musora-content-services/commit/917cb9dedce2a21c64af4e39eef013a30b236e77))
+
+
+### Bug Fixes
+
+* **agi:** offset default and slug clause on artistBySlug ([10cc6d0](https://github.com/railroadmedia/musora-content-services/commit/10cc6d0390a12715aeba01c5dc72ecdb53d0f236))
+* **agi:** remove count from AGI lessons functions ([5cf3bc3](https://github.com/railroadmedia/musora-content-services/commit/5cf3bc3ade98faa5200cd64ce4b0a9edaff29d7a))
+* **agi:** rollback total on fetchAGILessons ([a02be05](https://github.com/railroadmedia/musora-content-services/commit/a02be0592dd8b195da7971f13ea062674510c8bb))
+* lp lesson upserting ([#645](https://github.com/railroadmedia/musora-content-services/issues/645)) ([1a3094d](https://github.com/railroadmedia/musora-content-services/commit/1a3094d569bb868d62844755b876cee8bc63a365))
+* mobile login broken ([785e0fc](https://github.com/railroadmedia/musora-content-services/commit/785e0fc1fc1ce58840d0e85874f68b9c77342290))
+* **MU2-1305:** fix url and add postId for active discussions ([ac42889](https://github.com/railroadmedia/musora-content-services/commit/ac4288955e2d7c817e0a51aaf1364b75aec6eea1))
+* user id missing on auth key ([ba4068f](https://github.com/railroadmedia/musora-content-services/commit/ba4068ff3aa8005b1b3ed615b07d4ff7eb9f751b))
+
 ### [2.103.6](https://github.com/railroadmedia/musora-content-services/compare/v2.103.5...v2.103.6) (2025-12-12)
 
 ### [2.103.5](https://github.com/railroadmedia/musora-content-services/compare/v2.103.4...v2.103.5) (2025-12-12)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "musora-content-services",
-  "version": "2.103.6",
+  "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/forums/forums.ts

@@ -70,9 +70,10 @@ function transformLatestDiscussions(response: PaginatedResponse<ForumThread>): T
 
       return {
         id: thread.id,
-        url: `forums/post/${postId}`,
+        url: `forums/threads/${thread.category_id}/${thread.id}`,
         title: thread.title,
         post: postContent,
+        postId:postId,
         author: {
           id: thread.author!.id,
           name: thread.author!.display_name,

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)
+        })
+      )
+    })
+  })
 })