chore(deps): update dependencies - #1437
Conversation
Update multiple packages including claude-code, github/copilot, openai/codex, oh-my-pi packages, and other dependencies. Add new obsidian-headless package.
|
You do not have enough credits to review this pull request. Please purchase more credits to continue. |
📝 WalkthroughSummary by CodeRabbitChores
WalkthroughThis pull request updates multiple package dependencies to newer versions in Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Possibly related PRs
Suggested labels
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 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 |
Mesa DescriptionTL;DRUpdated multiple dependencies, including What changed?
Description generated by Mesa. Update settings |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@package.json`:
- Line 56: The package.json lists "obsidian-headless" in dependencies but not in
trustedDependencies; update the package.json object so that the plain-name
"obsidian-headless" appears in the trustedDependencies array alongside the other
plain-name deps (mirror how entries like "someOtherDep" are represented) to
ensure consistency between "dependencies" and "trustedDependencies". Ensure the
key name matches exactly ("obsidian-headless") and preserve version string
"^0.0.8".
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: fc35e725-dfba-45f8-97f0-0c5ec55df2bc
⛔ Files ignored due to path filters (1)
bun.lockis excluded by!**/*.lock
📒 Files selected for processing (1)
package.json
| "get-shit-done-cc": "^1.34.2", | ||
| "mcp-remote": "^0.1.38", | ||
| "mcporter": "^0.8.1", | ||
| "obsidian-headless": "^0.0.8", |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
set -euo pipefail
echo "Dependencies missing from trustedDependencies:"
comm -23 \
<(jq -r '.dependencies | keys[]' package.json | sort) \
<(jq -r '.trustedDependencies[]' package.json | sort)
echo
echo "If output includes obsidian-headless, decide whether to add it:"
echo 'jq '"'"'.trustedDependencies += ["obsidian-headless"] | .trustedDependencies |= unique'"'"' package.json'Repository: shunkakinoki/dotfiles
Length of output: 172
🏁 Script executed:
cat -n package.json | head -100Repository: shunkakinoki/dotfiles
Length of output: 3687
🏁 Script executed:
cat -n package.json | tail -50Repository: shunkakinoki/dotfiles
Length of output: 1472
Add obsidian-headless to trustedDependencies.
obsidian-headless is present in dependencies but missing from trustedDependencies. All other plain-name dependencies are included in both lists.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@package.json` at line 56, The package.json lists "obsidian-headless" in
dependencies but not in trustedDependencies; update the package.json object so
that the plain-name "obsidian-headless" appears in the trustedDependencies array
alongside the other plain-name deps (mirror how entries like "someOtherDep" are
represented) to ensure consistency between "dependencies" and
"trustedDependencies". Ensure the key name matches exactly ("obsidian-headless")
and preserve version string "^0.0.8".
There was a problem hiding this comment.
Code Review
This pull request updates several dependencies, including @anthropic-ai/claude-code, @github/copilot, and @openai/codex, and adds obsidian-headless to the project. The reviewer pointed out that obsidian-headless is missing from the trustedDependencies list, which is necessary for consistent behavior with Bun lifecycle scripts.
| "get-shit-done-cc": "^1.34.2", | ||
| "mcp-remote": "^0.1.38", | ||
| "mcporter": "^0.8.1", | ||
| "obsidian-headless": "^0.0.8", |
There was a problem hiding this comment.
The new dependency obsidian-headless is missing from the trustedDependencies list (lines 74-130). Since this project appears to maintain a comprehensive list of trusted dependencies to allow Bun lifecycle scripts, obsidian-headless should be added there to ensure consistent behavior and successful installation.
There was a problem hiding this comment.
Pull request overview
Updates the project’s Node/Bun dependencies to newer versions and introduces a new CLI dependency for headless Obsidian usage.
Changes:
- Bump versions for multiple existing dependencies (e.g., claude-code, GitHub Copilot, OpenAI Codex, oh-my-pi, openspec, openclaw).
- Add the new
obsidian-headlessdependency.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "get-shit-done-cc": "^1.34.2", | ||
| "mcp-remote": "^0.1.38", | ||
| "mcporter": "^0.8.1", | ||
| "obsidian-headless": "^0.0.8", |
There was a problem hiding this comment.
obsidian-headless was added to dependencies, but it is not included in the trustedDependencies allowlist later in this file (which otherwise appears to mirror the dependencies list). Please add obsidian-headless to trustedDependencies as well to keep the install/trust configuration consistent.
Update multiple packages including claude-code, github/copilot, openai/codex, oh-my-pi packages, and other dependencies. Add new obsidian-headless package.
Summary by cubic
Updates multiple dependencies to current minor/patch versions and adds
obsidian-headlessto extend headless note support. Lockfile refreshed for consistency.obsidian-headless^0.0.8@anthropic-ai/claude-codeto ^2.1.104@github/copilotto ^1.0.24@openai/codexto ^0.120.0@oh-my-pi/pi-coding-agentand@oh-my-pi/pi-nativesto ^14.1.0@fission-ai/openspecto ^1.3.0 and@sourcegraph/ampto ^0.0.1775969639-g317d38@typescript/native-previewto ^7.0.0-dev.20260411.1 andopenclawto ^2026.4.11Written for commit 8d04a6d. Summary will update on new commits.