Skip to content

fix(prompts): consolidate controller guidance without duplicate injection - #47

Merged
osovv merged 2 commits into
mainfrom
fix/consolidate-controller-guidance
Nov 8, 2026
Merged

osovv merged 2 commits into
mainfrom
fix/consolidate-controller-guidance

Conversation

@osovv

@osovv osovv commented Nov 8, 2026

Copy link
Copy Markdown
Owner

Related deduplication PR #44

Why the change

Replace competing controller instructions with the approved seven-section prompt while preserving repository memory, verification, delegation, and bounded recovery guidance.

Special things to note

  • This supersedes refactor(prompts): deduplicate vv-controller template sections against universal injection #44: it covers the four duplicated sections and the unique contract-reading/reroute rules, then consolidates the remaining wording; merge this version rather than applying both PRs.
  • Existing managed installations need vvoc sync and an OpenCode restart; the full controller template remains self-contained when injection is disabled, and other primary agents receive shared guidance without controller-only dispatch/recovery instructions.
  • bun run check passed (2297 tests, 7 live-host skips), bun run pack:check and GRACE lint passed; full live-host verification is blocked by the missing pinned OpenCode 2.2.5 binary, and model behavior was not evaluated.

Change outline

One wording source serves installation and runtime injection:

templates/agents/vv-controller.md
  task_and_scope · project_context · repository_memory
  evidence_discipline · working_cycle
  delegation_and_workflow · communication
  │
  ├─ install / sync → complete controller prompt
  └─ plugin setup → guidance.ts
       ├─ vv-controller → the same seven sections
       │    append only missing text; accept LF/CRLF without rewriting host text
       └─ other eligible primaries → four shared sections
            + shared authority and delivery fragments

Session profile → existing controller user-tail policy
Workflow plugin → existing workflow and result protocols

Memory remains explicit reading of the existing lessons/runbooks indexes; skill discovery belongs to OpenCode, and SDD file conventions remain in the existing skills.

- Separate, overlapping template and universal instruction wordings
+ Canonical wording with preserved counterexamples, drift thresholds and authority limits
+ Assembly checks: every controller section once, all four profiles, repeated hooks, CRLF
+ Shared-primary memory/evidence coverage and subagent exclusions
+ Sync and workflow scenario checks updated to the approved wording
+ Real-host probes identify primary requests by the retained repository_memory marker

osovv added 2 commits November 8, 2026 10:30
Resolve in favor of the consolidated seven-section controller guidance
(PRs #44 approach superseded per PR #47 description): the bundled
template stays the canonical wording source and system-context-injection
projects shared guidance from it via guidance.ts.
@osovv
osovv merged commit 96c5133 into main Nov 8, 2026
1 check passed
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