Help wanted :) about tween.js

[mk965]

@mikebolt @trusktr

I really want to join. Please feel free to contact me if I can be of help.

[trusktr]

Hi @mikebolt. I'm still here. If someone else is interested in collaborating, I can help ramping up and discussing. Please do reach out!

https://github.com/tweenjs/discuss hasn't been used in a long time

I went ahead and archived that after putting a link to https://github.com/tweenjs/tween.js/discussions there.

[cuixiping]

I like this lib and may contribute some code some time. I am using tweenjs for my games.

[trusktr]

@cuixiping @mk965 would be glad to have some help. If any ideas, please let me know. The issues list has some existing things to do. We need to fix

• #666

with higher priority.

[cuixiping]

@trusktr
As my test, similar to #659 , but have some differences.

• rename tween.esm.js to tween.mjs
• rename tween.cjs.js to tween.cjs
• modify package.json
  • set "main" as mjs file "dist/tween.mjs"
  • remove "module" field
  • modify file names in "exports" field

{
	...
	"type": "module",
	"main": "dist/tween.mjs",
	"types": "dist/tween.d.ts",
	"exports": {
		".": {
			"import": "./dist/tween.mjs",
			"require": "./dist/tween.cjs"
		}
	},
	...
}

My testing steps:

• create a new folder and run npm init and then npm install @tweenjs/tween.js
• create a index.mjs in this folder with content

import * as TWEEN from '@tweenjs/tween.js'
console.log(Object.keys(TWEEN));

• create a index.cjs in this folder with content

const TWEEN = require('@tweenjs/tween.js')
console.log(Object.keys(TWEEN));

• run node ./index.mjs works ok!
• run node ./index.cjs works ok!

My node.js version is 16.16.0

[trusktr]

Hey thanks @cuixiping! Can you test with the latest? .cjs.js was renamed to .cjs, but everything else should just work.

The new examples in examples/example-projects/nodejs-commonjs/ and examples/example-projects/nodejs-esmodules/ verify being able to import into Node.js with either CommonJS or ESM format, with code similar to what you wrote with console.log.

We did not need to rename the .esm.js to .mjs because .js is already treated as ESM with type: module in package.json. Maybe just tween.js would be a nicer name, but decided not to make a breaking change for that.

[trusktr]

@cuixiping @mk965 added you both to the repo! Will close this for now.

If anyone else is interested to volunteer some time on this, aiming to move towards a feature like Timeline for managing timelines with tweens after simplifying some things, please reach out!