fix: use commonjs for package type

[Blackbaud-SteveBrush]

AB#3939429

ng-packagr 21.2.2 sets type=module in package.json. This is a breaking change for projects that use ng-packagr to build, but ALSO include schematics. Schematics only support CommonJS. Component libraries by themselves can support either CommonJS or ESM.

Summary by CodeRabbit

• Chores
  • Standardized module resolution settings across multiple libraries for consistent runtime behavior.
• Bug Fixes / Behavior
  • Disabled application animations to simplify rendering and tests.
• Tests / Config
  • Simplified test setup and updated test runner JS loader configuration for more consistent test execution.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 755ca3b3-ecef-46f1-aaf6-5ed9c5dde2ff

📥 Commits

Reviewing files that changed from the base of the PR and between c6d6a96 and 31365a1.

📒 Files selected for processing (1)

• libs/stache/jasmine.json

✅ Files skipped from review due to trivial changes (1)

• libs/stache/jasmine.json

---

📝 Walkthrough

Walkthrough

Added explicit CommonJS "type" to four library package.json files; removed BrowserAnimationsModule from the playground AppModule; simplified an Angular unit test by removing NoopAnimationsModule from TestBed imports; updated Jasmine config to set jsLoader to "require".

Changes

Cohort / File(s)	Summary
CommonJS Module Type Declaration libs/clipboard/package.json, libs/code-block/package.json, libs/media/package.json, libs/stache/package.json	Added "type": "commonjs" to each package.json to explicitly set CommonJS semantics for .js files.
App Module Animation Removal apps/playground/src/app/app.module.ts	Removed BrowserAnimationsModule import and its entry from the AppModule imports array.
Test Import Simplification libs/stache/src/lib/modules/action-buttons/action-buttons.component.spec.ts	Removed NoopAnimationsModule from TestBed imports; test now imports only StacheActionButtonsModule and RouterTestingModule.
Jasmine Loader Config libs/stache/jasmine.json	Added jsLoader: "require" to the Jasmine config.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested labels

risk level (author): 1

Suggested reviewers

• johnhwhite

Poem

🐰 I nibbled through JSON fields with care,

Declared CommonJS in four little lairs,
Took animations out to make tests light,
Configured the loader — all snug and right,
🥕 Hop, build, and bounce into the night!

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'fix: use commonjs for package type' accurately describes the main change: adding 'type': 'commonjs' to multiple package.json files to address a breaking change from ng-packagr 21.2.2.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch commonjs

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}