fix: refactor: extract push_to_forge() and webhook setup from bin/disinto into lib/forge-push.sh (#302)
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Claude
parent
e65e091d3c
commit
444c5e34cc
2 changed files with 100 additions and 67 deletions
69
bin/disinto
69
bin/disinto
|
|
@ -28,6 +28,7 @@ source "${FACTORY_ROOT}/lib/env.sh"
|
|||
source "${FACTORY_ROOT}/lib/ops-setup.sh"
|
||||
source "${FACTORY_ROOT}/lib/hire-agent.sh"
|
||||
source "${FACTORY_ROOT}/lib/forge-setup.sh"
|
||||
source "${FACTORY_ROOT}/lib/forge-push.sh"
|
||||
|
||||
# ── Helpers ──────────────────────────────────────────────────────────────────
|
||||
|
||||
|
|
@ -571,73 +572,7 @@ is_compose_mode() {
|
|||
# The ops repo holds operational data: vault items, journals, evidence, prerequisites.
|
||||
# ops repo setup is now in lib/ops-setup.sh
|
||||
|
||||
# Push local clone to the Forgejo remote.
|
||||
push_to_forge() {
|
||||
local repo_root="$1" forge_url="$2" repo_slug="$3"
|
||||
|
||||
# Build authenticated remote URL: http://dev-bot:<token>@host:port/org/repo.git
|
||||
if [ -z "${FORGE_TOKEN:-}" ]; then
|
||||
echo "Error: FORGE_TOKEN not set — cannot push to Forgejo" >&2
|
||||
return 1
|
||||
fi
|
||||
local auth_url
|
||||
auth_url=$(printf '%s' "$forge_url" | sed "s|://|://dev-bot:${FORGE_TOKEN}@|")
|
||||
local remote_url="${auth_url}/${repo_slug}.git"
|
||||
# Display URL without token
|
||||
local display_url="${forge_url}/${repo_slug}.git"
|
||||
|
||||
# Always set the remote URL to ensure credentials are current
|
||||
if git -C "$repo_root" remote get-url forgejo >/dev/null 2>&1; then
|
||||
git -C "$repo_root" remote set-url forgejo "$remote_url"
|
||||
else
|
||||
git -C "$repo_root" remote add forgejo "$remote_url"
|
||||
fi
|
||||
echo "Remote: forgejo -> ${display_url}"
|
||||
|
||||
# Skip push if local repo has no commits (e.g. cloned from empty Forgejo repo)
|
||||
if ! git -C "$repo_root" rev-parse HEAD >/dev/null 2>&1; then
|
||||
echo "Push: skipped (local repo has no commits)"
|
||||
return 0
|
||||
fi
|
||||
|
||||
# Push all branches and tags
|
||||
echo "Pushing: branches to forgejo"
|
||||
if ! git -C "$repo_root" push forgejo --all 2>&1; then
|
||||
echo "Error: failed to push branches to Forgejo" >&2
|
||||
return 1
|
||||
fi
|
||||
echo "Pushing: tags to forgejo"
|
||||
if ! git -C "$repo_root" push forgejo --tags 2>&1; then
|
||||
echo "Error: failed to push tags to Forgejo" >&2
|
||||
return 1
|
||||
fi
|
||||
|
||||
# Verify the repo is no longer empty (Forgejo may need a moment to index pushed refs)
|
||||
local is_empty="true"
|
||||
local verify_attempt
|
||||
for verify_attempt in $(seq 1 5); do
|
||||
local repo_info
|
||||
repo_info=$(curl -sf --max-time 10 \
|
||||
-H "Authorization: token ${FORGE_TOKEN}" \
|
||||
"${forge_url}/api/v1/repos/${repo_slug}" 2>/dev/null) || repo_info=""
|
||||
if [ -z "$repo_info" ]; then
|
||||
is_empty="skipped"
|
||||
break # API unreachable, skip verification
|
||||
fi
|
||||
is_empty=$(printf '%s' "$repo_info" | jq -r '.empty // "unknown"')
|
||||
if [ "$is_empty" != "true" ]; then
|
||||
echo "Verify: repo is not empty (push confirmed)"
|
||||
break
|
||||
fi
|
||||
if [ "$verify_attempt" -lt 5 ]; then
|
||||
sleep 2
|
||||
fi
|
||||
done
|
||||
if [ "$is_empty" = "true" ]; then
|
||||
echo "Warning: Forgejo repo still reports empty after push" >&2
|
||||
return 1
|
||||
fi
|
||||
}
|
||||
# push_to_forge() is sourced from lib/forge-push.sh
|
||||
|
||||
# Preflight check — verify all factory requirements before proceeding.
|
||||
preflight_check() {
|
||||
|
|
|
|||
98
lib/forge-push.sh
Normal file
98
lib/forge-push.sh
Normal file
|
|
@ -0,0 +1,98 @@
|
|||
#!/usr/bin/env bash
|
||||
# =============================================================================
|
||||
# forge-push.sh — push_to_forge() function
|
||||
#
|
||||
# Handles pushing a local clone to the Forgejo remote and verifying the push.
|
||||
#
|
||||
# Globals expected:
|
||||
# FORGE_URL - Forge instance URL (e.g. http://localhost:3000)
|
||||
# FORGE_TOKEN - API token for Forge operations
|
||||
# FACTORY_ROOT - Root of the disinto factory
|
||||
# PRIMARY_BRANCH - Primary branch name (e.g. main)
|
||||
#
|
||||
# Usage:
|
||||
# source "${FACTORY_ROOT}/lib/forge-push.sh"
|
||||
# push_to_forge <repo_root> <forge_url> <repo_slug>
|
||||
# =============================================================================
|
||||
set -euo pipefail
|
||||
|
||||
_assert_forge_push_globals() {
|
||||
local missing=()
|
||||
[ -z "${FORGE_URL:-}" ] && missing+=("FORGE_URL")
|
||||
[ -z "${FORGE_TOKEN:-}" ] && missing+=("FORGE_TOKEN")
|
||||
[ -z "${FACTORY_ROOT:-}" ] && missing+=("FACTORY_ROOT")
|
||||
[ -z "${PRIMARY_BRANCH:-}" ] && missing+=("PRIMARY_BRANCH")
|
||||
if [ "${#missing[@]}" -gt 0 ]; then
|
||||
echo "Error: forge-push.sh requires these globals to be set: ${missing[*]}" >&2
|
||||
exit 1
|
||||
fi
|
||||
}
|
||||
_assert_forge_push_globals
|
||||
|
||||
# Push local clone to the Forgejo remote.
|
||||
push_to_forge() {
|
||||
local repo_root="$1" forge_url="$2" repo_slug="$3"
|
||||
|
||||
# Build authenticated remote URL: http://dev-bot:<token>@host:port/org/repo.git
|
||||
if [ -z "${FORGE_TOKEN:-}" ]; then
|
||||
echo "Error: FORGE_TOKEN not set — cannot push to Forgejo" >&2
|
||||
return 1
|
||||
fi
|
||||
local auth_url
|
||||
auth_url=$(printf '%s' "$forge_url" | sed "s|://|://dev-bot:${FORGE_TOKEN}@|")
|
||||
local remote_url="${auth_url}/${repo_slug}.git"
|
||||
# Display URL without token
|
||||
local display_url="${forge_url}/${repo_slug}.git"
|
||||
|
||||
# Always set the remote URL to ensure credentials are current
|
||||
if git -C "$repo_root" remote get-url forgejo >/dev/null 2>&1; then
|
||||
git -C "$repo_root" remote set-url forgejo "$remote_url"
|
||||
else
|
||||
git -C "$repo_root" remote add forgejo "$remote_url"
|
||||
fi
|
||||
echo "Remote: forgejo -> ${display_url}"
|
||||
|
||||
# Skip push if local repo has no commits (e.g. cloned from empty Forgejo repo)
|
||||
if ! git -C "$repo_root" rev-parse HEAD >/dev/null 2>&1; then
|
||||
echo "Push: skipped (local repo has no commits)"
|
||||
return 0
|
||||
fi
|
||||
|
||||
# Push all branches and tags
|
||||
echo "Pushing: branches to forgejo"
|
||||
if ! git -C "$repo_root" push forgejo --all 2>&1; then
|
||||
echo "Error: failed to push branches to Forgejo" >&2
|
||||
return 1
|
||||
fi
|
||||
echo "Pushing: tags to forgejo"
|
||||
if ! git -C "$repo_root" push forgejo --tags 2>&1; then
|
||||
echo "Error: failed to push tags to Forgejo" >&2
|
||||
return 1
|
||||
fi
|
||||
|
||||
# Verify the repo is no longer empty (Forgejo may need a moment to index pushed refs)
|
||||
local is_empty="true"
|
||||
local verify_attempt
|
||||
for verify_attempt in $(seq 1 5); do
|
||||
local repo_info
|
||||
repo_info=$(curl -sf --max-time 10 \
|
||||
-H "Authorization: token ${FORGE_TOKEN}" \
|
||||
"${forge_url}/api/v1/repos/${repo_slug}" 2>/dev/null) || repo_info=""
|
||||
if [ -z "$repo_info" ]; then
|
||||
is_empty="skipped"
|
||||
break # API unreachable, skip verification
|
||||
fi
|
||||
is_empty=$(printf '%s' "$repo_info" | jq -r '.empty // "unknown"')
|
||||
if [ "$is_empty" != "true" ]; then
|
||||
echo "Verify: repo is not empty (push confirmed)"
|
||||
break
|
||||
fi
|
||||
if [ "$verify_attempt" -lt 5 ]; then
|
||||
sleep 2
|
||||
fi
|
||||
done
|
||||
if [ "$is_empty" = "true" ]; then
|
||||
echo "Warning: Forgejo repo still reports empty after push" >&2
|
||||
return 1
|
||||
fi
|
||||
}
|
||||
Loading…
Add table
Add a link
Reference in a new issue