Skip to content

fix: parse git remote arguments safely - #60

Merged
EvanProgramming merged 1 commit into
mainfrom
codex/issue-52-git-remote
Sep 4, 2026
Merged

EvanProgramming merged 1 commit into
mainfrom
codex/issue-52-git-remote

Conversation

@EvanProgramming

Copy link
Copy Markdown
Owner

Fixes #52

Parse git_remote arguments with shlex and validate the supported empty-list, add name URL, and remove name forms before invoking Git. URLs retain punctuation and successful add/remove operations report deterministic results.

Validation:

  • ./venv/bin/python -m unittest tests.test_tools -v (real temporary Git repository add/list/remove)
  • make check
  • make lint
  • make test (127 tests)
  • git diff --check
  • PR head 1e6c1e4 is GPG-signed and GitHub verified.

@EvanProgramming
EvanProgramming merged commit eea5e94 into main Sep 4, 2026
3 checks passed
@EvanProgramming
EvanProgramming deleted the codex/issue-52-git-remote branch September 4, 2026 13:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: parse git_remote add name and URL as separate arguments

1 participant