🚀 Feature: Don't inline if doing so would increase code size

[JoshuaKGoldberg]

Bug Report Checklist

• I have tried restarting my IDE and the issue persists.
• I have pulled the latest main branch of the repository.
• I have searched for related issues and found none that matched my issue.

Overview

For projects that are using ts-function-inliner primarily to reduce code size, it may be preferable to skip inlining functions if the replaced call would actually result in more output code. Especially if the function can't be removed (e.g. it's exported) and/or is called in many places.

Let's add an option to skip inlining a function in that case. I'm not sure what it should be called: maybe, skipLargerReplacements: true?

Additional Info

For an example, take this function and two calls to it:

function shortName(a: number) {
  return a + 1.000000000000000000002;
}

shortName(3);
shortName(4);

Code block character length: 100

By default, ts-function-inliner would produce:

function shortName(a: number) {
  return a + 1.000000000000000000002;
}

3 + 1.000000000000000000002;
4 + 1.000000000000000000002;

Code block character length: 130

[JoshuaKGoldberg]

Hmm, I can't think of any case where someone wouldn't want this behavior enabled. I'm going to go ahead and make it always on. If someone wants an option added later, they can file a followup issue.

[github-actions]

🎉 This is included in version v0.2.0 🎉

The release is available on:

• GitHub releases
• npm package (@latest dist-tag)

Cheers! 📦🚀