fix: ensure optional deps are installed in CI

[soumeh01]

Changes

Add --prefer-offline --include=optional to all npm ci calls.

• --prefer-offline: use cached packages where available, reducing network usage on CI runners.
• --include=optional: ensure platform-specific esbuild binaries (e.g. @esbuild/linux-x64, @esbuild/darwin-arm64) are installed.
• --no-fund: Disables messages about funding/supporting packages

These are required by tsx, which is used to run TypeScript scripts such as copyright:check, download-tools, and check:links.

Screenshots

Checklist

• 🤖 This change is covered by unit tests (if applicable).
• 🤹 Manual testing has been performed (if necessary).
• 🛡️ Security impacts have been considered (if relevant).
• 📖 Documentation updates are complete (if required).
• 🧠 Third-party dependencies and TPIP updated (if required).

[qltysh]

Coverage Impact

This PR will not change total coverage.

🚦 See full report on Qlty Cloud »

🛟 Help

• Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

• Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

• File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

  • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

[jreineckearm]

@soumeh01 , what's the status of this? Also, some merge conflicts came up.

[jreineckearm]

LGTM.

But TBF: I unfortunately don't see much of a CI performance improvement. Probably because we don't use GH caches IIRC?

[soumeh01]

LGTM.

But TBF: I unfortunately don't see much of a CI performance improvement. Probably because we don't use GH caches IIRC?

Yes True, It looks like resource contention on the runner might also be affecting performance. At times, it runs significantly faster than what we are seeing now.