Signed-off-by: Zack Koppert <zkoppert@github.com>

fix: remove dead code

Co-authored-by: zkoppert <6935431+zkoppert@users.noreply.github.com>

Co-authored-by: zkoppert <6935431+zkoppert@users.noreply.github.com>

Signed-off-by: Zack Koppert <zkoppert@github.com>

This commit fixes an issue in the test where the mock commits setup was causing
an infinite loop. The changes include:

- Converting mock commits to a proper iterator to match real GitHub API behavior
- Fixing environment variable field names to align with actual main function usage
- Adding chunk_size to avoid issues in processing loops
- Adding additional assertions to verify early function exit

Tests that were previously hanging now complete successfully.

Signed-off-by: Zack Koppert <zkoppert@github.com>

Added test_markdown_writer_original_author_only.py to cover the specific case
where an original commit author is provided without a manager, ensuring that
the appropriate heading is written to the report markdown file.

Signed-off-by: Zack Koppert <zkoppert@github.com>

This change improves the clarity and maintainability of the codebase by:

- Adding an explicit `team_ownership_explicitly_specified` parameter to track when
  team ownership is provided explicitly rather than derived from commit history
- Updating logic in markdown_writer.py to prioritize this flag when determining
  report headings
- Adding a fallback message for cases where neither original author nor explicit
  team ownership is specified
- Setting flag value in measure_innersource.py based on owning_team parameter
- Adding comprehensive tests for the new parameter and updated behavior

This change makes the relationship between owning_team and report content more
explicit and handles edge cases more gracefully.

Signed-off-by: Zack Koppert <zkoppert@github.com>

…ficiency

This change improves memory usage and scalability for repositories with large
commit histories by:

- Replacing list(commits) with chunked processing to avoid loading the entire
  commit history into memory at once
- Using iterators directly to find the original commit author instead of
  loading all commits first
- Processing commits in manageable chunks based on the configured chunk_size,
  similar to how pull requests and issues are already handled
- Adding debug logging for commit processing progress

This approach ensures consistent memory usage regardless of repository size
and prevents potential out-of-memory issues when analyzing repositories with
extensive commit histories.

Signed-off-by: Zack Koppert <zkoppert@github.com>

…b-9f8941ab7116