musora-content-services 2.96.0 → 2.96.1

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.96.1](https://github.com/railroadmedia/musora-content-services/compare/v2.96.0...v2.96.1) (2025-12-09)
+
+
+### Bug Fixes
+
+* **agi:** interface return types ([cc0cfae](https://github.com/railroadmedia/musora-content-services/commit/cc0cfae1282266567ac46a6eb072f76ffde7032e))
+
 ## [2.96.0](https://github.com/railroadmedia/musora-content-services/compare/v2.95.5...v2.96.0) (2025-12-09)
 
 

package/package.json

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

package/src/services/content/artist.ts

@@ -15,6 +15,11 @@ export interface Artist {
   lessonCount: number
 }
 
+export interface Artists {
+  data: Artist[]
+  total: number
+}
+
 /**
  * Fetch all artists with lessons available for a specific brand.
  *
@@ -29,7 +34,7 @@ export interface Artist {
 export async function fetchArtists(
   brand: Brands | string,
   options: BuildQueryOptions = { sort: 'lower(name) asc' }
-): Promise<Artist[] | null> {
+): Promise<Artists> {
   const lessonFilter = await new FilterBuilder(`brand == "${brand}" && references(^._id)`, {
     bypassPermissions: true,
   }).buildFilter()

package/src/services/content/genre.ts

@@ -15,6 +15,11 @@ export interface Genre {
   thumbnail: string
 }
 
+export interface Genres {
+  data: Genre[]
+  total: number
+}
+
 /**
  * Fetch all genres with lessons available for a specific brand.
  *
@@ -29,7 +34,7 @@ export interface Genre {
 export async function fetchGenres(
   brand: Brands | string,
   options: BuildQueryOptions = { sort: 'lower(name) asc' }
-): Promise<Genre[]> {
+): Promise<Genres> {
   const lessonFilter = await new FilterBuilder(`brand == "${brand}" && references(^._id)`, {
     bypassPermissions: true,
   }).buildFilter()

package/src/services/content/instructor.ts

@@ -35,7 +35,7 @@ export interface Instructors {
 export async function fetchInstructors(
   brand: Brands | string,
   options: BuildQueryOptions
-): Promise<Instructor[]> {
+): Promise<Instructors> {
   const lessonFilter = await new FilterBuilder(`brand == "${brand}" && references(^._id)`, {
     bypassPermissions: true,
   }).buildFilter()

package/src/services/sync/fetch.ts

@@ -308,12 +308,7 @@ function serializeIds(ids: { id: RecordId }): { client_record_id: RecordId } {
 
 function deserializeRecord(record: SyncSyncable<BaseModel, 'client_record_id'> | null): SyncSyncable<BaseModel, 'id'> | null {
   if (record) {
-    const { client_record_id: id, ...rest } = record as any
-
-    if ('collection_type' in rest && rest.collection_type === 'self') {
-      rest.collection_type = null
-      rest.collection_id = null
-    }
+    const { client_record_id: id, ...rest } = record
 
     return {
       ...rest,

package/src/services/sync/models/ContentProgress.ts

@@ -2,9 +2,12 @@ import BaseModel from './Base'
 import { SYNC_TABLES } from '../schema'
 
 export enum COLLECTION_TYPE {
+  SELF = 'self',
+  GUIDED_COURSE = 'guided-course',
   LEARNING_PATH = 'learning-path-v2',
   PLAYLIST = 'playlist',
 }
+export const COLLECTION_ID_SELF = 0
 
 export enum STATE {
   STARTED = 'started',
@@ -13,8 +16,8 @@ export enum STATE {
 
 export default class ContentProgress extends BaseModel<{
   content_id: number
-  collection_type: COLLECTION_TYPE | null
-  collection_id: number | null
+  collection_type: COLLECTION_TYPE
+  collection_id: number
   state: STATE
   progress_percent: number
   resume_time_seconds: number | null
@@ -34,10 +37,10 @@ export default class ContentProgress extends BaseModel<{
     return this._getRaw('progress_percent') as number
   }
   get collection_type() {
-    return (this._getRaw('collection_type') as COLLECTION_TYPE) || null
+    return this._getRaw('collection_type') as COLLECTION_TYPE
   }
   get collection_id() {
-    return (this._getRaw('collection_id') as number) || null
+    return this._getRaw('collection_id') as number
   }
   get resume_time_seconds() {
     return (this._getRaw('resume_time_seconds') as number) || null
@@ -55,10 +58,10 @@ export default class ContentProgress extends BaseModel<{
   set progress_percent(value: number) {
     this._setRaw('progress_percent', Math.min(100, Math.max(0, value)))
   }
-  set collection_type(value: COLLECTION_TYPE | null) {
+  set collection_type(value: COLLECTION_TYPE) {
     this._setRaw('collection_type', value)
   }
-  set collection_id(value: number | null) {
+  set collection_id(value: number) {
     this._setRaw('collection_id', value)
   }
   set resume_time_seconds(value: number | null) {

package/src/services/sync/repositories/content-progress.ts

@@ -1,12 +1,12 @@
 import SyncRepository, { Q } from './base'
-import ContentProgress, { COLLECTION_TYPE, STATE } from '../models/ContentProgress'
+import ContentProgress, { COLLECTION_TYPE, COLLECTION_ID_SELF, STATE } from '../models/ContentProgress'
 
 export default class ProgressRepository extends SyncRepository<ContentProgress> {
   // null collection only
   async startedIds(limit?: number) {
     return this.queryAllIds(...[
-      Q.where('collection_type', null),
-      Q.where('collection_id', null),
+      Q.where('collection_type', COLLECTION_TYPE.SELF),
+      Q.where('collection_id', COLLECTION_ID_SELF),
 
       Q.where('state', STATE.STARTED),
       Q.sortBy('updated_at', 'desc'),
@@ -18,8 +18,8 @@ export default class ProgressRepository extends SyncRepository<ContentProgress>
   // null collection only
   async completedIds(limit?: number) {
     return this.queryAllIds(...[
-      Q.where('collection_type', null),
-      Q.where('collection_id', null),
+      Q.where('collection_type', COLLECTION_TYPE.SELF),
+      Q.where('collection_id', COLLECTION_ID_SELF),
 
       Q.where('state', STATE.COMPLETED),
       Q.sortBy('updated_at', 'desc'),
@@ -55,8 +55,8 @@ export default class ProgressRepository extends SyncRepository<ContentProgress>
     } = {}
   ) {
     const clauses: Q.Clause[] = [
-      Q.where('collection_type', null),
-      Q.where('collection_id', null),
+      Q.where('collection_type', COLLECTION_TYPE.SELF),
+      Q.where('collection_id', COLLECTION_ID_SELF),
 
       Q.or(Q.where('state', STATE.STARTED), Q.where('state', STATE.COMPLETED)),
       Q.sortBy('updated_at', 'desc'),
@@ -80,8 +80,8 @@ export default class ProgressRepository extends SyncRepository<ContentProgress>
   async mostRecentlyUpdatedId(contentIds: number[], collection: { type: COLLECTION_TYPE; id: number } | null = null) {
     return this.queryOneId(
       Q.where('content_id', Q.oneOf(contentIds)),
-      Q.where('collection_type', collection?.type ?? null),
-      Q.where('collection_id', collection?.id ?? null),
+      Q.where('collection_type', collection?.type ?? COLLECTION_TYPE.SELF),
+      Q.where('collection_id', collection?.id ?? COLLECTION_ID_SELF),
 
       Q.sortBy('updated_at', 'desc')
     )
@@ -93,8 +93,8 @@ export default class ProgressRepository extends SyncRepository<ContentProgress>
   ) {
     const clauses = [
       Q.where('content_id', contentId),
-      Q.where('collection_type', collection?.type ?? null),
-      Q.where('collection_id', collection?.id ?? null),
+      Q.where('collection_type', collection?.type ?? COLLECTION_TYPE.SELF),
+      Q.where('collection_id', collection?.id ?? COLLECTION_ID_SELF),
     ]
 
     return await this.queryOne(...clauses)
@@ -106,8 +106,8 @@ export default class ProgressRepository extends SyncRepository<ContentProgress>
   ) {
     const clauses = [
       Q.where('content_id', Q.oneOf(contentIds)),
-      Q.where('collection_type', collection?.type ?? null),
-      Q.where('collection_id', collection?.id ?? null),
+      Q.where('collection_type', collection?.type ?? COLLECTION_TYPE.SELF),
+      Q.where('collection_id', collection?.id ?? COLLECTION_ID_SELF),
     ]
 
     return await this.queryAll(...clauses)
@@ -118,8 +118,8 @@ export default class ProgressRepository extends SyncRepository<ContentProgress>
 
     const result = this.upsertOne(id, (r) => {
       r.content_id = contentId
-      r.collection_type = collection?.type ?? null
-      r.collection_id = collection?.id ?? null
+      r.collection_type = collection?.type ?? COLLECTION_TYPE.SELF
+      r.collection_id = collection?.id ?? COLLECTION_ID_SELF
 
       r.state = progressPct === 100 ? STATE.COMPLETED : STATE.STARTED
       r.progress_percent = progressPct
@@ -142,8 +142,8 @@ export default class ProgressRepository extends SyncRepository<ContentProgress>
         progressPercent: progressPct,
         progressStatus: progressPct === 100 ? STATE.COMPLETED : STATE.STARTED,
         bubble: true,
-        collectionType: collection?.type ?? null,
-        collectionId: collection?.id ?? null,
+        collectionType: collection?.type ?? COLLECTION_TYPE.SELF,
+        collectionId: collection?.id ?? COLLECTION_ID_SELF,
         resumeTimeSeconds: resumeTime ?? null,
         timestamp: Date.now()
       })
@@ -165,8 +165,8 @@ export default class ProgressRepository extends SyncRepository<ContentProgress>
           ProgressRepository.generateId(contentId, collection),
           (r: ContentProgress) => {
             r.content_id = contentId
-            r.collection_type = collection?.type ?? null
-            r.collection_id = collection?.id ?? null
+            r.collection_type = collection?.type ?? COLLECTION_TYPE.SELF
+            r.collection_id = collection?.id ?? COLLECTION_ID_SELF
 
             r.state = progressPct === 100 ? STATE.COMPLETED : STATE.STARTED
             r.progress_percent = progressPct
@@ -185,8 +185,8 @@ export default class ProgressRepository extends SyncRepository<ContentProgress>
         ProgressRepository.generateId(+contentId, collection),
         (r: ContentProgress) => {
           r.content_id = +contentId
-          r.collection_type = collection?.type ?? null
-          r.collection_id = collection?.id ?? null
+          r.collection_type = collection?.type ?? COLLECTION_TYPE.SELF
+          r.collection_id = collection?.id ?? COLLECTION_ID_SELF
 
           r.state = progressPct === 100 ? STATE.COMPLETED : STATE.STARTED
           r.progress_percent = progressPct
@@ -204,10 +204,6 @@ export default class ProgressRepository extends SyncRepository<ContentProgress>
     contentId: number,
     collection: { type: COLLECTION_TYPE; id: number } | null
   ) {
-    if (collection) {
-      return `${contentId}:${collection.type}:${collection.id}`
-    } else {
-      return `${contentId}`
-    }
+    return `${contentId}:${collection?.type || COLLECTION_TYPE.SELF}:${collection?.id || COLLECTION_ID_SELF}`
   }
 }

package/.claude/settings.local.json

@@ -1,9 +0,0 @@
-{
-  "permissions": {
-    "allow": [
-      "Bash(rg:*)",
-      "Bash(npm run lint:*)"
-    ],
-    "deny": []
-  }
-}