Skip to content

docs(v2): API reference and response corrections (2/3) - #310

Draft
SohamRatnaparkhi wants to merge 8 commits into
mainfrom
docs/pro-2457-api-response-cleanup
Draft

SohamRatnaparkhi wants to merge 8 commits into
mainfrom
docs/pro-2457-api-response-cleanup

Conversation

@SohamRatnaparkhi

@SohamRatnaparkhi SohamRatnaparkhi commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

The V2 API reference had wrong defaults, response shapes, and endpoint behavior, plus repeated notes that said the same thing two or three times. This PR keeps the original explanations and examples, fixes those claims against the backend, and removes the repetition.

Scope: api-reference/v2/ and essentials/v2/api-results.mdx. This is the API part of PRO-2457, next to Guides #309 and Cookbooks #311.

Fixes checked against the code:

  • SDKs: removed the Versioning section (no X-API-Version header exists). Polling now uses ready_for_ingestion.
  • Databases: the stats row now says it returns row counts. Added the metadata schema read endpoint. Removed the false MongoDB index warning on schema updates.
  • Ingest Context: the app_knowledge example uses kind, provider, external_id and fields. Item database and collection are optional.
  • Update Source Metadata: tenant replaced by database.
  • Query overview: the personalized recipe says to list both collections in collections when shared docs live elsewhere.
  • Error Responses: troubleshooting no longer names the deprecated tenant_metadata.

Repetition removed:

  • Query: the alias Note and the Default Behaviors block that restated the field table.
  • Duplicate warnings on Delete Database, Delete Collection, and Update Metadata Schema.
  • Duplicate notes on Submit Feedback and Error Responses.

Connector and webhook pages were read line by line and left as they are.

Navigation: the API Reference sidebar now opens with a "Start here" group holding the four calls most integrations are built on (Create Database, Ingest Context, Ingestion Status, Query) plus the overview. The resource groups still list every endpoint, so browsing by resource works as before. SDKs and Error Responses move to a Reference group. The overview page leads with the same four calls.

Validation: MDX compile, Mintlify build validation, broken links, hygiene, and the stale-claim check all pass.

🤖 Generated with Claude Code

Signed-off-by: SohamRatnaparkhi <soham@hydradb.com>
@mintlify

mintlify Bot commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated
cortex-ai 🟢 Ready View Preview Oct 8, 2026, 10:39 AM

💡 Tip: Enable Automations to automatically generate PRs for you.

@github-actions

github-actions Bot commented Oct 5, 2026

Copy link
Copy Markdown

✅ Mintlify Hygiene

No issues found.

@openhack-agent

openhack-agent Bot commented Oct 5, 2026 •

Copy link
Copy Markdown

✅ OpenHack Summary

Security review of docs(v2): API reference and response corrections (2/3). 37 changed files; 0 findings at or above the low reporting threshold.

P1: Critical 0   P2: High 0   P3: Medium 0   P4: Low 0

Confidence Score: 5/5

No reportable security findings were detected in this scan.

Security merge-readiness rubric: 1 = critical, 2 = high, 3 = medium, 4 = low, 5 = no reportable findings. This score reflects scan findings, not a guarantee of correctness or complete coverage.

Files Needing Attention: None

Important Files Changed
  • api-reference/v2/endpoint/configure-connector.mdx (modified)
  • api-reference/v2/endpoint/connectors-overview.mdx (modified)
  • api-reference/v2/endpoint/create-connector.mdx (modified)
  • api-reference/v2/endpoint/create-tenant.mdx (modified)
  • api-reference/v2/endpoint/delete-collection.mdx (modified)
  • api-reference/v2/endpoint/delete-connector.mdx (modified)
  • api-reference/v2/endpoint/delete-source.mdx (modified)
  • api-reference/v2/endpoint/delete-tenant.mdx (modified)
  • api-reference/v2/endpoint/fetch-content.mdx (modified)
  • api-reference/v2/endpoint/ingest-context.mdx (modified)
  • api-reference/v2/endpoint/list-connector-providers.mdx (modified)
  • api-reference/v2/endpoint/list-documents.mdx (modified)
  • api-reference/v2/endpoint/list-sub-tenants.mdx (modified)
  • api-reference/v2/endpoint/list-tenants.mdx (modified)
  • api-reference/v2/endpoint/list-webhook-deliveries.mdx (modified)
  • api-reference/v2/endpoint/query-overview.mdx (modified)
  • api-reference/v2/endpoint/query.mdx (modified)
  • api-reference/v2/endpoint/register-webhook.mdx (modified)
  • api-reference/v2/endpoint/retry-webhook-delivery.mdx (modified)
  • api-reference/v2/endpoint/source-relations.mdx (modified)
  • api-reference/v2/endpoint/source-status.mdx (modified)
  • api-reference/v2/endpoint/sources-overview.mdx (modified)
  • api-reference/v2/endpoint/subgraph.mdx (modified)
  • api-reference/v2/endpoint/submit-feedback.mdx (modified)
  • api-reference/v2/endpoint/tenant-stats.mdx (modified)

View all 37 changed files

Last reviewed commit: 1afa03f · View review on OpenHack


TIP: Mention @openhack-agent in a PR comment to request a review or ask a question. Use @openhack-agent fix all for every finding, or @openhack-agent fix unresolved threads for open review threads only.

