feat: extract completion validator into provider plugin #347

Merged
karsten merged 3 commits from feat/completion-validator-plugin into main 2026-11-11 20:31:18 +00:00
Owner

Implements the completion-validator extraction (not code review): minimal versioned provider contract/registry, startup PluginAPI registration, explicit completionCheck.provider selection/disable, unchanged legacy default. Task/nightly/delegated-step share one registry and preserve host authority and existing persisted shapes.

Plugin implementation published at 0df11da76fc8e5f15617fa09c7450e7f2ced5458 in plugins#91; pin includes exact new-file hash. Merge plugin PR before Core integration.

Verification: npm run typecheck passed; normal npm test passed 1897/1898, zero failures, one skipped. Four plugin tests passed. Added integration tests for selected provider, disabled evidence retention and actual failed worker precedence. Diff has no core default prompt additions, routing or model policy changes; dispatcher edits only pass registry to existing completion call. Evidence: development-evidence/completion-validator-plugin.md.

No rollout/config/restart yet. Concurrent compaction worker and original dirty checkouts untouched.

Implements the completion-validator extraction (not code review): minimal versioned provider contract/registry, startup PluginAPI registration, explicit completionCheck.provider selection/disable, unchanged legacy default. Task/nightly/delegated-step share one registry and preserve host authority and existing persisted shapes. Plugin implementation published at 0df11da76fc8e5f15617fa09c7450e7f2ced5458 in plugins#91; pin includes exact new-file hash. Merge plugin PR before Core integration. Verification: npm run typecheck passed; normal npm test passed 1897/1898, zero failures, one skipped. Four plugin tests passed. Added integration tests for selected provider, disabled evidence retention and actual failed worker precedence. Diff has no core default prompt additions, routing or model policy changes; dispatcher edits only pass registry to existing completion call. Evidence: development-evidence/completion-validator-plugin.md. No rollout/config/restart yet. Concurrent compaction worker and original dirty checkouts untouched.
feat: host completion validator provider plugins without changing task authority
All checks were successful
decision-interface-ci / full-validation (pull_request) Successful in 1m53s
d48cdbc51c
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
harness/core!347
No description provided.