Skip to content

fix: resolve import paths in additional .ts files - #77

Merged
ChronosSF merged 1 commit into
masterfrom
vkombov/resolve-import-paths
Jun 23, 2026
Merged

fix: resolve import paths in additional .ts files#77
ChronosSF merged 1 commit into
masterfrom
vkombov/resolve-import-paths

Conversation

@viktorkombov

@viktorkombov viktorkombov commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Description:
Fixes broken relative imports in generated StackBlitz samples where additional .ts files (e.g. grid-finjs.component.ts) reference other additional files (e.g. prevent-scroll.directive.ts).

  • Extend _modifyImports file filter to include .ts files without explicit fileExtension property
  • Fix provider import extraction to correctly handle arguments (e.g. provideRouter([]))

Root cause: _getAdditionalFiles creates LiveEditingFile instances without setting the fileExtension constructor parameter, so the fileExtension === 'ts' filter silently excluded them from import path resolution.

@ChronosSF
ChronosSF merged commit 1662be9 into master Jun 23, 2026
5 checks passed
@ChronosSF
ChronosSF deleted the vkombov/resolve-import-paths branch June 23, 2026 10:56
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.

3 participants