Skip to content

Conversation

@jupblb
Copy link
Member

@jupblb jupblb commented Oct 22, 2025

Our GraphQL API docs are severely outdated. They were originally introduced in https://github.com/sourcegraph/sourcegraph/commit/f97e8587f9eb8e4e56809e4ac372703a1124e1bd. Since then there have been only very minor changes.

The original HTML was generated using spectaql which does look cool but cannot be nicely integrated with this project's infrastructure. It would seem our options are:

  1. Create some processing pipeline which automatically builds spectaql resources and parses them in a way that is fitting this website.
  2. Just make user use API console, which is already suggested.

I tried very hard to work on option (1) so that we may have something maintainable but to no luck. I did consider using some other tool or suggesting using separate web endpoint for GraphQL documentation but then I noticed another problem. The original documentation was only generated for schema.graphql. When it's generated for all of our API the output is just humongous. It's not very easy to navigate using a sidebar like in case of the current UX.

Therefore, I suggest we just go with option (2). As a follow-up I recommend we upgrade graphiql dependency in sourcegraph/sourcegraph.

- Merge API Console and Documentation sections for better clarity
- Improve text formatting with proper line breaks
- Simplify documentation access instructions

Amp-Thread-ID: https://ampcode.com/threads/T-f15773dd-bae2-4f44-8ced-0227fd2eb907
@jupblb jupblb requested a review from mmanela October 22, 2025 13:29
@jupblb jupblb self-assigned this Oct 22, 2025
@vercel
Copy link

vercel bot commented Oct 22, 2025

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

Project Deployment Preview Comments Updated (UTC)
sourcegraph-docs Ready Ready Preview Comment Oct 22, 2025 1:29pm

@jupblb jupblb merged commit e61e4b3 into main Oct 23, 2025
7 checks passed
@jupblb jupblb deleted the michal/attempt-3 branch October 23, 2025 14:07
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.

3 participants