cmd(git): More commands

[tony]

cmd(git): More commands

Summary

This PR introduces the Manager/Cmd pattern for git subcommands, providing
typed Python objects instead of raw strings. It also includes numerous bug
fixes to existing git command implementations.

Changes

New: Manager/Cmd Pattern

Manager (collection-level)              Cmd (per-entity)
├── ls() -> QueryList[Cmd]              ├── run()
├── get(**kwargs) -> Cmd                ├── show()
├── filter(**kwargs) -> list[Cmd]       ├── remove()/delete()
├── add() / create()                    └── entity-specific operations
└── run()

New Manager Classes

Manager	Access	Cmd Class
GitBranchManager	git.branches	GitBranchCmd
GitRemoteManager	git.remotes	GitRemoteCmd
GitStashManager	git.stashes	GitStashEntryCmd
GitTagManager	git.tags	GitTagCmd
GitWorktreeManager	git.worktrees	GitWorktreeCmd
GitNotesManager	git.notes	GitNoteCmd
GitSubmoduleManager	git.submodules	GitSubmoduleEntryCmd
GitReflogManager	git.reflog	GitReflogEntryCmd

Enhanced Git.init()

• ref_format parameter for --ref-format (files/reftable)
• make_parents to auto-create parent directories (default: True)
• shared extended to support octal permissions (e.g., "0660")
• Improved parameter validation with clear error messages

Bug Fixes

Git.run() fixes

• Fix boolean config serialization ("true" if True -> "true" if v)
• Fix -C paths iteration (was appending list instead of iterating)
• Implement config_env parameter

Git.clone() fixes

• Fix reference_if_able flag name (was using --reference)
• Fix separate_git_dir quoting (was using !r instead of !s)

Git.fetch()/Git.pull() fixes

• Remove clone-only branch/origin parameters
• Implement recurse_submodules, recurse_submodules_default, negotiation_tip
• Fix --signoff flag spelling

Git.rebase() fixes

• Fix --no-rerere-autoupdate typo
• Fix whitespace options (ignore_whitespace instead of no_whitespace)

Git.init() fixes

• Remove invalid --default parameter
• Don't pass --shared when shared=False

Manager/Cmd fixes

• Forward **kwargs to underlying cmd.run() calls in all Manager/Cmd classes
• GitRemoteManager.ls(): Handle URLs with spaces in regex
• GitNoteCmd: Propagate custom ref to note commands
• GitNotesManager.merge(): Handle commit/abort forms correctly
• GitStashCmd.push(): Handle str paths correctly
• stash.pop(): Use stash@{N} format instead of pathspec separator
• branch.create(): Use git branch instead of checkout -b
• remote.add(): Wire fetch, track, and master parameters
• remote.show(): Only add --verbose when explicitly True

Documentation

• API documentation for all new Manager/Cmd classes
• Split git subcommand docs into separate pages
• Usage examples in README and quickstart

[codecov]

Codecov Report

❌ Patch coverage is 99.58203% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 53.18%. Comparing base (23ddb7e) to head (a402e4d).
⚠️ Report is 80 commits behind head on master.

Files with missing lines	Patch %	Lines
tests/cmd/test_git.py	99.57%	4 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           master     #465       +/-   ##
===========================================
+ Coverage   42.90%   53.18%   +10.28%     
===========================================
  Files          38       38               
  Lines        3627     5663     +2036     
  Branches      794     1063      +269     
===========================================
+ Hits         1556     3012     +1456     
- Misses       1718     2120      +402     
- Partials      353      531      +178     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[tony]

Code review

Found 4 issues:

1. GitTagCmd.run() accepts **kwargs in signature but does not forward them to self.cmd.run(), causing silent parameter loss

libvcs/src/libvcs/cmd/git.py

Lines 5941 to 5947 in 9421936

	return self.cmd.run(
	["tag", *local_flags],
	check_returncode=check_returncode,
	log_in_real_time=log_in_real_time,
	)

2. GitTagManager.run() accepts **kwargs in signature but does not forward them to self.cmd.run(), causing silent parameter loss

libvcs/src/libvcs/cmd/git.py

Lines 6121 to 6127 in 9421936

	return self.cmd.run(
	["tag", *local_flags],
	check_returncode=check_returncode,
	log_in_real_time=log_in_real_time,
	)

3. GitWorktreeManager.run() accepts **kwargs in signature but does not forward them to self.cmd.run(), causing silent parameter loss

libvcs/src/libvcs/cmd/git.py

Lines 6695 to 6701 in 9421936

	return self.cmd.run(
	["worktree", *local_flags],
	check_returncode=check_returncode,
	log_in_real_time=log_in_real_time,
	)

4. GitStashEntryCmd.run() accepts **kwargs in signature but does not forward them to self.cmd.run(), causing silent parameter loss

libvcs/src/libvcs/cmd/git.py

Lines 5553 to 5559 in 9421936

	*,
	# Pass-through to run()
	log_in_real_time: bool = False,
	check_returncode: bool | None = None,
	) -> str:
	"""Remove the upstream (tracking) information.

Context: Commit 7cf3f46 fixed this exact pattern in other Manager/Cmd classes (GitRemoteCmd, GitRemoteManager, GitBranchCmd, GitBranchManager, GitNotesManager), but these classes were missed. The correct pattern is shown in GitSubmoduleEntryCmd.run() (lines 2670-2675) where **kwargs is forwarded to self.cmd.run().

🤖 Generated with Claude Code

_{- If this code review was useful, please react with 👍. Otherwise, react with 👎.}

[tony]

Code review (updated)

Found 7 instances of missing **kwargs forwarding. These methods accept **kwargs in their signature but do not forward them to self.cmd.run(), causing silent parameter loss:

1. GitSubmoduleCmd.run() - lines 2451-2455

libvcs/src/libvcs/cmd/git.py

Lines 2450 to 2456 in 9421936

	return self.cmd.run(
	["submodule", *local_flags],
	check_returncode=check_returncode,
	log_in_real_time=log_in_real_time,
	)

2. GitStashEntryCmd.run() - lines 4554-4558

libvcs/src/libvcs/cmd/git.py

Lines 4553 to 4559 in 9421936

	return self.cmd.run(
	["stash", *local_flags],
	check_returncode=check_returncode,
	log_in_real_time=log_in_real_time,
	)

3. GitStashManager.run() - lines 5076-5080

libvcs/src/libvcs/cmd/git.py

Lines 5075 to 5081 in 9421936

	return self.cmd.run(
	["stash", *local_flags],
	check_returncode=check_returncode,
	log_in_real_time=log_in_real_time,
	)

4. GitTagCmd.run() - lines 5942-5946

libvcs/src/libvcs/cmd/git.py

Lines 5941 to 5947 in 9421936

	return self.cmd.run(
	["tag", *local_flags],
	check_returncode=check_returncode,
	log_in_real_time=log_in_real_time,
	)

5. GitTagManager.run() - lines 6122-6126

libvcs/src/libvcs/cmd/git.py

Lines 6121 to 6127 in 9421936

	return self.cmd.run(
	["tag", *local_flags],
	check_returncode=check_returncode,
	log_in_real_time=log_in_real_time,
	)

6. GitWorktreeCmd.run() - lines 6459-6463

libvcs/src/libvcs/cmd/git.py

Lines 6458 to 6464 in 9421936

	return self.cmd.run(
	["worktree", *local_flags],
	check_returncode=check_returncode,
	log_in_real_time=log_in_real_time,
	)

7. GitWorktreeManager.run() - lines 6696-6700

libvcs/src/libvcs/cmd/git.py

Lines 6695 to 6701 in 9421936

	return self.cmd.run(
	["worktree", *local_flags],
	check_returncode=check_returncode,
	log_in_real_time=log_in_real_time,
	)

Context: Commit 7cf3f46 established the pattern of forwarding **kwargs to self.cmd.run() for GitRemoteCmd, GitRemoteManager, GitBranchCmd, GitBranchManager, and GitNotesManager. The correct pattern is shown in GitSubmoduleEntryCmd.run() lines 2670-2675.

🤖 Generated with Claude Code

_{- If this code review was useful, please react with 👍. Otherwise, react with 👎.}