Skip to content

Record role tool sets and MCP memory gates - #910

Merged
seathatflowsinourveins merged 4 commits into
mainfrom
codex/ns2604-roles-table-20261009
Oct 9, 2026
Merged

seathatflowsinourveins merged 4 commits into
mainfrom
codex/ns2604-roles-table-20261009

Conversation

@seathatflowsinourveins

@seathatflowsinourveins seathatflowsinourveins commented Oct 9, 2026 •

Copy link
Copy Markdown
Collaborator

Scope

Role sets now cover their observed necessary servers with client-specific selection. Exec readers require EVALUATION, the narrowest existing contract covering all nine observed servers; conflicts with live lane exclusions go to the CC and intentional exclusions require a deciding file and decision. The record separates historical trial sets from live settings and aligns every named catalog disposition with its actual verdict row.

  • Base commit: 72e1f943d951e6aff4200729db88121a8d38d7da.
  • Previous reviewed head: 897b87229c2d59f93244ae6e4b9038de4a739469.
  • FIX2 head: 779fa242b4b3dd9c25f53295de49b3d0e5b2ab7d; one additional repair commit and one push, no rebase.
  • Lane: lane:foundation.
  • PR paths: docs/role-tool-sets.md, docs/role-tool-sets.json, docs/convergence-architecture.md, adoption/README.md, and documentation metadata in manifests/evidence.json. The final FIX2 delta updates only the two role records; earlier architecture/adoption links and manifest integrity remain part of the five-file PR.
  • Ten role rows, seven historical process contracts, seventeen tool records, thirteen pending/source-context slots.

The live Codex exclusions are bound per named lane to role-mcp-off.json v2 SHA256 77c91cdc0433f54ecacb070a4581ec7f20443d9d5835e7fb938f3994e6563107. CORE is explicitly the historical trial base; all four trial arms used role_mcp_off=skip. Claude's twelve-component strict selection contract needs grafana and agentsview additions for the CC/co-op; all observed necessary servers remain covered by role-specific contracts. Unknown coverage cannot pass a server reduction or shared-MCP topology decision.

Stage 1 uses valid array-index catalog verdict pointers: Context Mode and jCodeMunch are measured trade-offs, Headroom and GitHub CLI candidate:cli-cli are winners for their respective jobs, CBM and agentsview are conditional, and Hindsight is unqualified. Semble/browser links are layer context only. The coordination composition is pending and owned by orch-records. Hcom's tree and installed pins are distinguished, and grafana-observation pins the MCP connector. The architecture paragraph is appended to restore the test-enforced citation at line 134. Optional QMD cohort numbers are omitted because original observer files are not bound to this numerical packet. Each historical arm separately cites a hashed release.out:2 record for role_mcp_off=skip. The agentsview table is contiguous, api-actions has review-specific acceptance, and the Claude schema source identifies the 1000-byte span, start and term offsets. The profile readback came from the identity-launcher binary reporting Codex 0.161.0; launch metadata and native execution stay separately scoped.

SOTA sources

Evidence-class table

Claim Evidence class Retained source / result
Four historical lane PSS/process observations Retained native observation ROLE-MCP-TRIAL-REPORT-20261009.md SHA256 f0afc2c734b3ad1b70e68cd38d3c844c7f4b1338cc030c8fc40fc88db1e53989; after-task hashes and trial skip setting in JSON
Observed per-role required server coverage Retained native counter record plus reviewed selection contract Immutable adoption-now-e129759fe91ffbb1.json SHA256 e129759fe91ffbb133de2719a7d5194fe96ac65c44b222adf7fdd4d65af0e872; original plugin names preserved
Live lane exclusions, client semantics, catalog verdicts and pins Source review Hash-bound live source, native help/schema and pinned upstream source locators
Repository integrity and existing integration modules Local integration validate.py rc 0; test_upstream_surface_watch 108/114, test_adoption_docs_consistency 38/39, test_install_claude_profile 108/109, test_evidence_manifest 5/5 passed; eight skips

Each test module ran under nice/ionice with timeout 600. Counts above exclude skips from passed totals. Complete returned stdout/stderr and return codes are retained in the private verification record. These are existing repository checks, not vendor acceptance.

Per-tool PSS, fresh concurrent sharing, effective Claude registration readback, generic role/phase mappings and role 99's native client remain unmeasured. No runtime configuration, model route, provider trial or deployment changed. FIX2 incorporates the CC micro's 2 P2/7 P3 and the designated GPT micro's same table P3. The read-only completeness critic reports PASS WITH BOUNDARIES. Both designated exact-head reads, CI, required pre-cue tooling, the CC cue and owner decisions remain required for landing.

@seathatflowsinourveins seathatflowsinourveins added the lane:foundation Foundation lane: Claude/Codex setup, hosts, memory, RAG, research, workers label Oct 9, 2026
@seathatflowsinourveins
seathatflowsinourveins marked this pull request as ready for review October 9, 2026 05:23
@seathatflowsinourveins
seathatflowsinourveins merged commit 52ed488 into main Oct 9, 2026
38 of 49 checks passed
@seathatflowsinourveins
seathatflowsinourveins deleted the codex/ns2604-roles-table-20261009 branch October 9, 2026 05:23
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-09T05:27:07.159908Z 779fa24 Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lane:foundation Foundation lane: Claude/Codex setup, hosts, memory, RAG, research, workers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant