microlinkhq / oembed-spec / 35195189186

95%

push

github

fix: use @kikobeats/got to support Node.js 24.20 (#52)

* fix: use @kikobeats/got to support Node.js 24.20

got@11 is unmaintained and breaks on Node.js 24.20 (nodejs/node#64847):
a retried connection error rejects with ERR_SOCKET_CLOSED_BEFORE_CONNECTION
and then crashes the process with uncaught exceptions. @kikobeats/got is
got@11 with that fix backported.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01VZiY7VMgg1nEMkCezwMXEC

* chore: drop the accidental install.log

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01VZiY7VMgg1nEMkCezwMXEC

* ci: allow dependency build scripts to run

pnpm 12 fails the install when a package with a build script is not approved,
and it runs an implicit install before every script, which does not inherit the
--dangerously-allow-all-builds flag from the install step:

  Error: ERR_PNPM_IGNORED_BUILDS
  Ignored build scripts: simple-git-hooks@2.14.0, spawn-sync@1.0.15

The env form of the same flag covers every install in the job.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01VZiY7VMgg1nEMkCezwMXEC

* ci: allow builds for every pnpm command

pnpm test/run re-installs when workspace settings changed after install
(e.g. minimumReleaseAgeExclude written for a fresh dependency). That
implicit install did not get --dangerously-allow-all-builds and failed
with ERR_PNPM_IGNORED_BUILDS. Setting it as env covers all pnpm calls.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>

52 of 57 branches covered (91.23%)

Branch coverage included in aggregate %.

1 of 1 new or added line in 1 file covered. (100.0%)

176 of 182 relevant lines covered (96.7%)

1141.75 hits per line