chore: update openhound github collector reference docs - #377
chore: update openhound github collector reference docs#377jeff-matthews wants to merge 1 commit into
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 ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
|
Will take a look before the end of the week |
|
It looks like several node and edge kinds in the schema do not have corresponding description files in the OpenHound repository. Also, some documented edge kinds no longer exist in the schema. Unless those edges are expected to be added to the schema, I assume we can delete their documentation. @jaredcatkinson, are you planning to add descriptions for the undocumented node and edge kinds? There is also an issue with the documentation-generation script / OpenHound repositories causing metadata for nodes and edges to be missing. Before the OH repositories were released, they included a @d3vzer0, it looks like the Edges
Nodes
We do not need to store inbound and outbound edges for nodes because those can be derived from the edge definitions, avoiding redundant data. |
Summary
This is my first attempt at running the docs automation for the openhound-github collector repo.
I'm not sure if all of these changes are expected. It looks like the new docs have stripped out some mermaid diagrams and other info, but I'm not sure if that's expected. It certainly looks like the docs have been out of date since before the latest changes that prompted this PR: SpecterOps/openhound-github#22.
Here's a summary from the script I ran locally:
See terminal output for details:
github-collector-doc-gen.txt
Note
The docs automation tooling failed on my first attempt. I created the following proposed fix in the upstream repo: SpecterOps/og-docs-automation#2.
Related to #376