Skip to content

Conversation

@sod
Copy link
Contributor

@sod sod commented Jan 16, 2026

These TypeScript alias paths are not converted by tsc to relative imports. Either use a postprocessor like tsc-alias that converts those aliased path, or don't use them to begin with.

I opted in this PR with the option that doesn't add a dependency.

Fixes #62

These typscript alias paths are not converted by tsc to relative imports. Either use a postprocessor like `tsc-alias` that converts those aliased path, or don't use them to begin with.

I opted in this PR with the option that doesn't add a dependency.

Fixes amplitude#62
@macroscopeapp
Copy link

macroscopeapp bot commented Jan 16, 2026

Replace src/-prefixed absolute imports with relative parent-based imports in packages/node to fix broken Node resolution

Update exposure-related modules to use relative imports instead of src/-prefixed absolute paths, including cache and hash utilities and ExperimentUser type references.

📍Where to Start

Start with packages/node exposure module imports in packages/node/src/exposure/exposure.ts to see the pattern applied across files.


Macroscope summarized 92e424f.

@zhukaihan
Copy link
Collaborator

Merging. Test errors are expected as they don't have access to tokens. Running them on main.

@zhukaihan zhukaihan merged commit ed762a0 into amplitude:main Jan 17, 2026
3 of 7 checks passed
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.

Broken internal import (require("src/exposure/exposure")) fails with esbuild / Angular 21

2 participants