Skip to content

fix(security): resolve Dependabot alerts via npm overrides and lockfile refresh#41

Merged
ZappoMan merged 3 commits into
developfrom
dependbot-050926
May 10, 2026
Merged

fix(security): resolve Dependabot alerts via npm overrides and lockfile refresh#41
ZappoMan merged 3 commits into
developfrom
dependbot-050926

Conversation

@ZappoMan
Copy link
Copy Markdown
Contributor

@ZappoMan ZappoMan commented May 10, 2026

Summary

Addresses reported Dependabot / npm audit findings with minimal surface area: root overrides in the npm workspaces monorepo plus a regenerated package-lock.json so nested resolutions honor those overrides—no Expo SDK, React Native, or other major dependency upgrades.

Changes

Verification

  • npm audit0 vulnerabilities at the monorepo root
  • npm run test:unit:shared
  • npm run test:unit:mobile

Type of change

  • Feature
  • Bug fix (supply-chain / security)
  • Documentation
  • Refactoring
  • Other

Merge strategy reminder

⚠️ This PR targets develop: use Squash and merge.

Squash merging to main can cause conflicts when merging developmain later.

Checklist

  • Code follows the project's style guidelines
  • Self-review completed
  • Comments added for complex code (not needed—dependency pins only)
  • Documentation updated (not needed)
  • No new warnings generated (targeted dep resolution)
  • Tests added/updated (not needed—existing suites cover regressions)
  • Tests pass locally (test:unit:shared, test:unit:mobile)

Copilot AI review requested due to automatic review settings May 10, 2026 00:52
@ZappoMan ZappoMan changed the title fix dependbot issues fix(security): resolve Dependabot alerts via npm overrides and lockfile refresh May 10, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 10, 2026

🌐 Web preview: https://deploy.beakerstack.com/pr-41/
📱 Mobile preview: Channel pr-41

📱 Mobile Preview (OTA Updates)

No native code changes detected - using OTA updates only.

⚠️ Note: This app requires a Development Build (Expo Go will not work due to native Google OAuth).

Step 1: Install Development Build (one-time setup)

  1. Build: npm run mobile:build:dev:ios (or mobile:build:dev:android)
  2. Download from Expo dashboard
  3. Install: EAS_BUILD_PATH=~/Downloads/BeakerStack.ipa npm run mobile:install:dev:ios (iOS) or EAS_BUILD_PATH=~/Downloads/BeakerStack.apk npm run mobile:install:dev:android (Android)

Step 2: Load PR Preview Update

  1. Open the development build on your device/simulator
  2. Shake device (or Cmd+D on iOS / Cmd+M on Android) → "Enter URL manually"
  3. Paste the update URL: https://u.expo.dev/23c5e522-5341-4342-85f5-f2e46dd6087f?channel-name=pr-41
  4. The app will reload with the JavaScript bundle from channel pr-41

Note: You must use the full URL format - just entering the channel name (pr-41) will not work.

Alternative: For local development, use: cd apps/mobile && npx expo start --dev-client, then press 'i' for iOS simulator.

📖 See Mobile Build Testing Guide for detailed instructions.


Updated at: May 9, 2026 at 6:05 PM PDT

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the repository’s root npm overrides to address Dependabot-reported dependency issues by forcing patched versions across the workspace install graph.

Changes:

  • Added an override for @babel/plugin-transform-modules-systemjs to ensure a fixed version is used.
  • Added/updated overrides for postcss, fast-uri, and fast-xml-builder to enforce newer, presumably non-vulnerable versions.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 10, 2026

CI Coverage & Test Summary

Metric Coverage Covered / Total
Statements 96.81% 10487 / 10832
Branches 81.08% 1804 / 2225
Functions 89.74% 315 / 351
Lines 86.59% 9379 / 10832

Suites: 37 passed, 0 failed (37 total) · Tests: 407 passed, 0 failed (425 total)

✅ All reported test suites passed.

Coverage artifacts: coverage-summary, coverage-packages.


Updated at: May 9, 2026 at 6:08 PM PDT

@ZappoMan ZappoMan merged commit 853f54c into develop May 10, 2026
9 checks passed
ZappoMan added a commit that referenced this pull request May 10, 2026
Release: merge develop into main (#41#43)
@ZappoMan ZappoMan deleted the dependbot-050926 branch May 17, 2026 17:53
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