feat: add support for custom manifests in module replacements analysis #71

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

43081j merged 10 commits into e18e:main from dreyfus92:feat/bring-your-manifest

Aug 22, 2025

Contributor

dreyfus92 commented Aug 11, 2025

closes: #69

dreyfus92 added 3 commits

August 11, 2025 15:44


          feat: add support for custom manifests in module replacements analysis

462917f


          format

504c332


          remove unused express message check in custom manifests test

f7130b1

pkg-pr-new bot commented Aug 11, 2025 •

edited

Loading

Open in StackBlitz

npm i https://pkg.pr.new/e18e/cli/@e18e/cli@71

commit: 1c5eab4


          format

566566c

outslept reviewed

View reviewed changes

src/analyze/replacements.ts Outdated Show resolved Hide resolved

43081j reviewed

View reviewed changes

src/analyze/report.ts Outdated Show resolved Hide resolved

43081j reviewed

View reviewed changes

src/commands/analyze.meta.ts Outdated Show resolved Hide resolved

43081j reviewed

View reviewed changes

src/types.ts Outdated Show resolved Hide resolved

dreyfus92 added 3 commits

August 19, 2025 12:44


          Merge remote-tracking branch 'origin/main' into feat/bring-your-manifest

68c50ad


          Apply suggestions

68cc0df


          format

58046d1

43081j reviewed

View reviewed changes

src/analyze/attw.ts Outdated Show resolved Hide resolved

43081j reviewed

View reviewed changes

src/analyze/replacements.ts Outdated Show resolved Hide resolved

dreyfus92 mentioned this pull request

feat: add TypeScript configuration detection to file systems and update attw behavior #59

Merged

43081j reviewed

View reviewed changes

src/analyze/replacements.ts Outdated Show resolved Hide resolved

43081j reviewed

View reviewed changes

src/analyze/replacements.ts Outdated Show resolved Hide resolved

43081j reviewed

View reviewed changes

src/analyze/replacements.ts Outdated Show resolved Hide resolved


          Apply more suggestions.

9ee4c5f

43081j reviewed

View reviewed changes

src/analyze/replacements.ts Outdated Show resolved Hide resolved


          cleanup

6ea5f90

43081j reviewed

View reviewed changes

test/fixtures/custom-manifest.json Show resolved Hide resolved

43081j reviewed

View reviewed changes

src/test/custom-manifests.test.ts Outdated Show resolved Hide resolved


          tests: use fake modules in fixtures and convert to snapshot testing

1c5eab4

43081j approved these changes

View reviewed changes

43081j merged commit d6cf8fa into e18e:main

5 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet