fix(pre-commit-hooks): remove magic constants on pre-push hook

[g-roch]

Description

• Remove static reference to origin (work with other name)
• Now work when remote/HEAD isn't setted (e.g. on git init .; git remote add origin git@...)

Checklist

• I have read the contributing guidelines

Code Changes

• Add test cases to all the changes you introduce
• Run uv run poe all locally to ensure this change passes linter check and tests
• Manually test the changes:
  • Verify the feature/bug fix works as expected in real-world scenarios
  • Test edge cases and error conditions
  • Ensure backward compatibility is maintained
  • Document any manual testing steps performed
• Update the documentation for the changes

Documentation Changes

• Run uv run poe doc locally to ensure the documentation pages renders correctly
• Check and fix any broken links (internal or external) in the documentation

When running uv run poe doc, any broken internal documentation links will be reported in the console output like this:

INFO    -  Doc file 'config.md' contains a link 'commands/bump.md#-post_bump_hooks', but the doc 'commands/bump.md' does not contain an anchor '#-post_bump_hooks'.

Expected Behavior

• Expected it's work out-of-the-box, including when origin/HEAD isn't set (e.g. repository created by git init)
• Expected the name of remote (usually origin) isn't hard coded (or isn't to set in configuration file)

Steps to Test This Pull Request

1. git init test-repo
2. create empty repo in github or anything else
3. cz init
4. pre-commit install --hook-type commit-msg --hook-type pre-push
5. git init && git add . && git commit -m "feat: well formed commit message"
6. git remote add github_origin https://github.com/<some repo>.git
7. git push -u github_origin main

Additional Context

• No information found in document about previous behavior → no change in docs
• It's applied for any user run pre-commit autoupdate including if commitizen isn't updated (when new release published)

FIX #704

[bearomorphism]

Interesting. I didn't know these variables.

Additional details for PRE_COMMIT_TO_REF: https://github.com/pre-commit/pre-commit.com/blob/main/sections/advanced.md#pre-push

[bearomorphism]

Nice catch, thanks!

[KyleMylonakisProtopia]

After this change, invalid commit messages on main which previously were fine with commitizen are now tripping this check. Are the environment variable you added in the correct order, and is there supposed to be .. between them?

[Lee-W]

After this change, invalid commit messages on main which previously were fine with commitizen are now tripping this check. Are the environment variable you added in the correct order, and is there supposed to be .. between them?

Could you please create an issue and let us know how we can reproduce it? Thanks!

[KyleMylonakisProtopia]

Ok, I made a minumum reproducing repository: https://github.com/KyleMylonakisProtopia/commitizen-pr-1815-bug
Please fork this repository and you can reproduce the issue with the following commands:

 git clone git@github.com:KyleMylonakisProtopia/commitizen-pr-1815-bug.git
 cd commitizen-pr-1815-bug
 uv venv .venv
 source .venv/bin/activate
 uv pip install -e .
 pre-commit install
 git checkout -b new_branch
 touch new_file
 git add new_file
 git commit -m "feat: add new file"
 git push

For me this fails with the error:

(.venv) ➜  commitizen-pr-1815-bug git:(new_branch)     git push --set-upstream origin new_branch
commitizen check branch..................................................Failed
- hook id: commitizen-branch
- exit code: 14

commit validation: failed!
please enter a commit message in the commitizen format.
commit "2bbee8154e795c7d9f77fd9e190d6d7bf5b9f325": "improperly formatted commit that introduced
"pattern: (?s)(build|bump|chore|ci|docs|feat|fix|perf|refactor|revert|style|test)(\(\S+\))?!?: ([^\n\r]+)((\n\n.*)|(\s*))?$
commit "3d9318b341828644f23ff772a6f276be47bf825e": "first commit
"pattern: (?s)(build|bump|chore|ci|docs|feat|fix|perf|refactor|revert|style|test)(\(\S+\))?!?: ([^\n\r]+)((\n\n.*)|(\s*))?$


error: failed to push some refs to 'github.com:KyleMylonakisProtopia/commitizen-pr-1815-bug.git'

[KyleMylonakisProtopia]

See #1818