Skip to content

Docs update#236

Merged
montalvomiguelo merged 5 commits intomainfrom
docs-update-011926
Jan 19, 2026
Merged

Docs update#236
montalvomiguelo merged 5 commits intomainfrom
docs-update-011926

Conversation

@montalvomiguelo
Copy link
Copy Markdown
Collaborator

@montalvomiguelo montalvomiguelo commented Jan 19, 2026

Summary by CodeRabbit

  • Documentation
    • Expanded troubleshooting guide with clearer tunneling guidance and explicit instructions for enabling tunnel support, including an ngrok example.
    • Noted automatic configuration when tunneling is enabled and how custom server settings take precedence.
    • Broadened host allowances and CORS origin examples to cover localhost/loopback and Shopify domain variants for local and remote testing.
    • Improved configuration examples and code-block formatting for step-by-step setup.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Jan 19, 2026

Warning

Rate limit exceeded

@montalvomiguelo has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 14 minutes and 53 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 65aae1c and 1dcfc99.

📒 Files selected for processing (1)
  • docs/guide/troubleshooting.md
📝 Walkthrough

Walkthrough

Updated troubleshooting docs to show a Vite dev-server config that enables the Shopify plugin tunnel (shopify({ tunnel: true })), adds server.allowedHosts: ['.trycloudflare.com'], and expands server.cors.origin to include a localhost/loopback regex and /.+\.myshopify\.com$/.

Changes

Cohort / File(s) Summary
Documentation - Vite config example
docs/guide/troubleshooting.md
Rewrote Vite dev-server example to enable shopify({ tunnel: true }), add server.allowedHosts: ['.trycloudflare.com'], and replace defaultAllowedOrigins with explicit server.cors.origin entries (including a localhost/loopback regex and /.+\.myshopify\.com$/). Reordered and reformatted code blocks; added ngrok tunneling example and a "Tip Automatic Configuration (v4.1.0+)" note.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

🐰 I nudged the docs, a hop in the night,
tunnel on, localhost shining bright,
regex stitched, Cloudflare at the door,
devs can test without CORS uproar,
nibble on code and dream some more 🥕✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'Docs update' is too vague and generic, failing to convey what specific documentation changes were made, such as tunnel configuration or server settings. Use a more specific title that describes the actual changes, such as 'Add tunnel configuration and server settings examples to troubleshooting guide'.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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.

@netlify
Copy link
Copy Markdown

netlify bot commented Jan 19, 2026

Deploy Preview for bespoke-flan-5eacfc ready!

Name Link
🔨 Latest commit 1dcfc99
🔍 Latest deploy log https://app.netlify.com/projects/bespoke-flan-5eacfc/deploys/696e6614baca5000077dfe02
😎 Deploy Preview https://deploy-preview-236--bespoke-flan-5eacfc.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@docs/guide/troubleshooting.md`:
- Around line 125-142: The snippet is missing top-level imports and the closing
braces for the exported config; add imports for defineConfig, shopify and your
Tailwind plugin (referencing defineConfig, shopify, tailwindcss) at the top,
then close the default export object and the file (replace the stray
build.rollupOptions fragment with either a completed build.rollupOptions object
or a comment like "// build.rollupOptions omitted for brevity" if you intend to
continue it) so the file parses and the export default defineConfig({...}) is
properly closed.
🧹 Nitpick comments (1)
docs/guide/troubleshooting.md (1)

135-135: Consider strengthening the myshopify.com CORS pattern.

The current regex /\.myshopify\.com$/ doesn't enforce HTTPS or anchor at the start of the origin. Since Shopify stores use HTTPS, consider a more restrictive pattern:

-        /\.myshopify\.com$/,
+        /^https:\/\/[^\/]+\.myshopify\.com$/,

This ensures:

  • Only HTTPS connections are allowed
  • Proper start-of-string anchoring
  • Prevents potential edge cases with malformed origins

@montalvomiguelo montalvomiguelo merged commit bf227be into main Jan 19, 2026
11 checks passed
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