Skip to content

Restore lockfile platform packages for the v1.11.4 publish and Add ggshield secret scan to husky hooks - #271

Merged
Harshi-Shah-CS merged 5 commits into
stagingfrom
development
Oct 30, 2026
Merged

Harshi-Shah-CS merged 5 commits into
stagingfrom
development

Conversation

@Harshi-Shah-CS

Copy link
Copy Markdown
Contributor

fix(deps): restore platform-specific optional packages in package-lock.json (#270)
chore(hooks): add ggshield secret scan to husky hooks and drop secrets-scan workflow (#267)
chore: Sync main into staging (#266 release merge commit)

Harshi-Shah-CS and others added 5 commits October 29, 2026 18:51
Update deployment logs query to V2, Add rollback plan-not-available message and Upgrade cli-utilities with Snyk fixes
…s-scan workflow (+Claude)

GitGuardian's remote GitHub check already scans every PR, so the
secrets-scan.yml workflow is redundant; remove it and its .talismanrc entry.

Add Contentstack Security's standard ggshield block to the top of
.husky/pre-commit and .husky/pre-push. launch-cli is public, so a missing
ggshield warns instead of blocking; a secret ggshield finds still blocks.
pre-push reads the pushed refs once and pipes them to both ggshield and
secrets-scan.sh, since each reads stdin and the second would otherwise get
nothing and skip the Talisman/trufflehog scan. Both hook files get a
.talismanrc checksum: Talisman flags the setup-URL warning line.

#claude_code#
🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
chore(hooks): add ggshield secret scan to husky hooks and drop secrets-scan workflow
…k.json

The v1.11.4 publish workflow failed at `npm ci`: npm 11 (Node 24 in CI)
reported 50 packages missing from the lockfile - the per-platform builds of
rollup and @unrs/resolver-binding. npm 10 tolerates the gap, so local
installs and PR checks never caught it. #226 had restored these entries on
main, but the main -> development sync for this release kept development's
lockfile and dropped them.

Regenerated with `npm install --package-lock-only` on npm 11. The change only
adds the missing entries; no package is removed or re-versioned.

#claude_code#
🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
fix(deps): restore platform-specific optional packages in package-lock.json
@Harshi-Shah-CS
Harshi-Shah-CS requested review from a team as code owners October 30, 2026 09:44
@snyk-io

snyk-io Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

✅ Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
✅ Open Source Security 0 0 0 0 0 issues
✅ Licenses 0 0 0 0 0 issues
✅ Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@github-actions

Copy link
Copy Markdown

🔒 Security Scan Results

ℹ️ Note: Only vulnerabilities with available fixes (upgrades or patches) are counted toward thresholds.

Check Type Count (with fixes) Without fixes Threshold Result
🔴 Critical Severity 0 0 10 ✅ Passed
🟠 High Severity 0 10 25 ✅ Passed
🟡 Medium Severity 0 2 500 ✅ Passed
🔵 Low Severity 0 0 1000 ✅ Passed

⏱️ SLA Breach Summary

✅ No SLA breaches detected. All vulnerabilities are within acceptable time thresholds.

Severity Breaches (with fixes) Breaches (no fixes) SLA Threshold (with/no fixes) Status
🔴 Critical 0 0 15 / 30 days ✅ Passed
🟠 High 0 0 30 / 120 days ✅ Passed
🟡 Medium 0 0 90 / 365 days ✅ Passed
🔵 Low 0 0 180 / 365 days ✅ Passed

ℹ️ Vulnerabilities Without Available Fixes (Informational Only)

The following vulnerabilities were detected but do not have fixes available (no upgrade or patch). These are excluded from failure thresholds:

  • Critical without fixes: 0
  • High without fixes: 10
  • Medium without fixes: 2
  • Low without fixes: 0

✅ BUILD PASSED - All security checks passed

@Harshi-Shah-CS
Harshi-Shah-CS merged commit 85dda3a into staging Oct 30, 2026
7 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.

4 participants