Add path validation and logging for bbs2gh archive operations (Fixes …

[AakashSuresh2003]

…#1211)

• Did you write/update appropriate tests
• Release notes updated (if appropriate)
• Appropriate logging output
• Issue linked
• Docs updated (or issue created)
• New package licenses are added to ThirdPartyNotices.txt (if applicable)

[Copilot]

Pull request overview

This PR adds path validation and logging for bbs2gh archive operations to provide better error messages to users when invalid paths are specified. The changes address issue #1211 by validating that archive paths and BBS shared home directories exist before attempting migration operations.

• Adds validation for --archive-path to ensure the file exists when explicitly provided by users
• Adds validation for --bbs-shared-home to ensure the directory exists when running on the Bitbucket instance (not using SSH/SMB download)
• Adds logging to output the archive path before upload operations for better debugging
• Introduces DirectoryExists method to FileSystemProvider to support directory validation

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File	Description
src/bbs2gh/Commands/MigrateRepo/MigrateRepoCommandHandler.cs	Added path validation for archive files and BBS shared home directories in ValidateOptions, and added logging for archive path before upload operations
src/Octoshift/Services/FileSystemProvider.cs	Added DirectoryExists method to support directory existence validation, following the same pattern as existing FileExists method
src/OctoshiftCLI.Tests/bbs2gh/Commands/MigrateRepo/MigrateRepoCommandHandlerTests.cs	Added comprehensive test coverage including tests for validation failures, tests ensuring validation is skipped when using SSH/SMB, and test verifying archive path logging

[github-actions]

Unit Test Results

  1 files    1 suites   10m 23s ⏱️
960 tests 960 ✅ 0 💤 0 ❌
961 runs  961 ✅ 0 💤 0 ❌

Results for commit 1390b46.

♻️ This comment has been updated with latest results.

[Copilot]

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

[AakashSuresh2003]

Hi @brianaj,

Could you please review my PR and provide any suggestions?

[brianaj]

@AakashSuresh2003 were you able to functionally test it (screenshots would be nice)? And can you add release notes.

[AakashSuresh2003]

Hi @brianaj,

The functional test screenshot is attached below, and the release notes have been added
Please let me know if there’s anything else needed from my side.

[github-actions]

Package	Line Rate	Branch Rate	Complexity	Health
ado2gh	72%	70%	712	➖
Octoshift	83%	72%	1737	✔
bbs2gh	83%	78%	663	✔
gei	81%	72%	574	✔
Summary	81% (7734 / 9600)	73% (1837 / 2519)	3686	✔

[brianaj]

Thanks will work on shipping this in #1462 so I can run CI.