musora-content-services 2.107.4 → 2.107.5

package/.claude/settings.local.json

@@ -1,16 +1,9 @@
 {
   "permissions": {
     "allow": [
-      "Read(//app/musora-platform-backend/**)",
-      "Read(//app/musora-platform-frontend/**)",
-      "Bash(find:*)",
-      "Bash(sed:*)",
-      "Read(//app/**)",
-      "Bash(cat:*)",
-      "Bash(docker exec:*)",
-      "Bash(npm config:*)"
+      "Bash(rg:*)",
+      "Bash(npm run lint:*)"
     ],
-    "deny": [],
-    "ask": []
+    "deny": []
   }
 }

package/CHANGELOG.md

@@ -2,6 +2,13 @@
 
 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.107.5](https://github.com/railroadmedia/musora-content-services/compare/v2.107.4...v2.107.5) (2025-12-29)
+
+
+### Bug Fixes
+
+* establishes a positive progress validation ([#672](https://github.com/railroadmedia/musora-content-services/issues/672)) ([e9bc211](https://github.com/railroadmedia/musora-content-services/commit/e9bc211659262b282e1073c2746c3c8824371c35))
+
 ### [2.107.4](https://github.com/railroadmedia/musora-content-services/compare/v2.107.1...v2.107.4) (2025-12-22)
 
 

package/package.json

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

package/src/services/contentProgress.js

@@ -469,6 +469,13 @@ export async function contentStatusReset(contentId, collection = null) {
 }
 
 async function saveContentProgress(contentId, collection, progress, currentSeconds) {
+
+  // filter out contentIds that are setting progress lower than existing
+  const contentIdProgress = await getProgressDataByIds([contentId], collection)
+  if (progress <= contentIdProgress[contentId].progress) {
+    return
+  }
+
   const response = await db.contentProgress.recordProgress(
     contentId,
     collection,
@@ -483,6 +490,15 @@ async function saveContentProgress(contentId, collection, progress, currentSecon
   const hierarchy = await getHierarchy(contentId, collection)
 
   const bubbledProgresses = await bubbleProgress(hierarchy, contentId, collection)
+
+  // filter out contentIds that are setting progress lower than existing
+  const existingProgresses = await getProgressDataByIds(Object.keys(bubbledProgresses), collection)
+  for (const [bubbledContentId, bubbledProgress] of Object.entries(bubbledProgresses)) {
+    if (bubbledProgress <= existingProgresses[bubbledContentId].progress) {
+      delete bubbledProgresses[bubbledContentId]
+    }
+  }
+
   // BE bubbling/trickling currently does not work, so we utilize non-tentative pushing when learning path collection
   await db.contentProgress.recordProgressMany(bubbledProgresses, collection, collection?.type !== COLLECTION_TYPE.LEARNING_PATH)
 

package/src/services/sanity.js

@@ -927,7 +927,7 @@ export async function fetchLessonContent(railContentId, { addParent = false } =
     "instructor": ${instructorField},
     ${assignmentsField}
     video,
-    length_in_seconds,
+    "length_in_seconds": coalesce(soundslice[0].soundslice_length_in_second, length_in_seconds),
     mp3_no_drums_no_click_url,
     mp3_no_drums_yes_click_url,
     mp3_yes_drums_no_click_url,

package/.yarnrc.yml

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

package/check_content.js

@@ -1,30 +0,0 @@
-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

@@ -1,32 +0,0 @@
-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/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()
-    })
-  })
-})

package/test_owned_navigate.js

@@ -1,74 +0,0 @@
-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()