debug: Add react-grab#979
Conversation
WalkthroughThis PR adds react-grab debugging and development tools to the web package by introducing new npm packages and conditionally injecting development-time scripts into the Next.js application layout, enabled only in development environments. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes 🚥 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 |
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 `@packages/web/src/app/layout.tsx`:
- Around line 41-49: The dev-only Script tags in layout.tsx use
protocol-relative, unversioned CDN URLs; update the src values in the
process.env.NODE_ENV === "development" block (the Script components that load
react-grab and `@react-grab/mcp`) to explicit HTTPS and pinned versioned URLs that
match the versions in package.json (replace the //unpkg.com/... URLs with
https://unpkg.com/<package>@<version>/dist/....); keep the Script props
(strategy, crossOrigin if present) unchanged.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: c09b989a-3e89-4788-82ae-45de058e372d
⛔ Files ignored due to path filters (1)
yarn.lockis excluded by!**/yarn.lock,!**/*.lock
📒 Files selected for processing (2)
packages/web/package.jsonpackages/web/src/app/layout.tsx
|
@brendan-kellam your pull request is missing a changelog! |
Adds the new Meta message, Q.meta field, and Repository.metadata map that upstream zoekt added in sourcegraph/zoekt#962 and #979. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* chore: sync vendor/zoekt with upstream sourcegraph/zoekt Bumps the vendor/zoekt submodule to pull in 108 upstream commits from sourcegraph/zoekt. See sourcebot-dev/zoekt#10. The -shard_prefix flag on zoekt-git-index was renamed upstream to -shard_prefix_override; the backend indexer is updated accordingly. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * chore: add changelog entry for zoekt upstream sync Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * chore: regenerate web protos from upgraded zoekt Adds the new Meta message, Q.meta field, and Repository.metadata map that upstream zoekt added in sourcegraph/zoekt#962 and #979. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * chore: repoint vendor/zoekt at sourcebot-dev/zoekt@main merge commit sourcebot-dev/zoekt#10 merged as da9bf1a3. Updating the submodule pointer from the feature-branch tip (8566836f) to the merge commit on main so vendor/zoekt tracks the canonical history before merging. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Adds react-grab to our codebase at development time: https://x.com/aidenybai/status/2029603067927351506
Summary by CodeRabbit