musora-content-services 2.104.9 → 2.106.0

package/CHANGELOG.md

@@ -2,6 +2,26 @@
 
 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.106.0](https://github.com/railroadmedia/musora-content-services/compare/v2.105.0...v2.106.0) (2025-12-18)
+
+
+### Features
+
+* Add some validators prior to writing to watermelon ([#663](https://github.com/railroadmedia/musora-content-services/issues/663)) ([9eeedac](https://github.com/railroadmedia/musora-content-services/commit/9eeedac5a3cf8b4eedd091fcb8252ffd5bae6c85))
+
+## [2.105.0](https://github.com/railroadmedia/musora-content-services/compare/v2.104.9...v2.105.0) (2025-12-17)
+
+
+### Features
+
+* Remove xp and total_xp ([#656](https://github.com/railroadmedia/musora-content-services/issues/656)) ([e6238a6](https://github.com/railroadmedia/musora-content-services/commit/e6238a605e095dc81e152c25f0abf4885b2bb091))
+* update type for method progress card ([#662](https://github.com/railroadmedia/musora-content-services/issues/662)) ([ad2d6df](https://github.com/railroadmedia/musora-content-services/commit/ad2d6df35b321afd7df1a302ee6bfbcbce1a31b7))
+
+
+### Bug Fixes
+
+* Minor progress fixes ([#654](https://github.com/railroadmedia/musora-content-services/issues/654)) ([202cceb](https://github.com/railroadmedia/musora-content-services/commit/202ccebc85d119ce9a63007e600a8fd92c3f94d7))
+
 ### [2.104.9](https://github.com/railroadmedia/musora-content-services/compare/v2.104.8...v2.104.9) (2025-12-17)
 
 ### [2.104.8](https://github.com/railroadmedia/musora-content-services/compare/v2.104.7...v2.104.8) (2025-12-16)

package/package.json

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

package/src/contentTypeConfig.js

@@ -425,8 +425,6 @@ export let contentTypeConfig = {
       '"instructors": instructor[]->name',
       `"description": ${descriptionField}`,
       `"resource": ${resourcesField}`,
-      'xp',
-      'total_xp',
       `"lessons": child[]->{
                 "id": railcontent_id,
                 title,
@@ -515,11 +513,9 @@ export let contentTypeConfig = {
   pack: {
     fields: [
       '"lesson_count": coalesce(count(child[]->.child[]->), 0)',
-      'xp',
       `"description": ${descriptionField}`,
       '"instructors": instructor[]->{ "id": railcontent_id, name, "thumbnail_url": thumbnail_url.asset->url }',
       '"logo_image_url": logo_image_url.asset->url',
-      'total_xp',
       `"resources": ${resourcesField}`,
       '"thumbnail": thumbnail.asset->url',
       '"light_mode_logo": light_mode_logo_url.asset->url',
@@ -552,7 +548,6 @@ export let contentTypeConfig = {
       '"light_mode_logo": light_mode_logo_url.asset->url',
       '"dark_mode_logo": dark_mode_logo_url.asset->url',
       `"description": ${descriptionField}`,
-      'total_xp',
     ],
     childFields: [`"description": ${descriptionField}`],
   },
@@ -574,7 +569,6 @@ export let contentTypeConfig = {
                 title,
                 "type": _type,
                 "description": ${descriptionField},
-                xp,
                 web_url_path,
                 "url": web_url_path,
             }`,
@@ -586,8 +580,6 @@ export let contentTypeConfig = {
       '"instructors": instructor[]->name',
       `"description": ${descriptionField}`,
       `"resource": ${resourcesField}`,
-      'xp',
-      'total_xp',
       `"lessons": child[]->{
                 "id": railcontent_id,
                 title,

package/src/services/awards/award-callbacks.js

@@ -21,6 +21,7 @@ let progressUpdateCallback = null
  * - `awardId` - Unique Sanity award ID
  * - `name` - Display name of the award
  * - `badge` - URL to badge image
+ * - `contentType` - Content type ('guided-course' or 'learning-path-v2')
  * - `completed_at` - ISO timestamp
  * - `isCompleted` - Boolean indicating the award is completed (always true for granted awards)
  * - `completion_data.message` - Pre-generated congratulations message
@@ -72,6 +73,7 @@ export function registerAwardCallback(callback) {
       awardId: awardId,
       name: definition.name,
       badge: definition.badge,
+      contentType: definition.content_type,
       completed_at: completionData.completed_at,
       isCompleted: true,
       completionData: {

package/src/services/awards/award-query.js

@@ -136,9 +136,7 @@ export async function getContentAwards(contentId) {
         instructorName: def.instructor_name,
         progressPercentage: userProgress?.progress_percentage ?? 0,
         isCompleted: userProgress ? UserAwardProgressRepository.isCompleted(userProgress) : false,
-        completedAt: userProgress?.completed_at
-          ? new Date(userProgress.completed_at).toISOString()
-          : null,
+        completedAt: userProgress?.completed_at,
         completionData
       }
     })
@@ -168,8 +166,7 @@ export async function getContentAwards(contentId) {
  * - Badge and award images for display
  * - Completion date for "Earned on X" display
  * - `completionData.message` - Pre-generated congratulations text
- * - `completionData.practice_minutes` - Total practice time for this award
- * - `completionData.days_user_practiced` - Days spent earning this award
+ * - `completionData.XXX` - other fields are award type dependant
  *
  * Returns empty array `[]` on error (never throws).
  *
@@ -226,16 +223,14 @@ export async function getContentAwards(contentId) {
  */
 export async function getCompletedAwards(brand = null, options = {}) {
   try {
-    const allProgress = await db.userAwardProgress.getAll()
+    const allProgress = await db.userAwardProgress.getCompleted()
 
     const completed = allProgress.data.filter(p =>
       p.progress_percentage === 100 && p.completed_at !== null
     )
-
     let awards = await Promise.all(
       completed.map(async (progress) => {
         const definition = await awardDefinitions.getById(progress.award_id)
-
         if (!definition) {
           return null
         }
@@ -243,24 +238,24 @@ export async function getCompletedAwards(brand = null, options = {}) {
         if (brand && definition.brand !== brand) {
           return null
         }
-
-        const completionData = enhanceCompletionData(progress.completion_data)
-
+        const completionData = definition.type === awardDefinitions.CONTENT_AWARD ? enhanceCompletionData(progress.completion_data) : progress.completion_data;
+        const hasCertificate = definition.type === awardDefinitions.CONTENT_AWARD
         return {
           awardId: progress.award_id,
           awardTitle: definition.name,
+          awardType: definition.type,
           badge: definition.badge,
           award: definition.award,
           brand: definition.brand,
+          hasCertificate: hasCertificate,
           instructorName: definition.instructor_name,
           progressPercentage: progress.progress_percentage,
           isCompleted: true,
-          completedAt: new Date(progress.completed_at * 1000).toISOString(),
+          completedAt: progress.completed_at,
           completionData
         }
       })
     )
-
     awards = awards.filter(award => award !== null)
 
     awards.sort((a, b) => new Date(b.completedAt).getTime() - new Date(a.completedAt).getTime())
@@ -269,7 +264,6 @@ export async function getCompletedAwards(brand = null, options = {}) {
       const offset = options.offset || 0
       awards = awards.slice(offset, offset + options.limit)
     }
-
     return awards
   } catch (error) {
     console.error('Failed to get completed awards:', error)

package/src/services/awards/internal/award-definitions.js

@@ -9,6 +9,9 @@
 const STORAGE_KEY = 'musora_award_definitions_last_fetch'
 
 class AwardDefinitionsService {
+
+  CONTENT_AWARD = 'content-award'
+  EXP_AWARD  = 'exp-award'
   constructor() {
     /** @type {AwardDefinitionsMap} */
     this.definitions = new Map()
@@ -90,7 +93,7 @@ class AwardDefinitionsService {
         bypassPermissions: true,
       }).buildFilter()
 
-      const query = `*[_type == 'content-award'] {
+      const query = `*[_type in ['content-award', 'exp-award']] {
         _id,
         is_active,
         name,
@@ -107,7 +110,7 @@ class AwardDefinitionsService {
         'child_ids': content->child[${childFilter}]->railcontent_id,
       }`
 
-      const awards = await fetchSanity(query, true, { processNeedAccess: false })
+      const awards = await fetchSanity(query, true, { processNeedAccess: false, processPageType: false })
 
       this.definitions.clear()
       this.contentIndex.clear()

package/src/services/awards/internal/award-manager.js

@@ -99,7 +99,7 @@ export class AwardManager {
     const popupMessage = AwardMessageGenerator.generatePopupMessage(completionData)
 
     await db.userAwardProgress.recordAwardProgress(award._id, 100, {
-      completedAt: Date.now(),
+      completedAt: new Date().toISOString(),
       completionData,
       progressData,
       immediate: true

package/src/services/awards/internal/certificate-builder.js

@@ -39,10 +39,7 @@ export async function buildCertificateData(awardId) {
   return {
     userId: globalConfig.sessionConfig.userId,
     userName: userData?.display_name || userData?.name || 'User',
-    completedAt: userProgress.data.completed_at
-      ? new Date(userProgress.data.completed_at * 1000).toISOString()
-      : new Date().toISOString(),
-
+    completedAt: userProgress.data?.completed_at ?? new Date().toISOString(),
     awardId: awardDef._id,
     awardType: awardDef.type || 'content-award',
     awardTitle: awardDef.name,

package/src/services/awards/types.d.ts

@@ -65,6 +65,7 @@ export interface AwardCallbackPayload {
   awardId: string
   name: string
   badge: string
+  contentType: string
   completed_at: string
   isCompleted: boolean
   completion_data: AwardCompletionData

package/src/services/awards/types.js

@@ -37,10 +37,12 @@
  * @typedef {Object} AwardInfo
  * @property {string} awardId - Unique Sanity award ID
  * @property {string} awardTitle - Display name of the award
+ * @property {string} awardType - Type of the award
+ * @property {boolean} hasCertificate - flag to indicate if the award includes a downloadable certificate
  * @property {string} badge - URL to badge image
  * @property {string} award - URL to award image
- * @property {string} brand - Brand (drumeo, pianote, guitareo, singeo)
- * @property {string} instructorName - Name of the instructor
+ * @property {string} brand - Brand (drumeo, pianote, guitareo, singeo, playbass)
+ * @property {string|null} instructorName - Name of the instructor
  * @property {number} progressPercentage - Completion percentage (0-100). Progress is tracked per collection context for learning paths.
  * @property {boolean} isCompleted - Whether the award is fully completed
  * @property {string|null} completedAt - ISO timestamp of completion, or null if not completed
@@ -73,6 +75,7 @@
  * @property {string} awardId - Unique Sanity award ID
  * @property {string} name - Display name of the award
  * @property {string} badge - URL to badge image
+ * @property {string} contentType - Content type ('guided-course' or 'learning-path-v2')
  * @property {string} completed_at - ISO timestamp of completion
  * @property {AwardCompletionData} completion_data - Practice statistics
  */

package/src/services/contentProgress.js

@@ -6,6 +6,7 @@ import { getNextLessonLessonParentTypes } from '../contentTypeConfig.js'
 import { emitContentCompleted } from './progress-events'
 import {getDailySession} from "./content-org/learning-paths.ts";
 import {getToday} from "./dateUtils.js";
+import { fetchBrandsByContentIds } from './sanity.js'
 
 const STATE_STARTED = STATE.STARTED
 const STATE_COMPLETED = STATE.COMPLETED
@@ -316,48 +317,25 @@ async function getByIdsAndCollections(tuples, dataKey, defaultValue) {
 }
 
 export async function getAllStarted(limit = null) {
-  return db.contentProgress.startedIds(limit).then((r) => r.data.map((id) => parseInt(id)))
+  return db.contentProgress.startedIds(limit)
 }
 
 export async function getAllCompleted(limit = null) {
-  return db.contentProgress.completedIds(limit).then((r) => r.data.map((id) => parseInt(id)))
+  return db.contentProgress.completedIds(limit)
 }
 
 export async function getAllCompletedByIds(contentIds) {
   return db.contentProgress.completedByContentIds(normalizeContentIds(contentIds))
 }
 
+/**
+ * Fetches content **IDs** for items that were started or completed.
+ */
 export async function getAllStartedOrCompleted({
-  onlyIds = true,
   brand = null,
   limit = null,
 } = {}) {
-  const agoInSeconds = Math.floor(Date.now() / 1000) - 60 * 24 * 60 * 60 // 60 days in seconds
-  const filters = {
-    brand: brand ?? undefined,
-    updatedAfter: agoInSeconds,
-    limit: limit ?? undefined,
-  }
-
-  if (onlyIds) {
-    return db.contentProgress
-      .startedOrCompletedIds(filters)
-      .then((r) => r.data.map((id) => parseInt(id)))
-  } else {
-    return db.contentProgress.startedOrCompleted(filters).then((r) => {
-      return Object.fromEntries(
-        r.data.map((p) => [
-          p.content_id,
-          {
-            last_update: p.updated_at,
-            progress: p.progress_percent,
-            status: p.state,
-            brand: p.content_brand,
-          },
-        ])
-      )
-    })
-  }
+  return await _getAllStartedOrCompleted({ brand, limit }).then(recs => recs.map(rec => rec.content_id))
 }
 
 /**
@@ -378,11 +356,41 @@ export async function getAllStartedOrCompleted({
  * console.log(progressMap[123]); // => 52
  */
 export async function getStartedOrCompletedProgressOnly({ brand = undefined } = {}) {
-  return db.contentProgress.startedOrCompleted({ brand: brand }).then((r) => {
-    return Object.fromEntries(r.data.map((p) => [p.content_id, p.progress_percent]))
+  return _getAllStartedOrCompleted({ brand }).then((r) => {
+    return Object.fromEntries(r.map((p) => [p.content_id, p.progress_percent]))
   })
 }
 
+async function _getAllStartedOrCompleted({
+  brand = null,
+  limit = null,
+} = {}) {
+  const agoInSeconds = Math.floor(Date.now() / 1000) - 60 * 24 * 60 * 60 // 60 days in seconds
+  const baseFilters = {
+    updatedAfter: agoInSeconds,
+  }
+
+  if (!brand) {
+    return await db.contentProgress.startedOrCompleted({ ...baseFilters, limit }).then(r => r.data)
+  }
+
+  // content_brand can be null (i.e., when progress records created locally)
+  // TODO: eventually put content metadata into watermelon so we can
+  // always have brand info in progress records and avoid all this
+
+  // for now though, null-ish brands shouldn't be too numerous, so safe to have undefined limit
+  const [strictRecs, looseRecs] = await Promise.all([
+    db.contentProgress.startedOrCompleted({ ...baseFilters, brand, limit }),
+    db.contentProgress.startedOrCompleted({ ...baseFilters, brand: null, limit: undefined })
+  ]);
+
+  const map = await fetchBrandsByContentIds(looseRecs.data.map(r => r.content_id));
+  const filteredLooseRecs = looseRecs.data.filter(r => map[r.content_id] === brand).map(r => ({ ...r, content_brand: brand }));
+
+  const records = [...strictRecs.data, ...filteredLooseRecs].sort((a, b) => b.updated_at - a.updated_at).slice(0, limit || undefined);
+  return records;
+}
+
 /**
  * Record watch session
  * @return {string} sessionId - provide in future calls to update progress

package/src/services/progress-row/method-card.js

@@ -98,7 +98,7 @@ export async function getMethodCard(brand) {
       action = nextLesson
         ? getMethodActionCTA(nextLesson)
         : {
-            type: 'method',
+            type: 'method-complete',
             brand,
           }
     }

package/src/services/sanity.js

@@ -2083,3 +2083,26 @@ export async function fetchOwnedContent(
 
   return fetchSanity(query, true)
 }
+
+/**
+ * Fetch brands for given content IDs.
+ *
+ * @param {Array<number>} contentIds - Array of railcontent IDs
+ * @returns {Promise<Object>} - A promise that resolves to an object mapping content IDs to brands
+ */
+export async function fetchBrandsByContentIds(contentIds) {
+  if (!contentIds || contentIds.length === 0) {
+    return {}
+  }
+  const idsString = contentIds.join(',')
+  const query = `*[railcontent_id in [${idsString}]]{
+      railcontent_id,
+      brand
+    }`
+  const results = await fetchSanity(query, true)
+  const brandMap = {}
+  results.forEach((item) => {
+    brandMap[item.railcontent_id] = item.brand
+  })
+  return brandMap
+}

package/src/services/sync/errors/index.ts

@@ -47,3 +47,11 @@ export class SyncUnexpectedError extends SyncError {
     Object.setPrototypeOf(this, new.target.prototype)
   }
 }
+
+export class SyncValidationError extends SyncError {
+  constructor(message: string, value: any, expected: any) {
+    super('validationError', { message: message, value: value, expected: expected })
+    this.name = 'SyncValidationError'
+    Object.setPrototypeOf(this, new.target.prototype)
+  }
+}

package/src/services/sync/errors/validators.ts

@@ -0,0 +1,42 @@
+import { SyncValidationError } from './index'
+
+
+export function throwIfNotNumber(val: any) {
+  // note: this will accept decimal values
+  if (typeof val !== 'number') throw new SyncValidationError('Sync value is not a number: ' + val, typeof val, 'number');
+  return val
+}
+
+export function throwIfNotString(val: any) {
+  if (typeof val !== 'string') throw new SyncValidationError('Sync value is not a string: ' + val, typeof val, 'string');
+  return val
+}
+
+export function throwIfNotBoolean(val: any) {
+  if (typeof val !== 'boolean') throw new SyncValidationError('Sync value is not a boolean: ' + val, typeof val, 'boolean');
+  return val
+}
+
+export function throwIfNotNullableNumber(val: any) {
+  return val === null ? val : throwIfNotNumber(val)
+}
+
+export function throwIfNotNullableString(val: any) {
+  return val === null ? val : throwIfNotString(val)
+}
+
+export function throwIfOutsideRange(val: number, minimum?: number, maximum?: number) {
+  if (minimum !== undefined && val < minimum) throw new SyncValidationError('Sync value is less than minimum value ' + minimum + ': ' + val, val, null);
+  if (maximum !== undefined && val > maximum) throw new SyncValidationError('Sync value is greater than maximum value ' + maximum + ': ' + val, val, null);
+  return val
+}
+
+export function throwIfMaxLengthExceeded(val: string, maximum: number) {
+  if (val.length > maximum) throw new SyncValidationError('Sync value exceeds the maximum length ' + maximum + ': ' + val, val, null);
+  return val
+}
+
+export function throwIfInvalidEnumValue(val: string, enumClass: any) {
+  if (!Object.values(enumClass).includes(val)) throw new SyncValidationError('Sync value is invalid enum value: ' + val, val, enumClass);
+  return val
+}

package/src/services/sync/fetch.ts

@@ -34,7 +34,7 @@ type SyncPushFetchFailureResponse = SyncResponseBase & {
   failureType: 'fetch'
   isRetryable: boolean
 }
-type SyncPushFailureResponse = SyncResponseBase & {
+export type SyncPushFailureResponse = SyncResponseBase & {
   ok: false,
   failureType: 'error'
   originalError: Error

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

@@ -1,5 +1,6 @@
 import { SYNC_TABLES } from '../schema'
 import BaseModel from './Base'
+import { throwIfNotNumber } from '../errors/validators'
 
 export default class ContentLike extends BaseModel<{
   content_id: number
@@ -11,6 +12,6 @@ export default class ContentLike extends BaseModel<{
   }
 
   set content_id(value: number) {
-    this._setRaw('content_id', value)
+    this._setRaw('content_id', throwIfNotNumber(value))
   }
 }

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

@@ -1,5 +1,12 @@
 import BaseModel from './Base'
 import { SYNC_TABLES } from '../schema'
+import {
+  throwIfInvalidEnumValue,
+  throwIfNotNullableNumber,
+  throwIfNotNullableString,
+  throwIfNotNumber,
+  throwIfOutsideRange,
+} from '../errors/validators'
 
 export enum COLLECTION_TYPE {
   SELF = 'self',
@@ -16,6 +23,7 @@ export enum STATE {
 
 export default class ContentProgress extends BaseModel<{
   content_id: number
+  content_brand: string | null
   collection_type: COLLECTION_TYPE
   collection_id: number
   state: STATE
@@ -28,7 +36,7 @@ export default class ContentProgress extends BaseModel<{
     return this._getRaw('content_id') as number
   }
   get content_brand() {
-    return this._getRaw('content_brand') as string
+    return this._getRaw('content_brand') as string | null
   }
   get state() {
     return this._getRaw('state') as STATE
@@ -47,25 +55,37 @@ export default class ContentProgress extends BaseModel<{
   }
 
   set content_id(value: number) {
-    this._setRaw('content_id', value)
+    // unsigned int
+    throwIfNotNumber(value)
+    this._setRaw('content_id', throwIfOutsideRange(value, 0))
   }
-  set content_brand(value: string) {
-    this._setRaw('content_brand', value)
-  }
-  set state(value: STATE) {
-    this._setRaw('state', value)
+  set content_brand(value: string |  null) {
+    this._setRaw('content_brand', throwIfNotNullableString(value))
   }
+  // IMPORTANT: progress percent only moves forward and is clamped between 0 and 100
+  // also has implications for last-write-wins sync strategy
   set progress_percent(value: number) {
-    this._setRaw('progress_percent', Math.min(100, Math.max(0, value)))
+    // tinyint unsigned
+    throwIfNotNumber(value)
+    throwIfOutsideRange(value, 0, 100)
+    const percent = value === 0 ? 0 : Math.max(value, this.progress_percent)
+
+    this._setRaw('progress_percent', percent)
+    this._setRaw('state', percent === 100 ? STATE.COMPLETED : STATE.STARTED)
   }
   set collection_type(value: COLLECTION_TYPE) {
-    this._setRaw('collection_type', value)
+    // enum collection_type
+    this._setRaw('collection_type', throwIfInvalidEnumValue(value, COLLECTION_TYPE))
   }
   set collection_id(value: number) {
-    this._setRaw('collection_id', value)
+    // unsigned mediumint 16777215
+    throwIfNotNumber(value)
+    this._setRaw('collection_id', throwIfOutsideRange(value, 0, 16777215))
   }
   set resume_time_seconds(value: number | null) {
-    this._setRaw('resume_time_seconds', value !== null ? Math.max(0, value) : null)
+    // smallint unsigned
+    throwIfNotNullableNumber(value)
+    this._setRaw('resume_time_seconds', value !== null ? throwIfOutsideRange(value, 0, 65535) : value)
   }
 
 }

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

@@ -1,5 +1,14 @@
 import { SYNC_TABLES } from '../schema'
 import BaseModel from './Base'
+import {
+  throwIfMaxLengthExceeded,
+  throwIfNotBoolean,
+  throwIfNotNullableNumber,
+  throwIfNotNullableString,
+  throwIfNotNumber,
+  throwIfNotString,
+  throwIfOutsideRange,
+} from '../errors/validators'
 
 export default class Practice extends BaseModel<{
   manual_id: string | null
@@ -43,30 +52,44 @@ export default class Practice extends BaseModel<{
   }
 
   set manual_id(value: string | null) {
-    this._setRaw('manual_id', value)
+    // char(26)
+    throwIfNotNullableString(value)
+    this._setRaw('manual_id', value !== null ? throwIfMaxLengthExceeded(value, 26) : value)
   }
   set content_id(value: number | null) {
-    this._setRaw('content_id', value)
+    // int unsigned
+    throwIfNotNullableNumber(value)
+    this._setRaw('content_id', value !== null ? throwIfOutsideRange(value, 0) : value)
   }
   set date(value: string) {
-    this._setRaw('date', value)
+    this._setRaw('date', throwIfNotString(value))
   }
   set auto(value: boolean) {
-    this._setRaw('auto', value)
+    // tinyint(1)
+    this._setRaw('auto', throwIfNotBoolean(value))
   }
   set duration_seconds(value: number) {
-    this._setRaw('duration_seconds', value)
+    throwIfNotNumber(value)
+    this._setRaw('duration_seconds', throwIfOutsideRange(value, 0, 59999))
   }
   set title(value: string | null) {
-    this._setRaw('title', value)
+    // varchar(64)
+    throwIfNotNullableString(value)
+    this._setRaw('title', value !== null ? throwIfMaxLengthExceeded(value, 64) : value)
   }
   set thumbnail_url(value: string | null) {
-    this._setRaw('thumbnail_url', value)
+    // varchar(255)
+    throwIfNotNullableString(value)
+    this._setRaw('thumbnail_url', value !== null ? throwIfMaxLengthExceeded(value, 255) : value)
   }
   set category_id(value: number | null) {
-    this._setRaw('category_id', value)
+    // tinyint unsigned
+    throwIfNotNullableNumber(value)
+    this._setRaw('category_id', value !== null ? throwIfOutsideRange(value, 0, 255) : value)
   }
   set instrument_id(value: number | null) {
-    this._setRaw('instrument_id', value)
+    // tinyint unsigned
+    throwIfNotNullableNumber(value)
+    this._setRaw('instrument_id', value !== null ? throwIfOutsideRange(value, 0, 255) : value)
   }
 }

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

@@ -1,5 +1,6 @@
 import { SYNC_TABLES } from '../schema'
 import BaseModel from './Base'
+import { throwIfMaxLengthExceeded, throwIfNotString } from '../errors/validators'
 
 export default class PracticeDayNote extends BaseModel<{
   date: string
@@ -15,9 +16,10 @@ export default class PracticeDayNote extends BaseModel<{
   }
 
   set date(value: string) {
-    this._setRaw('date', value)
+    this._setRaw('date', throwIfNotString(value))
   }
   set notes(value: string) {
-    this._setRaw('notes', value)
+    throwIfNotString(value)
+    this._setRaw('notes', throwIfMaxLengthExceeded(value, 3000))
   }
 }

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

@@ -1,11 +1,18 @@
 import BaseModel from './Base'
 import { SYNC_TABLES } from '../schema'
 import type { CompletionData } from '../../awards/types'
+import {
+  throwIfMaxLengthExceeded,
+  throwIfNotNullableString,
+  throwIfNotNumber,
+  throwIfNotString,
+  throwIfOutsideRange,
+} from '../errors/validators'
 
 export default class UserAwardProgress extends BaseModel<{
   award_id: string
   progress_percentage: number
-  completed_at: number | null
+  completed_at: string | null
   progress_data: string | null
   completion_data: string | null
 }> {
@@ -20,7 +27,7 @@ export default class UserAwardProgress extends BaseModel<{
   }
 
   get completed_at() {
-    return this._getRaw('completed_at') as number | null
+    return this._getRaw('completed_at') as string | null
   }
 
   get progress_data() {
@@ -34,15 +41,19 @@ export default class UserAwardProgress extends BaseModel<{
   }
 
   set award_id(value: string) {
-    this._setRaw('award_id', value)
+    // varchar(255)
+    throwIfNotString(value)
+    this._setRaw('award_id', throwIfMaxLengthExceeded(value, 255))
   }
 
   set progress_percentage(value: number) {
-    this._setRaw('progress_percentage', value)
+    // int
+    throwIfNotNumber(value)
+    this._setRaw('progress_percentage', throwIfOutsideRange(value, 0, 100))
   }
 
-  set completed_at(value: number | null) {
-    this._setRaw('completed_at', value)
+  set completed_at(value: string | null) {
+    this._setRaw('completed_at', throwIfNotNullableString(value))
   }
 
   set progress_data(value: any) {

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

@@ -13,7 +13,7 @@ export interface CollectionParameter {
 export default class ProgressRepository extends SyncRepository<ContentProgress> {
   // null collection only
   async startedIds(limit?: number) {
-    return this.queryAllIds(...[
+    return this.queryAll(...[
       Q.where('collection_type', COLLECTION_TYPE.SELF),
       Q.where('collection_id', COLLECTION_ID_SELF),
 
@@ -21,12 +21,12 @@ export default class ProgressRepository extends SyncRepository<ContentProgress>
       Q.sortBy('updated_at', 'desc'),
 
       ...(limit ? [Q.take(limit)] : []),
-    ])
+    ]).then((r) => r.data.map((r) => r.content_id))
   }
 
   // null collection only
   async completedIds(limit?: number) {
-    return this.queryAllIds(...[
+    return this.queryAll(...[
       Q.where('collection_type', COLLECTION_TYPE.SELF),
       Q.where('collection_id', COLLECTION_ID_SELF),
 
@@ -34,7 +34,7 @@ export default class ProgressRepository extends SyncRepository<ContentProgress>
       Q.sortBy('updated_at', 'desc'),
 
       ...(limit ? [Q.take(limit)] : []),
-    ])
+    ]).then((r) => r.data.map((r) => r.content_id))
   }
 
   //this _specifically_ needs to get content_ids from ALL collection_types (including null)
@@ -50,17 +50,12 @@ export default class ProgressRepository extends SyncRepository<ContentProgress>
     return this.queryAll(...this.startedOrCompletedClauses(opts))
   }
 
-  // null collection only
-  async startedOrCompletedIds(opts: Parameters<typeof this.startedOrCompletedClauses>[0] = {}) {
-    return this.queryAllIds(...this.startedOrCompletedClauses(opts))
-  }
-
   // null collection only
   private startedOrCompletedClauses(
     opts: {
-      brand?: string
-      updatedAfter?: number
-      limit?: number
+      brand?: string | null
+      updatedAfter?: number,
+      limit?: number,
     } = {}
   ) {
     const clauses: Q.Clause[] = [
@@ -75,7 +70,7 @@ export default class ProgressRepository extends SyncRepository<ContentProgress>
       clauses.push(Q.where('updated_at', Q.gte(opts.updatedAfter)))
     }
 
-    if (opts.brand) {
+    if (typeof opts.brand != 'undefined') {
       clauses.push(Q.where('content_brand', opts.brand))
     }
 
@@ -146,7 +141,6 @@ export default class ProgressRepository extends SyncRepository<ContentProgress>
       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
 
       if (typeof resumeTime != 'undefined') {
@@ -193,7 +187,6 @@ export default class ProgressRepository extends SyncRepository<ContentProgress>
           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
         },
       ])

package/src/services/sync/repositories/user-award-progress.ts

@@ -5,7 +5,7 @@ import type { AwardDefinition, CompletionData } from '../../awards/types'
 import type { ModelSerialized } from '../serializers'
 
 type AwardProgressData = {
-  completed_at: number | null
+  completed_at: string | null
   progress_percentage: number
 }
 
@@ -18,7 +18,7 @@ export default class UserAwardProgressRepository extends SyncRepository<UserAwar
     return progress.progress_percentage >= 0 && !UserAwardProgressRepository.isCompleted(progress)
   }
 
-  static completedAtDate(progress: { completed_at: number | null }): Date | null {
+  static completedAtDate(progress: { completed_at: string | null }): Date | null {
     return progress.completed_at ? new Date(progress.completed_at) : null
   }
 
@@ -73,7 +73,7 @@ export default class UserAwardProgressRepository extends SyncRepository<UserAwar
     awardId: string,
     progressPercentage: number,
     options?: {
-      completedAt?: number | null
+      completedAt?: string | null
       progressData?: any
       completionData?: CompletionData | null
       immediate?: boolean
@@ -104,7 +104,7 @@ export default class UserAwardProgressRepository extends SyncRepository<UserAwar
     completionData: CompletionData
   ) {
     return this.recordAwardProgress(awardId, 100, {
-      completedAt: Date.now(),
+      completedAt: new Date().toISOString(),
       completionData,
       immediate: true
     })

package/src/services/sync/schema/index.ts

@@ -20,9 +20,9 @@ const contentProgressTable = tableSchema({
   name: SYNC_TABLES.CONTENT_PROGRESS,
   columns: [
     { name: 'content_id', type: 'number', isIndexed: true },
-    { name: 'content_brand', type: 'string', isIndexed: true },
-    { name: 'collection_type', type: 'string', isOptional: true, isIndexed: true },
-    { name: 'collection_id', type: 'number', isOptional: true, isIndexed: true },
+    { name: 'content_brand', type: 'string', isOptional: true, isIndexed: true },
+    { name: 'collection_type', type: 'string', isIndexed: true },
+    { name: 'collection_id', type: 'number', isIndexed: true },
     { name: 'state', type: 'string', isIndexed: true },
     { name: 'progress_percent', type: 'number' },
     { name: 'resume_time_seconds', type: 'number', isOptional: true },
@@ -61,7 +61,7 @@ const userAwardProgressTable = tableSchema({
   columns: [
     { name: 'award_id', type: 'string', isIndexed: true },
     { name: 'progress_percentage', type: 'number' },
-    { name: 'completed_at', type: 'number', isOptional: true, isIndexed: true },
+    { name: 'completed_at', type: 'string', isOptional: true, isIndexed: true },
     { name: 'progress_data', type: 'string', isOptional: true },
     { name: 'completion_data', type: 'string', isOptional: true },
     { name: 'created_at', type: 'number' },

package/src/services/user/types.d.ts

@@ -42,9 +42,7 @@ export interface User {
   revenuecat_origin_app_user_id: string | null
   is_drumeo_lifetime_member: number
   access_level: string
-  total_xp: number
   brand_method_levels: BrandMethodLevels
-  brand_total_xp: BrandTotalXp
   brand_minutes_practiced: BrandTimePracticed
   brand_seconds_practiced: BrandTimePracticed
   guitar_playing_since_year: number | null

package/src/services/user/types.js

@@ -46,9 +46,7 @@
  * @property {string|null} revenuecat_origin_app_user_id
  * @property {number} is_drumeo_lifetime_member
  * @property {string} access_level
- * @property {number} total_xp
  * @property {BrandMethodLevels} brand_method_levels
- * @property {BrandTotalXp} brand_total_xp
  * @property {BrandTimePracticed} brand_minutes_practiced
  * @property {BrandTimePracticed} brand_seconds_practiced
  * @property {number|null} guitar_playing_since_year

package/src/services/userActivity.js

@@ -982,9 +982,9 @@ function generateContentsMap(contents, playlistsContents) {
 export async function getProgressRows({ brand = 'drumeo', limit = 8 } = {}) {
   // TODO slice progress to a reasonable number, say 100
   const methodCardPromise = getMethodCard(brand)
-  const [recentPlaylists, progressContents, userPinnedItem] = await Promise.all([
+  const [recentPlaylists, nonPlaylistContentIds, userPinnedItem] = await Promise.all([
     fetchUserPlaylists(brand, { sort: '-last_progress', limit: limit }),
-    getAllStartedOrCompleted({ onlyIds: false, brand: brand, limit }),
+    getAllStartedOrCompleted({ brand: brand, limit }),
     getUserPinnedItem(brand),
   ])
 
@@ -995,7 +995,6 @@ export async function getProgressRows({ brand = 'drumeo', limit = 8 } = {}) {
   )
 
   // todo post v2: refactor this once we migrate playlist progress tracking to new system
-  const nonPlaylistContentIds = Object.keys(progressContents)
   if (userPinnedItem?.progressType === 'content') {
     nonPlaylistContentIds.push(userPinnedItem.id)
   }

package/test/awards/award-completion-flow.test.js

@@ -91,7 +91,8 @@ describe('Award Completion Flow - E2E Scenarios', () => {
         definition: expect.objectContaining({
           name: testAward.name,
           badge: testAward.badge,
-          award: testAward.award
+          award: testAward.award,
+          content_type: expect.any(String)
         }),
         completionData: expect.objectContaining({
           content_title: expect.any(String),

package/.claude/settings.local.json

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