Skip to content

fix: refresh ignored module lockfiles during deploy - #34

Merged
ralyodio merged 1 commit into
masterfrom
codex/fix-module-installs
Nov 1, 2026
Merged

ralyodio merged 1 commit into
masterfrom
codex/fix-module-installs

Conversation

@sh1pt-actions-fleet

Copy link
Copy Markdown
Contributor

The deploy script sets CI=true, which makes pnpm treat each module install as frozen by default. Per-module lockfiles are ignored and can be stale after package updates, so installs fail and modules silently fail to load. Allow those ephemeral module lockfiles to refresh from their package manifests during deploy.

@github-actions

github-actions Bot commented Nov 1, 2026

Copy link
Copy Markdown

ThreatCrush Security Scan

16 finding(s)

HIGH/CRITICAL: 13 | MEDIUM: 3

Severity Rule Location
HIGH sh-plaintext-download bin/deploy.sh:49
HIGH secret-generic-api-key mcp_modules/convert2doc/examples/usage-example.js:11
HIGH secret-generic-api-key mcp_modules/fake_json/docs/api.md:151
HIGH secret-generic-api-key mcp_modules/fake_json/README.md:82
HIGH secret-generic-api-key mcp_modules/seo-ranking/examples/basic-usage.js:10
HIGH secret-generic-api-key mcp_modules/seo-ranking/examples/basic-usage.js:22
HIGH secret-generic-api-key mcp_modules/seo-ranking/examples/basic-usage.js:38
HIGH secret-generic-api-key mcp_modules/seo-ranking/examples/basic-usage.js:166
HIGH secret-generic-api-key mcp_modules/seo-ranking/examples/basic-usage.js:187
HIGH secret-generic-api-key mcp_modules/seo-ranking/examples/basic-usage.js:215
HIGH secret-generic-api-key mcp_modules/seo-ranking/README.md:415
HIGH secret-generic-credential mcp_modules/webhook_verify/examples/basic-usage.js:10
HIGH sensitive-file-committed sample.env:1
MEDIUM sh-remote-script-execution bin/provision.sh:43
MEDIUM sh-remote-script-execution mcp_modules/wcag/bin/install.sh:111
MEDIUM sh-remote-script-execution mcp_modules/wcag/bin/install.sh:120

Snippets are redacted; ThreatCrush never prints matched credential material.

@ralyodio
ralyodio merged commit c8d24d2 into master Nov 1, 2026
10 checks passed
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.

2 participants