Skip to content

docs: Restructure API reference navigation#199

Open
sanjeed5 wants to merge 12 commits intogenerate-v1.xfrom
docs/api-reference-navigation
Open

docs: Restructure API reference navigation#199
sanjeed5 wants to merge 12 commits intogenerate-v1.xfrom
docs/api-reference-navigation

Conversation

@sanjeed5
Copy link
Contributor

@sanjeed5 sanjeed5 commented Jan 29, 2026

Updated PR description

Summary

  • Restructured the Python SDK API reference navigation into product-aligned sections: Overview, Tracing, Platform Client, Experiments, and Configuration
  • Added reference/overview with a simple decision table to help users find the right SDK surface area quickly
  • Split the platform client docs into per-resource pages under reference/client/ (datasets, datapoints, projects, sessions, events, metrics, configurations, tools, experiments)
  • Added a dedicated reference/tracing/ section (tracer, decorators, enrichment) and linked to existing tracer architecture + config model reference pages
  • Updated cross-references that previously pointed at the removed monolithic reference/api/client-apis page

Docs tooling / CI follow-ups included

  • Removed sphinx-tabs (unmaintained / docutils 0.22 breakage) and converted tabs to plain sections
  • Removed sphinx-search from docs/requirements.txt (package doesn’t exist on PyPI)
  • Updated docs preview workflow to install dependencies via docs/requirements.txt (instead of hardcoded installs)
  • Suppressed noisy ambiguous Python xref warnings and fixed a couple ambiguous type references that were generating Sphinx warnings

Test plan

  • Build docs locally from a fresh environment
  • Verify the new reference navigation renders and the new section index pages are reachable

sanjeed5 and others added 5 commits January 29, 2026 18:38
- Replace confusing nested "Core API" structure with clear product-aligned sections
- Add Overview page with decision table for choosing SDK components
- Split Platform Client APIs into individual pages (Datasets, Datapoints, etc.)
- Create dedicated Tracing section (HoneyHiveTracer, Decorators, Enrichment)
- Create dedicated Experiments and Configuration sections
- Remove duplicate orphaned files from reference root
- Simplify reference pages to pure autodoc (examples belong in tutorials)

Co-authored-by: Cursor <cursoragent@cursor.com>
…-apis references

Co-authored-by: Cursor <cursoragent@cursor.com>
…lts.rst

Co-authored-by: Cursor <cursoragent@cursor.com>
…ference warning

Co-authored-by: Cursor <cursoragent@cursor.com>
docutils 0.22 removed 'backrefs' attribute from nodes, which breaks
sphinx_tabs that does attrs.pop('backrefs'). Pin to <0.22 until
sphinx_tabs is updated to handle the change.

Co-authored-by: Cursor <cursoragent@cursor.com>
@sanjeed5 sanjeed5 force-pushed the docs/api-reference-navigation branch from b673ff9 to 2a23bb0 Compare January 29, 2026 19:49
@snyk-io
Copy link

snyk-io bot commented Jan 29, 2026

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

sanjeed5 and others added 3 commits January 30, 2026 02:28
Replace tab directives with sequential bold sections showing
different configuration approaches. Tabs were a cosmetic
feature that added a brittle dependency.

Files changed:
- docs/reference/api/tracer.rst
- docs/reference/configuration/config-options.rst
- docs/tutorials/advanced-configuration.rst

Co-authored-by: Cursor <cursoragent@cursor.com>
- Remove sphinx_tabs.tabs from conf.py extensions
- Remove sphinx-tabs from requirements.txt
- Remove non-existent sphinx-search package
- Remove docutils<0.22 pin (no longer needed)

sphinx-tabs was unmaintained and broken with docutils 0.22.
sphinx-search package never existed on PyPI.

Co-authored-by: Cursor <cursoragent@cursor.com>
Replace hardcoded pip install commands with single
pip install -r docs/requirements.txt for consistency
between local and CI builds.

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions
Copy link
Contributor

📚 Documentation Preview Built

Documentation preview is ready!

📦 Download Preview

Download documentation artifact

🔍 How to Review

  1. Download the artifact from the link above
  2. Extract the files
  3. Open index.html in your browser

✅ Validation Status

  • API validation: ✅ Passed
  • Build process: ✅ Successful
  • Import tests: ✅ All imports working

Preview generated for PR #199

Revert commit 03eda3d which added:
- Type annotation removal from trace_class (code change)
- Example files (not docs-related)

These belong in a separate PR, not in a docs restructure PR.

Co-authored-by: Cursor <cursoragent@cursor.com>
@sanjeed5 sanjeed5 marked this pull request as ready for review January 29, 2026 21:37
@cursor
Copy link

cursor bot commented Jan 29, 2026

You have run out of free Bugbot PR reviews for this billing cycle. This will reset on February 5.

To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

Fixed Sphinx ambiguous cross-reference warning for 'type' parameter in
trace_class decorator docstring by explicitly specifying the type hint
in the Args section. This prevents Sphinx from trying to cross-reference
to model .type attributes.

Co-authored-by: Cursor <cursoragent@cursor.com>
@sanjeed5 sanjeed5 force-pushed the docs/api-reference-navigation branch from 20dfa27 to b0a913e Compare January 29, 2026 22:00
Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions
Copy link
Contributor

📚 Documentation Preview Built

Documentation preview is ready!

📦 Download Preview

Download documentation artifact

🔍 How to Review

  1. Download the artifact from the link above
  2. Extract the files
  3. Open index.html in your browser

✅ Validation Status

  • API validation: ✅ Passed
  • Build process: ✅ Successful
  • Import tests: ✅ All imports working

Preview generated for PR #199

@sanjeed5 sanjeed5 requested a review from dhruv-hhai February 5, 2026 14:59
Remove the broad Sphinx warning suppression for Python cross-references so broken
API references surface during docs builds.

Co-authored-by: Cursor <cursoragent@cursor.com>
Overview
========

This reference documents the HoneyHive Python SDK's public API.
Copy link
Contributor

Choose a reason for hiding this comment

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

Add a ref to the python SDK version here if possible + link to the github releases for accessing changelog.

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.

2 participants