Make pr-reviewer binding provider/model-agnostic #194

Merged

kilo merged 1 commit from maintenance/pr-reviewer-binding into develop

2026-10-05 19:25:21 +00:00

kilo commented

2026-10-05 19:15:15 +00:00

Member

Owner-authorized maintenance (direct instruction, 2026-10-05): make the pr-reviewer contract provider/model-binding agnostic.

Change

Only .kilo/agents/pr-reviewer.md:

removed frontmatter model: openai/gpt-6.1-sol and variant: high (no replacement pin);
removed the prose asserting the GPT-6.1 Sol High binding and the installed Kilo 7.8.3 model listing;
inserted the canonical ## Reviewer binding policy: reviewer inherits the implementation session's provider family/model/reasoning configuration by default, with OpenAI→OpenAI and z.ai→z.ai provider-family affinity, explicit owner override support, no silent substitution, and precise reporting when a binding cannot be executed.

Preserved

Router-specific permission allowlists (uv run --no-sync checks, UV_OFFLINE=1 make check, canonical BioMedical-IT ls-remote URLs), the .kilo/rules/35-technical-recovery.md reference, sanitized-environment restrictions, checkout/read-only constraints, verdict semantics, the UX review dimension, and the stable JSON result schema (no new fields).

Validation

YAML frontmatter parses; remaining keys: description, mode, permission
no openai/gpt-6.1-sol, variant: high, GPT-6.1 or Kilo 7.8.3 references remain
git diff --name-only origin/develop...HEAD → only .kilo/agents/pr-reviewer.md
local python3 -m unittest tests.test_workflow_contract: 15 tests OK (the gpt-6.1-sol fixtures in execution/model tests are the router's own product tests, untouched)

Owner-authorized maintenance (direct instruction, 2026-10-05): make the `pr-reviewer` contract provider/model-binding agnostic. ## Change Only `.kilo/agents/pr-reviewer.md`: - removed frontmatter `model: openai/gpt-6.1-sol` and `variant: high` (no replacement pin); - removed the prose asserting the GPT-6.1 Sol High binding and the installed Kilo 7.8.3 model listing; - inserted the canonical `## Reviewer binding` policy: reviewer inherits the implementation session's provider family/model/reasoning configuration by default, with OpenAI→OpenAI and z.ai→z.ai provider-family affinity, explicit owner override support, no silent substitution, and precise reporting when a binding cannot be executed. ## Preserved Router-specific permission allowlists (`uv run --no-sync` checks, `UV_OFFLINE=1 make check`, canonical BioMedical-IT ls-remote URLs), the `.kilo/rules/35-technical-recovery.md` reference, sanitized-environment restrictions, checkout/read-only constraints, verdict semantics, the UX review dimension, and the stable JSON result schema (no new fields). ## Validation - YAML frontmatter parses; remaining keys: `description`, `mode`, `permission` - no `openai/gpt-6.1-sol`, `variant: high`, `GPT-6.1` or `Kilo 7.8.3` references remain - `git diff --name-only origin/develop...HEAD` → only `.kilo/agents/pr-reviewer.md` - local `python3 -m unittest tests.test_workflow_contract`: 15 tests OK (the `gpt-6.1-sol` fixtures in execution/model tests are the router's own product tests, untouched)

kilo added 1 commit

2026-10-05 19:15:15 +00:00

docs(workflow): make pr-reviewer binding provider/model-agnostic

ci / check (pull_request) Successful in 9m38s

Details

96e32722fc

Remove the hard-coded openai/gpt-6.1-sol + variant: high reviewer pin and the
Kilo 7.8.3 runtime-version assertion from the pr-reviewer contract. The
reviewer now inherits the implementation session's provider family, model and
reasoning/thinking configuration, with OpenAI-to-OpenAI and z.ai-to-z.ai
affinity by default, overridable only by explicit owner instruction.
Router-specific permission allowlists, UV-offline validation commands, the
35-technical-recovery reference, read-only behavior and the stable JSON result
schema are unchanged.

Owner-authorized maintenance; changes only .kilo/agents/pr-reviewer.md.

kilo merged commit d0df133ba1 into develop

2026-10-05 19:25:21 +00:00

kilo deleted branch maintenance/pr-reviewer-binding

2026-10-05 19:25:21 +00:00

kilo referenced this pull request from a commit

2026-10-05 19:25:23 +00:00

Merge pull request 'Make pr-reviewer binding provider/model-agnostic' (#194) from maintenance/pr-reviewer-binding into develop

kilo referenced this pull request

2026-10-05 22:58:51 +00:00

Return pre-implementation gated selections to SELECT without terminating the loop #195