chore: streamline toolchain, packaging and CI - #3
Open
Maximvdw wants to merge 6 commits into
Open
Conversation
npm is the package manager fleet-wide. Every CI workflow previously ran a bare `yarn install` or `npm install` rather than a frozen install, so lockfiles silently churned in CI. Yarn 1.22 is unmaintained; npm ships with Node 22 and 24, supports `overrides` for transitive CVE pinning, and can publish with `--provenance`. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FtN6Tx33QW5a6AtCH1zYQh
tsconfig.bundle.esm5.json used target "es6" while tsconfig.bundle.esm.json used "es2015" -- the same TypeScript alias, with the same module: es2020 -- so the esm5 output was pure duplication, and exports.import pointed at it while dist/esm went unreachable. scripts/finalize-esm.mjs now post-processes dist/esm so it is genuinely loadable as ES modules: it writes the per-directory "type" markers (without which both Node and webpack read the output as CommonJS and fail on the first import statement) and rewrites TypeScript's directory imports to explicit file paths, which Node's ESM resolver requires. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FtN6Tx33QW5a6AtCH1zYQh
Replaces a hand-forked main.yml with a thin caller. The fleet had 24 distinct variants of that file, Node was unpinned in 22 repositories, and 13 ran `npm install` rather than `npm ci`. Every job now installs its own dependency tree instead of depending on an actions/cache hit for node_modules, lint no longer waits on build, and the matrix covers Node 22 and 24. dependabot.yml and the auto-merge workflow arrive here for the first time outside openhps-core. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FtN6Tx33QW5a6AtCH1zYQh
TypeScript 5.9, ESLint 9 with a native flat config, typescript-eslint 8,
prettier 3, mocha 11, chai 6, c8, typedoc 0.28, @types/node 24, engines.node >=22.
Packaging is brought onto the fleet contract: `types` resolves to dist/types
rather than dist/cjs, the exports map lists `types` first in every condition and
points `import` at dist/esm, and tslib is declared -- the tsconfig sets both
importHelpers and noEmitHelpers, a combination that always emits an import of
tslib, which previously resolved only by hoisting.
@openhps/* peer ranges gain an upper bound (">=1.0.16 <2"), and every declared
peer now has a matching devDependency so CI tests what consumers receive;
scripts/check-peers.mjs enforces both as part of lint.
Coverage moves from nyc to c8: nyc 18's spawn-wrap breaks down on Node 22, running
the tests but reporting 0 passing and 0% coverage. cover:ci also writes a real
test-results.xml -- reporterEnabled had been sitting at the top level of
.mocharc.json where mocha ignores it, so the JUnit file CI consumed had never been
produced.
standard-version is replaced by commit-and-tag-version (unmaintained since 2023),
and eslint-plugin-deprecation by @typescript-eslint/no-deprecated (archived).
CONTRIBUTING.md and SECURITY.md move to the org-wide OpenHPS/.github repository.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FtN6Tx33QW5a6AtCH1zYQh
Formatting only. The lint script was `eslint src/**`, so test/ had never been seen by prettier; widening it to `eslint .` surfaced hundreds of formatting-only findings. Kept separate so the toolchain diff stays reviewable. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FtN6Tx33QW5a6AtCH1zYQh
react-native-wifi-reborn no longer declares WifiEntry inside a `WifiManager` namespace, so `WifiManager.WifiEntry` does not resolve as a type and the build failed on all three call sites. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FtN6Tx33QW5a6AtCH1zYQh
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Brings this module onto the shared OpenHPS toolchain baseline, applied by the
openhps-devtoolscodemod so every repository lands the same shape.Toolchain: TypeScript 5.9, ESLint 9 with a native flat config, typescript-eslint 8, prettier 3, mocha 11, chai 6, c8, typedoc 0.28,
@types/node24,engines.node >=22. npm replaces yarn, withnpm ciin CI instead of the barenpm install/yarn installthat silently rewrote lockfiles.Packaging:
typesresolves todist/types, the exports map liststypesfirst in every condition and pointsimportatdist/esm, and the redundantesm5target is gone — its tsconfig used targetes6while the esm one usedes2015, the same TypeScript alias.tslibis now declared, since the tsconfig sets bothimportHelpersandnoEmitHelpers, a combination that always emits an import of it.scripts/finalize-esm.mjsmakesdist/esmgenuinely loadable as ES modules: it writes the per-directorytypemarkers — without which both Node and webpack parse the output as CommonJS and fail on the firstimport— and rewrites TypeScript's directory imports to the explicit file paths Node's ESM resolver requires.Dependency policy:
@openhps/*peer ranges gain an upper bound (>=1.0.16 <2") and every declared peer has a matching devDependency, so CI tests what consumers receive.scripts/check-peers.mjs` enforces both as part of lint.CI: replaces a hand-forked
main.ymlwith a thin caller ontoOpenHPS/workflows. The fleet had 24 distinct variants of that file. Every job now installs its own dependency tree rather than depending on anactions/cachehit fornode_modules, lint no longer waits on build, and the matrix covers Node 22 and 24.Coverage moves from nyc to c8: nyc 18's spawn-wrap breaks down on Node 22, running the tests while reporting 0 passing and 0% coverage.
🤖 Generated with Claude Code
https://claude.ai/code/session_01FtN6Tx33QW5a6AtCH1zYQh