Skip to content

feat: support cjs#804

Open
sunrabbit123 wants to merge 2 commits intojhipster:mainfrom
sunrabbit123:feat/pr
Open

feat: support cjs#804
sunrabbit123 wants to merge 2 commits intojhipster:mainfrom
sunrabbit123:feat/pr

Conversation

@sunrabbit123
Copy link

@sunrabbit123 sunrabbit123 commented Jan 9, 2026

What changed with this PR:

just add tsdown and change package.json for to support cjs


https://www.npmjs.com/package/prettier-plugin-java-for-cjs?activeTab=code

thank you for making this project
wow, I surprise it

but this library is not support cjs, I failed use this on my project

@jtkiesel
Copy link
Contributor

jtkiesel commented Mar 15, 2026

@sunrabbit123 Thank you for your contribution and I apologize for my delay in taking a look at it! Would you mind rebasing your branch, which I think should resolve the merge conflicts? You'll also need to move your new file packages/prettier-plugin-java/tsdown.config.mts to the root tsdown.config.mts, since the repo has been flattened.

@sunrabbit123
Copy link
Author

well.. i failed install tree-sitter-java-orchard
스크린샷 2026-03-16 오후 1 45 40

because string-width-cjs@4.2.3 is not exsits
스크린샷 2026-03-16 오후 1 46 22

can i ask how to install it?

- Refactor package.json to support both ESM and CJS module formats.
- Update repository URL to point to the new location.
- Replace TypeScript build command with tsdown for improved build process.
- Add tsdown configuration file for build settings.
- Update dependencies to include tsdown.
- Add main and module fields to specify entry points for CommonJS and ESM.
- Update exports structure to support both import and require formats with correct type definitions.
@sunrabbit123
Copy link
Author

sunrabbit123 commented Mar 16, 2026

I rebased main without to merge lock file
if you can install tree-sitter-java-orchard library, you can run yarn install in my branch and push it

스크린샷 2026-03-16 오후 1 50 38

I allowed Allow edits by maintiainers @jtkiesel

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.

2 participants