Skip to content

Conversation

@cte
Copy link
Collaborator

@cte cte commented May 23, 2025

Description

I prefer ESM as a default. Claude 4 also took some liberties with emojis and refactoring.


Important

Convert bootstrap script to ESM format and update package.json script references.

  • Scripts:
    • Convert scripts/bootstrap.js to ESM format as scripts/bootstrap.mjs.
    • Update package.json to use bootstrap.mjs in preinstall, install, and install:all scripts.
  • Behavior:
    • The new ESM script checks for pnpm and installs it if not found, using a temporary package.json if necessary.
    • Exits early if BOOTSTRAP_IN_PROGRESS is set or if pnpm is already in use.

This description was created by Ellipsis for c08e91a. You can customize this summary. It will automatically update as commits are pushed.

@cte
Copy link
Collaborator Author

cte commented May 23, 2025

CC @KJ7LNW

@cte cte merged commit 78bb7e0 into main May 23, 2025
12 of 13 checks passed
@cte cte deleted the cte/bootstrap-mjs branch May 23, 2025 21:35
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap May 23, 2025
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap May 23, 2025
@@ -0,0 +1,5 @@
---
"roo-cline": patch
Copy link
Contributor

Choose a reason for hiding this comment

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

Typo in key: roo-cline appears to be incorrect. Did you mean roo-cli (or roo-code)?

Suggested change
"roo-cline": patch
"roo-cli": patch

@KJ7LNW
Copy link
Contributor

KJ7LNW commented May 23, 2025

CC @KJ7LNW

tested, works for me thanks

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

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants