-
Notifications
You must be signed in to change notification settings - Fork 5
BP-2211: Partial OpenGraph updates for v8.5.0 #158
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
BP-2211: Partial OpenGraph updates for v8.5.0 #158
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the WalkthroughThis PR refines BloodHound documentation by updating OpenGraph-related content. Changes include clarified terminology in the introduction, icon property updates across documentation files, and more nuanced FAQ content explaining partial support for custom nodes & edges with Search and Pathfinding features. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
StephenHinck
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@docs/opengraph/faq.mdx`:
- Line 34: The version string "v8.50" in the OpenGraph FAQ line should be
changed to a semver-consistent format; locate the sentence containing "v8.50" in
the OpenGraph nodes search paragraph and replace it with "v8.5.0" (or "v8.5" if
shorter) so the documentation uses standard semantic versioning.
📜 Review details
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (4)
docs/get-started/introduction.mdxdocs/home.mdxdocs/opengraph/faq.mdxdocs/opengraph/overview.mdx
🧰 Additional context used
🧠 Learnings (4)
📓 Common learnings
Learnt from: jeff-matthews
Repo: SpecterOps/bloodhound-docs PR: 89
File: docs/resources/edges/az-role-approver.mdx:14-14
Timestamp: 2025-10-27T15:00:33.251Z
Learning: In the bloodhound-docs repository, documentation content derived from HelpTexts in the code should not be editorially changed unless there's an egregious error. Minor stylistic improvements should be submitted as PRs to the source code instead of being made directly in the documentation.
Learnt from: StephenHinck
Repo: SpecterOps/bloodhound-docs PR: 67
File: docs/collect-data/enterprise-collection/privileged-collection.mdx:7-7
Timestamp: 2025-10-02T18:01:39.059Z
Learning: In the BloodHound documentation repository, "BloodHound" as a standalone name refers to the entire product family and is appropriate to use when content applies to all products in the family (Enterprise and Community Edition). "BloodHound Enterprise" should be used only when referring specifically to Enterprise-only features or capabilities.
📚 Learning: 2025-08-08T15:57:55.743Z
Learnt from: StephenHinck
Repo: SpecterOps/bloodhound-docs PR: 42
File: docs/install-data-collector/install-azurehound/system-requirements.mdx:70-73
Timestamp: 2025-08-08T15:57:55.743Z
Learning: For AzureHound docs (docs/install-data-collector/install-azurehound/system-requirements.mdx), prefer explicitly stating:
- Directory Reader must be permanently active (not PIM-eligible only).
- Microsoft Graph application permissions (Directory.Read.All, RoleManagement.Read.All) require admin consent.
- Azure Reader role phrasing: “on all Azure subscriptions, ideally assigned at the tenant root group (root management group) scope.”
Applied to files:
docs/get-started/introduction.mdx
📚 Learning: 2025-10-02T18:01:39.059Z
Learnt from: StephenHinck
Repo: SpecterOps/bloodhound-docs PR: 67
File: docs/collect-data/enterprise-collection/privileged-collection.mdx:7-7
Timestamp: 2025-10-02T18:01:39.059Z
Learning: In the BloodHound documentation repository, "BloodHound" as a standalone name refers to the entire product family and is appropriate to use when content applies to all products in the family (Enterprise and Community Edition). "BloodHound Enterprise" should be used only when referring specifically to Enterprise-only features or capabilities.
Applied to files:
docs/get-started/introduction.mdx
📚 Learning: 2025-08-22T22:40:24.105Z
Learnt from: StephenHinck
Repo: SpecterOps/bloodhound-docs PR: 53
File: docs/analyze-data/posture-page.mdx:33-41
Timestamp: 2025-08-22T22:40:24.105Z
Learning: BloodHound Enterprise has exactly four severity levels for attack paths: CRITICAL (95%–100%), HIGH (80%–94%), MODERATE (40%–79%), and LOW (0%–39%). There is no "NONE" or similar severity level for 0% exposure.
Applied to files:
docs/get-started/introduction.mdx
🪛 LanguageTool
docs/get-started/introduction.mdx
[style] ~7-~7: Consider a different adjective to strengthen your wording.
Context: ...ackers (red teams) use BloodHound for a deeper understanding of privileged relationshi...
(DEEP_PROFOUND)
🔇 Additional comments (3)
docs/home.mdx (1)
121-123: Icon update aligns with OpenGraph visuals.No concerns with this change.
docs/get-started/introduction.mdx (1)
7-12: Verify OpenGraph version/capability statements.Please confirm the “introduced in v8.0” timing and the listed capabilities/examples match current release notes and product behavior to avoid doc drift.
docs/opengraph/overview.mdx (1)
7-24: Confirm new icon names are valid in the icon set.Please double-check that
circle-nodesandbrackets-curlyresolve in the current Mintlify icon set to avoid missing icons at render time.
✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.
Purpose
This pull request (PR) cherry picks smaller, targeted commits from #155 to untie the proposed OpenGraph quickstart from the v8.5.0 release.
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.