Skip to content

Conversation

@hristoterezov
Copy link
Member

@hristoterezov hristoterezov commented Nov 3, 2025

Summary

  • Export random and transport modules from main entry point for consistent API access
  • Update CLAUDE.md to reflect current codebase state including TypeScript migration, test suite, and CI improvements

Changes

Export Consistency (index.ts)

All utility modules are now consistently exported from the main package entry point. Previously, random and transport were only available via subpath imports, creating inconsistency.

Users can now import everything from the main entry:

import { randomHexString, Transport, BrowserDetection } from '@jitsi/js-utils';

Subpath imports continue to work as an alternative:

import { randomHexString } from '@jitsi/js-utils/random';

Documentation Updates (CLAUDE.md)

Updated project documentation to accurately reflect:

  • Complete TypeScript migration (no longer hybrid JavaScript/TypeScript)
  • Comprehensive test suite with 100% coverage (182+ tests)
  • Enhanced CI workflow split into lint, test, and build jobs
  • Codecov integration for coverage tracking
  • Build output structure using dist/ directory
  • Removal of uri module
  • All available npm scripts and their purposes

Test plan

  • Build succeeds (npm run build)
  • Linter passes (npm run lint)
  • All tests pass
  • CI checks pass on PR

All utility modules should be consistently accessible from the main
package entry point to provide a unified API experience. Previously,
random and transport were only available via subpath imports, creating
inconsistency in how modules are accessed.
The documentation was outdated, referencing the project as a hybrid
JavaScript/TypeScript codebase when it has been fully migrated to
TypeScript. It also lacked information about the comprehensive test
suite, improved CI/CD workflow, and the recent removal of the uri module.

Updates include:
- Full TypeScript migration completion
- Comprehensive test suite with 100% coverage
- Split CI workflow (lint, test, build jobs)
- Codecov integration
- Updated build output structure (dist/ directory)
- Consistent module exports from main entry point
- Removal of uri module documentation
@codecov
Copy link

codecov bot commented Nov 3, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (d1d1fb8) to head (d35b81e).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #108   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            7         7           
  Lines         1870      1870           
  Branches        47        47           
=========================================
  Hits          1870      1870           

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d1d1fb8...d35b81e. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@hristoterezov hristoterezov merged commit bb1257f into master Nov 3, 2025
6 checks passed
@hristoterezov hristoterezov deleted the feat/consistent-exports branch November 3, 2025 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants