TS migration

[dubeyShivank]

Summary

This PR completes the full migration of the Millicast SDK from JavaScript to TypeScript, providing type safety, improved developer experience, and better IDE support for SDK consumers.

Key Changes

🔄 TypeScript Migration

• All SDK source files converted from .js to .ts
• Full type annotations for all classes, methods, and properties
• Strict TypeScript compilation with no errors
• Removed legacy JavaScript files from packages/millicast-sdk/src/

📦 Typed Event System

• Created MillicastEventEmitter - a generic typed event emitter class
• Added comprehensive event type interfaces:
• ViewerEvents - events for View (subscriber) class
• PublisherEvents - events for Publish class
• PeerConnectionEvents - WebRTC connection events
• PeerConnectionStatsEvents - statistics events
• SignalingEvents - WebSocket signaling events
• SDK consumers can now get full TypeScript intellisense for event handlers

📤 Standardized Exports

• Reorganized index.ts with clear sections:
• Core Modules: Director, Logger, PeerConnection, Signaling, Publish, View
• Named Exports: defaultApiEndpoint, signalingEvents, ConnectionType, webRTCEvents, VideoCodec, AudioCodec
• Type Exports: All TypeScript interfaces and types for SDK consumers
• 100% backwards compatible - all existing exports preserved

🧹 Code Quality Improvements

• Replaced all console.* calls with structured Logger abstraction
• Migrated ESLint to flat config (eslint.config.mjs) - removes deprecation warnings
• Fixed lint-staged configuration for pre-commit hooks
• Added new unit tests for FetchError, Signaling, and View classes

🏗️ Build & Tooling

• Updated Vite configuration for TypeScript builds
• Proper .d.ts type declaration file generation
• All demo projects updated to work with TypeScript SDK

Breaking Changes

None - This migration maintains full backwards compatibility with existing JavaScript/TypeScript consumers.

Testing
✅ 345 unit tests passing (3 skipped)
✅ All E2E tests passing
✅ TypeScript compilation clean
✅ ESLint passing
✅ Build succeeds for all 6 projects

---

[changeset-bot]

⚠️ No Changeset found

Latest commit: 0de4066

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[devin-ai-integration]

Devin Review found 1 new potential issue.

View 23 additional findings in Devin Review.

[devin-ai-integration]

🟡 View.onRtcDrmFetch has dead null check due to preceding fallback assignment

At View.ts:552, opts.headers is always assigned a non-null value via the || fallback: opts.headers = (opts.headers as Headers) || new Headers(). The very next line if (!opts.headers) at line 553 is therefore always false, making the opts.headers = new Headers() on line 554 dead code. While this doesn't cause a crash, it indicates the null-safety intent was incorrectly implemented. The original JS code (View.js:473-475) also had this pattern, but the TypeScript rewrite should have cleaned it up.

Suggested change

	opts.headers = (opts.headers as Headers) || new Headers();
	if (!opts.headers) {
	opts.headers = new Headers();
	}
	async onRtcDrmFetch (url: string, opts: RequestInit) {
	if (!opts.headers || !(opts.headers instanceof Headers)) {
	opts.headers = new Headers();
	}

---

Was this helpful? React with 👍 or 👎 to provide feedback.