Sanitize Repo, Debug Statements, License Mappings 2025-07-19

[juliojimenez]

No description provided.

[Copilot]

Pull Request Overview

This PR implements string and repository sanitization functionality to improve input validation and security. The changes focus on adding proper sanitization for repository names and uncomment previously disabled repository sanitization logic.

• Implements sanitize_repository function with validation for owner/repo format
• Adds comprehensive test coverage for both string and repository sanitization functions
• Enables repository sanitization in the main sanitization workflow

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
lib/sanitize.sh	Adds sanitize_repository function and enables repository sanitization in main workflow
test/simple.bats	Adds 11 new test cases covering string and repository sanitization edge cases

Comments suppressed due to low confidence (2)

test/simple.bats:330

• The test expects the dangerous command rm -rf / to remain in the output after sanitization. This suggests the sanitization may not be removing all dangerous shell commands, which could be a security concern that should be verified.

    [[ "$output" == "testcommand rm -rf /" ]]

test/simple.bats:337

• The test expects the @ symbol to be removed from the input string, but this should be verified against the actual sanitization logic to ensure the test expectation matches the implementation behavior.

    [[ "$output" == "test-string_with.safecharacters123" ]]