Skip to content

Conversation

@N-45div
Copy link

@N-45div N-45div commented Dec 6, 2025

Summary

Adds TypeDoc-based API documentation generation for the @solana/client package, addressing #85.

Changes

  • TypeDoc configuration: packages/client/typedoc.json
  • New scripts: docs and docs:json for generating API documentation
  • CI workflow: .github/workflows/docs.yml to auto-publish docs to GitHub Pages
  • Turbo integration: Added docs and docs:json tasks

Generated Output

  • Markdown docs: Human-readable API documentation
  • JSON API spec: Machine-readable api.json (1.4MB) for tooling integration

How to Test Locally

pnpm install
pnpm --filter @solana/client build
pnpm --filter @solana/client docs
# Check packages/client/docs/

Closes #85

- Add TypeDoc configuration for @solana/client package
- Add docs and docs:json scripts for generating API documentation
- Add CI workflow to build and publish docs to GitHub Pages
- Generate markdown and JSON API specs from TypeScript/JSDoc
- Add docs task to turbo.json

Closes solana-foundation#85
Copy link
Collaborator

@GuiBibeau GuiBibeau left a comment

Choose a reason for hiding this comment

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

i think deploying to github pages is not needed for now, we need the code to get the typedocs done but we'll have another place to export them soon

- Remove workflow_dispatch trigger
- Remove GitHub Pages deployment job
- Keep TypeDoc generation with artifact upload
- Docs will be deployed to solana.com in future PR
@N-45div N-45div requested a review from GuiBibeau December 16, 2025 17:14
@N-45div
Copy link
Author

N-45div commented Dec 16, 2025

Thanks for the feedback! I've updated the workflow:

  • Removed workflow_dispatch trigger
  • Removed GitHub Pages deployment
  • Docs are now generated and uploaded as artifacts for future integration with solana.com

Ready for re-review!

@GuiBibeau
Copy link
Collaborator

Incredible! Can you generate a patch changeset for this and we'll ship it? @N-45div

@N-45div
Copy link
Author

N-45div commented Dec 16, 2025

Incredible! Can you generate a patch changeset for this and we'll ship it? @N-45div

I have added the same !

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.

[Feature]: Publish OpenAPI specs

2 participants