musora-content-services 2.104.0 → 2.104.4

package/.claude/settings.local.json

@@ -7,7 +7,8 @@
       "Bash(sed:*)",
       "Read(//app/**)",
       "Bash(cat:*)",
-      "Bash(docker exec:*)"
+      "Bash(docker exec:*)",
+      "Bash(npm config:*)"
     ],
     "deny": [],
     "ask": []

package/CHANGELOG.md

@@ -2,6 +2,19 @@
 
 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.4](https://github.com/railroadmedia/musora-content-services/compare/v2.104.3...v2.104.4) (2025-12-12)
+
+### [2.104.3](https://github.com/railroadmedia/musora-content-services/compare/v2.104.0...v2.104.3) (2025-12-12)
+
+
+### Bug Fixes
+
+* **MU2-1309:** navigateTo missing on owned contents ([9b12213](https://github.com/railroadmedia/musora-content-services/commit/9b12213ad8061d60eada1603c0384d36b7a955f3))
+
+### [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/check_content.js

@@ -0,0 +1,30 @@
+const { initializeService } = require('./src/services/config.js');
+const { fetchByRailContentIds } = require('./src/services/sanity.js');
+require('dotenv/config');
+
+async function checkContent() {
+  initializeService({
+    sanityConfig: {
+      token: process.env.SANITY_TOKEN,
+      projectId: process.env.SANITY_PROJECT_ID,
+      dataset: process.env.SANITY_DATASET,
+      version: process.env.SANITY_VERSION || '2021-06-07',
+    },
+    railcontentConfig: {
+      token: process.env.RAILCONTENT_TOKEN,
+      userId: process.env.RAILCONTENT_USER_ID,
+      baseUrl: process.env.RAILCONTENT_BASE_URL,
+      authToken: process.env.RAILCONTENT_AUTH_TOKEN,
+    },
+    baseUrl: process.env.RAILCONTENT_BASE_URL,
+    localStorage: null,
+    isMA: false,
+  });
+
+  console.log('Checking railcontent_id: 421814');
+  const contents = await fetchByRailContentIds([421814]);
+  console.log('Results:', JSON.stringify(contents, null, 2));
+  console.log('Found:', contents.length, 'items');
+}
+
+checkContent().catch(console.error);

package/check_content.mjs

@@ -0,0 +1,32 @@
+import { initializeService } from './src/services/config.js';
+import { fetchByRailContentIds } from './src/services/sanity.js';
+import dotenv from 'dotenv';
+
+dotenv.config();
+
+async function checkContent() {
+  initializeService({
+    sanityConfig: {
+      token: process.env.SANITY_TOKEN,
+      projectId: process.env.SANITY_PROJECT_ID,
+      dataset: process.env.SANITY_DATASET,
+      version: process.env.SANITY_VERSION || '2021-06-07',
+    },
+    railcontentConfig: {
+      token: process.env.RAILCONTENT_TOKEN,
+      userId: process.env.RAILCONTENT_USER_ID,
+      baseUrl: process.env.RAILCONTENT_BASE_URL,
+      authToken: process.env.RAILCONTENT_AUTH_TOKEN,
+    },
+    baseUrl: process.env.RAILCONTENT_BASE_URL,
+    localStorage: null,
+    isMA: false,
+  });
+
+  console.log('Checking railcontent_id: 421814');
+  const contents = await fetchByRailContentIds([421814]);
+  console.log('Results:', JSON.stringify(contents, null, 2));
+  console.log('Found:', contents.length, 'items');
+}
+
+checkContent().catch(console.error);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "musora-content-services",
-  "version": "2.104.0",
+  "version": "2.104.4",
   "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/content.js

@@ -527,5 +527,10 @@ export async function getOwnedContent(brand, {
     };
   }
 
-  return data;
+  return await addContextToContent(() => data, {
+    dataField: 'entity',
+    addNavigateTo: true,
+    addProgressPercentage: true,
+    addProgressStatus: true,
+  });
 }

package/src/services/sanity.js

@@ -2068,7 +2068,8 @@ export async function fetchOwnedContent(
     showOnlyOwnedContent: true, // Key parameter: exclude membership content
   }).buildFilter()
 
-  const fieldsString = DEFAULT_FIELDS.join(',')
+  // Use 'tab-data' to include children field (needed for navigateTo calculation)
+  const fieldsString = await getFieldsForContentTypeWithFilteredChildren('tab-data', true)
 
   const query = buildEntityAndTotalQuery(filterWithRestrictions, fieldsString, {
     sortOrder: sortOrder,

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/test_owned_navigate.js

@@ -0,0 +1,74 @@
+import { initializeService } from './src/services/config.js'
+import { getOwnedContent } from './src/services/content.js'
+import dotenv from 'dotenv'
+
+dotenv.config()
+
+// Initialize service with credentials from .env
+initializeService({
+  sanityConfig: {
+    token: process.env.SANITY_API_TOKEN,
+    projectId: process.env.SANITY_PROJECT_ID,
+    dataset: process.env.SANITY_DATASET,
+    version: '2023-01-01',
+  },
+  railcontentConfig: {
+    email: process.env.RAILCONTENT_EMAIL,
+    password: process.env.RAILCONTENT_PASSWORD,
+    userId: parseInt(process.env.RAILCONTENT_USER_ID),
+    baseUrl: process.env.RAILCONTENT_BASE_URL,
+  },
+  localStorage: null,
+  isMA: false,
+})
+
+async function testOwnedContentNavigateTo() {
+  console.log('\n🧪 Testing owned content with navigateTo...\n')
+
+  try {
+    // Test for drumeo lessons
+    const result = await getOwnedContent('drumeo', {
+      type: ['course', 'guided-course', 'pack'],
+      limit: 5,
+    })
+
+    console.log('📊 Results:', {
+      total: result.total,
+      count: result.entity?.length || 0,
+    })
+
+    if (result.entity && result.entity.length > 0) {
+      result.entity.forEach((item, index) => {
+        console.log(`\n📦 Item ${index + 1}:`)
+        console.log(`  - ID: ${item.id}`)
+        console.log(`  - Title: ${item.title}`)
+        console.log(`  - Type: ${item.type}`)
+        console.log(`  - Has children: ${item.children ? '✅ YES' : '❌ NO'}`)
+        console.log(`  - Children count: ${item.children?.length || 0}`)
+        console.log(`  - Has navigateTo: ${item.navigateTo ? '✅ YES' : '❌ NO'}`)
+
+        if (item.navigateTo) {
+          console.log(`  - NavigateTo ID: ${item.navigateTo.id}`)
+          console.log(`  - NavigateTo Type: ${item.navigateTo.type}`)
+          console.log(`  - NavigateTo Title: ${item.children?.find(c => c.id === item.navigateTo.id)?.title || 'N/A'}`)
+        }
+
+        if (item.progressPercentage !== undefined) {
+          console.log(`  - Progress: ${item.progressPercentage}%`)
+        }
+        if (item.progressStatus) {
+          console.log(`  - Status: ${item.progressStatus}`)
+        }
+      })
+    } else {
+      console.log('\n⚠️  No owned content found for this user')
+    }
+
+    console.log('\n✅ Test completed successfully!')
+  } catch (error) {
+    console.error('\n❌ Test failed:', error.message)
+    console.error(error.stack)
+  }
+}
+
+testOwnedContentNavigateTo()