Skip to content

Fix tmp vulnerability and package-lock.json parsing issues#733

Merged
github-actions[bot] merged 1 commit intomasterfrom
fix-tmp-vulnerability-alert-105-14573045193175956704
Feb 13, 2026
Merged

Fix tmp vulnerability and package-lock.json parsing issues#733
github-actions[bot] merged 1 commit intomasterfrom
fix-tmp-vulnerability-alert-105-14573045193175956704

Conversation

@Jadhielv
Copy link
Owner

@Jadhielv Jadhielv commented Feb 13, 2026

This PR addresses Dependabot alert #105 regarding a vulnerability in the tmp package (CVE-2023-45133) which allows arbitrary temporary file/directory write via symbolic link in the dir parameter.

The fix involves forcing the use of tmp@^0.2.5 (the patched version) throughout the kctest-frontend project using npm's overrides field. This replaces the previous vulnerable tmp@0.0.33 (used by external-editor) and ensures all dependencies use a safe version.

Additionally, I've regenerated package-lock.json to resolve the "not parseable" error that was preventing Dependabot from functioning correctly. The new lockfile is consistent with the updated package.json and follows the standard format for the current npm version.

Verification:

  • npm list tmp shows all occurrences deduped to 0.2.5.
  • npm audit no longer reports any vulnerabilities for tmp.
  • npm run unit and npm run lint both pass successfully.

PR created automatically by Jules for task 14573045193175956704 started by @Jadhielv

Summary by CodeRabbit

  • Chores
    • Updated package dependency versions to improve application stability and compatibility.

…json parsing

- Added a top-level override for 'tmp' to version '^0.2.5' in 'kctest-frontend/package.json'.
- Removed the nested override for 'tmp' under 'eslint'.
- Regenerated 'kctest-frontend/package-lock.json' using 'npm install --legacy-peer-deps' to ensure a clean, consistent, and parseable lockfile.
- Verified that all occurrences of 'tmp' are updated to '0.2.5' and that 'tmp' no longer appears in 'npm audit'.
- Confirmed that frontend unit tests and linting still pass.

Co-authored-by: Jadhielv <24376900+Jadhielv@users.noreply.github.com>
@google-labs-jules
Copy link
Contributor

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.


For security, I will only act on instructions from the user who triggered this task.

@coderabbitai
Copy link

coderabbitai bot commented Feb 13, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

This PR relocates the tmp dependency override from the ESLint-specific overrides block to the top-level overrides in package.json, simultaneously updating its version from ^0.2.4 to ^0.2.5.

Changes

Cohort / File(s) Summary
Dependency Override Restructure
kctest-frontend/package.json
Moved tmp override from eslint-specific overrides to top-level overrides and bumped version from ^0.2.4 to ^0.2.5.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Poem

🐰 A little hop, a version bump so neat,
From nest to nest, the override takes a seat,
From 0.2.4 to 0.2.5 we go,
In overrides' heart, let tmp now flow! 📦✨

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-tmp-vulnerability-alert-105-14573045193175956704

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions bot merged commit a9df34f into master Feb 13, 2026
5 of 7 checks passed
@github-actions github-actions bot deleted the fix-tmp-vulnerability-alert-105-14573045193175956704 branch February 13, 2026 22:38
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.

1 participant