Skip to content

[codex] Use site URL in onboarding prompt#1

Merged
sbddesign merged 1 commit intomainfrom
codex/domain-onboarding-prompt
Apr 26, 2026
Merged

[codex] Use site URL in onboarding prompt#1
sbddesign merged 1 commit intomainfrom
codex/domain-onboarding-prompt

Conversation

@sbddesign
Copy link
Copy Markdown
Contributor

@sbddesign sbddesign commented Apr 26, 2026

Summary

  • Expands the copy-paste onboarding prompt with the configured site URL from getSiteUrl().
  • Keeps /llms.txt and API route instructions absolute so pasted agent instructions include the domain and do not require follow-up clarification.

Validation

  • pnpm lint
  • pnpm build

Notes

The working tree has other pre-existing uncommitted changes that were not included in this branch commit.

Summary by CodeRabbit

  • Bug Fixes
    • Onboarding prompt now dynamically adjusts API and llms.txt paths to reference your current site URL. The copied prompt content includes properly configured endpoints for your deployment without requiring manual modifications.

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 26, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
freeport Ready Ready Preview, Comment Apr 26, 2026 0:35am

Request Review

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 26, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: a4f80e28-3e8e-434e-a35b-5666533c1cf0

📥 Commits

Reviewing files that changed from the base of the PR and between 373250f and 098daab.

📒 Files selected for processing (1)
  • components/onboarding-prompt.tsx

📝 Walkthrough

Walkthrough

The OnboardingPrompt component now dynamically rewrites API and llms.txt file paths within the onboarding prompt template by prefixing them with the current site URL. The transformed prompt is used for both clipboard content and display rendering.

Changes

Cohort / File(s) Summary
Onboarding Prompt Transformation
components/onboarding-prompt.tsx
Added logic to derive a site-aware prompt by rewriting API and llms.txt paths with the current site URL prefix (stripped of trailing slashes). Both the clipboard copy operation and rendered prompt content now reference this transformed prompt instead of the static constant.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 A prompt takes flight on silver wings,
Rewritten paths that context brings,
With URLs now woven through,
The onboarding feels fresh and new,
Site-aware wisdom, tried and true! ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely summarizes the main change: incorporating the site URL into the onboarding prompt component.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/domain-onboarding-prompt

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.

@sbddesign sbddesign marked this pull request as ready for review April 26, 2026 12:36
@sbddesign sbddesign merged commit 0e2ef16 into main Apr 26, 2026
3 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.

1 participant