musora-content-services 2.158.3 → 2.159.0

package/.claude/settings.local.json

@@ -0,0 +1,12 @@
+{
+  "permissions": {
+    "allow": [
+      "Bash(rg:*)",
+      "Bash(npm run lint:*)",
+      "Bash(ls:*)",
+      "Bash(npm test *)",
+      "Bash(npx jest *)"
+    ],
+    "deny": []
+  }
+}

package/.github/workflows/automated-testing.yml

@@ -12,7 +12,7 @@ jobs:
       - uses: actions/checkout@v4
       - uses: actions/setup-node@v4
         with:
-          node-version: 20
+          node-version: 22
           cache: npm
       - name: Install dependencies
         run: npm ci
@@ -25,3 +25,23 @@ jobs:
           slug: railroadmedia/musora-content-services
           flags: unit
           fail_ci_if_error: true
+  integration-tests:
+    runs-on: ubuntu-latest
+    timeout-minutes: 5
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
+        with:
+          node-version: 22
+          cache: npm
+      - name: Install dependencies
+        run: npm ci
+      - name: Run integration tests
+        run: npm run test:integration
+      - name: Upload coverage to Codecov
+        uses: codecov/codecov-action@v5
+        with:
+          token: ${{ secrets.CODECOV_TOKEN }}
+          slug: railroadmedia/musora-content-services
+          flags: integration
+          fail_ci_if_error: true

package/CHANGELOG.md

@@ -2,6 +2,14 @@
 
 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.159.0](https://github.com/railroadmedia/musora-content-services/compare/v2.158.3...v2.159.0) (2026-05-12)
+
+
+### Features
+
+* **BEHSTP-304:** LP lesson downloads & refactors ([#955](https://github.com/railroadmedia/musora-content-services/issues/955)) ([536e90f](https://github.com/railroadmedia/musora-content-services/commit/536e90f2b7e4d405e22b3de497191aa5c61b5776))
+* **TP-1267:** audit live tests ([#962](https://github.com/railroadmedia/musora-content-services/issues/962)) ([aff540e](https://github.com/railroadmedia/musora-content-services/commit/aff540ec2e2f6f083e03406bc592eb54b4be8148))
+
 ### [2.158.3](https://github.com/railroadmedia/musora-content-services/compare/v2.158.2...v2.158.3) (2026-05-07)
 
 

package/README.md

@@ -120,8 +120,27 @@ https://railroadmedia.github.io/musora-content-services/
 ## Run tests
 Ensure that the setup process has been completed, including copying .env file from 1Password "musora-content-services .env" 
 and having jest installed (`npm install --save-dev jest`). To run the full test suite, simply run the following:
+
+### Unit tests only (fast, no external dependencies)
 ```
 npm test
 ```
-You can also filter down to specific tests with the `-- -t="..."` option. e.g. you can run the userContext test suite 
-with `npm test -- -t="userContext"` or just the contentLiked test with `npm test -- -t="contentLiked"`
+### Integration tests only (mocked external boundaries, runs in CI)
+```
+npm run test:integration
+```
+### Unit + integration (both suites)
+```
+npm run test:all
+```
+### Live tests (requires real credentials in .env, never runs in CI)
+```
+npm run test:live
+```
+### Filter to a specific test
+```
+npm test -- -t="contentLiked"
+```
+
+### Without coverage
+add flag `--coverage=false`

package/jest.config.js

@@ -117,10 +117,7 @@ export default {
   },
 
   // An array of regexp pattern strings, matched against all module paths before considered 'visible' to the module loader
-  modulePathIgnorePatterns: [
-    '<rootDir>/test/live',
-    '<rootDir>/test/integration'
-  ],
+  testMatch: ['<rootDir>/test/unit/**/*.test.[jt]s?(x)'],
 
   // Activates notifications for test results
   // notify: false,

package/jest.integration.config.js

@@ -0,0 +1,6 @@
+import baseConfig from './jest.config.js'
+export default {
+  ...baseConfig,
+  testMatch: ['<rootDir>/test/integration/**/*.test.[jt]s?(x)'],
+  coverageThreshold: {},
+}

package/jest.live.config.js

@@ -1,10 +1,6 @@
-/** @type {import('jest').Config} */
 import baseConfig from './jest.config.js'
-
 export default {
   ...baseConfig,
-  modulePathIgnorePatterns: [],
-  setupFilesAfterEnv: ['dotenv/config'],
-  testTimeout: 1000000,
+  testMatch: ['<rootDir>/test/live/**/*.test.[jt]s?(x)'],
   collectCoverage: false,
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "musora-content-services",
-  "version": "2.158.3",
+  "version": "2.159.0",
   "description": "A package for Musoras content services ",
   "main": "src/index.js",
   "type": "module",
@@ -8,7 +8,10 @@
     "build-index": "node tools/generate-index.cjs",
     "release": "standard-version",
     "doc": "jsdoc -c jsdoc.json --verbose",
-    "test": "jest"
+    "test": "jest",
+    "test:integration": "jest --config=jest.integration.config.js",
+    "test:all": "jest --config=jest.config.js; jest --config=jest.integration.config.js",
+    "test:live": "jest --config=jest.live.config.js"
   },
   "repository": {
     "type": "git",

package/src/contentTypeConfig.js

@@ -437,7 +437,7 @@ export let contentTypeConfig = {
       'railcontent_id',
       '"assignments": assignment[]{railcontent_id}',
       `"metadata": { brand, "type": _type, "parent_id":  coalesce(${parentReferenceField}->railcontent_id, 0) }`,
-      ],
+    ],
     childFields: [
       'railcontent_id',
       '"assignments": assignment[]{railcontent_id}',
@@ -498,6 +498,7 @@ export let contentTypeConfig = {
   'course-lesson': {
     fields: [`"resources": ${resourcesField}`],
   },
+
   download: {
     fields: [
       `"resources": ${resourcesField}`,
@@ -512,6 +513,7 @@ export let contentTypeConfig = {
       'video',
       ...playAlongMp3sFields,
       pcdForDownloadField,
+      '"learning_path_parent_id": *[_type == "learning-path-v2" && references(^._id)][0].railcontent_id',
       `...select(
         defined(live_event_start_time) => {
           ${getLiveFields(true).join(',')}
@@ -531,6 +533,7 @@ export let contentTypeConfig = {
       'video',
       ...playAlongMp3sFields,
       pcdForDownloadField,
+      '"learning_path_parent_id": *[_type == "learning-path-v2" && references(^._id)][0].railcontent_id',
       `...select(
         defined(live_event_start_time) => {
           ${getLiveFields(true).join(',')}
@@ -693,7 +696,7 @@ export let contentTypeConfig = {
   'new-and-scheduled': {
     fields: [
       'show_in_new_feed',
-      isLiveField()
+      isLiveField(),
     ],
   },
 }
@@ -812,7 +815,7 @@ export function artistOrInstructorNameAsArray(key = 'artists') {
 
 export async function getFieldsForContentTypeWithFilteredChildren(
   contentType,
-  asQueryString = true
+  asQueryString = true,
 ) {
   const childFields = getChildFieldsForContentType(contentType, true)
   const parentFields = getFieldsForContentType(contentType, false)
@@ -827,7 +830,7 @@ export async function getFieldsForContentTypeWithFilteredChildren(
         "children": child[${childFilter}]->{
           ${childFields}
         },
-      }`
+      }`,
     )
   }
   return asQueryString ? parentFields.toString() + ',' : parentFields
@@ -917,7 +920,7 @@ const filterHandlers = {
   length: (value) => {
     // Find the matching length option by name
     const lengthOption = Object.values(LengthFilterOptions).find(
-      (opt) => typeof opt === 'object' && opt.name === value
+      (opt) => typeof opt === 'object' && opt.name === value,
     )
 
     if (!lengthOption) return ''

package/src/index.d.ts

@@ -57,9 +57,6 @@ import {
 	getEnrichedLearningPaths,
 	getLearningPathLessonsByIds,
 	mapContentToParent,
-	mapContentsThatWereLastProgressedFromMethod,
-	mapLearningPathParentsTo,
-	onContentCompletedLearningPathActions,
 	resetAllLearningPaths,
 	startLearningPath,
 	updateDailySession
@@ -446,6 +443,7 @@ import {
 
 import {
 	getOnboardingRecommendedContent,
+	initializeOnboardingFlow,
 	startOnboarding,
 	updateOnboarding,
 	userOnboardingForBrand
@@ -727,6 +725,7 @@ declare module 'musora-content-services' {
 		guidedCourses,
 		hasAnyMethodV2IntroCompleted,
 		initializeEnvVar,
+		initializeOnboardingFlow,
 		initializeService,
 		isBucketUrl,
 		isContentLiked,
@@ -745,8 +744,6 @@ declare module 'musora-content-services' {
 		login,
 		logout,
 		mapContentToParent,
-		mapContentsThatWereLastProgressedFromMethod,
-		mapLearningPathParentsTo,
 		markAllNotificationsAsRead,
 		markContentAsInterested,
 		markContentAsNotInterested,
@@ -754,7 +751,6 @@ declare module 'musora-content-services' {
 		markNotificationAsUnread,
 		markThreadAsRead,
 		numberOfActiveUsers,
-		onContentCompletedLearningPathActions,
 		onProgressSaved,
 		openComment,
 		otherStats,

package/src/index.js

@@ -61,9 +61,6 @@ import {
 	getEnrichedLearningPaths,
 	getLearningPathLessonsByIds,
 	mapContentToParent,
-	mapContentsThatWereLastProgressedFromMethod,
-	mapLearningPathParentsTo,
-	onContentCompletedLearningPathActions,
 	resetAllLearningPaths,
 	startLearningPath,
 	updateDailySession
@@ -450,6 +447,7 @@ import {
 
 import {
 	getOnboardingRecommendedContent,
+	initializeOnboardingFlow,
 	startOnboarding,
 	updateOnboarding,
 	userOnboardingForBrand
@@ -726,6 +724,7 @@ export {
 	guidedCourses,
 	hasAnyMethodV2IntroCompleted,
 	initializeEnvVar,
+	initializeOnboardingFlow,
 	initializeService,
 	isBucketUrl,
 	isContentLiked,
@@ -744,8 +743,6 @@ export {
 	login,
 	logout,
 	mapContentToParent,
-	mapContentsThatWereLastProgressedFromMethod,
-	mapLearningPathParentsTo,
 	markAllNotificationsAsRead,
 	markContentAsInterested,
 	markContentAsNotInterested,
@@ -753,7 +750,6 @@ export {
 	markNotificationAsUnread,
 	markThreadAsRead,
 	numberOfActiveUsers,
-	onContentCompletedLearningPathActions,
 	onProgressSaved,
 	openComment,
 	otherStats,

package/src/services/content-org/learning-paths.ts

@@ -26,10 +26,16 @@ import { ContentProgress } from '../sync/models'
 import dayjs from 'dayjs'
 import { LEARNING_PATH_LESSON } from '../../contentTypeConfig'
 
+const excludeFromGeneratedIndex = [
+  'onLearningPathCompletedActions',
+  'mapContentsThatWereLastProgressedFromMethod',
+  'mapLearningPathParentsTo',
+]
+
 const BASE_PATH: string = `/api/content-org`
 const LEARNING_PATHS_PATH = `${BASE_PATH}/v1/user/learning-paths`
-let dailySessionPromise: Promise<DailySessionResponse|""> | null = null
-let activePathPromise: Promise<ActiveLearningPathResponse|""> | null = null
+let dailySessionPromise: Promise<DailySessionResponse | ''> | null = null
+let activePathPromise: Promise<ActiveLearningPathResponse | ''> | null = null
 
 interface ActiveLearningPathResponse {
   user_id: number
@@ -66,7 +72,7 @@ interface CollectionObject {
 export async function getDailySession(
   brand: string,
   userDate: Date,
-  forceRefresh: boolean = false
+  forceRefresh: boolean = false,
 ) {
   if (dailySessionPromise && !forceRefresh) {
     return dailySessionPromise
@@ -105,7 +111,7 @@ export async function getDailySession(
 export async function updateDailySession(
   brand: string,
   userDate: Date,
-  keepFirstLearningPath: boolean = false
+  keepFirstLearningPath: boolean = false,
 ) {
   const dateWithTimezone = formatLocalDateTime(userDate)
   const url: string = `${LEARNING_PATHS_PATH}/daily-session/create`
@@ -115,7 +121,7 @@ export async function updateDailySession(
     keepFirstLearningPath: keepFirstLearningPath,
   }
   try {
-    const response = (await POST(url, body)) as DailySessionResponse|''
+    const response = (await POST(url, body)) as DailySessionResponse | ''
 
     if (response || response === '') { // refresh cached value
       const urlGet: string = `${LEARNING_PATHS_PATH}/daily-session/get?brand=${brand}&userDate=${encodeURIComponent(dateWithTimezone)}`
@@ -173,8 +179,8 @@ export async function startLearningPath(brand: string, learningPathId: number) {
 
 async function dataPromiseGET(
   url: string,
-  forceRefresh: boolean
-): Promise<DailySessionResponse | ActiveLearningPathResponse | ""> {
+  forceRefresh: boolean,
+): Promise<DailySessionResponse | ActiveLearningPathResponse | ''> {
   if (url.includes('daily-session')) {
     if (!dailySessionPromise || forceRefresh) {
       dailySessionPromise = GET(url, {
@@ -203,8 +209,8 @@ export async function resetAllLearningPaths() {
       await Promise.all([
         db.contentProgress.eraseProgressMany(all.intros, null),
         ...all.learning_paths.map((id) =>
-          db.contentProgress.eraseProgress(id, {id, type: COLLECTION_TYPE.LEARNING_PATH})
-        )
+          db.contentProgress.eraseProgress(id, { id, type: COLLECTION_TYPE.LEARNING_PATH }),
+        ),
       ])
     }),
     POST(url, {}),
@@ -230,15 +236,15 @@ export async function getEnrichedLearningPath(learningPathId) {
       addProgressTimestamp: true,
       addResumeTimeSeconds: true,
       addNavigateTo: true,
-    }
+    },
   )) as any
   // add awards to LP parents only
   response = await addContextToLearningPaths(() => response, { addAwards: true })
   if (!response) return response
 
   response.children = mapContentToParent(
-      response.children,
-      {lessonType: LEARNING_PATH_LESSON, parentContentId: learningPathId}
+    response.children,
+    { lessonType: LEARNING_PATH_LESSON, parentContentId: learningPathId },
   )
   return response
 }
@@ -262,7 +268,7 @@ export async function getEnrichedLearningPaths(learningPathIds: number[]) {
       addProgressTimestamp: true,
       addResumeTimeSeconds: true,
       addNavigateTo: true,
-    }
+    },
   )) as any
   // add awards to LP parents only
   response = await addContextToLearningPaths(() => response, { addAwards: true })
@@ -271,8 +277,8 @@ export async function getEnrichedLearningPaths(learningPathIds: number[]) {
 
   response.forEach((learningPath) => {
     learningPath.children = mapContentToParent(
-        learningPath.children,
-        {lessonType: LEARNING_PATH_LESSON, parentContentId: learningPath.id}
+      learningPath.children,
+      { lessonType: LEARNING_PATH_LESSON, parentContentId: learningPath.id },
     )
   })
   return response
@@ -299,8 +305,8 @@ export async function getLearningPathLessonsByIds(contentIds, learningPathId) {
  * @param options.parentContentId
  */
 export function mapContentToParent(
-   lessons: any,
-  options?: { lessonType?: string; parentContentId?: number }
+  lessons: any,
+  options?: { lessonType?: string; parentContentId?: number },
 ) {
   if (!lessons || (Array.isArray(lessons) && lessons.length === 0)) return lessons
 
@@ -363,7 +369,7 @@ interface fetchLearningPathLessonsResponse {
 export async function fetchLearningPathLessons(
   learningPathId: number,
   brand: string,
-  userDate: Date
+  userDate: Date,
 ) {
   const learningPath = await getEnrichedLearningPath(learningPathId)
   if (!learningPath || learningPath.children?.length === 0) return null
@@ -420,7 +426,7 @@ export async function fetchLearningPathLessons(
   if (previousContentIds.length !== 0) {
     previousLPDailies = await getLearningPathLessonsByIds(
       previousContentIds,
-      previousLearningPathId
+      previousLearningPathId,
     )
   }
   if (nextContentIds.length !== 0) {
@@ -429,7 +435,7 @@ export async function fetchLearningPathLessons(
         lessons.map((lesson) => ({
           ...lesson,
           in_next_learning_path: learningPath.progressStatus === STATE.COMPLETED,
-        }))
+        })),
     )
   }
 
@@ -456,7 +462,7 @@ export async function fetchLearningPathLessons(
  * @returns {Promise<number[]>} Array with completed content IDs
  */
 export async function fetchLearningPathProgressCheckLessons(
-  contentIds: number[]
+  contentIds: number[],
 ): Promise<number[]> {
   let query = await getAllCompletedByIds(contentIds)
   let completedProgress = query.data.map((progress) => progress.content_id)
@@ -467,6 +473,7 @@ interface completeMethodIntroVideo {
   intro_video_response: SyncWriteDTO<ContentProgress, any> | null
   active_path_response: ActiveLearningPathResponse
 }
+
 /**
  * Handles completion of method intro video and other related actions.
  * @param introVideoId - The method intro video content ID. If not provided, does not `complete` intro video.
@@ -476,14 +483,14 @@ interface completeMethodIntroVideo {
  * @returns {Promise<Object>} response.active_path_response - The set active learning path response.
  */
 export async function completeMethodIntroVideo(
-  introVideoId: number|null,
-  brand: string
+  introVideoId: number | null,
+  brand: string,
 ): Promise<completeMethodIntroVideo> {
   let response = {} as completeMethodIntroVideo
 
   const [intro_video_response, methodStructure] = await Promise.all([
     introVideoId ? completeIfNotCompleted(introVideoId) : Promise.resolve(null),
-    fetchMethodV2Structure(brand)
+    fetchMethodV2Structure(brand),
   ])
   response.intro_video_response = intro_video_response
 
@@ -492,7 +499,7 @@ export async function completeMethodIntroVideo(
   response.active_path_response = await methodIntroVideoCompleteActions(
     brand,
     firstLearningPathId,
-    new Date()
+    new Date(),
   )
 
   return response
@@ -511,6 +518,7 @@ interface completeLearningPathIntroVideo {
   lesson_import_response: SyncWriteDTO<ContentProgress, any> | null
   update_dailies_response: DailySessionResponse | null
 }
+
 /**
  * Handles completion of learning path intro video and other related actions.
  * @param introVideoId - The learning path intro video content ID.
@@ -527,14 +535,14 @@ export async function completeLearningPathIntroVideo(
   introVideoId: number,
   learningPathId: number,
   lessonsToImport: number[] | null,
-  brand: string
+  brand: string,
 ) {
   let response = {} as completeLearningPathIntroVideo
   const collection: CollectionObject = { id: learningPathId, type: COLLECTION_TYPE.LEARNING_PATH }
 
   const [anyIntroComplete, activePath] = await Promise.all([
     hasAnyMethodV2IntroCompleted(),
-    getActivePath(brand)
+    getActivePath(brand),
   ])
 
   let lateMethodSetup = false
@@ -561,7 +569,7 @@ export async function completeLearningPathIntroVideo(
 }
 
 async function completeIfNotCompleted(
-  contentId: number
+  contentId: number,
 ): Promise<SyncWriteDTO<ContentProgress, any> | null> {
   const introVideoStatus = await getProgressState(contentId)
 
@@ -570,20 +578,14 @@ async function completeIfNotCompleted(
 
 async function resetIfPossible(
   contentId: number,
-  collection: CollectionParameter = null
+  collection: CollectionParameter = null,
 ): Promise<SyncWriteDTO<ContentProgress, any> | null> {
   const status = await getProgressState(contentId, collection)
 
   return status !== '' ? await contentStatusReset(contentId, collection) : null
 }
 
-export async function onContentCompletedLearningPathActions(
-  contentId: number,
-  collection: CollectionObject | null
-) {
-  if (collection?.type !== COLLECTION_TYPE.LEARNING_PATH) return
-  if (contentId !== collection?.id) return
-
+export async function onLearningPathCompletedActions(contentId: number) {
   const learningPathId = contentId
   const learningPath = await getEnrichedLearningPath(learningPathId)
 
@@ -628,7 +630,7 @@ export async function mapContentsThatWereLastProgressedFromMethod(objects: any[]
 
   let filtered = objects.filter((obj) => trueIds.includes(obj.id))
 
-  filtered = await mapLearningPathParentsTo(filtered, {type: true, parent_id: true})
+  filtered = await mapLearningPathParentsTo(filtered, { type: true, parent_id: true })
 
   // Map each filtered item back into the total contents object
   objects = objects.map((item) => {
@@ -639,7 +641,10 @@ export async function mapContentsThatWereLastProgressedFromMethod(objects: any[]
 
 }
 
-export async function mapLearningPathParentsTo(objects: any[], fieldsToMap?: {type?: boolean, parent_id?: boolean}): Promise<object[]> {
+export async function mapLearningPathParentsTo(objects: any[], fieldsToMap?: {
+  type?: boolean,
+  parent_id?: boolean
+}): Promise<object[]> {
   const ids = objects.map((obj: any) => obj.id) as number[]
   const hierarchy = await fetchParentChildRelationshipsFor(ids, COLLECTION_TYPE.LEARNING_PATH)
 
@@ -654,7 +659,7 @@ export async function mapLearningPathParentsTo(objects: any[], fieldsToMap?: {ty
     const parent_id = parentMap.get(obj.id) ?? undefined
     return mapContentToParent(obj, {
       lessonType: fieldsToMap.type ? LEARNING_PATH_LESSON : undefined,
-      parentContentId: fieldsToMap.parent_id ? parent_id : undefined
+      parentContentId: fieldsToMap.parent_id ? parent_id : undefined,
     })
   })
 }