fix(#120): detect VCS host for releases API and derive action-repo #122
Closed
rodin
wants to merge 2 commits from
issue-120 into feature/github-support
pull from: issue-120
merge into: :feature/github-support
:main
:ci/cleanup
:ci-selfreview-gate
:issue-150
:issue-157
:issue-141
:issue-154
:review-bot-dev-loop
:issue-143
:issue-146
:pr-153
:review-bot-issue-130-work
:issue-148
:issue-139
:issue-137
:review-bot-fixes
:review-bot-issue-133
:review-bot-issue-130
:issue-130
:github-support
:issue-123-work
:issue-123
:review-bot-issue-120
:fix/125-readme-cli-example
:issue-125
:issue-124
:issue-120
:feature/github-support
:review-bot-issue-116
:review-bot-issue-115
:review-bot-issue-114
:review-bot-issue-96
:review-bot-issue-107
:review-bot-issue-82
:review-bot-issue-95
:review-bot-issue-92
:review-bot-issue-94
:review-bot-issue-81
:review-bot-issue-91
:review-bot-issue-97
:issue-80-c-file-reader
:issue-80-b-pr-reader
:issue-80-a-client
:review-bot-issue-80
:review-bot-issue-87
:review-bot-issue-79
:review-bot-issue-84
:review-bot-issue-78
:issue-73
:issue-70
:issue-68
:issue-66
:issue-64
:issue-60-remote-personas
:issue-60
:issue-57
:allow-deps
:feat/aicore-provider-v2
:issue-51
:ci/pr-ready-gate
:fix/stale-commit-check
:feat/aicore-provider
:fix/response-body-truncation
:fix/json-repair
:fix/sonnet-reviewer
:fix/consistent-path-escape
:feat/inline-review-comments
:feat/6-update-existing-review
:fix/19-context-overflow
:feat/18-anthropic-api
:fix/url-escaping-and-shadow
:fix/quick-wins
:fix/context-and-encapsulation
:docs/code-review-report
:ci/release-workflow
2
Commits
| Author | SHA1 | Message | Date | |
|---|---|---|---|---|
|
|
db13078196 |
chore: use dry-run in review.yml (GHE has no releases yet, validate infra)
PR Ready Gate / clear-labels (pull_request) Successful in 1s
CI / test (pull_request) Successful in 20s
CI / review (anthropic--claude-4.6-sonnet, sonnet, SONNET_REVIEW_TOKEN) (pull_request) Successful in 29s
CI / review (gpt-5, security, ., rodin/security-patterns, SECURITY_REVIEW.md, SECURITY_REVIEW_TOKEN) (pull_request) Successful in 1m2s
CI / review (gpt-5, gpt, GPT_REVIEW_TOKEN) (pull_request) Successful in 1m31s
The self-test workflow on github.concur.com runs the action with: - gitea-url=https://gitea.weiker.me (binary download source) - dry-run=true (avoids PR# mismatch between GHE and Gitea) This validates: - Binary download and checksum verification works from a GitHub runner - GITHUB_SERVER_URL/GITHUB_REPOSITORY env vars are correctly passed - AiCore provider authenticates and LLM call succeeds When strat/review-bot has its own releases, remove gitea-url override and dry-run. |
||
|
|
3ac5e5dcca |
fix(#120): detect VCS host for releases API and derive action-repo
CI / test (pull_request) Successful in 20s
CI / review (anthropic--claude-4.6-sonnet, sonnet, SONNET_REVIEW_TOKEN) (pull_request) Successful in 29s
CI / review (gpt-5, security, ., rodin/security-patterns, SECURITY_REVIEW.md, SECURITY_REVIEW_TOKEN) (pull_request) Successful in 1m40s
CI / review (gpt-5, gpt, GPT_REVIEW_TOKEN) (pull_request) Successful in 2m3s
Both composite actions hardcoded: - Gitea's /api/v1/ releases endpoint path - 'rodin/review-bot' as the action source repo - .gitea version used GITEA_URL/GITEA_REPO env vars instead of GITHUB_SERVER_URL/GITHUB_REPOSITORY Fix: - Add 'action-repo' input (default: empty) to allow explicit override - Auto-detect VCS host from server_url: URLs containing 'gitea' use /api/v1/ (Gitea format), all others use /api/v3/ (GitHub format) - Set smart default action-repo: 'rodin/review-bot' on Gitea, 'strat/review-bot' on GitHub - Pass server-url and action-repo as step outputs to avoid re-computing - Fix .gitea action's 'Run review' env to use GITHUB_SERVER_URL and GITHUB_REPOSITORY (matching .github version and review-bot binary env var expectations) - Add .github/workflows/review.yml for self-testing on github.concur.com Backward compatible: existing Gitea callers using default inputs continue to resolve rodin/review-bot via /api/v1/ unchanged. Closes #120 |