fix: proper asset attribution

[tivac]

Description

Changes how assets are associated with bundle entries to split apart dynamic vs static. Also updates @modular-css/rollup-rewriter to only use the dynamic assets.

Motivation and Context

Assets being attached to the bundle object were being conflated, leading to over-inclusion when using @modular-css/rollup-rewriter. Now it only handles the dynamic assets and leaves static assets up to the consumer.

How Has This Been Tested?

Tests, and running against a large rollup project with tons of chunks.

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• All new and existing tests passed.

[codecov]

Codecov Report

Merging #554 into master will increase coverage by 0.09%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #554      +/-   ##
==========================================
+ Coverage   98.94%   99.03%   +0.09%     
==========================================
  Files          46       46              
  Lines        1041     1039       -2     
  Branches      160      163       +3     
==========================================
- Hits         1030     1029       -1     
+ Misses         11       10       -1

Impacted Files	Coverage Δ
packages/rollup/rollup.js	100% <100%> (+0.81%)	⬆️
packages/rollup-rewriter/rewriter.js	100% <100%> (ø)	⬆️

---

Continue to review full report at Codecov.

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