musora-content-services 2.167.2 → 2.167.3

package/.agent/decisions/2026-07-20-mu2-1724-songs-lessons-counts.md

@@ -0,0 +1,33 @@
+---
+date: 2026-07-20
+branch: chore/MU2-1724-songs-lessons-counts
+pr: https://github.com/railroadmedia/musora-content-services/pull/1012
+status: open
+tags: [[chore]]
+---
+
+# Song and lesson counts query
+
+## Context
+Needed a way to get the total number of songs vs. individual playable lessons in Sanity, globally or scoped to a brand. Content types don't map 1:1 onto "song" vs "lesson" — songs include `song`, `play-along`, `jam-track`, `song-tutorial-lesson` per `SONG_TYPES` in `contentTypeConfig.js`, and a "lesson" is any published, top-level (no `parent_type` / empty `parent_content_reference`) document with a `railcontent_id`, regardless of `_type`. The final filter definitions came out of a discussion with teammates, converging on `status == 'published' && defined(railcontent_id)` plus the parent-less check for lessons.
+
+## Decision
+Added `fetchSongAndLessonCounts(brand?)` in `src/services/content/counts.ts`, returning `{ songs, lessons, total }`.
+- Filter logic built with the `Filters` helper (`src/lib/sanity/filter.ts`) and `query()` builder (`src/lib/sanity/query.ts`) instead of hand-interpolated GROQ strings, for consistency with `artist.ts`/`genre.ts`/`instructor.ts`.
+- Query executed via `SanityClient.executeQuery` (`src/infrastructure/sanity/SanityClient.ts`) rather than the legacy `fetchSanity` in `sanity.js` — `executeQuery` returns the raw object result directly, which fits an object-shaped `{songs, lessons}` GROQ query better than `fetchSanity`'s array-oriented `isList` handling.
+- `total` is computed in application code (`songs + lessons`) after the query resolves, not inside GROQ, keeping the query itself to two independent counts.
+- Kept `content.ts` as a types-only file (its original purpose) — the new function and its `SongAndLessonCounts` interface live in the new `counts.ts` file instead, matching the one-file-per-concern convention in `src/services/content/`.
+
+## Alternatives Considered
+- Hand-built GROQ query with `${brand}` string interpolation (the version worked out iteratively with the Sanity MCP) — rejected in favor of the `Filters`/`query` builder for composability and to match existing code style.
+- A fluent `.then().then()` chain for the `total` calculation — tried during review, rejected as unnecessary indirection versus a plain `await` + object literal; reverted to match house style (`artist.ts`, `sanity.js` use plain `await` throughout).
+- Putting the function directly in `content.ts` — rejected once it became clear that file is types-only elsewhere in the codebase; moved to a dedicated `counts.ts`.
+
+## Process Notes
+- `course` vs `course-lesson`: `course` is the parent course entity, `course-lesson` is the individual playable video inside it — easy to conflate when first scoping "lesson" types.
+- Lesson types span several groupings in `contentTypeConfig.js` (`individualLessonsTypes`, `entertainmentLessonTypes`, `skill-pack-lesson`, `guided-course-lesson`, `pack-bundle-lesson`, live/student archive types) — rather than enumerate all `_type` values, the final query avoids the type-list entirely and instead filters on the structural "no parent" condition, which is more robust to new content types being added later.
+- `pack-bundle-lesson` appears in `SINGLE_PARENT_TYPES` but nowhere else in `contentTypeConfig.js` — possibly legacy/unused; not specifically verified against the live Sanity schema.
+
+## Consequences
+- New public function `fetchSongAndLessonCounts` exported from `src/services/content/counts.ts`; needs `npm run build-index` to surface in the generated package index/types.
+- Introduces the first consumer of `SanityClient.executeQuery` outside `src/infrastructure/sanity/` — establishes a pattern for future count/aggregate-style queries to use the client class over `fetchSanity`.

package/.agent/decisions/2026-07-20-query-builder-tostring.md

@@ -0,0 +1,37 @@
+---
+date: 2026-07-20
+branch: chore/query-builder-tostring
+pr: https://github.com/railroadmedia/musora-content-services/pull/1013
+status: open
+tags: [[chore]]
+related: [[2026-07-20-mu2-1724-songs-lessons-counts]]
+---
+
+# QueryBuilder toString()
+
+## Context
+`fetchSongAndLessonCounts` (added in `chore/MU2-1724-songs-lessons-counts`) interpolates `query().and(filter)` builders directly into a GROQ template literal, e.g. `` count(${query().and(songsFilter).build()}) ``. The trailing `.build()` at each interpolation site is boilerplate — the same pattern repeats in `artist.ts`, `genre.ts`, and `instructor.ts` wherever a builder's output lands inside a template literal.
+
+## Decision
+Added `toString(): string` to `QueryBuilder` (`src/lib/sanity/query.ts`), implemented as `return builder.build()`. JS automatically calls `toString()` during string coercion — template literals, `+` concat, `String(x)` — so a builder can now be interpolated directly: `` count(${query().and(filter)}) ``. This mirrors PHP's `Stringable` interface / Java's `toString()` override.
+
+Swept the existing call sites to drop the now-redundant `.build()`:
+- `counts.ts`: both `count(${...})` interpolations.
+- `artist.ts`, `genre.ts`, `instructor.ts`: the `data`/`total` variables that are built once and interpolated later into a `q` template literal.
+
+Left `.build()` in place at the one call shape that isn't string interpolation: `fetchArtistBySlug`/`fetchGenreBySlug`/`fetchInstructorBySlug` build `q` and pass it directly as `fetchSanity(q, ...)`'s first argument. Runtime-wise this would also work fine without `.build()` (`encodeURIComponent` inside `fetchSanity` coerces via `ToString` too), but keeping the explicit call there reads clearer for a variable that's a function argument rather than a string-interpolation target.
+
+`.build()` itself was not removed or deprecated — this is purely additive, both call styles remain valid.
+
+## Alternatives Considered
+- Removing `.build()` everywhere unconditionally, including the `fetchArtistBySlug`-style direct-argument calls — rejected to keep those call sites' typing/intent explicit (a `q` meant to be "the query string" reads oddly as a `QueryBuilder` object until it's coerced at the call boundary).
+- `Symbol.toPrimitive` instead of plain `toString()` — unnecessary; nothing here needs numeric coercion, and plain `toString()` fully covers the template-literal/`String()`/concat cases that matter.
+
+## Process Notes
+- Caveat surfaced during review: `JSON.stringify(builder)` does NOT trigger `toString()` — it serializes the object's own enumerable properties instead. Anywhere a builder might end up inside `JSON.stringify`, this would silently produce the wrong output. Not currently a problem (no such call site exists), but worth remembering if a future call site does this.
+- Equality checks (`===`) never coerce, so comparing two builders directly still won't compare their query strings.
+- This PR is stacked on `chore/MU2-1724-songs-lessons-counts` (PR #1012, unmerged) rather than targeting `main` directly, since it builds on that branch's `counts.ts`. PR base should be retargeted to `main` once #1012 merges.
+
+## Consequences
+- `QueryBuilder` consumers can now interpolate a builder directly into a template literal without remembering to call `.build()` — reduces boilerplate at future call sites.
+- No behavior change: verified via the full test suite (70 suites) after the sweep, plus an intentional break-and-revert of both `toString()` and the `counts.ts` total-summation logic to confirm the new tests actually catch regressions.

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.167.3](https://github.com/railroadmedia/musora-content-services/compare/v2.167.2...v2.167.3) (2026-07-22)
+
+
+### Bug Fixes
+
+* add gen-version script ([#1015](https://github.com/railroadmedia/musora-content-services/issues/1015)) ([c047e38](https://github.com/railroadmedia/musora-content-services/commit/c047e38f5a20d34dc9a69e2eb3cf3f31861aa8f1))
+* **BR-819:** add learning path id for learning path intro in recent a… ([#1008](https://github.com/railroadmedia/musora-content-services/issues/1008)) ([267423f](https://github.com/railroadmedia/musora-content-services/commit/267423f525ee12824acfe45e771fd2271d566ed6))
+
 ### [2.167.2](https://github.com/railroadmedia/musora-content-services/compare/v2.167.1...v2.167.2) (2026-07-17)
 
 

package/package.json

@@ -1,10 +1,11 @@
 {
   "name": "musora-content-services",
-  "version": "2.167.2",
+  "version": "2.167.3",
   "description": "A package for Musoras content services ",
   "main": "src/index.js",
   "type": "module",
   "scripts": {
+    "prepare": "node scripts/gen-version.cjs",
     "build-index": "node tools/generate-index.cjs",
     "release": "standard-version",
     "doc": "jsdoc -c jsdoc.json --verbose",

package/src/contentTypeConfig.js

@@ -433,6 +433,7 @@ export let contentTypeConfig = {
       `"badge" : ${contentAwardField}.badge.asset->url`,
       `"badge_rear" : ${contentAwardField}.badge_rear.asset->url`,
       `"badge_logo" : ${contentAwardField}.logo.asset->url`,
+      '"learning_path_id": *[_type == "learning-path-v2" && references(^._id)][0].railcontent_id',
     ],
     includeChildFields: true,
   },

package/src/lib/sanity/query.ts

@@ -32,6 +32,7 @@ export interface QueryBuilder {
   dereference(): QueryBuilder
   postFilter(expr: string): QueryBuilder
   build(): string
+  toString(): string
 
   _state(): QueryBuilderState
 }
@@ -148,6 +149,10 @@ export const query = (selector?: string): QueryBuilder => {
       `.trim()
     },
 
+    toString() {
+      return builder.build()
+    },
+
     _state() {
       return state
     },

package/src/services/config.js

@@ -72,6 +72,21 @@ const excludeFromGeneratedIndex = []
  *   appEnv: 'local'
  * });
  */
+export async function getMCSVersion() {
+  try {
+    const { MCS_VERSION } = await import('../version-info.js')
+    return MCS_VERSION
+  } catch {
+    return null
+  }
+}
+
+if (process.env.NODE_ENV === 'development') {
+  getMCSVersion().then(v => v && console.log('[MCS] last tagged version:', v))
+} else {
+  getMCSVersion().then(v => v && console.log('[MCS] version:', v))
+}
+
 export function initializeService(config) {
   globalConfig.sanityConfig = config.sanityConfig
   globalConfig.railcontentConfig = config.railcontentConfig

package/src/services/content/artist.ts

@@ -50,7 +50,6 @@ export async function fetchArtists(
       `"lesson_count": ${await f.lessonCount(brand)}`
     )
     .postFilter(postFilter)
-    .build()
 
   const q = `{
     "data": ${data},
@@ -152,9 +151,8 @@ export async function fetchArtistLessons(
     .order(sort)
     .slice(offset, limit)
     .select((await getFieldsForContentTypeWithFilteredChildren(contentType, true)) as string)
-    .build()
 
-  const total = query().and(restrictions).build()
+  const total = query().and(restrictions)
 
   const q = `{
     "data": ${data},

package/src/services/content/counts.ts

@@ -0,0 +1,47 @@
+import { SONG_TYPES } from '../../contentTypeConfig.js'
+import { SanityClient } from '../../infrastructure/sanity/SanityClient'
+import { Brands } from '../../lib/brands'
+import { Filters as f } from '../../lib/sanity/filter'
+import { query } from '../../lib/sanity/query'
+
+const sanityClient = new SanityClient()
+
+export interface SongAndLessonCounts {
+  songs: number
+  lessons: number
+  total: number
+}
+
+/**
+ * @param {Brands|string} [brand] - Filters by brand; omit for a global count.
+ * @returns {Promise<SongAndLessonCounts>}
+ */
+export async function fetchSongAndLessonCounts(
+  brand?: Brands | string
+): Promise<SongAndLessonCounts> {
+  const songsFilter = await f.combineAsync(
+    f.typeIn(SONG_TYPES),
+    f.statusIn(['published']),
+    f.defined('railcontent_id'),
+    brand ? f.brand(brand) : f.empty
+  )
+
+  const lessonsFilter = await f.combineAsync(
+    f.defined('railcontent_id'),
+    f.statusIn(['published']),
+    f.combineOr(f.notDefined('parent_type'), 'count(parent_content_reference) == 0'),
+    brand ? f.brand(brand) : f.empty
+  )
+
+  const q = `{
+    "songs": count(${query().and(songsFilter)}),
+    "lessons": count(${query().and(lessonsFilter)})
+  }`
+
+  const counts = (await sanityClient.executeQuery<Omit<SongAndLessonCounts, 'total'>>(q)) ?? {
+    songs: 0,
+    lessons: 0,
+  }
+
+  return { ...counts, total: counts.songs + counts.lessons }
+}

package/src/services/content/genre.ts

@@ -50,7 +50,6 @@ export async function fetchGenres(
       `"lesson_count": ${await f.lessonCount(brand)}`
     )
     .postFilter(postFilter)
-    .build()
 
   const q = `{
     "data": ${data},
@@ -151,9 +150,8 @@ export async function fetchGenreLessons(
     .order(sort)
     .slice(offset, limit)
     .select((await getFieldsForContentTypeWithFilteredChildren(contentType, true)) as string)
-    .build()
 
-  const total = query().and(restrictions).build()
+  const total = query().and(restrictions)
 
   const q = `{
     "data": ${data},

package/src/services/content/instructor.ts

@@ -51,7 +51,6 @@ export async function fetchInstructors(
       `"lesson_count": ${await f.lessonCount(brand)}`
     )
     .postFilter(postFilter)
-    .build()
 
   const q = `{
     "data": ${data},
@@ -151,9 +150,8 @@ export async function fetchInstructorLessons(
     .order(sort)
     .slice(offset, limit)
     .select((await getFieldsForContentTypeWithFilteredChildren(contentType, true)) as string)
-    .build()
 
-  const total = query().and(restrictions).build()
+  const total = query().and(restrictions)
 
   const q = `{
     "data": ${data},

package/src/services/userActivity.js

@@ -526,6 +526,7 @@ export async function getRecentActivity({ page = 1, limit = 5, tabName = null }
       thumbnail: content.thumbnail || null,
       title: content.title || practice.title,
       parent_id: content.parent_id || null,
+      learningPathId: content.learning_path_id || null,
       navigateTo: content.navigateTo || null,
       sanityType: content.type || null,
       artist_name: content.artist_name || null,
@@ -820,6 +821,7 @@ async function formatPracticeMeta(practices = []) {
       sanity_type: content?.type || null,
       content_slug: content?.slug || null,
       parent_id: content?.parent_id || null,
+      learningPathId: content?.learning_path_id || null,
       navigateTo: content?.navigateTo || null,
       artist_name: content?.artist_name || null,
     }

package/src/version-info.js

@@ -0,0 +1 @@
+export const MCS_VERSION = '2.167.3'

package/.claude/settings.local.json

@@ -1,53 +0,0 @@
-{
-  "permissions": {
-    "allow": [
-      "Bash(npx jest *)",
-      "Bash(npx tsc *)",
-      "Skill(counselors)",
-      "Bash(counselors ls *)",
-      "Bash(counselors groups *)",
-      "Bash(counselors run *)",
-      "Bash(npm test *)",
-      "Bash(gh pr *)",
-      "Bash(gh api *)",
-      "Bash(mkdir -p /tmp/pr-review-v2)",
-      "Read(//tmp/pr-review-v2/**)",
-      "Bash(cat /home/alesevero/railenvironment/applications/musora-content-services/AGENTS.md)",
-      "Bash(echo \"no AGENTS.md\")",
-      "Bash(echo \"exit=$?\")",
-      "Bash(git checkout *)",
-      "Skill(pr)",
-      "Skill(create-decision)",
-      "mcp__github__pull_request_read",
-      "mcp__github__push_files",
-      "Bash(git push *)",
-      "mcp__github__create_pull_request",
-      "mcp__github__search_pull_requests",
-      "Bash(git commit *)",
-      "Bash(git add *)",
-      "mcp__github__list_pull_requests"
-    ]
-  },
-  "model": "sonnet",
-  "enabledMcpjsonServers": [
-    "atlassian",
-    "figma",
-    "google-workspace",
-    "snowflake",
-    "aws",
-    "hex",
-    "sanity",
-    "mysql",
-    "slack",
-    "langfuse",
-    "chrome-devtools",
-    "railway",
-    "github",
-    "asana"
-  ],
-  "disabledMcpjsonServers": [
-    "nightwatch",
-    "1password"
-  ],
-  "effortLevel": "medium"
-}

package/test/initializeTests.js

@@ -1,60 +0,0 @@
-import { globalConfig, initializeService } from '../src'
-import { LocalStorageMock } from './localStorageMock'
-const railContentModule = require('../src/services/railcontent.js')
-const awardDefsModule = require('../src/services/awards/internal/award-definitions.js')
-let token = null
-let userId = process.env.RAILCONTENT_USER_ID ?? null
-
-export async function initializeTestService(useLive = false, isAdmin = false) {
-  let token, userId
-  if (useLive && !token && process.env.RAILCONTENT_BASE_URL) {
-    const baseUrl = `${process.env.RAILCONTENT_BASE_URL}/api/user-management-system`
-    const response = await fetch(`${baseUrl}/v1/sessions`, {
-      method: 'POST',
-      headers: {
-        'X-Client-Platform': 'mobile',
-        'Content-Type': 'application/json',
-        Authorization: null,
-      },
-      body: JSON.stringify({
-        email: process.env.RAILCONTENT_EMAIL,
-        password: process.env.RAILCONTENT_PASSWORD,
-        device_name: 'test',
-        device_token: '',
-        platform: '',
-      }),
-    })
-
-    let data = await response.json() // Parse the JSON body
-    token = data['token']
-    userId = data['user']['id']
-  }
-
-  const config = {
-    sanityConfig: {
-      token: process.env.SANITY_API_TOKEN,
-      projectId: process.env.SANITY_PROJECT_ID,
-      dataset: process.env.SANITY_DATASET,
-      useCachedAPI: process.env.SANITY_USE_CACHED_API === 'true' || true,
-      version: '2021-06-07',
-      debug: process.env.DEBUG === 'true' || false,
-      useDummyRailContentMethods: true,
-    },
-    railcontentConfig: {
-      baseUrl: process.env.RAILCONTENT_BASE_URL || 'https://test.musora.com',
-      token: token,
-      userId: userId,
-      authToken: token,
-    },
-    sessionConfig: { token: token, userId: userId, authToken: token },
-    baseUrl: process.env.RAILCONTENT_BASE_URL,
-    localStorage: new LocalStorageMock(),
-    isMA: true,
-  }
-  jest.spyOn(awardDefsModule.awardDefinitions, 'initialize').mockResolvedValue()
-  initializeService(config)
-  // Mock user permissions
-  let permissionsMock = jest.spyOn(railContentModule, 'fetchUserPermissionsData')
-  let permissionsData = { permissions: [108, 91, 92], isAdmin: isAdmin }
-  permissionsMock.mockImplementation(() => permissionsData)
-}

package/test/integration/awards/award-exclusion-handling.test.ts

@@ -1,275 +0,0 @@
-import { awardManager } from '../../../src/services/awards/internal/award-manager.js'
-import { awardEvents } from '../../../src/services/awards/internal/award-events.js'
-import { mockAwardDefinitions, getAwardByContentId } from '../../mockData/award-definitions.js'
-import { globalConfig } from '../../../src/services/config.js'
-import { LocalStorageMock } from '../../localStorageMock.js'
-import { setupDefaultMocks, setupAwardEventListeners } from '../../unit/awards/helpers/index'
-import { mockCompletionStates, mockAllCompleted, mockNoneCompleted } from '../../unit/awards/helpers/completion-mock'
-
-jest.mock('../../../src/services/sanity.js', () => ({
-  ...jest.requireActual('../../../src/services/sanity'),
-  default: { fetch: jest.fn() },
-  fetchSanity: jest.fn()
-}))
-
-jest.mock('../../../src/services/railcontent.js', () => ({
-  ...jest.requireActual('../../../src/services/railcontent'),
-  fetchUserPermissionsData: jest.fn().mockResolvedValue({ permissions: [108, 91, 92], isAdmin: false })
-}))
-
-jest.mock('../../../src/services/sync/repository-proxy.ts', () => {
-  const mockFns = {
-    contentProgress: {
-      getOneProgressByContentId: jest.fn(),
-      getSomeProgressByContentIds: jest.fn(),
-      queryOne: jest.fn(),
-      queryAll: jest.fn()
-    },
-    practices: {
-      sumPracticeMinutesForContent: jest.fn()
-    },
-    userAwardProgress: {
-      hasCompletedAward: jest.fn(),
-      recordAwardProgress: jest.fn(),
-      getByAwardId: jest.fn(),
-      completeAward: jest.fn()
-    }
-  }
-  return { default: mockFns, ...mockFns }
-})
-
-import sanityClient, { fetchSanity } from '../../../src/services/sanity.js'
-import db_ from '../../../src/services/sync/repository-proxy'
-const db = db_ as any
-import { awardDefinitions } from '../../../src/services/awards/internal/award-definitions.js'
-
-describe('Award Content Exclusion Handling - E2E Scenarios', () => {
-  let listeners
-
-  beforeEach(async () => {
-    jest.clearAllMocks()
-    globalConfig.localStorage = new LocalStorageMock()
-    awardEvents.removeAllListeners()
-
-    sanityClient.fetch = jest.fn().mockResolvedValue(mockAwardDefinitions)
-    setupDefaultMocks(db, fetchSanity, { practiceMinutes: 150 })
-
-    db.userAwardProgress.completeAward = jest.fn().mockResolvedValue({ data: {}, status: 'synced' })
-
-    db.contentProgress.getSomeProgressByContentIds.mockResolvedValue({
-      data: [{ created_at: Math.floor(Date.now() / 1000) - 86400 * 7 }]
-    })
-
-    await awardDefinitions.refresh()
-
-    listeners = setupAwardEventListeners(awardEvents)
-  })
-
-  afterEach(() => {
-    awardEvents.removeAllListeners()
-    awardDefinitions.clear()
-  })
-
-  describe('Scenario: Guided course with excluded intro video (416446 - 1 eligible lesson)', () => {
-    const testAward = getAwardByContentId(416446)
-    const courseId = 416446
-
-    test('completing the single eligible lesson grants award at 100%', async () => {
-      mockAllCompleted(db)
-
-      await awardManager.onContentCompleted(courseId)
-
-      expect(db.userAwardProgress.recordAwardProgress).toHaveBeenCalledWith(
-        expect.any(String),
-        100,
-        expect.objectContaining({
-          completedAt: expect.any(String),
-          immediate: true
-        })
-      )
-      expect(listeners.granted).toHaveBeenCalledTimes(1)
-    })
-
-    test('shows 0% progress when eligible lesson not completed', async () => {
-      mockNoneCompleted(db)
-
-      await awardManager.onContentCompleted(courseId)
-
-      expect(db.userAwardProgress.recordAwardProgress).toHaveBeenCalledWith(
-        testAward._id,
-        0,
-        expect.objectContaining({
-          progressData: expect.any(Object)
-        })
-      )
-    })
-  })
-
-  describe('Scenario: Course with 4 eligible lessons (417049 - intro excluded)', () => {
-    const testAward = getAwardByContentId(417049)
-    const courseId = 417049
-
-    test('completing 1 of 4 lessons shows 25% progress', async () => {
-      mockCompletionStates(db, [417045])
-
-      await awardManager.onContentCompleted(courseId)
-
-      expect(db.userAwardProgress.recordAwardProgress).toHaveBeenCalledWith(
-        testAward._id,
-        25,
-        expect.objectContaining({
-          progressData: expect.any(Object)
-        })
-      )
-    })
-
-    test('completing 2 of 4 lessons shows 50% progress', async () => {
-      mockCompletionStates(db, [417045, 417046])
-
-      await awardManager.onContentCompleted(courseId)
-
-      expect(db.userAwardProgress.recordAwardProgress).toHaveBeenCalledWith(
-        testAward._id,
-        50,
-        expect.objectContaining({
-          progressData: expect.any(Object)
-        })
-      )
-    })
-
-    test('must complete all 4 eligible lessons to earn award', async () => {
-      mockCompletionStates(db, [417045, 417046, 417047, 417048])
-
-      await awardManager.onContentCompleted(courseId)
-
-      expect(db.userAwardProgress.recordAwardProgress).toHaveBeenCalledWith(
-        expect.any(String),
-        100,
-        expect.objectContaining({
-          completedAt: expect.any(String),
-          immediate: true
-        })
-      )
-      expect(listeners.granted).toHaveBeenCalledTimes(1)
-    })
-  })
-
-  describe('Scenario: Large course with 23 eligible lessons (416464 - intro excluded)', () => {
-    const testAward = getAwardByContentId(416464)
-    const courseId = 416464
-    const eligibleLessons = [
-      416467, 416468, 416469, 416470, 416471, 416472, 416473,
-      416474, 416475, 416476, 416477, 416478, 416479, 416480, 416481,
-      416482, 416483, 416484, 416485, 416486, 416487, 416488, 416489
-    ]
-
-    test('shows 0% progress when no eligible lessons completed', async () => {
-      mockNoneCompleted(db)
-
-      await awardManager.onContentCompleted(courseId)
-
-      expect(db.userAwardProgress.recordAwardProgress).toHaveBeenCalledWith(
-        testAward._id,
-        0,
-        expect.objectContaining({
-          progressData: expect.any(Object)
-        })
-      )
-    })
-
-    test('completing 12 of 23 lessons shows ~52% progress', async () => {
-      mockCompletionStates(db, eligibleLessons.slice(0, 12))
-
-      await awardManager.onContentCompleted(courseId)
-
-      expect(db.userAwardProgress.recordAwardProgress).toHaveBeenCalledWith(
-        testAward._id,
-        52,
-        expect.objectContaining({
-          progressData: expect.any(Object)
-        })
-      )
-    })
-
-    test('must complete all 23 eligible lessons to earn award', async () => {
-      mockCompletionStates(db, eligibleLessons)
-
-      await awardManager.onContentCompleted(courseId)
-
-      expect(db.userAwardProgress.recordAwardProgress).toHaveBeenCalledWith(
-        expect.any(String),
-        100,
-        expect.objectContaining({
-          completedAt: expect.any(String),
-          immediate: true
-        })
-      )
-      expect(listeners.granted).toHaveBeenCalledTimes(1)
-    })
-  })
-
-  describe('Scenario: Course type without excluded content (417039 - all 3 lessons count)', () => {
-    const testAward = getAwardByContentId(417039)
-    const courseId = 417039
-
-    test('first lesson counts toward progress', async () => {
-      mockCompletionStates(db, [417035])
-
-      await awardManager.onContentCompleted(courseId)
-
-      expect(db.userAwardProgress.recordAwardProgress).toHaveBeenCalledWith(
-        testAward._id,
-        33,
-        expect.objectContaining({
-          progressData: expect.any(Object)
-        })
-      )
-    })
-
-    test('all 3 lessons must be completed to earn award', async () => {
-      mockAllCompleted(db)
-
-      await awardManager.onContentCompleted(courseId)
-
-      expect(db.userAwardProgress.recordAwardProgress).toHaveBeenCalledWith(
-        expect.any(String),
-        100,
-        expect.objectContaining({
-          completedAt: expect.any(String),
-          immediate: true
-        })
-      )
-    })
-  })
-
-  describe('Scenario: Learning path content type (417140)', () => {
-    const testAward = getAwardByContentId(417140)
-    const courseId = 417140
-
-    test('all child content counts toward progress', async () => {
-      mockCompletionStates(db, testAward.child_ids.slice(0, 11))
-
-      await awardManager.onContentCompleted(courseId)
-
-      expect(db.userAwardProgress.recordAwardProgress).toHaveBeenCalledWith(
-        testAward._id,
-        50,
-        expect.objectContaining({
-          progressData: expect.any(Object)
-        })
-      )
-    })
-  })
-
-  describe('Scenario: Skill pack content type (418000)', () => {
-    const courseId = 418000
-
-    test('completing all lessons grants award', async () => {
-      mockAllCompleted(db)
-
-      await awardManager.onContentCompleted(courseId)
-
-      expect(listeners.granted).toHaveBeenCalledTimes(1)
-    })
-  })
-})