Skip to content

chore: repo maintenance (navigation) - #412

Open
jeff-matthews wants to merge 1 commit into
mainfrom
housekeeping-navigation
Open

chore: repo maintenance (navigation)#412
jeff-matthews wants to merge 1 commit into
mainfrom
housekeeping-navigation

Conversation

@jeff-matthews

@jeff-matthews jeff-matthews commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Summary

This pull request (PR) adds missing .mdx files to the navigation and removes an unused/unnecessary .mdx file.

This is pre-work for modularizing the navigation using split configuration, which is a pattern that will make restructuring the doc site and future maintenance easier.

Summary by CodeRabbit

  • Documentation
    • Added API reference pages for Jamf OpenGraph, collection uploads, and Cypher saved queries.
    • Updated navigation to make the new API documentation available.
    • Updated the security documentation link to point directly to SharpHound hardening guidance.
    • Removed the redundant BloodHound and collectors security overview page.

@jeff-matthews jeff-matthews self-assigned this Aug 28, 2026
@coderabbitai

coderabbitai Bot commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 48e97c63-eaff-43d5-8c11-7034e553a119

📥 Commits

Reviewing files that changed from the base of the PR and between 0b5ed19 and ab927c7.

📒 Files selected for processing (3)
  • docs/docs.json
  • docs/manage-bloodhound/overview.mdx
  • docs/manage-bloodhound/securing-bloodhound-and-collectors/overview.mdx
💤 Files with no reviewable changes (1)
  • docs/manage-bloodhound/securing-bloodhound-and-collectors/overview.mdx

Included review availability: 4 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.


Walkthrough

The documentation navigation now includes four API reference pages. The security guidance link now targets SharpHound hardening. The previous BloodHound and collectors security overview page was removed.

Changes

Documentation navigation

Layer / File(s) Summary
API reference navigation
docs/docs.json
Added Jamf OpenGraph, collection uploads, and Cypher saved-query reference pages to the navigation.
Security guidance navigation
docs/manage-bloodhound/overview.mdx, docs/manage-bloodhound/securing-bloodhound-and-collectors/overview.mdx
Changed the security guidance link to the SharpHound hardening page and removed the previous security overview page.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Merge Risk: ⚪ Minimal · up to ab927

This localized documentation navigation maintenance change has no actionable merge-blocking risk remaining and is merge-ready after normal checks and review.

Poem

A rabbit checks the links at night
Four API pages hop in sight
SharpHound guards the path ahead
Old overview rests in bed
Documentation blooms bright

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately identifies repository maintenance focused on documentation navigation. It matches the primary changes: adding navigation entries, updating a documentation link, and removing an un…
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Title check

Explanation

The title accurately identifies repository maintenance focused on documentation navigation. It matches the primary changes: adding navigation entries, updating a documentation link, and removing an unused page.

Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (2 skipped: 2 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch housekeeping-navigation

Comment @coderabbitai help to get the list of available commands.

@jeff-matthews jeff-matthews added the maintenance Work related to repo upkeep and housekeeping label Aug 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Work related to repo upkeep and housekeeping

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant