Epic #322: connector skill contracts — 6/6 complete#394
Open
bradtaylorsf wants to merge 3 commits into
Open
Conversation
29d4aee to
68d428e
Compare
68d428e to
5cf24c4
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Session Summary
Branch:
session/epic-322-epic-connector-skill-contracts-teach-every-connector-enforce-every-payloadEpic: #322
Issues completed: 6 (6 succeeded, 0 failed)
Recovered after interrupted finalization: yes
Issues
Closes #316
Closes #317
Closes #318
Closes #319
Closes #320
Closes #321
Verification
bash scripts/test.sh— PASS (2534 passed, 2 skipped)uv run pytest tests/test_orchestrator_connector_contracts.py tests/test_candidate_roster_backtest.py tests/test_skills_content.py tests/test_doctor.py tests/research_agent/tools/test_registry.py tests/test_orchestrator_loop.py::test_connector_search_passthrough_covers_registered_payload_contracts -q— PASS (259 passed)uv run research doctor --json— PASS (ok: true; registry contract, skill, planner allowlist, and TaskKind coherence allok)CA; FEC empty candidate sweep repairs tokind=candidates_enumerate; sanctionskindspayload validates.Review Fix
Independent review found that validated connector schema knobs accepted by connector
search()functions could still be dropped by the dispatch passthrough whitelist. This branch addskinds,provider, andtimeoutto the whitelist plus a registry-wide regression test so planner-visible connector knobs cannot silently disappear before dispatch.Notes
This PR was recovered onto current
origin/mainafter the original alpha-loop finalization force-pushed an incomplete session branch. The recovered branch is based onorigin/mainand contains the two successful batch commits with thedocs/CONFIG.mdconflict resolved by keeping both current config documentation and the new candidate-roster backtest guidance.Automated by alpha-loop, recovered and independently verified by Codex.