Skip to content

fix(ci): repair CI (lockfile, CVE overrides, SPDX headers)#21

Open
igorjs wants to merge 9 commits into
mainfrom
chore/pnpm-lockfile-settings
Open

fix(ci): repair CI (lockfile, CVE overrides, SPDX headers)#21
igorjs wants to merge 9 commits into
mainfrom
chore/pnpm-lockfile-settings

Conversation

@igorjs

@igorjs igorjs commented Jun 30, 2026

Copy link
Copy Markdown
Owner

Repairs main CI end to end after the repo-config hardening apply: lockfile regen (autoInstallPeers/tarball mismatch), CVE overrides (ws>=8.21.0, undici>=7.28.0, brace-expansion>=5.0.6), SPDX headers on bin/tests/scripts, and a .license-checker.allow for the dep-license gate. Verified: lint-and-test, deny + spdx, and scan all pass.

Closes #22

The repo-config apply wrote autoInstallPeers and lockfileIncludeTarballUrl into pnpm-workspace.yaml without regenerating the lockfile, so pnpm install --frozen-lockfile (CI) failed with ERR_PNPM_LOCKFILE_CONFIG_MISMATCH. Regenerate to match; no dependency versions change.

Signed-off-by: Igor Santos <oss@mail.igorjs.io>
@github-actions github-actions Bot added chore Maintenance size:l Large change (250-999 lines) labels Jun 30, 2026
@github-actions

Copy link
Copy Markdown
Contributor

PR Quality Gate: no linked issue

Every PR must reference an issue that describes the motivation for the change.

Add a closing reference to your PR description:

Closes #N

Replace N with the issue number. If no issue exists yet, please open one first.

@github-actions github-actions Bot closed this Jun 30, 2026
@github-actions github-actions Bot added dco:passed All commits have Signed-off-by cla:signed CLA signed by contributor labels Jun 30, 2026
…ders

The dependency audit + osv-scan flagged high CVEs pulled in via @igorjs/pure-test: ws (<8.21.0), undici (<7.28.0), brace-expansion (<5.0.6). Pin them via pnpm overrides to patched versions and regenerate the lockfile. Also add the SPDX-License-Identifier header to the bin/tests/scripts .mjs/.ts files the license gate flagged (the repo's own header script only covers src/*.ts).

Signed-off-by: Igor Santos <oss@mail.igorjs.io>
@igorjs igorjs changed the title chore(deps): regenerate lockfile to match hardened pnpm-workspace fix(ci): repair CI (lockfile, CVE overrides, SPDX headers) Jul 1, 2026
license.yml runs license-checker --onlyAllow against .license-checker.allow and hard-fails when it is absent. pure-test never got the repo-config bootstrap (that was enabled only on gh-actions-clean-workflow), so add a permissive-OSS allowlist here. The repo has zero production dependencies today, so the list only needs to cover future additions.

Signed-off-by: Igor Santos <oss@mail.igorjs.io>
@igorjs igorjs reopened this Jul 1, 2026
@github-actions github-actions Bot added fix Bug fix test Test changes labels Jul 1, 2026
@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

PR Quality Gate: no linked issue

Every PR must reference an issue that describes the motivation for the change.

Add a closing reference to your PR description:

Closes #N

Replace N with the issue number. If no issue exists yet, please open one first.

1 similar comment
@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

PR Quality Gate: no linked issue

Every PR must reference an issue that describes the motivation for the change.

Add a closing reference to your PR description:

Closes #N

Replace N with the issue number. If no issue exists yet, please open one first.

@github-actions github-actions Bot closed this Jul 1, 2026
@igorjs igorjs reopened this Jul 1, 2026
@socket-security

socket-security Bot commented Jul 1, 2026

Copy link
Copy Markdown

Warning

Review the following alerts detected in dependencies.

According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.

Action Severity Alert  (click "▶" to expand/collapse)
Warn High
Obfuscated code: npm @emnapi/runtime is 90.0% likely obfuscated

Confidence: 0.90

Location: Package overview

From: pnpm-lock.yamlnpm/miniflare@4.20260603.0npm/@emnapi/runtime@1.11.1

ℹ Read more on: This package | This alert | What is obfuscated code?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev.

Suggestion: Packages should not obfuscate their code. Consider not using packages with obfuscated code.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/@emnapi/runtime@1.11.1. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

Warn High
Obfuscated code: npm yargs is 90.0% likely obfuscated

Confidence: 0.90

Location: Package overview

From: pnpm-lock.yamlnpm/c8@11.0.0npm/yargs@17.7.3

ℹ Read more on: This package | This alert | What is obfuscated code?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev.

Suggestion: Packages should not obfuscate their code. Consider not using packages with obfuscated code.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/yargs@17.7.3. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

View full report

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Maintenance cla:signed CLA signed by contributor dco:passed All commits have Signed-off-by fix Bug fix size:l Large change (250-999 lines) test Test changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI: repair main after pnpm hardening rollout

1 participant