fix(#230): Correct documentation errors from code review
- Fix CRITICAL: Correct 5 environment variable names to match actual config (VALKEY_HOST not ORCHESTRATOR_VALKEY_HOST, CLAUDE_API_KEY not ORCHESTRATOR_CLAUDE_API_KEY, etc.) - Fix CRITICAL: Correct quality gate profiles table to match actual gate-config service (minimal = tests only, not typecheck+lint; add agent type defaults) - Fix IMPORTANT: Add missing gateProfile optional field to spawn request docs Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
@@ -58,6 +58,7 @@ Monitored via `apps/web/` (Agent Dashboard).
|
|||||||
{
|
{
|
||||||
"taskId": "string (required)",
|
"taskId": "string (required)",
|
||||||
"agentType": "worker | reviewer | tester",
|
"agentType": "worker | reviewer | tester",
|
||||||
|
"gateProfile": "strict | standard | minimal | custom (optional)",
|
||||||
"context": {
|
"context": {
|
||||||
"repository": "https://git.example.com/repo.git",
|
"repository": "https://git.example.com/repo.git",
|
||||||
"branch": "main",
|
"branch": "main",
|
||||||
@@ -134,11 +135,11 @@ orchestrator:events → Pub/sub channel for lifecycle events
|
|||||||
|
|
||||||
## Quality Gate Profiles
|
## Quality Gate Profiles
|
||||||
|
|
||||||
| Profile | Pre-commit | Post-commit |
|
| Profile | Default For | Gates |
|
||||||
| -------- | ---------------------- | --------------------------------------------- |
|
| -------- | ----------- | --------------------------------------------------------------------- |
|
||||||
| strict | typecheck, lint, tests | coverage (85%), build, integration, AI review |
|
| strict | reviewer | typecheck, lint, tests, coverage (85%), build, integration, AI review |
|
||||||
| standard | typecheck, lint, tests | coverage (85%), build |
|
| standard | worker | typecheck, lint, tests, coverage (85%) |
|
||||||
| minimal | typecheck, lint | build |
|
| minimal | tester | tests only |
|
||||||
|
|
||||||
## Development
|
## Development
|
||||||
|
|
||||||
@@ -175,14 +176,14 @@ pnpm --filter @mosaic/orchestrator lint
|
|||||||
|
|
||||||
Environment variables loaded via `@nestjs/config`. Key variables:
|
Environment variables loaded via `@nestjs/config`. Key variables:
|
||||||
|
|
||||||
| Variable | Description |
|
| Variable | Description |
|
||||||
| ------------------------------ | ---------------------------- |
|
| ------------------- | -------------------------------------- |
|
||||||
| `ORCHESTRATOR_PORT` | HTTP port (default: 3001) |
|
| `ORCHESTRATOR_PORT` | HTTP port (default: 3001) |
|
||||||
| `ORCHESTRATOR_CLAUDE_API_KEY` | Claude API key for agents |
|
| `CLAUDE_API_KEY` | Claude API key for agents |
|
||||||
| `ORCHESTRATOR_VALKEY_HOST` | Valkey/Redis host |
|
| `VALKEY_HOST` | Valkey/Redis host (default: localhost) |
|
||||||
| `ORCHESTRATOR_VALKEY_PORT` | Valkey/Redis port |
|
| `VALKEY_PORT` | Valkey/Redis port (default: 6379) |
|
||||||
| `ORCHESTRATOR_COORDINATOR_URL` | Quality Coordinator base URL |
|
| `COORDINATOR_URL` | Quality Coordinator base URL |
|
||||||
| `ORCHESTRATOR_DOCKER_ENABLED` | Enable Docker sandbox |
|
| `SANDBOX_ENABLED` | Enable Docker sandbox (true/false) |
|
||||||
|
|
||||||
## Related Documentation
|
## Related Documentation
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user