feat(gitpush): support force-with-lease push #10

Merged

genji1024 merged 1 commit from 8-feat/gitpush-force-with-lease into main

2026-09-07 22:55:35 +09:00

bot-genji1024 commented

2026-09-07 22:40:38 +09:00

Collaborator

Closes #8

概要

gitpush-mcp の push に force フラグを追加し、--force-with-lease に対応する。private-note#10 の15コミット squash（b83aa26）を force-push するために必要（c463 指示による実装）。

変更内容

dockerfiles/gitpush-mcp/server.py — push(..., force=False) 追加、真のときのみ --force-with-lease

方針（#8 本文どおり）:

既定 force=false は従来どおり plain push
force=true のみ --force-with-lease --set-upstream
--force ではなく --force-with-lease で、リモート想定外進行時の誤上書きを防ぐ
permission 変更なし（pusher のみ gitpush_* allow）。force 実行可否はオーナー判断、bot は独断で force しない

動作確認

python3 -m py_compile dockerfiles/gitpush-mcp/server.py — 成功
本ブランチ自体を gitpush-mcp plain push で push 成功
force=true の実 force-push は、本 PR マージ後の新 gitpush-mcp イメージでのみ検証可能（現稼働イメージは旧コード）。マージ後に private-note#10 の b83aa26 で検証する

手動確認手順

本 PR マージ後に gitpush-mcp イメージを再ビルド: docker compose -f compose.yaml build gitpush-mcp
docker compose up -d gitpush-mcp で再起動
private-note-fix ローカル（b83aa26）から push(..., force=true) で force-with-lease が通ることを確認
Forgejo 側で migrate/postgres-minio が単一コミットに整理されたことを確認

備考

#7（file-push deny）とは独立。本 PR は #8 のみ閉じる
トークン scrub・askpass 認証は既存維持

Closes #8 ## 概要 gitpush-mcp の `push` に `force` フラグを追加し、`--force-with-lease` に対応する。private-note#10 の15コミット squash（`b83aa26`）を force-push するために必要（c463 指示による実装）。 ## 変更内容 - `dockerfiles/gitpush-mcp/server.py` — `push(..., force=False)` 追加、真のときのみ `--force-with-lease` 方針（#8 本文どおり）: - 既定 `force=false` は従来どおり plain push - `force=true` のみ `--force-with-lease --set-upstream` - `--force` ではなく `--force-with-lease` で、リモート想定外進行時の誤上書きを防ぐ - permission 変更なし（pusher のみ `gitpush_*` allow）。force 実行可否はオーナー判断、bot は独断で force しない ## 動作確認 - [x] `python3 -m py_compile dockerfiles/gitpush-mcp/server.py` — 成功 - [x] 本ブランチ自体を gitpush-mcp plain push で push 成功 - [ ] `force=true` の実 force-push は、本 PR マージ後の新 gitpush-mcp イメージでのみ検証可能（現稼働イメージは旧コード）。マージ後に private-note#10 の `b83aa26` で検証する ## 手動確認手順 1. 本 PR マージ後に gitpush-mcp イメージを再ビルド: `docker compose -f compose.yaml build gitpush-mcp` 2. `docker compose up -d gitpush-mcp` で再起動 3. private-note-fix ローカル（`b83aa26`）から `push(..., force=true)` で force-with-lease が通ることを確認 4. Forgejo 側で `migrate/postgres-minio` が単一コミットに整理されたことを確認 ## 備考 - #7（file-push deny）とは独立。本 PR は #8 のみ閉じる - トークン scrub・askpass 認証は既存維持

bot-genji1024 added 1 commit

2026-09-07 22:40:38 +09:00

feat(gitpush): support force-with-lease push dae6c3c431

- Add force flag to push tool for history cleanup
  - add force bool param default False keeps plain push
  - use force-with-lease only to avoid clobbering remote
  - keep token scrub and askpass auth unchanged

bot-genji1024 requested review from genji1024

2026-09-07 22:40:46 +09:00

bot-genji1024 referenced this pull request

2026-09-07 22:40:55 +09:00

forgejo-mcp / github-mcp 経由の push を制限し gitpush-mcp に一本化する方針 #7

bot-genji1024 referenced this pull request

2026-09-07 22:40:58 +09:00

gitpush-mcp で force-push (with-lease) に対応する #8