musora-content-services 2.107.3 → 2.107.5

package/.claude/settings.local.json

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

package/CHANGELOG.md

@@ -2,6 +2,22 @@
 
 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.107.5](https://github.com/railroadmedia/musora-content-services/compare/v2.107.4...v2.107.5) (2025-12-29)
+
+
+### Bug Fixes
+
+* establishes a positive progress validation ([#672](https://github.com/railroadmedia/musora-content-services/issues/672)) ([e9bc211](https://github.com/railroadmedia/musora-content-services/commit/e9bc211659262b282e1073c2746c3c8824371c35))
+
+### [2.107.4](https://github.com/railroadmedia/musora-content-services/compare/v2.107.1...v2.107.4) (2025-12-22)
+
+
+### Bug Fixes
+
+* **AGI:** contentType optional ([deae293](https://github.com/railroadmedia/musora-content-services/commit/deae2934c26ea0b9ee4d68c736fbb5753616a5e9))
+* **AGI:** return children on AGI lessons functions ([#671](https://github.com/railroadmedia/musora-content-services/issues/671)) ([92ed791](https://github.com/railroadmedia/musora-content-services/commit/92ed791ce563aa4f1bf16cb41d2b6fe421504064))
+* **onboarding:** call BE for recommendation ([#670](https://github.com/railroadmedia/musora-content-services/issues/670)) ([a876736](https://github.com/railroadmedia/musora-content-services/commit/a8767363ff3428638bde477cd6c30e5bf61179b2))
+
 ### [2.107.3](https://github.com/railroadmedia/musora-content-services/compare/v2.107.2...v2.107.3) (2025-12-19)
 
 

package/package.json

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

package/src/index.d.ts

@@ -155,6 +155,7 @@ import {
 	createPost,
 	deletePost,
 	fetchCommunityGuidelines,
+	fetchPost,
 	fetchPosts,
 	jumpToPost,
 	likePost,
@@ -168,6 +169,7 @@ import {
 	deleteThread,
 	fetchFollowedThreads,
 	fetchLatestThreads,
+	fetchThread,
 	fetchThreads,
 	followThread,
 	lockThread,
@@ -310,6 +312,14 @@ import {
 	jumpToContinueContent
 } from './services/sanity.js';
 
+import {
+	generateCommentUrl,
+	generateContentUrl,
+	generateContentUrlWithDomain,
+	generateForumPostUrl,
+	generatePlaylistUrl
+} from './services/urlBuilder.ts';
+
 import {
 	confirmEmailChange,
 	deleteAccount,
@@ -543,6 +553,7 @@ declare module 'musora-content-services' {
 		fetchPlayAlongsCount,
 		fetchPlaylist,
 		fetchPlaylistItems,
+		fetchPost,
 		fetchPosts,
 		fetchRecent,
 		fetchRecentActivitiesActiveTabs,
@@ -563,6 +574,7 @@ declare module 'musora-content-services' {
 		fetchSongById,
 		fetchSongsInProgress,
 		fetchTabData,
+		fetchThread,
 		fetchThreads,
 		fetchTopComment,
 		fetchTopLevelParentId,
@@ -580,6 +592,11 @@ declare module 'musora-content-services' {
 		findIncompleteLesson,
 		followThread,
 		generateAuthSessionUrl,
+		generateCommentUrl,
+		generateContentUrl,
+		generateContentUrlWithDomain,
+		generateForumPostUrl,
+		generatePlaylistUrl,
 		getActiveDiscussions,
 		getActivePath,
 		getAllCompleted,

package/src/index.js

@@ -159,6 +159,7 @@ import {
 	createPost,
 	deletePost,
 	fetchCommunityGuidelines,
+	fetchPost,
 	fetchPosts,
 	jumpToPost,
 	likePost,
@@ -172,6 +173,7 @@ import {
 	deleteThread,
 	fetchFollowedThreads,
 	fetchLatestThreads,
+	fetchThread,
 	fetchThreads,
 	followThread,
 	lockThread,
@@ -314,6 +316,14 @@ import {
 	jumpToContinueContent
 } from './services/sanity.js';
 
+import {
+	generateCommentUrl,
+	generateContentUrl,
+	generateContentUrlWithDomain,
+	generateForumPostUrl,
+	generatePlaylistUrl
+} from './services/urlBuilder.ts';
+
 import {
 	confirmEmailChange,
 	deleteAccount,
@@ -542,6 +552,7 @@ export {
 	fetchPlayAlongsCount,
 	fetchPlaylist,
 	fetchPlaylistItems,
+	fetchPost,
 	fetchPosts,
 	fetchRecent,
 	fetchRecentActivitiesActiveTabs,
@@ -562,6 +573,7 @@ export {
 	fetchSongById,
 	fetchSongsInProgress,
 	fetchTabData,
+	fetchThread,
 	fetchThreads,
 	fetchTopComment,
 	fetchTopLevelParentId,
@@ -579,6 +591,11 @@ export {
 	findIncompleteLesson,
 	followThread,
 	generateAuthSessionUrl,
+	generateCommentUrl,
+	generateContentUrl,
+	generateContentUrlWithDomain,
+	generateForumPostUrl,
+	generatePlaylistUrl,
 	getActiveDiscussions,
 	getActivePath,
 	getAllCompleted,

package/src/services/contentProgress.js

@@ -469,6 +469,13 @@ export async function contentStatusReset(contentId, collection = null) {
 }
 
 async function saveContentProgress(contentId, collection, progress, currentSeconds) {
+
+  // filter out contentIds that are setting progress lower than existing
+  const contentIdProgress = await getProgressDataByIds([contentId], collection)
+  if (progress <= contentIdProgress[contentId].progress) {
+    return
+  }
+
   const response = await db.contentProgress.recordProgress(
     contentId,
     collection,
@@ -483,6 +490,15 @@ async function saveContentProgress(contentId, collection, progress, currentSecon
   const hierarchy = await getHierarchy(contentId, collection)
 
   const bubbledProgresses = await bubbleProgress(hierarchy, contentId, collection)
+
+  // filter out contentIds that are setting progress lower than existing
+  const existingProgresses = await getProgressDataByIds(Object.keys(bubbledProgresses), collection)
+  for (const [bubbledContentId, bubbledProgress] of Object.entries(bubbledProgresses)) {
+    if (bubbledProgress <= existingProgresses[bubbledContentId].progress) {
+      delete bubbledProgresses[bubbledContentId]
+    }
+  }
+
   // BE bubbling/trickling currently does not work, so we utilize non-tentative pushing when learning path collection
   await db.contentProgress.recordProgressMany(bubbledProgresses, collection, collection?.type !== COLLECTION_TYPE.LEARNING_PATH)
 

package/src/services/forums/posts.ts

@@ -23,8 +23,26 @@ export interface CreatePostParams {
  * @throws {HttpError} - If the request fails.
  */
 export async function createPost(threadId: number, params: CreatePostParams): Promise<ForumPost> {
+  const { generateForumPostUrl } = await import('../urlBuilder.ts')
+  const { fetchThread } = await import('./threads.ts')
+
+  // Fetch thread to get category_id for URL generation
+  const thread = await fetchThread(threadId, params.brand)
+
+  // Generate forum post URL
+  const contentUrl = generateForumPostUrl({
+    brand: params.brand,
+    thread: {
+      category_id: thread.category_id,
+      id: threadId
+    }
+  }, false)
+
   const httpClient = new HttpClient(globalConfig.baseUrl)
-  return httpClient.post<ForumPost>(`${baseUrl}/v1/threads/${threadId}/posts`, params)
+  return httpClient.post<ForumPost>(`${baseUrl}/v1/threads/${threadId}/posts`, {
+    ...params,
+    content_url: contentUrl
+  })
 }
 
 /**
@@ -40,6 +58,19 @@ export async function updatePost(postId: number, params: CreatePostParams): Prom
   return httpClient.put<ForumPost>(`${baseUrl}/v1/posts/${postId}`, params)
 }
 
+/**
+ * Fetches a single forum post by ID.
+ *
+ * @param {number} postId - The ID of the post to fetch.
+ * @param {string} brand - The brand context (e.g., "drumeo", "singeo").
+ * @returns {Promise<ForumPost>} - A promise that resolves to the forum post.
+ * @throws {HttpError} - If the HTTP request fails.
+ */
+export async function fetchPost(postId: number, brand: string): Promise<ForumPost> {
+  const httpClient = new HttpClient(globalConfig.baseUrl)
+  return httpClient.get<ForumPost>(`${baseUrl}/v1/posts/${postId}?brand=${brand}`)
+}
+
 export interface FetchPostParams {
   page?: number
   limit?: number
@@ -91,8 +122,25 @@ export async function fetchPosts(
  * @throws {HttpError} - If the request fails.
  */
 export async function likePost(postId: number, brand: string): Promise<void> {
+  const { generateForumPostUrl } = await import('../urlBuilder.ts')
+
+  // Fetch post to get thread info for URL generation
+  const post = await fetchPost(postId, brand)
+
+  // Generate forum post URL
+  const contentUrl = generateForumPostUrl({
+    brand,
+    thread: {
+      category_id: post.thread.category_id,
+      id: post.thread.id
+    }
+  }, false)
+
   const httpClient = new HttpClient(globalConfig.baseUrl)
-  return httpClient.post<void>(`${baseUrl}/v1/posts/${postId}/likes`, { brand })
+  return httpClient.post<void>(`${baseUrl}/v1/posts/${postId}/likes`, {
+    brand,
+    content_url: contentUrl
+  })
 }
 
 /**

package/src/services/forums/threads.ts

@@ -89,6 +89,19 @@ export async function markThreadAsRead(threadId: number, brand: string): Promise
   return httpClient.put<void>(`${baseUrl}/v1/threads/${threadId}/read?brand=${brand}`, {})
 }
 
+/**
+ * Fetches a single forum thread by ID.
+ *
+ * @param {number} threadId - The ID of the thread to fetch.
+ * @param {string} brand - The brand context (e.g., "drumeo", "singeo").
+ * @returns {Promise<ForumThread>} - A promise that resolves to the forum thread.
+ * @throws {HttpError} - If the HTTP request fails.
+ */
+export async function fetchThread(threadId: number, brand: string): Promise<ForumThread> {
+  const httpClient = new HttpClient(globalConfig.baseUrl)
+  return httpClient.get<ForumThread>(`${baseUrl}/v1/threads/${threadId}?brand=${brand}`)
+}
+
 export interface FetchThreadParams {
   is_followed?: boolean,
   page?: number,

package/src/services/railcontent.js

@@ -249,8 +249,38 @@ export async function restoreComment(commentId) {
  * @returns {Promise<*|null>}
  */
 export async function replyToComment(commentId, comment) {
+  const { generateCommentUrl } = await import('./urlBuilder.ts')
+  const { fetchByRailContentIds } = await import('./sanity.js')
+
+  // Fetch parent comment to get content info
+  const parentComment = await fetchComment(commentId)
+
+  if (!parentComment?.content) {
+    const url = `/api/content/v1/comments/${commentId}/reply`
+    return await POST(url, { comment })
+  }
+
+  // Fetch content from Sanity to get parentId and correct type
+  const contents = await fetchByRailContentIds([parentComment.content.id])
+  const content = contents?.[0]
+
+  // Generate content URL
+  const contentUrl = content ? generateCommentUrl({
+    id: commentId,
+    content: {
+      id: content.id,
+      type: content.type,
+      parentId: content.parentId || content.parent_id,
+      brand: content.brand
+    }
+  }, false) : null
+
+  const data = {
+    comment: comment,
+    ...(contentUrl && { content_url: contentUrl })
+  }
   const url = `/api/content/v1/comments/${commentId}/reply`
-  return await POST(url, { comment })
+  return await POST(url, data)
 }
 
 /**
@@ -259,8 +289,28 @@ export async function replyToComment(commentId, comment) {
  * @returns {Promise<*|null>}
  */
 export async function createComment(railcontentId, comment) {
+  const { generateContentUrl } = await import('./urlBuilder.ts')
+  const { fetchByRailContentIds } = await import('./sanity.js')
+
+  // Fetch content to get type and brand info
+  const contents = await fetchByRailContentIds([railcontentId])
+  const content = contents?.[0]
+
+  // Generate content URL
+  const contentUrl = content ? generateContentUrl({
+    id: content.id,
+    type: content.type,
+    parentId: content.parentId || content.parent_id,
+    brand: content.brand
+  }) : null
+
+  const data = {
+    comment: comment,
+    content_id: railcontentId,
+    ...(contentUrl && { content_url: contentUrl })
+  }
   const url = `/api/content/v1/comments/store`
-  return await POST(url, { comment, content_id: railcontentId })
+  return await POST(url, data)
 }
 
 /**
@@ -286,8 +336,35 @@ export async function unassignModeratorToComment(commentId) {
  * @returns {Promise<*|null>}
  */
 export async function likeComment(commentId) {
+  const { generateCommentUrl } = await import('./urlBuilder.ts')
+  const { fetchByRailContentIds } = await import('./sanity.js')
+
+  // Fetch comment to get content info
+  const comment = await fetchComment(commentId)
+
+  if (!comment?.content) {
+    const url = `/api/content/v1/comments/${commentId}/like`
+    return await POST(url, null)
+  }
+
+  // Fetch content from Sanity to get parentId and correct type
+  const contents = await fetchByRailContentIds([comment.content.id])
+  const content = contents?.[0]
+
+  // Generate content URL
+  const contentUrl = content ? generateCommentUrl({
+    id: commentId,
+    content: {
+      id: content.id,
+      type: content.type,
+      parentId: content.parentId || content.parent_id,
+      brand: content.brand
+    }
+  }, false) : null
+
   const url = `/api/content/v1/comments/${commentId}/like`
-  return await POST(url, null)
+  const data = contentUrl ? { content_url: contentUrl } : {}
+  return await POST(url, data)
 }
 
 /**

package/src/services/reporting/reporting.ts

@@ -11,6 +11,10 @@ import { HttpClient } from '../../infrastructure/http/HttpClient'
 import { globalConfig } from '../config.js'
 import { ReportResponse, ReportableType, IssueTypeMap, ReportIssueOption } from './types'
 import { Brands } from '../../lib/brands'
+import { generateContentUrl, generatePlaylistUrl, generateForumPostUrl, generateCommentUrl } from '../urlBuilder.ts'
+import {fetchByRailContentId} from "../../index";
+import {fetchByRailContentIds} from "../sanity";
+import {addContextToContent} from "../contentAggregator";
 
 /**
  * Parameters for submitting a report with type-safe issue values
@@ -26,6 +30,14 @@ export type ReportParams<T extends ReportableType = ReportableType> = {
   details?: string
   /** Brand context (required: drumeo, pianote, guitareo, singeo, playbass) */
   brand: Brands | string
+  /** Full URL to the reported content (generated via urlBuilder) */
+  contentUrl?: string
+  /** Content data for URL generation (only needed if contentUrl not provided) */
+  content?: {
+    id: number
+    type: string
+    parentId?: number
+  }
 }
 
 /**
@@ -86,12 +98,68 @@ export async function report<T extends ReportableType>(
     requestBody.details = params.details
   }
 
-  const response = await httpClient.post<ReportResponse>(
+  // Generate content_url for reports (relative URL - backend adds domain)
+  if (params.type === 'content') {
+    // Fetch content and add navigateTo for courses/packs/etc
+    const contents = await addContextToContent(
+      fetchByRailContentIds,
+      [params.id],
+      { addNavigateTo: true }
+    )
+    const content = contents?.[0]
+
+    if (content) {
+      requestBody.content_url = generateContentUrl({
+        id: content.id,
+        type: content.type,
+        parentId: content.parentId || content.parent_id,
+        brand: content.brand,
+        navigateTo: content.navigateTo
+      })
+    }
+  } else if (params.type === 'playlist') {
+    requestBody.content_url = generatePlaylistUrl({
+      id: params.id
+    })
+  } else if (params.type === 'forum_post') {
+    const { fetchPost } = await import('../forums/posts.ts')
+    const post = await fetchPost(params.id, params.brand)
+
+    if (post?.thread) {
+      requestBody.content_url = generateForumPostUrl({
+        brand: params.brand,
+        thread: {
+          category_id: post.thread.category_id,
+          id: post.thread.id
+        }
+      })
+    }
+  } else if (params.type === 'comment') {
+    const { fetchComment } = await import('../railcontent.js')
+    const comment = await fetchComment(params.id)
+
+    if (comment?.content) {
+      const contents = await fetchByRailContentIds([comment.content.id])
+      const content = contents?.[0]
+
+      if (content) {
+        requestBody.content_url = generateCommentUrl({
+          id: comment.id,
+          content: {
+            id: content.id,
+            type: content.type,
+            parentId: content.parentId || content.parent_id,
+            brand: params.brand
+          }
+        })
+      }
+    }
+  }
+
+  return await httpClient.post<ReportResponse>(
     '/api/user-reports/v1/reports',
     requestBody
   )
-
-  return response
 }
 
 /**

package/src/services/sanity.js

@@ -927,7 +927,7 @@ export async function fetchLessonContent(railContentId, { addParent = false } =
     "instructor": ${instructorField},
     ${assignmentsField}
     video,
-    length_in_seconds,
+    "length_in_seconds": coalesce(soundslice[0].soundslice_length_in_second, length_in_seconds),
     mp3_no_drums_no_click_url,
     mp3_no_drums_yes_click_url,
     mp3_yes_drums_no_click_url,

package/src/services/urlBuilder.ts

@@ -0,0 +1,297 @@
+/**
+ * @module UrlBuilder
+ * @description URL generation for content across Musora platform
+ *
+ * This is the SINGLE SOURCE OF TRUTH for URL generation.
+ * Used by:
+ * - musora-platform-frontend (via import)
+ * - Mobile apps (via import)
+ * - Backend receives these URLs from frontend and stores them in DB
+ *
+ * Port of: musora-platform-frontend/src/shared/utils/content.utils.ts:generateContentUrl
+ * Related: musora-platform-backend/app/Modules/Content/Builders/UrlBuilder.php (deprecated fallback)
+ */
+
+import { globalConfig } from './config.js'
+import { Brands } from '../lib/brands.js'
+
+/**
+ * Brand type - accepts enum values or string
+ */
+export type Brand = Brands | string
+
+/**
+ * Parameters for generating content URLs
+ */
+export interface ContentUrlParams {
+  /** Content ID (required) */
+  id: number | string
+  /** Content type (required) */
+  type: string
+  /** Parent content ID (optional) */
+  parentId?: number
+  /** Navigation target (optional) */
+  navigateTo?: {
+    id: number
+  }
+  /** Brand (drumeo, pianote, guitareo, singeo, playbass) */
+  brand?: Brand
+}
+
+/**
+ * Forum post object for URL generation
+ */
+export interface ForumPostUrlParams {
+  /** Brand (drumeo, pianote, etc) */
+  brand: Brand
+  /** Thread information */
+  thread: {
+    /** Thread category ID */
+    category_id: number
+    /** Thread ID */
+    id: number
+  }
+}
+
+/**
+ * Playlist object for URL generation
+ */
+export interface PlaylistUrlParams {
+  /** Playlist ID */
+  id: number
+}
+
+/**
+ * Comment object for URL generation
+ */
+export interface CommentUrlParams {
+  /** Comment ID */
+  id: number
+  /** Content information */
+  content: {
+    /** Content ID */
+    id: number
+    /** Content type */
+    type: string
+    /** Parent content ID (optional) */
+    parentId?: number
+    /** Brand */
+    brand: Brand
+  }
+}
+
+/**
+ * Generate a frontend URL for content
+ *
+ * @param params - Content parameters
+ * @returns The generated URL path
+ *
+ * @example
+ * generateContentUrl({ id: 123, type: 'song', brand: 'drumeo' })
+ * // Returns: "/drumeo/songs/transcription/123"
+ *
+ * @example
+ * generateContentUrl({ id: 456, type: 'course-part', parentId: 789, brand: 'pianote' })
+ * // Returns: "/pianote/lessons/course/789/456"
+ *
+ * @example
+ * generateContentUrl({ id: 123, type: 'pack-bundle', navigateTo: { id: 456 }, brand: 'guitareo' })
+ * // Returns: "/guitareo/lessons/pack/123/456"
+ */
+export function generateContentUrl({
+  id,
+  type,
+  parentId,
+  navigateTo,
+  brand = 'drumeo',
+}: ContentUrlParams): string {
+  // Special case: method homepage
+  if (type === 'method') {
+    return `/${brand}/method`
+  }
+
+  // Return fallback if required params missing
+  if (!id || !type) {
+    return '#'
+  }
+
+  // Special cases that don't follow the standard pattern
+  if (type === 'live') {
+    return `/${brand}/lessons/${id}/live`
+  }
+
+  if (type === 'pack') {
+    return `/${brand}/lessons/pack/overview/${id}`
+  }
+
+  if (type === 'pack-bundle') {
+    if (navigateTo?.id) {
+      return `/${brand}/lessons/pack/${id}/${navigateTo.id}`
+    }
+    // Fallback to overview if navigateTo is missing
+    return `/${brand}/lessons/pack/overview/${id}`
+  }
+
+  // Helper function to build URL with common parameters
+  const buildUrl = (typeSegments: string[]): string => {
+    const contentId = navigateTo ? `${id}/${navigateTo.id}` : id
+    const parentSegment = parentId ? `/${parentId}` : ''
+    return `/${brand}/${typeSegments.join('/')}${parentSegment}/${contentId}`
+  }
+
+  // Determine page type (songs, method, or lessons)
+  const songTypes = [
+    'song',
+    'song-tutorial',
+    'song-tutorial-lesson',
+    'transcription',
+    'play-along',
+    'jam-track',
+  ]
+
+  const methodTypes = ['learning-path-v2', 'learning-path-lesson-v2']
+
+  let pageType: string
+  if (songTypes.includes(type)) {
+    pageType = 'songs'
+  } else if (methodTypes.includes(type)) {
+    pageType = 'method'
+  } else {
+    pageType = 'lessons'
+  }
+
+  // Content type routing - maps specific types to URL segments
+  const contentTypeRoutes: Record<string, string> = {
+    // Lesson types
+    'course-lesson': 'course',
+    'guided-course-lesson': 'course',
+    'guided-course': 'course',
+    'pack-bundle-lesson': 'pack',
+    'documentary-lesson': 'documentary',
+    'skill-pack-lesson': 'skill-pack',
+
+    // Method types
+    'learning-path-lesson-v2': 'lesson',
+    'learning-path-v2': 'lesson',
+
+    // Song types
+    song: 'transcription',
+    'song-tutorial': 'tutorial',
+    'song-tutorial-lesson': 'tutorial',
+  }
+
+  // Use specific route if available, otherwise fall back to type as-is
+  const contentTypeSegment = contentTypeRoutes[type] || type
+
+  return buildUrl([pageType, contentTypeSegment])
+}
+
+/**
+ * Generate a full URL with domain
+ *
+ * @param params - Content parameters (same as generateContentUrl)
+ * @returns Full URL with domain from globalConfig.frontendUrl
+ *
+ * @example
+ * generateContentUrlWithDomain({ id: 123, type: 'song' })
+ * // Returns: "https://www.musora.com/drumeo/songs/transcription/123"
+ */
+export function generateContentUrlWithDomain(params: ContentUrlParams): string {
+  const path = generateContentUrl(params)
+
+  if (path === '#') {
+    return '#'
+  }
+
+  return globalConfig.frontendUrl + path
+}
+
+/**
+ * Generate URL for a forum post
+ *
+ * @param post - Forum post object
+ * @param withDomain - Include domain from globalConfig.frontendUrl
+ * @returns Forum post URL
+ *
+ * @example
+ * generateForumPostUrl({ brand: 'drumeo', thread: { category_id: 12, id: 456 }})
+ * // Returns: "/drumeo/forums/threads/12/456"
+ */
+export function generateForumPostUrl(
+  post: ForumPostUrlParams,
+  withDomain: boolean = false
+): string {
+  const path = `/${post.brand}/forums/threads/${post.thread.category_id}/${post.thread.id}`
+
+  if (withDomain) {
+    return globalConfig.frontendUrl + path
+  }
+
+  return path
+}
+
+/**
+ * Generate URL for a user playlist
+ *
+ * @param playlist - Playlist object
+ * @param withDomain - Include domain from globalConfig.frontendUrl
+ * @returns Playlist URL
+ *
+ * @example
+ * generatePlaylistUrl({ id: 123 })
+ * // Returns: "/playlists/123"
+ */
+export function generatePlaylistUrl(
+  playlist: PlaylistUrlParams,
+  withDomain: boolean = false
+): string {
+  const path = `/playlists/${playlist.id}`
+
+  if (withDomain) {
+    return globalConfig.frontendUrl + path
+  }
+
+  return path
+}
+
+/**
+ * Generate URL for a comment (content URL with anchor)
+ *
+ * @param comment - Comment object
+ * @param withDomain - Include domain from globalConfig.frontendUrl
+ * @returns Comment URL with anchor
+ *
+ * @example
+ * generateCommentUrl({
+ *   id: 789,
+ *   content: { id: 123, type: 'song', brand: 'drumeo' }
+ * })
+ * // Returns: "/drumeo/songs/transcription/123#comment-789"
+ */
+export function generateCommentUrl(
+  comment: CommentUrlParams,
+  withDomain: boolean = false
+): string {
+  if (!comment.content) {
+    return '#'
+  }
+
+  const contentUrl = generateContentUrl({
+    id: comment.content.id,
+    type: comment.content.type,
+    parentId: comment.content.parentId,
+    brand: comment.content.brand,
+  })
+
+  if (contentUrl === '#') {
+    return '#'
+  }
+
+  const path = `${contentUrl}#comment-${comment.id}`
+
+  if (withDomain) {
+    return globalConfig.frontendUrl + path
+  }
+
+  return path
+}