musora-content-services 2.98.0 → 2.99.0

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.99.0](https://github.com/railroadmedia/musora-content-services/compare/v2.98.0...v2.99.0) (2025-12-09)
+
+
+### Features
+
+* **BEH-1421:** Content Migration : child lesson type migrations ([#624](https://github.com/railroadmedia/musora-content-services/issues/624)) ([77300ed](https://github.com/railroadmedia/musora-content-services/commit/77300eddabc4deed46c26f7b1291d9540a3ef486))
+
 ## [2.98.0](https://github.com/railroadmedia/musora-content-services/compare/v2.94.7...v2.98.0) (2025-12-09)
 
 

package/package.json

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

package/src/contentTypeConfig.js

@@ -6,21 +6,19 @@ export const AWSUrl = 'https://s3.us-east-1.amazonaws.com/musora-web-platform'
 export const CloudFrontURl = 'https://d3fzm1tzeyr5n3.cloudfront.net'
 
 // This is used to pull related content by license, so we only show "consumable" content
-export const SONG_TYPES = ['song', 'play-along', 'jam-track', 'song-tutorial-children']
+export const SONG_TYPES = ['song', 'play-along', 'jam-track', 'song-tutorial-lesson']
 // Oct 2025: It turns out content-meta categories are not really clear
 // THis is used for the page_type field as a post processor so we include parents and children
 // Duplicated in SanityGateway.php if you update this, update that
 export const SONG_TYPES_WITH_CHILDREN = [
   'song',
-  'song-part',
   'play-along',
-  'play-along-part',
   'jam-track',
   'song-tutorial',
-  'song-tutorial-children',
+  'song-tutorial-lesson',
 ]
 // Single hierarchy refers to only one element in the hierarchy has video lessons, not that they have a single parent
-export const SINGLE_PARENT_TYPES = ['course-part', 'pack-bundle-lesson', 'song-tutorial-children']
+export const SINGLE_PARENT_TYPES = ['course-lesson', 'pack-bundle-lesson', 'song-tutorial-lesson']
 
 export const genreField = `genre[]->{
   name,
@@ -175,7 +173,7 @@ export const showsTypes = {
 
 export const coachLessonsTypes = [
   'course',
-  'course-part',
+  'course-lesson',
   'coach-stream',
   'student-focus',
   'quick-tips',
@@ -183,7 +181,7 @@ export const coachLessonsTypes = [
   'semester-pack',
   'question-and-answer',
   'song-tutorial',
-  'song-tutorial-children',
+  'song-tutorial-lesson',
   'workout',
 ]
 
@@ -192,15 +190,8 @@ export const childContentTypeConfig = {
 }
 
 export const singleLessonTypes = ['quick-tips', 'rudiment']
-export const practiceAlongsLessonTypes = ['workout'] // challenges ->workouts
+export const practiceAlongsLessonTypes = ['workout']
 export const performancesLessonTypes = ['performance']
-export const documentariesLessonTypes = [
-  'tama',
-  'sonor',
-  'history-of-electronic-drums',
-  'paiste-cymbals',
-  'backstage-secret',
-]
 export const liveArchivesLessonTypes = [
   'podcast',
   'coach-stream',
@@ -235,16 +226,18 @@ export const skillLessonTypes = ['skill-pack']
 
 export const showsLessonTypes = [
   'boot-camp',
-  'diy-drum-experiment',
-  'exploring-beats',
-  'in-rhythm',
-  'rhythmic-adventures-of-captain-carson',
-  'rhythms-from-another-planet',
   'study-the-greats',
+  // TODO these require clearing the information with Chris, they weren't part of the old system, but are part of the migration document as SHOWS
+  // 'gear-guide',
+  // 'odd-times',
+  // 'podcast',
+  // 'spotlight',
+  // 'study-the-greats',
+  // 'performance',
 ]
 export const entertainmentLessonTypes = [
-  'specials', // TODO: new type
-  ...documentariesLessonTypes,
+  'special',
+  'documentary-lesson',
   ...showsLessonTypes,
 ]
 export const collectionLessonTypes = [...coursesLessonTypes, ...showsLessonTypes]
@@ -255,7 +248,7 @@ export const lessonTypesMapping = {
   'live archives': liveArchivesLessonTypes,
   performances: performancesLessonTypes,
   'student archives': studentArchivesLessonTypes,
-  documentaries: documentariesLessonTypes,
+  documentaries: ['documentary-lesson'],
   courses: ['course'],
   'guided courses': ['guided-course'],
   'tiered courses': ['tiered-course'],
@@ -296,20 +289,20 @@ export const progressTypesMapping = {
     ...liveArchivesLessonTypes,
     ...performancesLessonTypes,
     ...studentArchivesLessonTypes,
-    ...documentariesLessonTypes,
+    'documentary-lesson',
     'live',
     'pack-bundle-lesson',
   ],
   course: ['course'],
   show: showsLessonTypes,
-  'song tutorial': [...tutorialsLessonTypes, 'song-tutorial-children'],
+  'song tutorial': [...tutorialsLessonTypes, 'song-tutorial-lesson'],
   songs: transcriptionsLessonTypes,
   'play along': playAlongLessonTypes,
   'guided course': ['guided-course'],
   pack: ['pack', 'semester-pack'],
   'learning path': ['learning-path-v2'],
   'jam track': jamTrackLessonTypes,
-  'course video': ['course-part'],
+  'course video': ['course-lesson'],
 }
 
 export const songs = {
@@ -341,9 +334,9 @@ export const filterTypes = {
 export const recentTypes = {
   lessons: [
     ...individualLessonsTypes,
-    'course-part',
+    'course-lesson',
     'pack-bundle-lesson',
-    'guided-course-part',
+    'guided-course-lesson',
     'quick-tips',
   ],
   songs: [...SONG_TYPES],
@@ -420,7 +413,7 @@ export let contentTypeConfig = {
       },
     },
   },
-  'song-tutorial-children': {
+  'song-tutorial-lesson': {
     fields: [`"resources": ${resourcesField}`],
   },
   'guided-course': {
@@ -633,7 +626,7 @@ export let contentTypeConfig = {
   live: { ...contentWithInstructorsField, slug: 'live-streams' },
   solo: { ...contentWithInstructorsField, slug: 'solos' },
   'boot-camp': contentWithInstructorsField,
-  'gear-guids': contentWithInstructorsField,
+  'gear-guide': contentWithInstructorsField,
   performance: contentWithInstructorsField,
   challenges: contentWithInstructorsField,
   'on-the-road': contentWithInstructorsField,

package/src/filterBuilder.js

@@ -67,6 +67,7 @@ export class FilterBuilder {
 
     if (this.debug) console.log('baseFilter', this.filter)
     const filter = this._applyContentStatuses()
+      ._removeDeprecatedContent()
       ._applyPermissions()
       ._applyPublishingDateRestrictions()
       ._trimAmpersands().filter // just in case
@@ -83,6 +84,13 @@ export class FilterBuilder {
     return new Date(now.getFullYear(), now.getMonth(), now.getDate(), now.getHours(), 1)
   }
 
+  _removeDeprecatedContent() {
+    this._andWhere(
+      `!defined(${this.prefix}deprecated_railcontent_id)`
+    )
+    return this
+  }
+
   _applyContentStatuses() {
     // This must be run before _applyPublishDateRestrictions()
     if (this.bypassStatuses) return this

package/src/services/userActivity.js

@@ -922,7 +922,7 @@ async function extractPinnedItemsAndSortAllItems(
 }
 
 function generateContentsMap(contents, playlistsContents) {
-  const excludedTypes = new Set(['pack-bundle', 'guided-course-part'])
+  const excludedTypes = new Set(['pack-bundle', 'guided-course-lesson'])
   const existingShows = new Set()
   const contentsMap = new Map()
   const childToParentMap = {}
@@ -1097,7 +1097,8 @@ async function processContentItem(content) {
     const progressTimestamp = content.progressTimestamp
     const wasPinned = content.pinned ?? false
     if (content.progressStatus === 'completed') {
-      // this could be handled more gracefully if their was a parent content type for shows
+      // this could be handled more gracefully if there was a parent content type for shows
+      // Update Dec 3rd. We updated almost everything to the DocumentaryType :D, but there's still a few
       const nextByProgress = findIncompleteLesson(progressOnItems, content.id, content.type)
       content = shows.find((lesson) => lesson.id === nextByProgress)
       content.completed_children = completedShows

package/.claude/settings.local.json

@@ -1,14 +0,0 @@
-{
-  "permissions": {
-    "allow": [
-      "Read(//app/musora-platform-backend/**)",
-      "Read(//app/musora-platform-frontend/**)",
-      "Bash(find:*)",
-      "Bash(sed:*)",
-      "Read(//app/**)",
-      "Bash(cat:*)"
-    ],
-    "deny": [],
-    "ask": []
-  }
-}

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