fix/v02-patch: pr_files_diff, template_name, repo_update #26

Merged

mathias merged 6 commits from fix/v02-patch into main 2026-05-16 22:03:29 +00:00

Conversation 0 Commits 6 Files Changed 12

+653 -287

6 Commits

Author	SHA1	Message	Date
mathiasandClaude Opus 4.7	3cccbfb8cb	chore: re-sync context adapters after rebase ... Upstream .context/PROJECT.md gained a branch-protection rule + an extra agent instruction. Pure regeneration via scripts/context-sync.sh to make task check pass before force-push. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-17 00:02:08 +02:00
mathiasandmathias	3648373333	fix: merge repo_update — add archived+template, keep default_branch+confirm from main	2026-05-16 23:54:16 +02:00
mathias	eeefc626ed	feat(repo_update): tool for archiving + metadata patches ... Adds a repo_update tool exposing PATCH /api/v1/repos/{owner}/{name} with optional pointer fields (archived, description, private, website, template). Only fields set by the caller are sent on the wire, so the server patches exactly what was asked for. Originally needed to archive ingestion-svc cleanly instead of leaving a README tombstone, and to flip template-go-{agent,web} to template=true so create_project_from_template stops failing the "is not marked as template" guard. Wire-level enforcement of "at least one field" returns ErrValidation before any network call, preventing no-op PATCHes. private=false (making a repo public) is allowed but flagged in the tool description with a "verify intent before calling" warning. The earlier issue draft suggested an ntfy confirmation hook for that path — out of scope for this PR; the warning string is the minimum that fits inside the tool surface today. Wires NewRepoUpdate into cmd/gitea-mcp/main.go alongside the rest of the repo_* family. Closes #12	2026-05-16 23:54:16 +02:00
mathias	5545d6ab4b	fix(create_project_from_template): accept per-call template_name override ... The template name was hardcoded into the binary at startup via NewCreateProjectFromTemplate("mathias", "template-go-web"), so generating from a different template (e.g. template-go-agent) required a code change and restart. The constructor already parameterised it correctly — the gap was at the tool's input schema, which never exposed template_name to the caller. Adds an optional template_name input field. When set, it overrides the server-configured default for that call only; when omitted, behavior is unchanged. Template owner stays server-configured — only the repo name is per-call. Server-side validation already verifies the resolved template exists and is marked as a template repo, so no enum constraint is added — keeps the door open for future templates (go-ml, go-service, ...) without redeploys. Adds TestCreateProjectTemplateNameOverride verifying the override directs both the template lookup and the /generate POST. Closes #24	2026-05-16 23:24:16 +02:00
mathias	9013c8ff9c	fix(pr_files_diff): copy per-file diff bytes to break buffer aliasing ... splitUnifiedDiff used bytes.Buffer to accumulate each file's diff, then stored buf.Bytes() into the result map and called buf.Reset() to start the next file. bytes.Buffer.Bytes() returns the buffer's internal backing slice; Reset() resets length to 0 but reuses the same backing array. As a result, every map entry aliased the same storage, so all files ended up showing the LAST file's diff content. Fix: copy the bytes into a fresh slice before storing in the map. Adds TestPRFilesDiffPerFileIsolation as a regression test that asserts each file entry contains its OWN diff --git header and none of the other files' headers. Verified failing on the prior code, passing after the fix. Closes #25	2026-05-16 23:24:16 +02:00
mathiasandClaude Opus 4.7	f26f922c96	chore: re-sync context adapters with upstream root ... Derived adapters drifted from canonical root .context/AGENT.md after the pgvector default change landed upstream. Pure regeneration via scripts/context-sync.sh, no manual edits. Required to make task check pass before the feature commits on this branch. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-16 23:24:16 +02:00