@openhack-agent openhack-agent Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

OpenHack reviewed this commit. See the OpenHack Summary for the confidence score and fix actions.

Read every API Reference page and api-results line by line, keeping the
original voice and cutting only repetition and wrong claims:

- Query: drop the alias Note and the Default Behaviors block that restated
  the field table; split the indexing and collection-scope advice; link
  Recommended configurations by name.
- Query overview: new description, removed the duplicate Tip, and the
  personalized recipe now says to list both collections when shared docs
  live elsewhere. Thinking mode always includes graph context.
- Submit Feedback: removed a Note that repeated the field rules.
- Error Responses: removed the E6001 Note that repeated the table, and the
  troubleshooting bullet no longer names the deprecated tenant_metadata.
- Labels end with colons throughout.

Connector and webhook pages were read and left as they are.

Refs PRO-2457

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Signed-off-by: SohamRatnaparkhi <soham@hydradb.com>
Signed-off-by: SohamRatnaparkhi <soham@hydradb.com>
@SohamRatnaparkhi

Copy link
Copy Markdown
Contributor Author

@greptileai Please review the current head, including the V2 OpenAPI changes.

@greptile-apps

greptile-apps Bot commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

[Low risk] Documentation corrections and clarifications across API reference.

The PR appears safe to merge; no new actionable issue or outstanding previous finding remains.

Summary

The PR corrects v2 API request and response guidance, examples, and OpenAPI schemas, then makes retrieval results easier to approach before the advanced graph fields.

  • Aligns ingestion, query, status, metadata, and error documentation with the revised examples and specification.
  • Introduces context, chunks, and connected subgraphs in plainer language while retaining the detailed response reference in an expandable section.
  • Greptile automatically discovered a related ticket that helped explain the purpose of this PR: it calls for a final pass on v2 API response examples and the OpenAPI specification.

Reviews (4) · Last reviewed commit: "docs(v2): explain API results before adv..." · Reviewed by Greptile

Comment thread api-reference/v2/endpoint/subgraph.mdx Outdated
Comment thread api-reference/v2/endpoint/query.mdx
Comment thread api-reference/v2/endpoint/update-metadata-schema.mdx
@greptile-apps

This comment has been minimized.

Signed-off-by: SohamRatnaparkhi <soham@hydradb.com>
@SohamRatnaparkhi

Copy link
Copy Markdown
Contributor Author

@greptileai Please review the current head.

  • The PATCH metadata-schema example now adds a VARCHAR field without dense/sparse search flags. Ignored enable_match fields are also removed from all OpenAPI examples.
  • Query defaults and unsupported filtering flags in the linked Guides are corrected in the separate Guides PR docs(v2): Guides cleanup (1/3) #309, as requested by PRO-2457's three-PR scope. Guides is the first stage; the API Reference is the second. Keeping those guide files in docs(v2): Guides cleanup (1/3) #309 avoids duplicating changes across the independent PRs.
  • The subgraph route statement is verified against published Python and TypeScript SDK 2.1.7, rather than inferred from this older OpenAPI snapshot: Python hydra_db/context/raw_client.py calls context/subgraph with id in query parameters; TypeScript dist/api/resources/context/client/Client.js does the same. The backend registers both /context/:id/subgraph and /context/subgraph. The wording now identifies the verified SDK version explicitly. The OpenAPI page's path route and the SDK's query-string route are both supported.
  • Rendered-preview inspection also exposed success examples containing a non-null error. Their schemas now model error as an object or null and use a null success example. Scope aliases carry matching values, and the ingest sample's success count matches its item count.

Mintlify validation, MDX/frontmatter, Python/JSON parsing, and local schema reference checks pass.

Comment thread api-reference/v2/endpoint/fetch-content.mdx
Signed-off-by: SohamRatnaparkhi <soham@hydradb.com>
Signed-off-by: SohamRatnaparkhi <soham@hydradb.com>
@SohamRatnaparkhi

Copy link
Copy Markdown
Contributor Author

@greptileai Please reassess the current head and the inline responses. The fetch schema now accepts the null fields emitted by content and URL modes; those samples pass schema validation. SDK route and separate Guides-scope findings have implementation evidence in their threads.

Signed-off-by: SohamRatnaparkhi <soham@hydradb.com>
@SohamRatnaparkhi

Copy link
Copy Markdown
Contributor Author

@greptileai Please review current head 4d42afe. This pass explains context and chunks before advanced terms, clarifies relationship/subgraph terminology, and keeps the full API Results response reference in an expandable section before the prompt-formatting examples. It preserves request examples and endpoint contracts. Local build validation and hosted validation, hygiene, and link checks pass.

Most integrations are create database, ingest, check status, query.
The sidebar now opens with a "Start here" group holding exactly those
four endpoints and the overview, and the overview page leads with the
same four calls before the endpoint groups and the full inventory.
The resource groups still list every endpoint, so browsing by resource
works as before. SDKs and Error Responses move to a Reference group.

Refs PRO-2457

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Signed-off-by: SohamRatnaparkhi <soham@hydradb.com>

This branch was successfully deployed

1 active deployment
staging — 1afa03f1 Deployed Oct 8, 2026 by mintlify[bot]
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.

1 participant