Remove node fs usage from core business logic - use ReaderAdapter pattern

- Update LicenseService to use ReaderAdapter instead of direct fs calls
- Convert detectLicenseFromFile to async method using ReaderAdapter
- Update GitHubRepositoryProvider to use ReaderAdapter for logo detection
- Refactor all related test files to use ReaderAdapter mocks instead of fs mocks
- Fix TypeScript import type issues for proper decorator metadata
- Update GitHub repository provider tests with proper ReaderAdapter mocking

Node fs is now only used in reader/renderer components within core package. CLI and parser utilities retain legitimate fs usage for their specific purposes.

Co-authored-by: neilime <314088+neilime@users.noreply.github.com>

Author: Copilot