Dead Code Analysis Report¶

Note: This page is a point-in-time status snapshot and may be out of date relative to the current source. PRs welcome.

Date: November 25, 2025 Status: Analysis Complete Codebase: /apps/web/src/

📊 Executive Summary¶

This comprehensive analysis identified dead code across the Mentat Protocol web application. The codebase contains approximately 42 source files with several areas requiring cleanup and consolidation.

Key Findings: - ~250-300 lines of truly dead code can be safely removed - ~1,100 lines of skeleton/planned code (should be kept but clearly marked) - 10+ unused API wrapper functions (bypassed by direct apiClient calls) - 3-4 duplicate type definitions across multiple files - 1 completely unused component (AuthModal.vue)

🎯 Priority Actions¶

High Priority - Safe to Delete¶

✅ Unused API Functions (~100 lines)
fetchUserMarkets() - Never imported
fetchDraftDetail() - Components use mockApi
createDraft() - Never imported
updateDraft() - Never imported
fetchCuratorQueue() - Direct apiClient used
claimDraft() - Direct apiClient used
approveDraft() - Direct apiClient used
requestChanges() - Direct apiClient used
rejectDraft() - Direct apiClient used
fetchDraftActions() - Never imported
✅ AuthModal.vue (~70 lines)
Never imported in any component
Only mentioned in documentation
✅ Unused Helper Functions (~15 lines)
getNetworkBadgeText() - Never imported
isTestnetEnvironment() - Never imported
✅ validateDraft() in aiAgents.ts
Exported but never imported

Medium Priority - Consolidate¶

Duplicate Type Definitions (~100 lines)
MarketState (3 different definitions)
Outcome/MarketOutcome (3 versions)
MarketDetail (2 versions)
ProofJob & CreatorMetric (duplicated in api.ts)
Documentation Location
Move stores/README.md to /docs

Low Priority - Keep but Mark Clearly¶

Skeleton Code (Planned Features)
services/solanaProgram.ts (480 lines)
composables/useSolana.ts (118 lines)
stores/trading.ts (499 lines)
Add clear status markers/comments

🔍 Detailed Findings¶

1. UNUSED FILES¶

1.1 Skeleton Services (KEEP - Future Use)¶

File: src/services/solanaProgram.ts (480 lines)

// Status: SKELETON - Awaiting Solana program deployment
// TODO: Implement once programs deployed to devnet

- Why Unused: Solana programs not yet deployed - Status: 13 TODO comments throughout - Action: KEEP - Add status comment at top of file

File: src/composables/useSolana.ts (118 lines)

// Status: READY - Not yet integrated
// TODO: Integrate for blockchain RPC calls

- Why Unused: Not integrated into components yet - Action: KEEP - Will be needed for blockchain features

File: src/stores/trading.ts (499 lines)

// Status: SKELETON - Awaiting backend integration
// All methods throw "not yet implemented" errors

- Why Unused: Only in README.md, not in components - Action: KEEP - Part of planned trading features

1.2 Truly Unused (DELETE)¶

File: src/components/AuthModal.vue (~70 lines) - Why Dead: Never imported, authentication simplified to wallet-only - Evidence: Only found in README.md - Action: DELETE - Auth flow clarified in AUTH-FLOW-CLARIFICATION.md

File: src/stores/README.md (448 lines) - Why Wrong Location: Documentation in source directory - Action: MOVE to /docs directory

2. UNUSED EXPORTS¶

2.1 API Service (`src/services/api.ts`)¶

Function: fetchUserMarkets() (line 63)

export async function fetchUserMarkets(userId: string): Promise<MarketListItem[]>

- Status: Exported but never imported - Usage: None - Action: DELETE

Function: fetchDraftDetail() (line 87)

export async function fetchDraftDetail(id: string): Promise<DraftDetail>

- Status: Components use mockApi.getDraftDetail() instead - Usage: None - Action: DELETE

Function: createDraft() (line 92)

export async function createDraft(draft: any): Promise<any>

- Status: Never imported - Usage: None - Action: DELETE

Function: updateDraft() (line 97)

export async function updateDraft(id: string, draft: any): Promise<any>

- Status: Never imported - Usage: None - Action: DELETE

