Skip to content

fix: strip @infragistics/ scope from imports in generated code-viewer JSON#1082

Merged
ChronosSF merged 1 commit into
vnextfrom
dtsvetkov/strip-official-package-from-generated-jsons
Jun 17, 2026
Merged

fix: strip @infragistics/ scope from imports in generated code-viewer JSON#1082
ChronosSF merged 1 commit into
vnextfrom
dtsvetkov/strip-official-package-from-generated-jsons

Conversation

@dobromirts

Copy link
Copy Markdown
Contributor

The CI/CD pipeline rewrites all igniteui-react and igniteui-dockmanager import strings in source .tsx/.ts files to their licensed @infragistics/ scoped equivalents before running the build. Since gulp updateBrowser runs as part of the build, getSamples reads the already rewritten source files and updateCodeViewer embeds them into the generated code-viewer JSON files. As a result, end users in the docs site code viewer would see@infragistics/igniteui-react-* imports packages that require a license and are not publicly available on npm.

@dobromirts dobromirts marked this pull request as ready for review June 17, 2026 08:48
@ChronosSF ChronosSF requested a review from Copilot June 17, 2026 15:48

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR prevents licensed @infragistics/igniteui-* scoped package names (introduced by the CI rewrite step) from leaking into the docs site’s generated code-viewer JSON, so end users see the public igniteui-* package imports instead.

Changes:

  • Strip @infragistics/ scope from igniteui-* imports when copying sample sources into ./src/samples.
  • Strip @infragistics/ scope from the main TSX sample source before writing it into the code-viewer JSON.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread browser/tasks/gulp-samples.js
@ChronosSF ChronosSF merged commit cc86e08 into vnext Jun 17, 2026
5 checks passed
@ChronosSF ChronosSF deleted the dtsvetkov/strip-official-package-from-generated-jsons branch June 17, 2026 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants