Skip to content

fix: canonicalize GitHub credential paths#194

Merged
RexWzh merged 6 commits into
masterfrom
rex/update-hermes
May 7, 2026
Merged

fix: canonicalize GitHub credential paths#194
RexWzh merged 6 commits into
masterfrom
rex/update-hermes

Conversation

@RexWzh

@RexWzh RexWzh commented May 7, 2026

Copy link
Copy Markdown
Member

Summary

  • canonicalize GitHub credential paths so explicit --repo owner/name and current-remote resolution use the same git credential key
  • keep repo-perms masked token/source visibility from the previous branch work
  • add mock coverage for https, ssh, and scp-style GitHub remotes with and without .git

Testing

  • python -m pytest -q tests/mock-cli-tests/gh/test_chattool_gh_basic.py tests/mock-cli-tests/gh/test_chattool_gh_actions_diagnostics.py tests/code-tests/tools/github/test_cli.py
  • chattool gh repo-perms
  • chattool gh repo-perms --repo cubenlp/ChatTool

@github-actions

github-actions Bot commented May 7, 2026

Copy link
Copy Markdown
Contributor

Preview available at: https://chattool.wzhecnu.cn/dev/

@RexWzh RexWzh merged commit f63e2c8 into master May 7, 2026
3 checks passed
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.

1 participant