feat(opencode): switch consumers from repo-search to codebase-memory (0.1.70) - #90
Merged
Conversation
Chart 0.1.70 stage 1 of 2: swap the opencode.json MCP entry from repo-search to the verified makeitwork-codebase-memory backend, and rework the kimi/kimi-256k subagent routing rules to the new contract (discovery via codebase-memory when indexed; exact contents, private repos, freshness, writes via GitHub; subagents never index).
Chart 0.1.70 stage 2 of 2: replace the cache-first repo-search bullets in the makeitwork and default primary-agent definitions with the codebase-memory discovery contract (index at /repos/<repo>/current, re-index on SHA moves, derived-state caveats, GitHub MCP for exact contents and HEAD verification). xnoto, README, and architecture doc follow in the final commit.
Chart 0.1.70 final stage: replace the xnoto primary agent's cache-first bullet (keeping the xnoto project-path conventions), reword the chart README Living-knowledge paragraph, and update both agent-instruction- architecture mentions to the codebase-memory contract.
Adversarial review findings: restore base line wrapping in default.md outside the replaced bullet (transcription drift); replace the repo-search mandatory-first-read rule in the charts root AGENTS.md with the codebase-memory discovery contract.
Adversarial review finding: restore the base line wrapping and apply only the two terminology substitutions (repo-search -> codebase-memory graph index; repo-search -> codebase-memory routing rule) instead of rewrapping surrounding passages.
…epair Adversarial review finding: the shared primary cache-first bullet appears in six additional primary-agent files beyond the three originally identified; replace it with the codebase-memory discovery contract in career, grillmaster, and homerepair (homesteader, lawnmowerman, teacher follow).
…/teacher Completes the adversarial-review scope fix: the last three of the six additional primary-agent files carrying the shared cache-first bullet now use the codebase-memory discovery contract. All packaged agents are now consistent with the opencode.json entry swap.
Chart CI passedRepository hygiene and Helm validation passed. View the workflow run. |
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
1 participant
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.
Summary
Stage 3 of the owner-approved repo-search deprecation (2026-09-09): swaps the OpenCode consumer from
makeitwork-repo-searchto the deployed-and-verifiedmakeitwork-codebase-memorybackend. Chart0.1.69 → 0.1.70.files/opencode.jsondrops therepo-searchentry and addscodebase-memoryat the verified endpointhttp://mcp-makeitwork-codebase-memory-proxy.mcp.svc:8080/mcp. Every packaged agent carrying the shared cache-first repo-search bullet —makeitwork,default,xnoto,career,grillmaster,homerepair,homesteader,lawnmowerman,teacher(nine primaries) plus thekimi/kimi-256ksubagent routing rules — now carries the codebase-memory discovery contract: index/query at/repos/<repo>/current(xnoto keeps its/repos/xnoto-<repository>/currentconventions), re-index when the indexed project's embedded worktree SHA is missing or stale (seconds per repo), discovery viasearch_graph/search_code/trace_path/get_architecture, exact file contents and pre-branch HEAD verification via GitHub MCP (whole-file reads are impossible through the index backend). Subagents never runindex_repository; primaries own indexing. The chart README "Living knowledge" paragraph,docs/agent-instruction-architecture.md(two mentions), and the repository rootAGENTS.mdfirst-read rule are updated to the same contract.Fixes # N/A — owner-directed migration; backend verification evidence in
agent-knowledgedocs/agents/makeitwork/codebase-memory-mcp-gateway-assessment.md.Type of change
Validation
package opencode-serverare the validation authority.Review gate record: adversarial review round 1 (head
5f00674) found: (Critical ×2) line-wrap drift outside intended hunks indefault.mdandagent-instruction-architecture.md; (High) six additional primary-agent files (career,grillmaster,homerepair,homesteader,lawnmowerman,teacher) still carrying the repo-search bullet whileopencode.jsondrops the entry; (High) rootAGENTS.mdstill mandating repo-search. All four resolved at headsc5bf39b–1c17e16(wrap restoration, minimal substitutions, six bullet replacements, AGENTS.md rule). Follow-up reviewer rounds held only on input-formality grounds without inspecting the branch; dispositions are verified against the authoritative PR file diffs and repo-widerepo-searchreference sweep, attached below as comments before merge. A docs-writer consistency pass was attempted but hit a provider quota limit; the adversarial review covered the documentation-consistency scope.Sequencing: the
makeitwork-repo-searchMCPServer stays deployed until AFTER this chart rolls out and a fresh OpenCode session verifies the new backend end-to-end; its removal (and remaining repo-search reference cleanup) is the explicit owner-directed follow-up.Impact and rollout
Producer:
opencode-serverchart0.1.70(packaged config, agents, docs, rootAGENTS.mdguidance). Consumers: all OpenCode agent sessions after rollout. Automatic after confirmed merge: main workflow publishes the immutable OCI chart and opens/updates thekustomize-clusterversion-pin PR (auto-merge on checks). Separate gates: pin-PR checks/auto-merge,opencodeApplication reconciliation, Deployment rollout, and a representative fresh-session verification ofcodebase-memorydiscovery through OpenCode before the repo-search removal PR. Rollback: select immutable0.1.69through the canonical GitOps Application; repo-search remains deployed until the removal PR, so rollback restores the previous consumer contract with zero data loss.Safety and secrets
AI-assisted change: an OpenCode agent materially produced this change; adversarial review was dispatched with all Critical/High findings resolved. Reviewers should verify the per-file diffs match the stated hunks only, and that no
repo-searchreferences remain in packaged chart content.