BED-6876 - add SharpHound and SharpHoundCommon version to log

[mykeelium]

Description

Include the SharpHound and SharpHoundCommon Version in the Logs

Motivation and Context

BED-6876

How Has This Been Tested?

This was tested by creating a build and running the code and inspecting the logs to ensure the changes were available.

Screenshots (if appropriate):

Types of changes

• Chore (a change that does not modify the application functionality)
• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• Documentation updates are needed, and have been made accordingly.
• I have added and/or updated tests to cover my changes.
• All new and existing tests passed.
• My changes include a database migration.

Summary by CodeRabbit

• Improvements
  • Startup logging now shows detailed version information for the application and its shared library components, making it easier to verify what’s running and troubleshoot environment/version issues.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

Adds two startup log statements that report the executing SharpHound assembly version and the SharpHound Common library assembly version via reflection; also adds using System.Reflection;. No control-flow or public API changes.

Changes

Cohort / File(s)	Summary
Version Logging Addition src/Sharphound.cs	Adds using System.Reflection; and two log statements reporting SharpHound (Assembly.GetExecutingAssembly().GetName().Version) and SharpHound Common (Assembly.GetAssembly(typeof(CommonLib)).GetName().Version) assembly versions at startup.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

• Verify reflection calls retrieve the intended assemblies and versions.
• Confirm log placement at startup and formatting.
• Quick check for any potential null reference when resolving Assembly.GetAssembly(typeof(CommonLib)).

Possibly related PRs

• SHCE version bump to 2.8.0 #183 — Adds runtime logging of assembly versions; directly related to reported version values.
• [BED-6485] chore: bump version to v2.7.2 #175 — Changes the SharpHound.Common assembly version referenced by the new logging statements.

Suggested reviewers

• ktstrader
• MikeX777

Poem

🐰 I hop through code with curious eyes,

I read my versions under autumn skies,
A whisper in logs, two numbers align,
SharpHound and Common, stamped in a line,
I twitch my nose and bless this little sign.

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and specifically describes the main change: adding SharpHound and SharpHoundCommon version information to logs, with direct reference to the issue tracker (BED-6876).
Description check	✅ Passed	The pull request description follows the template structure and includes all key sections: Description, Motivation and Context, testing details, and completed checklist. The author provided meaningful information in each section.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch mcuomo/BED-6876

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 781da1e and 0d1afc1.

📒 Files selected for processing (1)

• src/Sharphound.cs (2 hunks)

🚧 Files skipped from review as they are similar to previous changes (1)

• src/Sharphound.cs

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 094ead7 and 781da1e.

📒 Files selected for processing (1)

• src/Sharphound.cs (2 hunks)

🔇 Additional comments (1)

src/Sharphound.cs (1)

19-19: LGTM!

The System.Reflection namespace import is correctly placed and necessary for the version logging functionality added below.