Skip to content

docs(v2): cookbook corrections (3/3) - #311

Draft
SohamRatnaparkhi wants to merge 9 commits into
mainfrom
docs/pro-2457-cookbooks-cleanup
Draft

SohamRatnaparkhi wants to merge 9 commits into
mainfrom
docs/pro-2457-cookbooks-cleanup

Conversation

@SohamRatnaparkhi

@SohamRatnaparkhi SohamRatnaparkhi commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Removes repeated openings, boilerplate, and unsupported product claims from all 12 V2 cookbooks and their index while retaining the walkthroughs and examples. Independent of #309 and #310.

  • Corrects database readiness, collection scopes, metadata/ACL guidance, memory usage, and processing-state handling.
  • Requires caller ACL principals in Glean queries, makes source filters match ingested free-form metadata, routes the UI through an authenticated backend, propagates ACL and collection arguments in tenant searches, and raises on onboarding indexing failures.
  • Fixes travel category grouping and TypeScript mode typing. Adds the missing Gmail decoding helpers and recruiter profile lookup, and removes calls to unimplemented tutorial methods. Recruiter hard criteria require an application predicate before results are returned; search statistics use only accepted candidates.
  • Adds a quickstart prerequisite and replaces unexplained terms in the cookbook index with concrete descriptions.
  • Describes application scoring and sample batching as application choices. Makes JSON request examples valid and labels abbreviated responses accurately.

Validation: Mintlify build and broken-link checks, MDX/frontmatter compilation, documentation hygiene, and Python/JSON example parsing passed. Extracted Gmail decoding, travel, preference-learning, recruiter lookup, interview-preparation, and ranking examples passed local checks with sample data. UI search dispatch, tenant permission/ACL/collection propagation, recruiter filtering and statistics, filter payloads, failure paths, and response conversion were checked through SDK 2.1.7 with mocked HTTP. External services were mocked; production integrations were not exercised.

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 7, 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): cookbook corrections (3/3). 13 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
  • cookbooks/v2/ai-chief-of-staff.mdx (modified)
  • cookbooks/v2/ai-linkedin-recruiter.mdx (modified)
  • cookbooks/v2/ai-onboarding-agent.mdx (modified)
  • cookbooks/v2/ai-travel-planner.mdx (modified)
  • cookbooks/v2/competitive-intelligence-agent.mdx (modified)
  • cookbooks/v2/cookbook-01-build-cursor-for-docs.mdx (modified)
  • cookbooks/v2/cookbook-04-build-notion-ai.mdx (modified)
  • cookbooks/v2/cookbook-10-ai-financial-analyst.mdx (modified)
  • cookbooks/v2/customer-support-agent.mdx (modified)
  • cookbooks/v2/glean-clone.mdx (modified)
  • cookbooks/v2/hydradb-cookbook-06.mdx (modified)
  • cookbooks/v2/index.mdx (modified)
  • cookbooks/v2/internal-search-perplexity.mdx (modified)

Last reviewed commit: 8f5d69e · 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.

Every cookbook was read line by line and its code checked against the
backend, the TS SDK 2.1.7, and the Python SDK 2.1.5. The voice, structure
and examples stay; wrong code, filler and invented claims go.

Code fixes:
- Queries and status checks now name the collection the data was written
  to. Several cookbooks searched or polled the default collection.
- Memory curl examples use multipart with a JSON-stringified memories
  field instead of a JSON body.
- upsert is the string "true" in TS, as the SDK types it.
- TS reads camelCase chunk fields (sourceTitle, chunkContent,
  additionalMetadata); Python reads SDK attributes, not dict keys.
- Filters use the equals and contains_any operators, and schema-backed
  filter keys are declared in database_metadata_schema.
- Database readiness polls infra.ready_for_ingestion; indexing polls
  indexing_status instead of fixed sleeps.
- Travel planner groups itinerary chunks by additional_metadata.category,
  since an item's type is not returned.
- Recency examples send an event time, which recency_bias reads.

Removed: invented ROI and latency claims, a fake "20 per request" API
limit (now pacing advice), dead session code, and filler words.

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 corrected cookbook examples.

@greptile-apps

greptile-apps Bot commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

[Low risk] Documentation corrections across cookbook guides.

The PR appears safe to merge based on the current review.

Summary

The PR corrects V2 cookbook examples and product descriptions across 12 guides and their index. Since the previous review, the Internal IT Support card was narrowed to match the tutorial’s Slack output.

  • The revised card no longer promises source links.
  • No new actionable issue was identified in the change since the previous review.

Reviews (6) · Last reviewed commit: "docs(v2): match IT support card to tutor..." · Reviewed by Greptile

Comment thread cookbooks/v2/ai-travel-planner.mdx
Comment thread cookbooks/v2/hydradb-cookbook-06.mdx
Comment thread cookbooks/v2/cookbook-10-ai-financial-analyst.mdx
Comment thread cookbooks/v2/ai-onboarding-agent.mdx
Comment thread cookbooks/v2/ai-linkedin-recruiter.mdx
Comment thread cookbooks/v2/ai-linkedin-recruiter.mdx Outdated
Comment thread cookbooks/v2/glean-clone.mdx Outdated
Comment thread cookbooks/v2/glean-clone.mdx Outdated
Comment thread cookbooks/v2/cookbook-10-ai-financial-analyst.mdx
Comment thread cookbooks/v2/competitive-intelligence-agent.mdx
Signed-off-by: SohamRatnaparkhi <soham@hydradb.com>
@SohamRatnaparkhi

Copy link
Copy Markdown
Contributor Author

@greptileai Please review the latest head, including the final removal of repeated cookbook openings and unsupported claims.

Comment thread cookbooks/v2/cookbook-04-build-notion-ai.mdx Outdated
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. Valid ACL, metadata-filter, indexing, and criteria issues are fixed and tested. Execution through the published SDK confirms the existing camelCase response reads. The implementation evidence also confirms object-valued nested metadata, equality filtering without an enable_match gate, and the 0.4 recency default.

Comment thread cookbooks/v2/glean-clone.mdx
Comment thread cookbooks/v2/ai-linkedin-recruiter.mdx Outdated
Signed-off-by: SohamRatnaparkhi <soham@hydradb.com>
@SohamRatnaparkhi

Copy link
Copy Markdown
Contributor Author

@greptileai Please review the current head dd0d50a. Both findings from the previous review are fixed: Glean UI search uses an authenticated backend callback, tenant search passes caller ACL and collection in the correct positions, and recruiter search insights are recomputed from accepted candidates. Runtime checks cover these paths, including permission denial and empty accepted results; the hosted validation and hygiene checks pass. Details are in the inline replies.

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

Copy link
Copy Markdown
Contributor Author

@greptileai Please review current head dd4740f. This pass gives the cookbook index a quickstart prerequisite and replaces unexplained terminology in its descriptions. Tutorial code is unchanged. Local and hosted build validation, hygiene, and link checks pass.

Comment thread cookbooks/v2/index.mdx Outdated
Signed-off-by: SohamRatnaparkhi <soham@hydradb.com>
@SohamRatnaparkhi

Copy link
Copy Markdown
Contributor Author

@greptileai Please review current head 8f5d69e. The Internal IT Support card now describes source titles alongside Slack search results, matching the tutorial output. This addresses the source-link finding; tutorial code is unchanged.

This branch was successfully deployed

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