Function: fetchCuratorQueue() (line 118)

export async function fetchCuratorQueue(): Promise<DraftSummary[]>

- Status: CuratorConsole uses direct apiClient.get() calls - Usage: None - Action: DELETE or refactor CuratorConsole to use it

Function: claimDraft() (line 125)

export async function claimDraft(draftId: string, curatorId: string): Promise<void>

- Status: CuratorConsole uses direct apiClient - Usage: None - Action: DELETE

Function: approveDraft() (line 130)

export async function approveDraft(draftId: string, curatorId: string, comments: string): Promise<void>

- Status: Never imported - Usage: None - Action: DELETE

Function: requestChanges() (line 138)

export async function requestChanges(draftId: string, curatorId: string, changes: any[], comments: string): Promise<void>

- Status: Never imported - Usage: None - Action: DELETE

Function: rejectDraft() (line 149)

export async function rejectDraft(draftId: string, curatorId: string, reason: string): Promise<void>

- Status: Never imported - Usage: None - Action: DELETE

Function: fetchDraftActions() (line 156)

export async function fetchDraftActions(draftId: string): Promise<CurationAction[]>

- Status: Never imported - Usage: None - Action: DELETE

2.2 Network Config (`src/config/network.ts`)¶

Function: getNetworkBadgeText() (line 57)

export function getNetworkBadgeText(network: NetworkConfig): string

- Status: Exported but never imported - Usage: Components use network.displayName directly - Action: DELETE

Function: isTestnetEnvironment() (line 65)

export function isTestnetEnvironment(): boolean

- Status: Exported but never imported - Usage: Components use network.isTestnet directly - Action: DELETE

2.3 AI Agents (`src/services/aiAgents.ts`)¶

Function: validateDraft() (line 130)

export async function validateDraft(draft: MarketDraft): Promise<ValidationResult>

- Status: Exported but never imported - Usage: None - Action: DELETE if not planning to use

3. DUPLICATE TYPE DEFINITIONS¶

3.1 MarketState Type (3 Definitions!)¶

Location 1: src/types/index.ts (line 1)

export type MarketState = 'draft' | 'pending' | 'active' | 'locked' | 'resolved' | 'invalid' | 'disputed';

Location 2: src/types/api.ts (line 47)

export type MarketState = 'draft' | 'pending_launch' | 'active' | 'locked' | 'resolved' | 'invalid' | 'disputed';

Note: Different - has pending_launch instead of pending

Location 3: src/types/trading.ts (line 29)

export enum MarketState {
  OPEN = 0,
  LOCKED = 1,
  RESOLVED = 2,
  DISPUTED = 3,
  INVALID = 4,
  CLOSED = 5,
}

Note: Completely different - enum with numeric values for on-chain

Issue: Three conflicting definitions Action: - Keep api.ts version as source of truth (matches backend) - Update types/index.ts to re-export from api.ts - Keep trading.ts enum separate (it's for on-chain state)

3.2 Outcome Interface (3 Versions)¶

Location 1: src/types/index.ts (line 10)

export interface Outcome {
  index: number;
  text: string;
  odds?: number;
}

Location 2: src/types/api.ts (line 49)

export interface MarketOutcome {
  id: string;
  text: string;
  order: number;
}

Location 3: src/types/trading.ts (line 38)

export interface Outcome {
  index: number;
  text: string;
  price: number;
  shares: bigint;
}

Issue: Three different structures for outcomes Action: - Use MarketOutcome from api.ts for API responses - Create frontend Outcome type that extends/adapts MarketOutcome - Keep trading.ts Outcome separate (has on-chain data)

3.3 MarketDetail Interface (2 Versions)¶

Location 1: src/types/index.ts (line 34)

export interface MarketDetail extends Market {
  description: string;
  sources: Source[];
  trading_data?: TradingData;
  participants?: number;
  volume?: number;
  created_at?: string;
}

Location 2: src/types/api.ts (line 83)

export interface MarketDetail {
  id: string;
  question_text: string;
  description?: string;
  // ... 18 more fields
}

Issue: Two different structures Action: - Keep api.ts version as source of truth - Use adapters.ts to transform to frontend version - Consider removing types/index.ts version

3.4 ProofJob & CreatorMetric (Duplicated)¶

Location 1: src/types/index.ts (lines 119, 128)

export interface ProofJob {
  id: string;
  market_id: string;
  // ...
}

export interface CreatorMetric {
  total_markets: number;
  active_markets: number;
  // ...
}

Location 2: src/services/api.ts (lines 168, 177)

interface ProofJob {
  id: string;
  market_id: string;
  // ... (identical)
}

interface CreatorMetric {
  total_markets: number;
  active_markets: number;
  // ... (identical)
}

Issue: Exact duplicates Action: Remove from services/api.ts, import from types/index.ts

4. ARCHITECTURE INCONSISTENCY¶

CuratorConsole Direct API Calls¶

File: src/views/CuratorConsole.vue

Issue: Uses direct apiClient.get() calls instead of wrapper functions:

// Line 45 - Direct call
const response = await apiClient.get<DraftSummary[]>('/curator/queue');

// Should use:
import { fetchCuratorQueue } from '@/services/api';
const drafts = await fetchCuratorQueue();

Other direct calls: - Line 61: apiClient.post('/curator/claim') - Line 89: apiClient.post('/curator/approve') - Line 104: apiClient.post('/curator/request-changes')

Action: - Option A: Refactor CuratorConsole to use api.ts wrapper functions - Option B: Delete unused wrapper functions from api.ts (current approach)

Recommendation: Option A is better for consistency, but Option B is pragmatic given current state.

5. COMMENTED CODE¶

Anchor Imports (Keep - Planned)¶

File: src/services/solanaProgram.ts

// Lines 2-3
// import { AnchorProvider, Program, Idl } from '@coral-xyz/anchor';
// import type { MarketFactory } from '../idl/market_factory';

// Lines 46-47
// private marketFactoryProgram: Program<MarketFactory> | null = null;
// private marketSettlementProgram: Program | null = null;

// Lines 64-68
// if (!this.marketFactoryProgram || !this.marketSettlementProgram) {
//   throw new Error('Solana programs not initialized. Call initialize() first.');
// }

Status: Waiting for Solana program deployment Action: KEEP - Add clear TODO comment explaining why commented

6. ROUTES ANALYSIS¶

File: src/router/index.ts

All routes are properly linked and accessible: - ✅ / → DiscoveryHub (home page) - ✅ /market/:id → MarketDetail (linked from cards) - ✅ /create → CreatorStudio (header nav) - ✅ /curate → CuratorConsole (header nav) - ✅ /proofs → ProofSubmission (header nav) - ✅ /account → AccountDashboard (header nav - removed from UI but route exists)

Note: /account route exists but link was removed from header in auth flow simplification. Consider if this route should be deleted or re-added.

7. UNUSED CSS¶

File: src/styles/global.css

All CSS classes are actively used: - ✅ .card, .card__title, .card__meta - Used in all views - ✅ .grid, .grid.two, .grid.three - Layout grids - ✅ .pill, .pill.draft, etc. - Status badges - ✅ button.cta, button.ghost - Buttons - ✅ .view-content, .view-header - View containers

Action: No cleanup needed

8. EMPTY DIRECTORIES¶

Directory: src/assets/ - Status: Empty (only . and ..) - Action: Consider adding .gitkeep or removing

📈 Cleanup Impact¶

Immediate Removals¶

Type	Lines	Files
Unused API functions	~100	1
AuthModal component	~70	1
Unused helpers	~15	1
validateDraft function	~20	1
Total	~205	4

Consolidations¶

Type	Lines Affected	Files
MarketState types	~30	3
Outcome interfaces	~40	3
MarketDetail interfaces	~60	2
Duplicate types	~20	2
Total	~150	10

Grand Total¶

Lines to remove: ~205
Lines to consolidate: ~150
Total cleanup: ~355 lines (8-9% of codebase)
Skeleton code to keep: ~1,100 lines

✅ Action Plan¶

Phase 1: Safe Deletions (Start Here)¶

✅ Delete unused API wrapper functions
✅ Delete AuthModal.vue component
✅ Delete unused helper functions from network.ts
✅ Delete validateDraft() from aiAgents.ts
✅ Move stores/README.md to /docs

Phase 2: Type Consolidation¶

⏳ Consolidate MarketState definitions
⏳ Consolidate Outcome interfaces
⏳ Consolidate MarketDetail interfaces
⏳ Remove duplicate ProofJob/CreatorMetric from api.ts

Phase 3: Documentation¶

⏳ Add status comments to skeleton files
⏳ Update TODOs with issue tracking numbers
⏳ Create PLANNED-FEATURES.md document

🎯 Files Requiring Status Markers¶

Add clear markers to skeleton code:

/**
 * IMPLEMENTATION STATUS: SKELETON
 *
 * This service is waiting for:
 * - Solana programs to be deployed to devnet
 * - Program IDL files to be generated
 * - Program IDs to be configured
 *
 * Once ready, uncomment Anchor imports and implement methods.
 *
 * Tracking: Issue #XXX
 */

Files needing markers: 1. src/services/solanaProgram.ts 2. src/composables/useSolana.ts 3. src/stores/trading.ts 4. src/types/trading.ts

🔧 Testing After Cleanup¶

Build Test¶

npm run build
# Should succeed with no errors

Type Check¶

npx tsc --noEmit
# Should pass with no errors

Runtime Test¶

npm run dev
# Test all routes still work:
# - / (home)
# - /market/:id
# - /create
# - /curate
# - /proofs

Features to Verify¶

✅ Market discovery works
✅ Market detail pages load
✅ Creator studio functions
✅ Curator console functions
✅ Proof submission works
✅ Network indicators show correctly

📊 Conclusion¶

The codebase is well-structured with most "dead code" actually being planned features waiting for backend/blockchain implementation.

True dead code: ~205 lines (can be safely removed) Duplicate code: ~150 lines (should be consolidated) Skeleton code: ~1,100 lines (keep but mark clearly)

Primary issues: 1. CuratorConsole bypasses API wrapper layer 2. Multiple conflicting type definitions 3. One truly unused component (AuthModal) 4. Lack of clear status markers on skeleton code

Recommended approach: - Delete true dead code (Phase 1) - Consolidate types (Phase 2) - Add clear markers (Phase 3) - Create PLANNED-FEATURES.md to track skeleton code

Last Updated: November 25, 2025 Analysis Tool: Manual + grep + file structure analysis Confidence: High (95%+)

Dead Code Analysis Report¶

📊 Executive Summary¶

🎯 Priority Actions¶

High Priority - Safe to Delete¶

Medium Priority - Consolidate¶

Low Priority - Keep but Mark Clearly¶

🔍 Detailed Findings¶

1. UNUSED FILES¶

1.1 Skeleton Services (KEEP - Future Use)¶

1.2 Truly Unused (DELETE)¶

2. UNUSED EXPORTS¶

2.1 API Service (src/services/api.ts)¶

2.2 Network Config (src/config/network.ts)¶

2.3 AI Agents (src/services/aiAgents.ts)¶

3. DUPLICATE TYPE DEFINITIONS¶

3.1 MarketState Type (3 Definitions!)¶

3.2 Outcome Interface (3 Versions)¶

3.3 MarketDetail Interface (2 Versions)¶

3.4 ProofJob & CreatorMetric (Duplicated)¶

4. ARCHITECTURE INCONSISTENCY¶

CuratorConsole Direct API Calls¶

5. COMMENTED CODE¶

Anchor Imports (Keep - Planned)¶

6. ROUTES ANALYSIS¶

7. UNUSED CSS¶

8. EMPTY DIRECTORIES¶

📈 Cleanup Impact¶

Immediate Removals¶

Consolidations¶

Grand Total¶

✅ Action Plan¶

Phase 1: Safe Deletions (Start Here)¶

Phase 2: Type Consolidation¶

Phase 3: Documentation¶

🎯 Files Requiring Status Markers¶

🔧 Testing After Cleanup¶

Build Test¶

Type Check¶

Runtime Test¶

Features to Verify¶

📊 Conclusion¶

2.1 API Service (`src/services/api.ts`)¶

2.2 Network Config (`src/config/network.ts`)¶

2.3 AI Agents (`src/services/aiAgents.ts`)¶