From 99ab710e582a59666755a617ab89f1354cdf4cb3 Mon Sep 17 00:00:00 2001
From: taofeeq991 <taofeeqfolawiyo@gmail.com>
Date: Sun, 25 Oct 2026 13:53:51 +0100
Subject: [PATCH] fix: remove duplicate imports and dead code across
 module/courses (#520, #521, #522)

- credentials.ts: remove duplicate getValidToken import (#520)
- module page: remove redundant type-guard render blocks, keep single
  parseModuleContent call with discriminated-union narrowing (#522)
- types/course.ts: remove duplicate ModuleContent definition, duplicate
  Module.content property, and now-unused type guards / parseModuleContent
- package.json: #521 no change needed (duplicates not present)
---
 .../[courseId]/modules/[moduleId]/page.tsx    | 19 -----------
 src/lib/api/credentials.ts                    |  1 -
 src/types/course.ts                           | 34 -------------------
 3 files changed, 54 deletions(-)

diff --git a/src/app/courses/[courseId]/modules/[moduleId]/page.tsx b/src/app/courses/[courseId]/modules/[moduleId]/page.tsx
index adcf7f3..fff91f5 100644
--- a/src/app/courses/[courseId]/modules/[moduleId]/page.tsx
+++ b/src/app/courses/[courseId]/modules/[moduleId]/page.tsx
@@ -19,7 +19,6 @@ import {
 } from "lucide-react";
 import Link from "next/link";
 import { useState } from "react";
-import { isTextContent, isVideoContent, isInteractiveContent } from "@/types/course";
 
 export default function ModulePage({
   params,
@@ -109,24 +108,6 @@ export default function ModulePage({
           <p className="text-gray-500 mb-6 dark:text-gray-400">{module.description}</p>
 
           <div className="prose prose-gray max-w-none dark:prose-invert">
-            {/* #498 — Render module content with proper type handling */}
-            {isTextContent(module.content) && (
-              <div dangerouslySetInnerHTML={{ __html: DOMPurify.sanitize(module.content) }} />
-            )}
-            {isVideoContent(module.content) && (
-              <div className="aspect-video rounded-lg overflow-hidden">
-                <iframe
-                  src={module.content.url}
-                  title={module.title}
-                  className="w-full h-full"
-                  allowFullScreen
-                />
-              </div>
-            )}
-            {isInteractiveContent(module.content) && (
-              <div className="rounded-lg border border-gray-200 p-4">
-                <p className="text-sm text-gray-500">Interactive content</p>
-              </div>
             {content.type === "video" ? (
               <div className="aspect-video w-full bg-black rounded-lg overflow-hidden flex items-center justify-center mb-6">
                 {content.url ? (
diff --git a/src/lib/api/credentials.ts b/src/lib/api/credentials.ts
index 69afc30..b5f2dfa 100644
--- a/src/lib/api/credentials.ts
+++ b/src/lib/api/credentials.ts
@@ -22,7 +22,6 @@ export interface VerifyCredentialResult {
   verifiedAt?: string;
   error?: VerifyCredentialError;
 }
-import { getValidToken } from "./auth";
 
 /**
  * Fetch all credentials for the authenticated user.
diff --git a/src/types/course.ts b/src/types/course.ts
index 6f058cd..74f3414 100644
--- a/src/types/course.ts
+++ b/src/types/course.ts
@@ -13,15 +13,6 @@ export interface Course {
   createdAt: string;
 }
 
-// #498 — Content payload union: text is markdown/HTML string, video is a URL,
-// interactive is a config object. This replaces the previous `content: string`
-// which forced casting in parseModuleContent.
-export type ModuleContent =
-  | string // text/markdown content
-  | { type: "video"; url: string; duration?: number }
-  | { type: "interactive"; config: Record<string, unknown> };
-
-
 export interface Module {
   id: string;
   courseId: string;
@@ -29,36 +20,11 @@ export interface Module {
   description: string;
   order: number;
   contentType: "text" | "video" | "interactive";
-  content: ModuleContent;
   content: string | ModuleContent;
   estimatedMinutes: number;
   isCompleted?: boolean;
 }
 
-// #498 — Type guard for Module.content
-export function isTextContent(content: ModuleContent): content is string {
-  return typeof content === "string";
-}
-
-export function isVideoContent(
-  content: ModuleContent
-): content is { type: "video"; url: string; duration?: number } {
-  return typeof content === "object" && content !== null && "type" in content && content.type === "video";
-}
-
-export function isInteractiveContent(
-  content: ModuleContent
-): content is { type: "interactive"; config: Record<string, unknown> } {
-  return typeof content === "object" && content !== null && "type" in content && content.type === "interactive";
-}
-
-// #498 — Safe parser that returns string content or empty string
-export function parseModuleContent(content: ModuleContent): string {
-  if (isTextContent(content)) return content;
-  if (isVideoContent(content)) return content.url;
-  return "";
-}
-
 export interface CourseEnrollment {
   id: string;
   courseId: string;