musora-content-services 2.94.5 → 2.94.7

package/.claude/settings.local.json

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

package/.yarnrc.yml

@@ -0,0 +1 @@
+nodeLinker: node-modules

package/CHANGELOG.md

@@ -2,6 +2,25 @@
 
 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.94.7](https://github.com/railroadmedia/musora-content-services/compare/v2.94.1...v2.94.7) (2025-12-04)
+
+
+### Bug Fixes
+
+* bad import ([#611](https://github.com/railroadmedia/musora-content-services/issues/611)) ([ac9ee7b](https://github.com/railroadmedia/musora-content-services/commit/ac9ee7bb8d9617e319dcdedb47c5d7e3c7973732))
+* fallback for method card when no activ epath ([#610](https://github.com/railroadmedia/musora-content-services/issues/610)) ([8458d97](https://github.com/railroadmedia/musora-content-services/commit/8458d9754d293ae28cb134546305264e638c8bbd))
+* **MU2E2-183/T3PS-956:** Fix create/edit forum category ([070f7e5](https://github.com/railroadmedia/musora-content-services/commit/070f7e5754c0e2e9536311a76610df938f4e5146))
+* Patch permissions filter (round 2) ([#616](https://github.com/railroadmedia/musora-content-services/issues/616)) ([f8509a5](https://github.com/railroadmedia/musora-content-services/commit/f8509a5178dfdd17ac06f5779a39960ae1a8130b))
+* refactor fetchLearningPathProgressCheckLessons to exclude duplicates ([#615](https://github.com/railroadmedia/musora-content-services/issues/615)) ([0f0f51e](https://github.com/railroadmedia/musora-content-services/commit/0f0f51e2148a9e611bb3cbc9d103f12952e00056))
+* watermelon fixes for content progress ([#614](https://github.com/railroadmedia/musora-content-services/issues/614)) ([2c2fa8e](https://github.com/railroadmedia/musora-content-services/commit/2c2fa8e68d5b3174ca62c8b9a900370947e44725))
+
+### [2.94.6](https://github.com/railroadmedia/musora-content-services/compare/v2.94.5...v2.94.6) (2025-12-03)
+
+
+### Bug Fixes
+
+* Patch permissions filter (round 2) ([#616](https://github.com/railroadmedia/musora-content-services/issues/616)) ([f8509a5](https://github.com/railroadmedia/musora-content-services/commit/f8509a5178dfdd17ac06f5779a39960ae1a8130b))
+
 ### [2.94.5](https://github.com/railroadmedia/musora-content-services/compare/v2.94.4...v2.94.5) (2025-12-03)
 
 

package/package.json

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

package/src/services/forums/categories.ts

@@ -20,7 +20,7 @@ export async function fetchForumCategories(brand: string): Promise<ForumCategory
 }
 
 export interface CreateForumCategoryParams {
-  name: string
+  title: string
   description: string
   weight: number
   brand: string
@@ -44,7 +44,7 @@ export async function createForumCategory(
 export interface UpdateForumCategoryParams {
   id: number
   brand: string
-  name: string
+  title: string
   weight: number
   description?: string
   icon?: string

package/src/services/permissions/PermissionsV2Adapter.ts

@@ -178,7 +178,7 @@ export class PermissionsV2Adapter extends PermissionsAdapter {
 
     // Content with no permissions is accessible to all
     // A content has "no permissions" if BOTH permission and permission_v2 are empty/undefined
-    clauses.push(`((!defined(${prefix}permission) || count(${prefix}permission) == 0) && (!defined(${prefix}permission_v2) || count(${prefix}permission_v2) == 0))`)
+    clauses.push(`(!defined(${prefix}permission_v2) || count(${prefix}permission_v2) == 0)`)
 
     // User has matching permissions
     if (userPermissionIds.length > 0) {
@@ -204,12 +204,10 @@ export class PermissionsV2Adapter extends PermissionsAdapter {
   ): string {
     if (isDereferencedContext) {
       // In dereferenced context, check the permission array directly
-      return `((count((${prefix}permission[]._ref)[@ in *[_type == 'permission' && railcontent_id in ${arrayToRawRepresentation(permissions)}]._id]) > 0)
-      || count(array::intersects(${prefix}permission_v2, ${arrayToRawRepresentation(permissions)})) > 0)`
+      return `array::intersects(${prefix}permission_v2, ${arrayToRawRepresentation(permissions)})`
     }
     // In standard context, use references() function
-    return `(references(*[_type == 'permission' && railcontent_id in ${arrayToRawRepresentation(permissions)}]._id) ||
-     count(array::intersects(permission_v2, ${arrayToRawRepresentation(permissions)})) > 0)`
+    return `array::intersects(permission_v2, ${arrayToRawRepresentation(permissions)})`
   }
 
   /**

package/test/reporting.test.js

@@ -0,0 +1,132 @@
+/**
+ * Tests for Reporting Service
+ *
+ * Note: These tests use mocked HTTP calls. For integration tests,
+ * ensure the backend API is running at the configured baseUrl.
+ */
+
+import { initializeTestService } from './initializeTests.js'
+import {
+  reportContent,
+  reportForumPost,
+  submitReport
+} from '../src/services/reporting/reporting.ts'
+
+// Mock HttpClient to avoid actual API calls in tests
+jest.mock('../src/infrastructure/http/HttpClient', () => {
+  return {
+    HttpClient: jest.fn().mockImplementation(() => ({
+      post: jest.fn().mockResolvedValue({
+        report_id: 12345,
+        message: 'Report submitted successfully',
+        is_duplicate: false
+      })
+    }))
+  }
+})
+
+describe('Reporting Service', () => {
+  beforeEach(() => {
+    initializeTestService()
+  })
+
+  describe('reportContent', () => {
+    test('should submit a content report with video_not_working category', async () => {
+      const result = await reportContent({
+        contentId: 123,
+        category: 'video_not_working',
+        description: 'Video freezes at 2:30'
+      })
+
+      expect(result).toBeDefined()
+      expect(result.report_id).toBe(12345)
+      expect(result.message).toBe('Report submitted successfully')
+      expect(result.is_duplicate).toBe(false)
+    })
+
+    test('should submit a content report without description', async () => {
+      const result = await reportContent({
+        contentId: 456,
+        category: 'incorrect_content'
+      })
+
+      expect(result).toBeDefined()
+      expect(result.report_id).toBeDefined()
+    })
+
+    test('should submit a content report with other category and description', async () => {
+      const result = await reportContent({
+        contentId: 789,
+        category: 'other',
+        description: 'Audio quality is very poor'
+      })
+
+      expect(result).toBeDefined()
+      expect(result.report_id).toBeDefined()
+      expect(result.message).toBeDefined()
+    })
+  })
+
+  describe('reportForumPost', () => {
+    test('should submit a forum post report with required brand', async () => {
+      const result = await reportForumPost({
+        postId: 555,
+        brand: 'drumeo',
+        category: 'spam'
+      })
+
+      expect(result).toBeDefined()
+      expect(result.report_id).toBeDefined()
+    })
+
+    test('should submit a forum post report with description', async () => {
+      const result = await reportForumPost({
+        postId: 666,
+        brand: 'pianote',
+        category: 'harassment',
+        description: 'User is harassing other members'
+      })
+
+      expect(result).toBeDefined()
+      expect(result.report_id).toBeDefined()
+      expect(result.message).toBeDefined()
+    })
+  })
+
+  describe('submitReport', () => {
+    test('should submit a generic report for content', async () => {
+      const result = await submitReport({
+        reportableType: 'content',
+        reportableId: 999,
+        category: 'technical_issue',
+        description: 'Page not loading'
+      })
+
+      expect(result).toBeDefined()
+      expect(result.report_id).toBeDefined()
+    })
+
+    test('should submit a generic report for playlist', async () => {
+      const result = await submitReport({
+        reportableType: 'playlist',
+        reportableId: 777,
+        category: 'incorrect_content'
+      })
+
+      expect(result).toBeDefined()
+      expect(result.report_id).toBeDefined()
+    })
+
+    test('should submit a generic report for comment', async () => {
+      const result = await submitReport({
+        reportableType: 'comment',
+        reportableId: 888,
+        category: 'inappropriate',
+        description: 'Comment contains offensive language'
+      })
+
+      expect(result).toBeDefined()
+      expect(result.report_id).toBeDefined()
+    })
+  })
+})