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

Merged

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

2026-10-05 19:16:49 +00:00

kilo commented

2026-10-05 19:16:43 +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

Repository-specific permission allowlists (uv run --no-sync checks, UV_OFFLINE=1 make check, canonical Creatidy/model-intelligence ls-remote URLs), the .kilo/rules/50-technical-remediation.md references and checkout-isolation semantics, read-only behavior, verdict semantics, 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: 14 tests OK

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 Repository-specific permission allowlists (`uv run --no-sync` checks, `UV_OFFLINE=1 make check`, canonical Creatidy/model-intelligence ls-remote URLs), the `.kilo/rules/50-technical-remediation.md` references and checkout-isolation semantics, read-only behavior, verdict semantics, 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`: 14 tests OK

kilo added 1 commit

2026-10-05 19:16:43 +00:00

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

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.
Repository-specific permission allowlists, UV-offline validation commands, the
rule 50 references, read-only behavior and the stable JSON result schema are
unchanged.

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

kilo merged commit ceded05e2c into develop

2026-10-05 19:16:49 +00:00

kilo deleted branch maintenance/pr-reviewer-binding

2026-10-05 19:16:49 +00:00

kilo referenced this pull request from a commit

2026-10-05 19:16:51 +00:00

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