Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

@bodynar
Copy link

@bodynar bodynar commented Nov 14, 2025

This pull request makes a small update to the documentation links in the generative UI section to improve navigation.

  • Changed the href attributes for the "Backend Tools", "Frontend Tools", and "Agent State" cards to use relative paths without a leading dot, ensuring consistent and correct routing in the documentation.

These links aren't working right now

Summary by CodeRabbit

  • Documentation
    • Updated navigation links in documentation by standardizing relative URL paths.

@changesets-bot-copilotkit
Copy link

⏭️ Changeset Not Required

Latest commit: 2a8992e

No changes in this PR affected the @copilitkit/* packages. Merging this PR will not cause a version bump for any packages.

Changeset is not required for this PR.

@vercel
Copy link

vercel bot commented Nov 14, 2025

@bodynar is attempting to deploy a commit to the CopilotKit Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 14, 2025

Walkthrough

Updated three relative URL references in a documentation MDX file by removing "./" prefix from href attributes, changing explicit relative paths (./backend-tools, ./frontend-tools, ./agentic) to implicit relative paths (backend-tools, frontend-tools, agentic).

Changes

Cohort / File(s) Summary
Documentation URL References
docs/content/docs/langgraph/generative-ui/index.mdx
Removed "./" prefix from three Card href attributes, changing explicit relative paths to implicit relative paths while maintaining target resolution

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

preview

Suggested reviewers

  • mme
  • ataibarkai
  • ranst91
  • tylerslaton

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title is vague and generic, using 'Update index.mdx' which doesn't convey the actual purpose of the changes—fixing documentation links in the generative UI section. Consider using a more descriptive title like 'Fix generative UI documentation card links' that clearly indicates the bug being fixed.
✅ 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.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1ca902c and 2a8992e.

📒 Files selected for processing (1)
  • docs/content/docs/langgraph/generative-ui/index.mdx (1 hunks)
🔇 Additional comments (1)
docs/content/docs/langgraph/generative-ui/index.mdx (1)

39-39: I need to verify that the "./" prefix was indeed present before (confirming it was changed) and understand how the documentation routing system handles these paths.

Let me verify the file history and search for build configuration to understand the path resolution:

Let me run a script to check the git history, build configuration, and verify the changes:Based on my verification, I have sufficient information to generate the final review comment. The script confirmed that all target files exist, and the href changes follow proper web standards for relative path resolution. Since this is documentation in an MDX file using regular HTML href attributes (not Next.js Link components), the changes from "./backend-tools" to "backend-tools" are both valid and equivalent according to HTML URL resolution standards. The PR states the previous links were broken, and this change fixes them.

All three target documentation files confirmed to exist. Path resolution changes are valid and consistent with documentation build practices.

The changes remove the "./" prefix from relative hrefs, which is correct for documentation routing. All three target files exist at the expected paths:

  • backend-tools.mdx
  • frontend-tools.mdx
  • agentic.mdx

For regular HTML anchor tags (not Next.js Link components), href="backend-tools" and href="./backend-tools" resolve identically in modern browsers. The PR description indicates the previous "./" paths were causing broken links, confirming this change addresses the actual issue.


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.

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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.