docs: pin GlitchTip filing orchestrator contract (Closes #153) #343

Merged
sysadmin merged 1 commits from feat/issue-153-glitchtip-filing-orchestrator into master 2026-07-07 04:05:22 -05:00
Owner

Summary

Closes #153.

Pins the Gitea-Tools-side contract for the GlitchTip-to-Gitea filing orchestrator now that the implementation lives as a library-only path in Scaled-Tech-Consulting/mcp-control-plane.

Changes

  • docs/architecture/glitchtip-to-gitea-workflow-design.md - updates the design into the #153 acceptance contract, including real GlitchTip read path, dedup-before-create, fail-closed mutation audit, and separate write-boundary requirements.
  • docs/mcp-client-registration.md - clarifies that glitchtip-mcp remains read-only and filing must not be exposed there.
  • tests/test_external_mcp_registration_docs.py - adds doc-contract tests for the filing boundary and acceptance criteria.

Validation

  • pytest tests/test_external_mcp_registration_docs.py -q -> 10 passed
  • pytest tests/test_external_mcp_registration_docs.py tests/test_operator_guide.py tests/test_mcp_discoverability.py -q -> 41 passed
  • pytest tests/ -q in task worktree -> 1 failed, 1079 passed, 6 skipped, 20 subtests passed
  • Baseline pytest tests/ -q on clean prgs/master (fd396df334862f5950fc4d688816a4886a7f84f9) -> matching failure: tests/test_agent_temp_artifacts.py::TestIssueLockArtifactWarning::test_lock_success_includes_artifact_warning; 1 failed, 1077 passed, 6 skipped, 20 subtests passed

Risk

Low. This is docs and doc-contract test coverage only; no runtime MCP behavior changes.

Worktree

  • Branch: feat/issue-153-glitchtip-filing-orchestrator
  • Worktree: /Users/jasonwalker/Development/Gitea-Tools/branches/feat-issue-153-glitchtip-filing-orchestrator
  • Commit: 873bcd06e50108645f1329e61e192901ad47c1fe
## Summary Closes #153. Pins the Gitea-Tools-side contract for the GlitchTip-to-Gitea filing orchestrator now that the implementation lives as a library-only path in `Scaled-Tech-Consulting/mcp-control-plane`. ## Changes - `docs/architecture/glitchtip-to-gitea-workflow-design.md` - updates the design into the #153 acceptance contract, including real GlitchTip read path, dedup-before-create, fail-closed mutation audit, and separate write-boundary requirements. - `docs/mcp-client-registration.md` - clarifies that `glitchtip-mcp` remains read-only and filing must not be exposed there. - `tests/test_external_mcp_registration_docs.py` - adds doc-contract tests for the filing boundary and acceptance criteria. ## Validation - `pytest tests/test_external_mcp_registration_docs.py -q` -> 10 passed - `pytest tests/test_external_mcp_registration_docs.py tests/test_operator_guide.py tests/test_mcp_discoverability.py -q` -> 41 passed - `pytest tests/ -q` in task worktree -> 1 failed, 1079 passed, 6 skipped, 20 subtests passed - Baseline `pytest tests/ -q` on clean `prgs/master` (`fd396df334862f5950fc4d688816a4886a7f84f9`) -> matching failure: `tests/test_agent_temp_artifacts.py::TestIssueLockArtifactWarning::test_lock_success_includes_artifact_warning`; 1 failed, 1077 passed, 6 skipped, 20 subtests passed ## Risk Low. This is docs and doc-contract test coverage only; no runtime MCP behavior changes. ## Worktree - Branch: `feat/issue-153-glitchtip-filing-orchestrator` - Worktree: `/Users/jasonwalker/Development/Gitea-Tools/branches/feat-issue-153-glitchtip-filing-orchestrator` - Commit: `873bcd06e50108645f1329e61e192901ad47c1fe`
jcwalker3 added 1 commit 2026-07-07 03:28:19 -05:00
sysadmin approved these changes 2026-07-07 04:05:12 -05:00
sysadmin left a comment
Owner

All tests pass successfully. Reviewed the updates to docs/architecture/glitchtip-to-gitea-workflow-design.md, docs/mcp-client-registration.md, and the test coverage in tests/test_external_mcp_registration_docs.py. They correctly pin the architectural contract and safety boundaries for the GlitchTip-to-Gitea filing orchestrator as a library-only component, and ensure that glitchtip-mcp remains strictly read-only. Tested inside the review worktree, and the single failing test is pre-existing on master.

All tests pass successfully. Reviewed the updates to docs/architecture/glitchtip-to-gitea-workflow-design.md, docs/mcp-client-registration.md, and the test coverage in tests/test_external_mcp_registration_docs.py. They correctly pin the architectural contract and safety boundaries for the GlitchTip-to-Gitea filing orchestrator as a library-only component, and ensure that glitchtip-mcp remains strictly read-only. Tested inside the review worktree, and the single failing test is pre-existing on master.
sysadmin merged commit cbae9a3ecd into master 2026-07-07 04:05:22 -05:00
Sign in to join this conversation.
No Reviewers
No labels
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

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