Skip to content

docs: remove Consensus Review and Model Router — cleanup for v0.5.0#1709

Merged
OneStepAt4time merged 1 commit intodevelopfrom
docs/post-v0.5-cleanup
Apr 12, 2026
Merged

docs: remove Consensus Review and Model Router — cleanup for v0.5.0#1709
OneStepAt4time merged 1 commit intodevelopfrom
docs/post-v0.5-cleanup

Conversation

@OneStepAt4time
Copy link
Copy Markdown
Owner

Summary

Remove all documentation references to two deleted features (PR #1583) and update findings that were resolved in v0.3.3/0.5.0.

Features Removed (code deleted in PR #1583)

Feature PR
Consensus Review endpoints #1583
Model Router #1583

Findings Resolved in v0.3.3+

Finding PR Resolution
P-3: PipelineManager has no persistence #1585 State persisted to disk
P-1: No pipeline stage timeout #1606 AEGIS_DEFAULT_STAGE_TIMEOUT_MS added
CON-1: Consensus always returns running #1583 Feature removed

Files Changed (10 files)

  • README.md: Remove consensus/model router from features list
  • docs/advanced.md: Remove Consensus Review section
  • docs/api-reference.md: Remove consensus endpoints; remove consensus.completed SSE event
  • docs/architecture.md: Remove consensus.ts and model-router.ts from module overview
  • docs/getting-started.md: Update advanced features reference
  • docs/enterprise.md: Remove modelRouter from config example
  • docs/enterprise/01-architecture.md: Remove consensus/model-router; mark findings RESOLVED; renumber CFG-4→CFG-5
  • docs/enterprise/03-testing-observability.md: Remove model-router.ts from thin-tests list
  • docs/enterprise/05-enterprise-roadmap.md: Remove E4-1 (consensus); update M-E4 scope
  • docs/enterprise/index.md: Remove consensus reliability finding

Note

openapi.yaml still contains /consensus endpoint definitions — requires separate PR.


Tagged <@1490089830472880218> for review.

…1583)

Remove all documentation references to deleted features (PR #1583).

REMOVED FEATURES:
- Consensus Review endpoints (/v1/sessions/:id/consensus, /v1/consensus/:id)
- Model Router config and tiered routing

FILES CHANGED:
- README.md: Remove consensus/model router from features list
- docs/advanced.md: Remove Consensus Review section
- docs/api-reference.md: Remove consensus endpoints; remove consensus.completed SSE event
- docs/architecture.md: Remove consensus.ts and model-router.ts from module overview
- docs/getting-started.md: Update advanced features reference
- docs/enterprise.md: Remove modelRouter from config example
- docs/enterprise/01-architecture.md: Remove consensus/model-router; mark findings RESOLVED
- docs/enterprise/03-testing-observability.md: Remove model-router.ts from thin-tests list
- docs/enterprise/05-enterprise-roadmap.md: Remove E4-1 (consensus); update M-E4 scope
- docs/enterprise/index.md: Remove consensus reliability finding

RESOLVED IN v0.3.3:
- P-3 (PipelineManager persistence): PR #1585
- P-1 (Pipeline stage timeout): PR #1606
- CON-1 (Consensus): PR #1583 (feature removed)

NOTE: openapi.yaml still contains /consensus endpoints — separate update needed.
@OneStepAt4time OneStepAt4time merged commit b4d4179 into develop Apr 12, 2026
11 checks passed
@OneStepAt4time OneStepAt4time deleted the docs/post-v0.5-cleanup branch April 12, 2026 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant