mosaic/stack
1
0
Fork 0
Code Issues 10 Pull Requests Actions Packages Projects Releases 2 Wiki Activity

[ORCH-114] Quality gate callbacks #249

New Issue
Closed
opened 2026-02-02 21:39:47 +00:00 by jason.woltje · 0 comments
jason.woltje commented 2026-02-02 21:39:47 +00:00
Owner
Copy Link

Call coordinator quality gates before commit/push.

Acceptance Criteria

  • src/coordinator/quality-gates.service.ts implemented
  • Pre-commit quality check (before git commit)
  • Post-commit quality check (before git push)
  • Parse quality gate response
  • Block commit/push if rejected
  • Return rejection details to agent

Dependencies

  • Blocked by: ORCH-113 (complete)

Technical Notes

Coordinator runs: typecheck, lint, tests, coverage. Orchestrator calls coordinator.

Implementation

See scratchpad: docs/scratchpads/orch-114-gates.md

Files Created:

  1. src/coordinator/quality-gates.service.ts - NestJS service with dependency injection
  2. src/coordinator/quality-gates.service.spec.ts - 22 comprehensive test cases

Test Coverage: 91.66% branch, 100% line (exceeds 85% requirement)

Integration: Updated coordinator.module.ts to export QualityGatesService.

Call coordinator quality gates before commit/push. ## Acceptance Criteria - [x] src/coordinator/quality-gates.service.ts implemented - [x] Pre-commit quality check (before git commit) - [x] Post-commit quality check (before git push) - [x] Parse quality gate response - [x] Block commit/push if rejected - [x] Return rejection details to agent ## Dependencies - Blocked by: ORCH-113 (complete) ## Technical Notes Coordinator runs: typecheck, lint, tests, coverage. Orchestrator calls coordinator. ## Implementation See scratchpad: docs/scratchpads/orch-114-gates.md Files Created: 1. src/coordinator/quality-gates.service.ts - NestJS service with dependency injection 2. src/coordinator/quality-gates.service.spec.ts - 22 comprehensive test cases Test Coverage: 91.66% branch, 100% line (exceeds 85% requirement) Integration: Updated coordinator.module.ts to export QualityGatesService.
jason.woltje added this to the M6-AgentOrchestration (0.0.6) milestone 2026-02-02 21:39:47 +00:00
jason.woltje added the orchestrator label 2026-02-02 21:39:47 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
ai
AI/LLM integration
 api
Backend/API work
 api
auth
Authentication
 database
database
Database/schema work
 devops
Docker/deployment
 docs
Documentation
 frontend
graph
knowledge-module
migration
Data migration
 orchestrator
Orchestrator service (apps/orchestrator/)
 p0
Critical priority
 p1
High priority
 p2
Medium priority
 p3
Low priority
 performance
Performance work
 phase-1
phase-2
phase-3
phase-4
phase-5
plugin
MoltBot plugin work
 search
security
Security-related
 setup
Initial setup
 testing
Testing/QA
 web
Frontend work
No Label orchestrator
Milestone
No items
No Milestone M6-AgentOrchestration (0.0.6)
Projects
Clear projects
No project
Assignees
Clear assignees
jason.woltje
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: mosaic/stack#249
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?