fix: pluginLLMsPostprocess: honor "base" config #526
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
✅ Deploy Preview for lynx-doc ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify project configuration. |
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR refactors the llms-postprocess plugin to support configurable URL prefixes and base paths, enabling dynamic link generation for different deployment environments. The changes convert hardcoded absolute URLs to a parameterized approach and update AGENTS.md files to use relative links.
Key changes:
- Adds
prefixandbaseparameters topostprocessLLMsfunction for flexible URL construction - Introduces
COMMON_OPTIONSconstant for consistent markdown formatting options - Converts all links in AGENTS.md files from absolute (
https://lynxjs.org/next/...) to relative paths (/guide/...)
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
plugins/llms-postprocess/src/postprocess.ts |
Adds URL parameterization with prefix/base parameters, introduces COMMON_OPTIONS, implements link normalization and absolute path construction logic |
plugins/llms-postprocess/src/index.ts |
Updates function call to pass hardcoded prefix and config-derived base parameter |
plugins/llms-postprocess/scripts/main.ts |
Updates function call with hardcoded prefix and version-specific base path |
docs/public/zh/AGENTS.md |
Converts all absolute documentation links to relative paths |
docs/public/AGENTS.md |
Converts all absolute documentation links to relative paths |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
hzy
force-pushed
the
p/hzy/docs-mcp-server-3
branch
2 times, most recently
from
12eba1e to
f409e75
Compare
Change-Id: I608aa99fec95a9d3c09284f92a86bc7391af164b
hzy
force-pushed
the
p/hzy/docs-mcp-server-3
branch
from
f409e75 to
379d5a8
Compare
colinaaa
approved these changes
Nov 11, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Change-Id: I608aa99fec95a9d3c09284f92a86bc7391af164b