Skip to content

fix: open in llm dropdown #23145

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Jul 24, 2025
Merged

fix: open in llm dropdown #23145

merged 3 commits into from
Jul 24, 2025

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

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
2 participants