Skip to content

feat!: compile to JS before publishing#791

Merged
arbrandes merged 1 commit intoopenedx:frontend-basefrom
arbrandes:frontend-base-compile-before-publishing
Feb 13, 2026
Merged

feat!: compile to JS before publishing#791
arbrandes merged 1 commit intoopenedx:frontend-basefrom
arbrandes:frontend-base-compile-before-publishing

Conversation

@arbrandes
Copy link
Contributor

@arbrandes arbrandes commented Feb 6, 2026

Description

Configure the package to compile TypeScript and copy SCSS and image assets (PNG, SVG) to dist/ before publishing, rather than publishing raw source files. This allows us to use tsc-alias for @src imports.

Also use a more modern export map to decouple the internal file structure from the package's API, and add a build step to CI.

BREAKING CHANGE

Consuming projects may need to update their imports or SASS @use lines.

Depends on

openedx/frontend-base#164

LLM Usage Notice

Built with assistance from Claude Opus 4.6.

Configure the package to compile TypeScript and copy SCSS and image
assets (PNG, SVG) to dist/ before publishing, rather than publishing raw
source files. This allows us to use tsc-alias for @src imports.

Also use a more modern export map to decouple the internal file
structure from the package's API, and add a build step to CI.

BREAKING CHANGE: Consuming projects may need to update their imports or
SASS @use lines.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@arbrandes arbrandes force-pushed the frontend-base-compile-before-publishing branch from 957c19e to 33f22f7 Compare February 11, 2026 15:14
@arbrandes arbrandes marked this pull request as ready for review February 11, 2026 15:17
@codecov
Copy link

codecov bot commented Feb 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (frontend-base@35283df). Learn more about missing BASE report.

Additional details and impacted files
@@               Coverage Diff                @@
##             frontend-base     #791   +/-   ##
================================================
  Coverage                 ?   88.58%           
================================================
  Files                    ?      158           
  Lines                    ?     1297           
  Branches                 ?      214           
================================================
  Hits                     ?     1149           
  Misses                   ?      144           
  Partials                 ?        4           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 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.

Copy link
Contributor

@brian-smith-tcril brian-smith-tcril left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@arbrandes arbrandes merged commit 70576cf into openedx:frontend-base Feb 13, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants