fix: upgrade tmp to 0.2.6 (CVE-2026-44705)#1174
Conversation
Automated dependency upgrade by OrbisAI Security
📝 WalkthroughWalkthroughAdds a ChangesPackage override
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
alfred-openspec
left a comment
There was a problem hiding this comment.
Requesting changes because this does not remove the vulnerable tmp from the dependency tree.
I checked the branch lockfile and ran pnpm audit --audit-level low --json: the PR adds top-level tmp@0.2.6, but vulnerable tmp@0.0.33 still remains via @inquirer/prompts > @inquirer/editor > external-editor > tmp, so CVE-2026-44705 still reports against this PR. OpenSpec does not import tmp directly, so the new runtime dependency does not remediate the actual path.
Please update the upstream dependency chain or add a focused pnpm override/resolution so pnpm-lock.yaml no longer contains tmp@0.0.33, then re-run audit.
…from tree Adding tmp@0.2.6 as a direct dependency left the vulnerable tmp@0.0.33 installed transitively via @inquirer/prompts > @inquirer/editor > external-editor. Use pnpm.overrides to force the entire dependency tree to resolve tmp to ^0.2.6, removing the vulnerable version entirely. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Addressed. Pls review |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
package.json (1)
62-66: Confirm CVE-2026-44705 is addressed by thetmpoverrideCVE-2026-44705 affects
tmpversions< 0.2.6, and the advisory’s first patched version istmp@0.2.6. Thepackage.jsonpnpm.overridesforcingtmpto^0.2.6(lines 62-66) aligns with that fix.
Optional: pin to exact"0.2.6"instead of"^0.2.6"for fully deterministic installs.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@package.json` around lines 62 - 66, The pnpm.overrides entry forcing "tmp" to "^0.2.6" should be confirmed to remediate CVE-2026-44705 (which is fixed in tmp@0.2.6); ensure the override in package.json under the pnpm.overrides key indeed applies to all dependency trees (run pnpm list or pnpm why tmp) and, if you want fully deterministic installs, change the override value for "tmp" from "^0.2.6" to the exact "0.2.6" string so the tmp dependency is pinned; reference the pnpm.overrides object and the "tmp" override when making the change.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In `@package.json`:
- Around line 62-66: The pnpm.overrides entry forcing "tmp" to "^0.2.6" should
be confirmed to remediate CVE-2026-44705 (which is fixed in tmp@0.2.6); ensure
the override in package.json under the pnpm.overrides key indeed applies to all
dependency trees (run pnpm list or pnpm why tmp) and, if you want fully
deterministic installs, change the override value for "tmp" from "^0.2.6" to the
exact "0.2.6" string so the tmp dependency is pinned; reference the
pnpm.overrides object and the "tmp" override when making the change.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: c043cf5c-15a9-4345-8a46-fc18d492c931
⛔ Files ignored due to path filters (1)
pnpm-lock.yamlis excluded by!**/pnpm-lock.yaml
📒 Files selected for processing (1)
package.json
Summary
Upgrade tmp from 0.0.33 to 0.2.6 to fix CVE-2026-44705.
Vulnerability
CVE-2026-44705pnpm-lock.yamlDescription: tmp has Path Traversal via unsanitized prefix/postfix that enables directory escape
Evidence
Scanner confirmation: trivy rule
CVE-2026-44705flagged this pattern.Production code: This file is in the production codebase, not test-only code.
Threat Model Context
This is a Node.js library - vulnerabilities affect downstream consumers who use this package.
Changes
package.jsonpnpm-lock.yamlVerification
This change addresses a pattern flagged by static analysis. The code path handles user-influenced input and the fix reduces the attack surface against both manual and automated exploitation.
Automated security fix by OrbisAI Security
Summary by CodeRabbit