Skip to content

Reduce npm deprecation noise and modernize build/te…#67

Merged
esgn merged 1 commit into
masterfrom
66-toolchain
Apr 23, 2026
Merged

Reduce npm deprecation noise and modernize build/te…#67
esgn merged 1 commit into
masterfrom
66-toolchain

Conversation

@esgn
Copy link
Copy Markdown
Member

@esgn esgn commented Apr 23, 2026

Close #66

Summary

This PR performs a focused build/test toolchain cleanup after the WFS stabilization work, with the goal of reducing install/build warning noise and improving maintainability without changing application behavior.

What changed

  • audited current npm deprecation warnings and mapped each to direct vs transitive sources
  • simplified the build pipeline by removing duplicate TypeScript compilation in build (mcp-build already runs tsc)
  • migrated Jest config from jest.config.ts to jest.config.mjs (no behavior change, less tooling overhead)
  • removed unused dev dependencies:
    • @types/cors
    • @types/supertest
    • geojson
    • ts-node
  • added an override for babel-plugin-istanbul@^8.0.0 to remove old test-exclude@6 chain
    • this removes inflight@1.0.6 and glob@7.2.3
  • modernized TS module resolution:
    • main config moved to module/moduleResolution: NodeNext
    • test config kept Jest-compatible via module: ESNext + moduleResolution: bundler
  • fixed newly surfaced strict typing issues (noImplicitAny) in:
    • src/gpf/wfs-schema-catalog.ts
    • src/helpers/wfs_engine/properties.ts
  • updated publish workflow hygiene:
    • actions/checkout@v4
    • npm ci instead of npm install

Validation

  • npm run typecheck
  • npm run build
  • npm test ✅ (19 suites, 106 tests)

Warning impact

Clean install warning count reduced from 4 -> 2 (npm ci --ignore-scripts).

Removed:

  • inflight@1.0.6
  • glob@7.2.3

Remaining:

  • glob@10.5.0 (Jest 30 transitive chain)
  • node-domexception@1.0.0 (node-fetch chain, direct + transitive via @camptocamp/ogc-client)

Prod-only install (npm ci --omit=dev --ignore-scripts) now shows only:

  • node-domexception@1.0.0

@esgn esgn merged commit 9d4b328 into master Apr 23, 2026
2 checks passed
@esgn esgn deleted the 66-toolchain branch April 23, 2026 14:18
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.

Review build warnings and modernize the test/build toolchain

1 participant