Skip to content

Conversation

@olliethedev
Copy link
Collaborator

@olliethedev olliethedev commented Nov 21, 2025

Changes in this PR:

  • ai chat plugin using AI-SDK v5

Note

Extracts a shared markdown renderer and UI primitives into the UI package, updates the blog plugin to use them with new attribution, and sets OPENAI_API_KEY for e2e tasks.

  • UI:
    • Shared Markdown Renderer: Adds @workspace/ui/components/markdown-content with math support, syntax highlighting, code block toolbar/copy, heading anchors, task lists, and pluggable LinkComponent/ImageComponent; includes markdown-content.css and export in packages/ui.
    • New Primitives: Introduces dropdown-menu, scroll-area, sheet, and tooltip components.
    • Attribution: Adds BetterStackAttribution component.
  • Blog Plugin:
    • Replaces custom markdown renderer with wrapper around shared MarkdownContent (imports styles and highlight theme).
    • Switches attribution to BetterStackAttribution.
  • Config/Tooling:
    • turbo.json: Adds OPENAI_API_KEY to e2e:* task env.
    • Updates packages/ui dependencies/peer deps to support markdown rendering and styling.

Written by Cursor Bugbot for commit 9921177. This will update automatically on new commits. Configure here.

@vercel
Copy link

vercel bot commented Nov 21, 2025

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

Project Deployment Review Updated (UTC)
better-stack-docs Ready Ready Preview, Comment Dec 14, 2025 8:12pm

## Available Documentation Pages
${pages.map((page) => `- ${page.data.title}: /docs${page.url}.mdx`).join('\n')}
${pages.map((page) => `- ${page.data.title}: /docs${page.url==="/" ? "index" : page.url}.mdx`).join('\n')}
Copy link

Choose a reason for hiding this comment

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

Bug: Malformed URL generated for root documentation page

When page.url equals "/", the template literal /docs${page.url==="/" ? "index" : page.url}.mdx produces /docsindex.mdx instead of /docs/index.mdx. The ternary correctly substitutes "index" for the root page, but the template is missing the / separator before the substitution. Non-root pages work correctly because their URLs like "/installation" already start with /. The correct template would be /docs/${page.url==="/" ? "index" : page.url.slice(1)}.mdx or similar to ensure consistent slash handling.

Fix in Cursor Fix in Web

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