docs: define bootstrap review path for self-hosted MCP fixes (Closes #557) #564

Merged

sysadmin merged 1 commits from docs/issue-557-bootstrap-review-path into master 2026-07-08 21:52:44 -05:00

Conversation 3 Commits 1 Files Changed 4

+307 -1

jcwalker3 commented 2026-07-08 21:36:50 -05:00

Owner

Copy Link

Copy Source

Summary

Closes #557.

Defines the narrow controller-authorized bootstrap review path for Gitea-Tools MCP workflow fixes that cannot be reviewed by a live daemon still running broken master (bootstrap deadlock).

Changes

• docs/bootstrap-review-path.md — durable authorization marker, required audits, allowed/forbidden actions, landing procedure, post-land daemon restart + canonical re-verify
• Links from docs/llm-workflow-runbooks.md and skills/llm-project-workflow/SKILL.md
• tests/test_bootstrap_review_path_docs.py — doc contract tests

Explicit non-goals

Does not weaken normal reviewer/merger gates for ordinary PRs.

Worktree

branches/docs-issue-557-bootstrap-review-path @ 565363d only.

Tests

pytest tests/test_bootstrap_review_path_docs.py -q
# 7 passed

## Summary Closes #557. Defines the narrow controller-authorized **bootstrap review path** for Gitea-Tools MCP workflow fixes that cannot be reviewed by a live daemon still running broken `master` (bootstrap deadlock). ## Changes - `docs/bootstrap-review-path.md` — durable authorization marker, required audits, allowed/forbidden actions, landing procedure, post-land daemon restart + canonical re-verify - Links from `docs/llm-workflow-runbooks.md` and `skills/llm-project-workflow/SKILL.md` - `tests/test_bootstrap_review_path_docs.py` — doc contract tests ## Explicit non-goals Does **not** weaken normal reviewer/merger gates for ordinary PRs. ## Worktree `branches/docs-issue-557-bootstrap-review-path` @ `565363d` only. ## Tests ``` pytest tests/test_bootstrap_review_path_docs.py -q # 7 passed ```

jcwalker3 added 1 commit 2026-07-08 21:36:51 -05:00

docs: define bootstrap review path for self-hosted MCP fixes (Closes #557) ... 565363d5f4

Document the narrow controller-authorized path for landing MCP workflow
fixes when live daemons cannot canonically review themselves, with
required audits, allowed/forbidden actions, and post-land restart steps.

sysadmin commented 2026-07-08 21:51:56 -05:00

Owner

Copy Link

Copy Source

repo: Scaled-Tech-Consulting/Gitea-Tools
pr: #564
issue: #557
reviewer_identity: sysadmin
profile: prgs-reviewer
session_id: 35511-bf807139fe99
worktree: /Users/jasonwalker/Development/Gitea-Tools/branches/review-queue-mergeable
phase: claimed
candidate_head: 565363d5f4
target_branch: master
target_branch_sha: none
last_activity: 2026-07-09T02:51:56Z
expires_at: 2026-07-09T04:51:56Z
blocker: none

<!-- mcp-review-lease:v1 --> repo: Scaled-Tech-Consulting/Gitea-Tools pr: #564 issue: #557 reviewer_identity: sysadmin profile: prgs-reviewer session_id: 35511-bf807139fe99 worktree: /Users/jasonwalker/Development/Gitea-Tools/branches/review-queue-mergeable phase: claimed candidate_head: 565363d5f45a4c43362b064bd75c93122c7efb82 target_branch: master target_branch_sha: none last_activity: 2026-07-09T02:51:56Z expires_at: 2026-07-09T04:51:56Z blocker: none

sysadmin approved these changes 2026-07-08 21:52:14 -05:00

sysadmin left a comment

Owner

Copy Link

Copy Source

Independent review (sysadmin / prgs-reviewer)

• Author is jcwalker3 (not self-review)
• Scope is documentation + contract tests for bootstrap review path (#557)
• Does not weaken normal gates; narrow bootstrap authorization only
• Mergeable; head pinned 565363d

APPROVE for merge.

## Independent review (sysadmin / prgs-reviewer) - Author is jcwalker3 (not self-review) - Scope is documentation + contract tests for bootstrap review path (#557) - Does not weaken normal gates; narrow bootstrap authorization only - Mergeable; head pinned 565363d **APPROVE** for merge.

sysadmin commented 2026-07-08 21:52:34 -05:00

Owner

Copy Link

Copy Source

adopted_at: 2026-07-09T02:52:34Z
adopted_by_identity: sysadmin
adopted_by_profile: prgs-merger
adopted_from_session_id: 35511-bf807139fe99
adopted_from_profile: prgs-reviewer
adopted_from_reviewer_identity: sysadmin
adopted_from_comment_id: 7560
adoption_reason: merger-handoff-approved-head

repo: Scaled-Tech-Consulting/Gitea-Tools
pr: #564
issue: #557
reviewer_identity: sysadmin
profile: prgs-merger
session_id: 35507-64c0ad907fa3
worktree: /Users/jasonwalker/Development/Gitea-Tools/branches/review-queue-mergeable
phase: adopted
candidate_head: 565363d5f4
target_branch: master
target_branch_sha: none
last_activity: 2026-07-09T02:52:34Z
expires_at: 2026-07-09T04:52:34Z
blocker: none

<!-- mcp-review-lease-adoption:v1 --> adopted_at: 2026-07-09T02:52:34Z adopted_by_identity: sysadmin adopted_by_profile: prgs-merger adopted_from_session_id: 35511-bf807139fe99 adopted_from_profile: prgs-reviewer adopted_from_reviewer_identity: sysadmin adopted_from_comment_id: 7560 adoption_reason: merger-handoff-approved-head <!-- mcp-review-lease:v1 --> repo: Scaled-Tech-Consulting/Gitea-Tools pr: #564 issue: #557 reviewer_identity: sysadmin profile: prgs-merger session_id: 35507-64c0ad907fa3 worktree: /Users/jasonwalker/Development/Gitea-Tools/branches/review-queue-mergeable phase: adopted candidate_head: 565363d5f45a4c43362b064bd75c93122c7efb82 target_branch: master target_branch_sha: none last_activity: 2026-07-09T02:52:34Z expires_at: 2026-07-09T04:52:34Z blocker: none

sysadmin merged commit 4bcdd350f6 into master 2026-07-08 21:52:44 -05:00

Sign in to join this conversation.

Reviewers

No Reviewers

sysadmin

Labels

Clear labels

allocator

Controller-owned work allocator

anti-stomp

Prevent concurrent LLM session stomping

architecture

Architecture / structural design

bug

chore

codex

OpenAI Codex client / workflow session surface

concurrency

Concurrent session safety

contamination

Workflow or session contamination incident

control-plane

MCP control-plane coordination and allocation authority

dashboard

MCP operational dashboard/queue view

database

Internal coordination storage (SQLite/Postgres)

design

Design / investigation, no implementation

documentation

Docs / runbooks

enhancement

New feature or improvement

gitea

Gitea MCP workflow

glitchtip

GlitchTip integration

important

incident

Operational or process incident requiring durable audit trail

incident-bridge

Sentry-to-Gitea incident bridging

integration

Integration testing

jenkins

Jenkins integration

labels

Label taxonomy management

leases

Lease adopt/release/expire lifecycle

mcp

MCP server / tooling

mcp-health

MCP namespace and runtime health

mcp-menu

MCP menu surface

multi-project

Work spanning multiple monitoring projects or Gitea repos

mutating

Mutating action; requires gating

nice-to-have

observability

Observability, metrics, traces, error reporting

portability

Cross-platform / portability

preflight

Shared preflight gates before mutation

protected-branch

Protected branch / stable-branch policy concern

queue

Work queue visibility and allocation

read-only

Read-only, no mutation

reconnect

MCP client reconnect/reload recovery path

recovery

Recovery paths for stale/foreign leases

refactor

Code refactor / restructure

release

Release / versioning

reliability

Reliability / failure handling

resumable-review

Persist and resume prepared review verdicts across sessions

reviewer

Reviewer workflow tooling

roadmap

Roadmap / umbrella issue

safety

Safety rails and fail-closed mutation guards

security

Security / trust boundary

self-hosted

Self-hosted infrastructure integration

sentry

Sentry error monitoring integration

stale-runtime

Stale backend daemon / runtime-vs-master parity failures

status:blocked

Issue is blocked

status:in-progress

Issue is being worked on

status:pr-open

Issue has an open pull request

status:ready

Issue is ready for work

terminal-lock

Terminal review lock (#332) path

testing

Tests / test coverage

tracker

Issue tracker hygiene / meta

type:bug

Bug or defect

type:feature

Feature or enhancement

type:feature

Feature or enhancement

type:guardrail

Safety gate or guardrail

visibility

Workflow state visibility for LLMs/operators

workflow

Cross-tool workflow

workflow-hardening

LLM workflow coordination hardening

workflow-hardening

LLM workflow coordination hardening

No labels

Milestone

No items

No Milestone

Projects

Clear projects

No projects

Assignees

Clear assignees

jcwalker3 sysadmin

No Assignees

2 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Scaled-Tech-Consulting/Gitea-Tools#564