Skip to content

fix: avoid non-existent variables#703

Merged
RiskeyL merged 1 commit intolanggenius:mainfrom
hyoban:3-10-fix-variable
Mar 11, 2026
Merged

fix: avoid non-existent variables#703
RiskeyL merged 1 commit intolanggenius:mainfrom
hyoban:3-10-fix-variable

Conversation

@hyoban
Copy link
Copy Markdown
Member

@hyoban hyoban commented Mar 10, 2026

No description provided.

@hyoban hyoban requested a review from RiskeyL as a code owner March 10, 2026 03:39
Copilot AI review requested due to automatic review settings March 10, 2026 03:39
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Mar 10, 2026
@dosubot dosubot bot added the bug Something isn't working label Mar 10, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates MDX documentation to render placeholder braces (e.g., {api_key}, {id}) as literal text instead of being parsed as MDX/JSX expressions that can trigger “variable not defined” errors during build.

Changes:

  • Escape placeholder braces in Markdown tables (e.g., \{api_key\}, \{id\}).
  • Render brace-containing placeholders safely inside HTML/JSX tables using JSX string expressions (e.g., {"workflow_{id}"}, {'\"message_{id}\"'}).
  • Minor formatting cleanup in one CN doc line (indentation/line alignment).

Reviewed changes

Copilot reviewed 18 out of 18 changed files in this pull request and generated no comments.

Show a summary per file
File Description
zh/use-dify/workspace/api-extension/api-extension.mdx Escapes {api_key} placeholder in header table to avoid MDX expression parsing; minor formatting adjustment.
zh/use-dify/monitor/integrations/integrate-weave.mdx Escapes {id} placeholders in Markdown tables so they render literally.
zh/use-dify/monitor/integrations/integrate-langsmith.mdx Escapes {id} placeholders in Markdown tables (workflow/message name rows).
zh/use-dify/monitor/integrations/integrate-langfuse.mdx Uses JSX string rendering in HTML table cells to display workflow_{id} / message_{id} literally.
ja/use-dify/workspace/api-extension/api-extension.mdx Escapes {api_key} placeholder in header table to avoid MDX expression parsing.
ja/use-dify/monitor/integrations/integrate-weave.mdx Escapes {id} placeholders in Markdown tables so they render literally.
ja/use-dify/monitor/integrations/integrate-phoenix.mdx Uses JSX string rendering in HTML table cell to display workflow_{id} literally.
ja/use-dify/monitor/integrations/integrate-opik.mdx Uses JSX string rendering in HTML table cell to display workflow_{id} literally.
ja/use-dify/monitor/integrations/integrate-langsmith.mdx Uses JSX string rendering in HTML table cells for workflow_{id} and "message_{id}".
ja/use-dify/monitor/integrations/integrate-langfuse.mdx Uses JSX string rendering in HTML table cells to display workflow_{id} / message_{id} literally.
ja/use-dify/monitor/integrations/integrate-arize.mdx Uses JSX string rendering in HTML table cell to display workflow_{id} literally.
en/use-dify/workspace/api-extension/api-extension.mdx Escapes {api_key} placeholder in header table to avoid MDX expression parsing.
en/use-dify/monitor/integrations/integrate-weave.mdx Escapes {id} placeholders in Markdown tables so they render literally.
en/use-dify/monitor/integrations/integrate-phoenix.mdx Uses JSX string rendering in HTML table cell to display workflow_{id} literally.
en/use-dify/monitor/integrations/integrate-opik.mdx Uses JSX string rendering in HTML table cell to display workflow_{id} literally.
en/use-dify/monitor/integrations/integrate-langsmith.mdx Uses JSX string rendering in HTML table cells for workflow_{id} and "message_{id}".
en/use-dify/monitor/integrations/integrate-langfuse.mdx Uses JSX string rendering in HTML table cells to display workflow_{id} / message_{id} literally.
en/use-dify/monitor/integrations/integrate-arize.mdx Uses JSX string rendering in HTML table cell to display workflow_{id} literally.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Mar 11, 2026
@RiskeyL RiskeyL merged commit a958852 into langgenius:main Mar 11, 2026
4 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working lgtm This PR has been approved by a maintainer size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants