Skip to content

docs(collections): explain connector cleanup on deletion - #340

Closed
adityamishra-hydradb wants to merge 2 commits into
mainfrom
vorflux/adityamishra/eng-3301-collection-delete-stops-connectors-docs-dco
Closed

adityamishra-hydradb wants to merge 2 commits into
mainfrom
vorflux/adityamishra/eng-3301-collection-delete-stops-connectors-docs-dco

Conversation

@adityamishra-hydradb

@adityamishra-hydradb adityamishra-hydradb commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

For humans

The delete-collection reference now states that deletion removes dedicated connectors and only matching resources from mixed connectors.

Linear: https://linear.app/hydradb/issue/ENG-3301

Merge this PR only after the application change reaches production.

For agents

  • Base revision: 82850ca5440e56a8a967f147cf6768659ac6ba80 from main.
  • Validation passed:
    • npm run hygiene:strict
    • npm run verify:agents-endpoints
    • npx mintlify validate
  • verify:agents-endpoints checked 54 endpoint mentions.
  • No staging or production service was accessed.
  • Merge order: application first, dashboard second, this docs PR after the application change reaches production.

Decisions taken without review

  • The reference uses one direct behavior sentence because the endpoint semantics changed but the request and response schemas did not.
  • The docs PR stays separate so reviewers can enforce the production-first publishing order.

Session Details


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need.


Devin Review

Signed-off-by: adityamishra-hydradb <321510987+adityamishra-hydradb@users.noreply.github.com>
@mintlify

mintlify Bot commented Oct 27, 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 28, 2026, 12:12 AM

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

@github-actions

Copy link
Copy Markdown

✅ Mintlify Hygiene

No issues found.

@openhack-agent

openhack-agent Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

✅ OpenHack Summary

Security review of docs(collections): explain connector cleanup on deletion. 1 changed file; 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/delete-collection.mdx (modified)

Last reviewed commit: 9a4bd9e · 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.

@greptile-apps

greptile-apps Bot commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[Low impact] The documentation change appears safe to merge after the application change reaches production.

Summary

Updates the delete-collection reference to say that deletion also disables matching connector default routes.

  • The delete page explains which connectors lose collection routes.

Reviews (3) · Last reviewed commit: "docs(collections): clarify mixed connect..." · Reviewed by Greptile

@adityamishra-hydradb

Copy link
Copy Markdown
Contributor Author

@greptileai review

@adityamishra-hydradb

Copy link
Copy Markdown
Contributor Author

/devin review

@devin-ai-integration

Copy link
Copy Markdown
Contributor

Starting Devin Review.

Devin Review

@devin-ai-integration devin-ai-integration Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Note

Newer findings are available below. Devin Review posted a newer report on this PR, in addition to the findings presented here.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

@vorflux

vorflux Bot commented Oct 28, 2026

Copy link
Copy Markdown
Contributor

/devin review

@vorflux

vorflux Bot commented Oct 28, 2026

Copy link
Copy Markdown
Contributor

@greptileai review

1 similar comment
@adityamishra-hydradb

Copy link
Copy Markdown
Contributor Author

@greptileai review

@adityamishra-hydradb

Copy link
Copy Markdown
Contributor Author

/devin review

@devin-ai-integration

Copy link
Copy Markdown
Contributor

Starting Devin Review.

Devin Review

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Devin Review found 1 new potential issue.

Devin Review

Comment thread api-reference/v2/endpoint/delete-collection.mdx
@adityamishra-hydradb

Copy link
Copy Markdown
Contributor Author

Superseded by #342. The replacement contains one DCO-signed commit and avoids a prohibited force-push.

This branch was successfully deployed

1 active deployment
staging — 9a4bd9e8 Deployed Oct 28, 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