Repository navigation
docs(v2): API reference and response corrections (2/3) - #310
SohamRatnaparkhi wants to merge 12 commits into
Conversation
Signed-off-by: SohamRatnaparkhi <soham@hydradb.com>
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Automations to automatically generate PRs for you. |
✅ Mintlify HygieneNo issues found. |
✅ OpenHack SummarySecurity review of docs(v2): API reference and response corrections (2/3). 52 changed files; 0 findings at or above the low reporting threshold. Confidence Score: 5/5No 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
Last reviewed commit: e75296c · View review on OpenHack
|
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>
045b2cf to
62b470f
Compare
Signed-off-by: SohamRatnaparkhi <soham@hydradb.com>
|
@greptileai Please review the current head, including the V2 OpenAPI changes. |
|
This comment has been minimized.
This comment has been minimized.
Signed-off-by: SohamRatnaparkhi <soham@hydradb.com>
|
@greptileai Please review the current head.
Mintlify validation, MDX/frontmatter, Python/JSON parsing, and local schema reference checks pass. |
Signed-off-by: SohamRatnaparkhi <soham@hydradb.com>
Signed-off-by: SohamRatnaparkhi <soham@hydradb.com>
|
@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>
|
@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>
…Connectors - Webhooks: Overview, first in the Webhooks group, lists every webhook endpoint including the signing-secret calls, the typical sequence, and how deliveries are signed and retried (exponential backoff, 16 attempts or 48 hours on Cloud, per domain/webhooks/retry.go). - The connector API pages link to the custom instructions section of the Connectors guide, where that content now lives. Refs PRO-2457 Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Signed-off-by: SohamRatnaparkhi <soham@hydradb.com>
- Code tabs: every connector and webhook endpoint now shows Python SDK, TypeScript SDK, and cURL, like the rest of the reference. Update Metadata Schema, Update Source Metadata, and an Ingest Context example used raw HTTP in their SDK tabs; they now call the SDK. Every new SDK call was run against a local mock server and type-checked against SDK 2.1.7, and sends the same request as its cURL. - Callouts: 40 routine Note, Info, Tip, and Warning boxes become plain text. Warnings remain only where data or access is at risk: replacing a connector resource, deleting a database, deleting a source that is still indexing, and an acl_warning on a connector resource. - Overview: drops the core concepts table, the endpoint groups table, the SDK setup code, and the status code table, each covered on its own page; the endpoint inventory is grouped by resource, each group linking to its overview. 2,202 words to 1,655. - Query: the tuning list that repeated the Query guide becomes a link; Behavior notes becomes Common mistakes. - Error Responses: notes that both SDKs already retry 408, 429, and 5xx twice with exponential backoff (Python also 409), and how to raise it. - Wording: no "material", no per-user or one-user framing of collections and memories; connector examples write to one company_docs collection; no arrows in prose. Refs PRO-2457 Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Signed-off-by: SohamRatnaparkhi <soham@hydradb.com>
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/andessentials/v2/api-results.mdx. This is the API part of PRO-2457, next to Guides #309 and Cookbooks #311.Fixes checked against the code:
X-API-Versionheader exists). Polling now usesready_for_ingestion.app_knowledgeexample useskind,provider,external_idandfields. Itemdatabaseandcollectionare optional.tenantreplaced bydatabase.collectionswhen shared docs live elsewhere.tenant_metadata.Repetition removed:
Connector and webhook pages were read line by line. A new Webhooks: Overview page, first in the Webhooks group, lists every webhook endpoint including the signing-secret calls, the typical sequence, and how deliveries are signed and retried. The connector pages link to the custom instructions section of the Connectors guide.
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.
Latest pass, applying the same principles as the Guides:
408,429, and5xxwith exponential backoff, and how to raise the count.company_docscollection.mainafter docs(ingest): blank scans and text-free photos fail with E2002 (PRO-2456) #313 and docs(app-sources): clarify classes, threads, and ingestion flow #315.Validation: MDX compile, Mintlify build validation, broken links, hygiene, and the stale-claim check all pass.
🤖 Generated with Claude Code