Skip to content

Conversation

@sarahsanders-docker
Copy link
Contributor

@sarahsanders-docker sarahsanders-docker commented Jul 23, 2025

Description

  • The original implementation was trying to pass the entire content of the documentation page through the URL parameters
  • Web servers/browsers have limits on URLs
  • Instead of passing the whole page as a URL parameter, the fix prompts Claude with a reference to the markdown file, putting the fetching and parsing onto the LLM, not the browser (which can't handle long files/pages)
  • Removing ChatGPT option, as OpenAI appears to no longer support reading RAW markdown files (I tested this using our site and several other docs sites)

Related issues or tickets

Fixes #23144

Reviews

  • Editorial review

Testing

Claude
image

@netlify
Copy link

netlify bot commented Jul 23, 2025

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit cbfb62c
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/688238a5eb4a94000889ec1c
😎 Deploy Preview https://deploy-preview-23145--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions bot added the hugo Updates related to hugo label Jul 23, 2025
@sarahsanders-docker sarahsanders-docker added the status/do-not-merge Pull requests that are awaiting some event or decision before they can be merged. label Jul 23, 2025
@ArthurFlag
Copy link
Contributor

ArthurFlag commented Jul 24, 2025

The trade-off is that we cannot use these features locally or from preview deployments, so maybe we should remove these 2 buttons from non-production builds. Wdyt?

@sarahsanders-docker
Copy link
Contributor Author

The trade-off is that we cannot use these features locally or from preview deployments, so maybe we should remove these 2 buttons from non-production builds. Wdyt?

added the production environment variable in the latest commit! good idea :)

@sarahsanders-docker sarahsanders-docker merged commit 86dccb6 into main Jul 24, 2025
15 checks passed
@sarahsanders-docker sarahsanders-docker deleted the fix-open-llms branch July 24, 2025 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hugo Updates related to hugo status/do-not-merge Pull requests that are awaiting some event or decision before they can be merged. status/review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] “Open in ChatGPT” link fails with 414/431 error

3 participants