Skip to content

docs(v2): API reference cleanup and working examples (2/3) - #310

Merged
SohamRatnaparkhi merged 35 commits into
mainfrom
docs/pro-2457-api-response-cleanup
Oct 22, 2026
Merged

SohamRatnaparkhi merged 35 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 exposed unreleased controls, described nested responses as opaque objects, and repeated long explanations. This PR corrects the public contract and makes the endpoint pages easier to scan.

  • Preserve metadata and metadata_filters. Hide unlaunched attributes, graph pruning and temporal intent controls, and internal query source blocks.
  • Type nested graph responses, source filters, and list comments. Correct nullable fields, Bearer authorization, and structured/Markdown query output descriptions.
  • Explain automatic collection creation. Shorten error explanations and three-column tables. Remove legacy aliases from handwritten tables while retaining deprecated fields in generated schemas.
  • Collapse file formats, link shared metadata guidance, remove duplicate response payloads, and show distinct knowledge/memory deletion responses.
  • Add a CI guard for schema references, hidden fields, customer metadata keys, endpoint bindings, and handwritten response examples.

The application source change is hydradb-application#2552. Its shared export policy produces this exact OpenAPI snapshot. Merge that source PR first so future generation preserves these corrections.

Validation passed for all 53 V2 API pages in the browser, 44 endpoint bindings, 81 JSON examples, and 58 response shapes. Affected pages were rechecked after each correction, including the two pages that retain combined list/delete requests. Mintlify build and hygiene pass. No V2 link or anchor errors were found; four pre-existing V1/archive anchor warnings remain outside this change.

Full TypeScript SDK generation and generated-code checks passed with the pinned generator in an isolated setup using a smaller Node heap. Ten Python/TypeScript helper tests pass, including real generated serialization, null preservation, and provider authentication over local HTTP. The repository generator configuration is unchanged. No SDK was published. Browser and SDK evidence are committed under .audit/.

Earlier production testing in this PR covered the 44 V2 operations using cURL and SDK 2.1.7 with authenticated GitHub connector fixtures. That is separate from this final schema and browser pass. Slack authentication and a real OpenAI model call remain unverified.

V2 only. Sidebar shortcuts are separate in #325. Part of PRO-2457.

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 21, 2026, 8:21 PM

💡 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). 52 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/add-connector-resource.mdx (modified)
  • api-reference/v2/endpoint/configure-connector.mdx (modified)
  • api-reference/v2/endpoint/connector-resources.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-resource.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/delete-webhook.mdx (modified)
  • api-reference/v2/endpoint/discover-connector-resources.mdx (modified)
  • api-reference/v2/endpoint/fetch-content.mdx (modified)
  • api-reference/v2/endpoint/get-connector-status.mdx (modified)
  • api-reference/v2/endpoint/get-connector.mdx (modified)
  • api-reference/v2/endpoint/get-webhook-delivery.mdx (modified)
  • api-reference/v2/endpoint/get-webhook.mdx (modified)
  • api-reference/v2/endpoint/ingest-context.mdx (modified)
  • api-reference/v2/endpoint/list-connector-providers.mdx (modified)
  • api-reference/v2/endpoint/list-connectors.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)

View all 52 changed files

Last reviewed commit: e75296c · 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

RetriggerView in GreptileConfidence Score: 5/5

[Medium impact] The changes since the last review appear safe to merge.

Summary

Cleans up the V2 API reference, updates the public OpenAPI contract, and adds checks for schema references and handwritten examples.

  • V2 context and query pages describe the released request fields and response shapes.
  • Connector pages show working examples and explain how resources and synced data fit together.
  • Database pages explain collection creation, readiness, and metadata schema rules more directly.
  • Webhook pages add examples and explain registration, signing, delivery history, and retries.
  • CI checks selected V2 documentation and API contract details.
  • Shared V2 guides and navigation use clearer contract details and examples.

Reviews (23) · Last reviewed commit: "docs(v2): retain combined list and delet..." · Reviewed by Greptile

Comment thread api-reference/v2/endpoint/subgraph.mdx Outdated
Comment thread api-reference/v2/endpoint/query.mdx Outdated
Comment thread api-reference/v2/endpoint/update-metadata-schema.mdx Outdated
@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>
Signed-off-by: SohamRatnaparkhi <soham@hydradb.com>
Signed-off-by: SohamRatnaparkhi <soham@hydradb.com>
Signed-off-by: SohamRatnaparkhi <soham@hydradb.com>
Signed-off-by: SohamRatnaparkhi <soham@hydradb.com>
Signed-off-by: SohamRatnaparkhi <soham@hydradb.com>
Signed-off-by: SohamRatnaparkhi <soham@hydradb.com>
Signed-off-by: SohamRatnaparkhi <soham@hydradb.com>
Signed-off-by: SohamRatnaparkhi <soham@hydradb.com>
Comment thread api-reference/v2/openapi.json Outdated
Comment thread api-reference/v2/openapi.json Outdated
Signed-off-by: SohamRatnaparkhi <soham@hydradb.com>
Signed-off-by: SohamRatnaparkhi <soham@hydradb.com>
Signed-off-by: SohamRatnaparkhi <soham@hydradb.com>
Signed-off-by: SohamRatnaparkhi <soham@hydradb.com>
@SohamRatnaparkhi
SohamRatnaparkhi force-pushed the docs/pro-2457-api-response-cleanup branch from 4ad0379 to cac3ba5 Compare October 21, 2026 20:20
@SohamRatnaparkhi
SohamRatnaparkhi merged commit 796bc2b into main Oct 22, 2026
9 checks passed
@SohamRatnaparkhi
SohamRatnaparkhi deleted the docs/pro-2457-api-response-cleanup branch October 22, 2026 04:26

This branch was successfully deployed

1 active deployment
staging — cac3ba55 Deployed Oct 21, 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