Skip to content

[Fix] Update cleanup script#823

Open
akaashrp wants to merge 3 commits intomlc-ai:mainfrom
akaashrp:cleanup
Open

[Fix] Update cleanup script#823
akaashrp wants to merge 3 commits intomlc-ai:mainfrom
akaashrp:cleanup

Conversation

@akaashrp
Copy link
Copy Markdown
Collaborator

@akaashrp akaashrp commented May 4, 2026

No description provided.

@akaashrp akaashrp linked an issue May 4, 2026 that may be closed by this pull request
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates cleanup-index-js.sh to replace pathToFileURL calls with a dummy path in both the library and source map files to support ESM SSR environments. The review feedback suggests improving regex precision by escaping literal dots and combining multiple sed commands to increase efficiency and prevent overwriting backup files.

Comment thread cleanup-index-js.sh Outdated
Comment thread cleanup-index-js.sh Outdated
akaashrp and others added 2 commits May 5, 2026 03:18
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
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.

Build not working : require is not defined in ES module scope

1 participant