Repository navigation
Consolidate skills lifecycle, routing and SDK 0.160.1 qualification - #795
Draft
seathatflowsinourveins wants to merge 9 commits into
Draft
seathatflowsinourveins wants to merge 9 commits into
seathatflowsinourveins wants to merge 9 commits into
Conversation
|
Dependency limit exceeded — report not shown. This pull request scan exceeded the 10,000-dependency limit applied to this scan, so the results are incomplete and may be inaccurate. To avoid reporting false positives, Socket has not posted a report. Upgrade your plan to raise the dependency limit and get complete reports, or view the partial scan in the dashboard. Socket is always free for open source. If this is a non-commercial open source project, contact us to request a free Team account. |
seathatflowsinourveins
force-pushed
the
codex/ns2604-skills-r2-workflow-20261006
branch
from
October 7, 2026 01:27
7a29550 to
0618864
Compare
This was referenced Oct 7, 2026
5 tasks done
seathatflowsinourveins
added a commit
that referenced
this pull request
Oct 7, 2026
### Scope Route catalog lookup through QMD's lexical queries and bounded document retrieval; use SocratiCode for semantic catalog search at the main checkout's projectPath. Keep reranking available and prohibit qmd embed/pull. Update current maintenance recipes and native instruction carriers through the repository renderers. Existing Serena consumers read its manual before navigation, and jCodeMunch consumers obtain the current guide before a typed route with the actual caller model and execution off. Four existing Claude roles gain only Serena's manual tool. Reviewer/builder keep their no-menu boundary, and a focused read of a known path and line remains valid. This landing repair appends the preregistration's Amendment4 to bind the deliberately changed role bodies and compacts lane-authored RTK explanations within the unchanged compact and startup budgets. The entire composed pre-RTK prefix, upstream awareness, owner's blocks, exact SKILL line, seven exceptions, models and effort are preserved. Prior amendment rows, observations, seals and RUNBOOK bytes remain unchanged. The canonical code-graph rename is prepared as a separate patch and is excluded from this head under command-center ruling081016Z. Main's existing names remain in both templates, the map, fixtures and carriers. The protected lane token requires the owner's word before the rename can land. Until that follow-up, this host's Claude code-graph access is through the vendor's hook channel only; the stale MCP carrier id is a recorded limitation and is not claimed fixed. - Base commit: `c44993b379da25fae923dbbe70188978af5104aa` (verified native source; #803 already landed). - Lane: `lane:foundation`; draft. - Head: `9985e468fa4997d22630714b2fa6a76666e3cda9`. Native bytes: compact 8076 <8192, rendered 9142, three-file startup 20101 <=20103; full Claude block 4087 <=4100. These are byte gates, not token-use measurements. - Owned paths: token-lane carriers and their paired handbook text, minimal manual grants and mirrors, Codex template/rendered copies, native loader/routing/sequence tests, current recipes and generated handbook, append-only decisions/preregistration amendment, follow-up receipts and checksum/registry projections. - ROLE-CARRIER-GAPS and ROLE-GRANTS are separate future work and stay out of this change. Landing path: explicit command-center cue070358Z and fallback ruling081016Z permit one pushed rebase/content repair onto then-current main while #802/#813/#830 land. The co-op performs the new-head delta read with its CI result; the command center ACKs; 5f lands. A further pushed rebase requires the separate dated landing-conflict cue. Host configuration, cutover and fresh-session/working-day acceptance belong to their owners. ## SOTA sources - [QMD v2.8.3 lexical query](https://github.com/tobi/qmd/blob/v2.8.3/src/mcp/server.ts#L294), [typed searches/rerank](https://github.com/tobi/qmd/blob/v2.8.3/src/mcp/server.ts#L321) and [document retrieval](https://github.com/tobi/qmd/blob/v2.8.3/src/mcp/server.ts#L412): native lexical subqueries, retrieval and optional rerank. Rerank defaults true (:334-335). [README maintenance syntax:1033-1037](https://github.com/tobi/qmd/blob/v2.8.3/README.md#L1033-L1037) supplies bounded commands. - [SocratiCode v1.15.0 codebase_search](https://github.com/giancarloerra/SocratiCode/blob/v1.15.0/src/index.ts#L138): semantic search at an absolute projectPath. Owner ruling224125Z selects the routing and preserves rerank-on acceptance; this PR performs no host cutover. - [Serena c6fbd1c5 manual order](https://github.com/oraios/serena/blob/c6fbd1c5932df2494ffa0020af5a9fbe80b82143/src/serena/resources/config/prompt_templates/system_prompt.yml#L5-L6), [manual tool:28-40](https://github.com/oraios/serena/blob/c6fbd1c5932df2494ffa0020af5a9fbe80b82143/src/serena/tools/workflow_tools.py#L28-L40) and [project/session binding:44-49](https://github.com/oraios/serena/blob/c6fbd1c5932df2494ffa0020af5a9fbe80b82143/src/serena/tools/config_tools.py#L44-L49): manual first, active cwd project and returned session id for switches. - [jCodeMunch 1.108.330, 28803366, typed route:446-463](https://github.com/jgravelle/jcodemunch-mcp/blob/288033668f0425ab0547f420dd647c14bd186c7f/src/jcodemunch_mcp/server.py#L446-L463), [guide:4743-4752](https://github.com/jgravelle/jcodemunch-mcp/blob/288033668f0425ab0547f420dd647c14bd186c7f/src/jcodemunch_mcp/server.py#L4743-L4752) and [policy:101-128](https://github.com/jgravelle/jcodemunch-mcp/blob/288033668f0425ab0547f420dd647c14bd186c7f/src/jcodemunch_mcp/cli/policy.py#L101-L128): guide-first and typed task/model route. These files are unchanged at target1.108.331/d94049d0. The executable schema corrects the guide's query example; no new adaptive-tier setting or upstream rebuild. - `native-agent-stack@c44993b:tests/test_token_e2e_preregistration.py:637-663,905-909`: later dated role-pin amendment and sealed RUNBOOK contracts. Amendment4 preserves every earlier row/seal and extends the current-row selector. The command center records merge chronology at landing; this is a structural repair and authorizes no run. - [DeusData/codebase-memory-mcp v0.11.0 release](https://github.com/DeusData/codebase-memory-mcp/releases/tag/v0.11.0), the repository's pinned component, and the command center's native-client read-back identify the canonical server name. Its proposed client-key/wire-id/owner-token correction is kept in the separate patch, preserving every payload and strict fixture. Ruling081016Z explicitly keeps main's names in this publication while the owner's instruction-token decision remains open. - RTK explanation provenance: `native-agent-stack@2d849ba` (#726) and `@50b9579` (#389), `adoption/templates/codex.AGENTS.template.md` after its exceptions marker. The eight explanations shorten while all facts and protected bytes remain. Command-center064832Z accepted their class and scoped wording. - Native helpers at `native-agent-stack@c44993b`: `tools/adoption/managed_block.py:169` (`codex_block`), `tools/adoption/codex_roles.py:287` (`f4_block`), `scripts/host_receipts.py:710` (`register_file`), `docs/lanes.md:94-128`. Current generated inventory/handbook and checksum bindings use those repository tools; historical receipts and the frozen catalog passage are preserved. ### Evidence-class table | Claim | Evidence class | Command / receipt | | --- | --- | --- | | Documented native routing/manual/guide interfaces | source_review | Pinned upstream files above | | Carrier rendering, protected bytes, new body pins and tool ids | local_integration | Existing native helpers, hashes and required modules | | Permission/sequence/byte-mutant/frozen-row controls | synthetic | Existing tests with assertions preserved | | Deployed use and token savings | Pending owner evidence | No model, host apply or working-day measurement in this repair | ### Local commands run ```text $ CI=true nice -n 19 ionice -c3 python3 -B -m unittest tests.test_token_lanes_session_start tests.test_token_lanes_subagent_start tests.test_scaffold_repo tests.test_new_wsl_handbook tests.test_adoption_docs_consistency tests.test_codex_agents tests.test_codex_roles tests.test_codex_worker_lane tests.test_token_e2e_preregistration tests.test_new_wsl_client_config.RecordTests tests.test_install_claude_profile tests.test_managed_block tests.test_task_model_routing tests.test_new_wsl_client_config.MapTests tests.test_new_wsl_client_config.AgentGapTests tests.test_new_wsl_client_config.RenderTests tests.test_new_wsl_client_config.ApplyTests.test_the_first_run_writes_what_the_wired_pieces_name_and_nothing_else exit 0; Ran 595 tests in 109.929s; OK (skipped=14) $ CI=true nice -n 19 ionice -c3 python3 -B -m unittest tests.test_install_claude_profile tests.test_adoption_docs_consistency tests.test_new_wsl_client_config.RecordTests exit 0; Ran 150 tests in 16.306s; OK (skipped=2) $ CI=true nice -n 19 ionice -c3 node examples/claude-native/workflows/test-envelope.mjs exit 0; SUMMARY passed=254 failed=0 total=254 $ CI=true nice -n 19 ionice -c3 python3 -B tools/adoption/new_wsl_client_config.py --check --markdown exit 0 $ CI=true nice -n 19 ionice -c3 python3 -B scripts/build_new_wsl_handbook.py --check exit 0 ``` The passing 595-test run is retained on unchanged source/assertion inputs from the pre-window checkpoint; the 150-test run checks the concrete relocation-fixture and workflows-README changes in the new main base. Their counts overlap and are not added. The fresh Node run passes 254/254. Strict checksums in both Codex directories and the Claude hook directory, plus `git diff --check`, exit 0. All three Amendment-4 role digests were re-derived in both copies and are unchanged from the previous published head. ```text $ CI=true nice -n 19 ionice -c3 python3 -B scripts/validate.py exit 0; 69 components, 10,482 hashed files, 4 profiles, 224 receipts, status passed ``` The earlier landing composition used source `630b6ece8485a7710ea51321682d5a12e364e25c` and hot commit `5703ef1061b982078038425fada888fd35153868`; its native checks and source-critic result are retained. The authorized locator follow-up changes only `catalogs/foundation/upstream-surface-dispositions.json` and that file's registry row: the `otel.environment` citation now points to the command at `examples/codex-native/README.md:214`. Source commit `2b36cfee` is followed by registry-last commit `9985e468fa4997d22630714b2fa6a76666e3cda9`, with no further rebase. Claude/Codex instruction bytes, role hashes, historical records, namespace and byte budgets remain unchanged. The registry starts from exact main `c44993b379da25fae923dbbe70188978af5104aa`, registers the same 67 changed-file rows through the native producer, and preserves main's receipt and convergence arrays. The two-file correction passed the 114-test citation module, the two exact failing methods and a second native validator run. The worktree is clean; the shifted-citation scan found no other current target to repair. Historical before/after references remain intact. ```text $ CI=true nice -n 19 ionice -c3 python3 -B -m unittest tests.test_upstream_surface_watch exit 0; 114 tests, 6 skipped $ CI=true nice -n 19 ionice -c3 python3 -B -m unittest tests.test_upstream_surface_watch.DispositionCitationTests.test_every_cited_line_names_the_key tests.test_upstream_surface_watch.DispositionCitationTests.test_a_dotted_key_has_its_parent_near_the_citation exit 0; 2 tests $ CI=true nice -n 19 ionice -c3 python3 -B scripts/validate.py exit 0; integrity and scope passed ``` The hosted run at5703 returned two failures in11,495 tests because the documentation locator moved. A reviewed locator still gates when its repository test fails. This follow-up closes that exact contract; it adds no assertion waiver or namespace change. Read-only overlap metadata checked all 47 open PRs, including full file pagination where needed. Source overlaps: #645, #706, #709, #754, #769, #770, #775, #776, #795, #810, #821, #826, #829. Shared registry/checksum paths use the hot-file protocol; no peer branch is changed. Earlier failed CI at3ba ran11,381tests and failed six role-body-pin subcases; its complete failed log is retained privately. The failed 560-test and earlier 595-test preparation runs also remain retained. This repair uses a dated amendment, preserves the older source evidence and strict current-body comparison, and performs no full-suite or provider acceptance run. Validator results are integrity and scope evidence only. ### Decision record `docs/decisions/2026-10-06-qmd-lexical-catalog-instructions.md` and `docs/decisions/2026-10-07-serena-jcodemunch-native-navigation-wiring.md`, with appended clarifications and the new landing follow-up, explain the behavior and limits. The preregistration README gains Amendment4 only; its RUNBOOK and earlier sealed records stay byte-identical. No prior result is recast as a new run. ### Host evidence Repository-only change. No live instruction file, MCP registration, hook trust, client setting, gateway setting/key or model session changed. Source/render checks and synthetic fixtures remain distinct from the configuration owner's read-back and organic-use acceptance. ### Checklist - [x] No GitHub Actions or paid service change. - [x] No credentials, raw conversations or live client configuration committed. - [x] Models, effort, owner's blocks and historical observations preserved. - [x] Peer-owned worktrees and source preserved. - [x] Native validation passed; changed checksums/registry committed last at the final head.
This branch has not been deployed
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
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.
Scope
Consolidate the skills lifecycle and round-two routing work into this draft. The central lifecycle gains native-stack-research and the vendor-generated hf-cli record; workers inherit central held/pruned decisions and explicitly exclude unsupported generated payloads. The routing manifest, renderer/validator, preload variants, foundation path rule, recorder and routing-hook sources preserve unresolved delivery gates. The worker keeps its exact SDK/runtime guard while qualifying 0.160.1.
d9eb68750311d5f888adec140723da174f9fe10c.lane:foundation.Landing path
Co-op delta GPT read of the frozen consolidated head, followed by command-center Claude adjudication and ACK, then 5f's assigned foundation queue and landing cue. This is the command-center-approved retained PR under FOLDS-RULED review023012Z. The command center and 5f assign its queue position; this lane never merges or closes the originals. No DIRTY-only rebase or new draft.
SOTA sources
native-agent-stack@ecfa1127:tests/test_install_claude_profile.py:1289-1346. Held grants remain plan inputs..github/workflows/sdk.yml; the worker's exact initialize guard remains.native-agent-stack@d9eb68750311d5f888adec140723da174f9fe10c:tools/sota-convergence/landscape-sweep/README.md; hot-file registration: docs/lanes.md at the frozen base.Fold comparisons and overlaps
Frozen source comparisons are comparison.json; exact unions are overlaps.md. Non-hot source files are #719 193/203, #792 4/11, #793 7/7, #794 20/22 and #817 3/5 identical; no original file is missing. Registry and ledger are rebuilt natively rather than merged.
Overlaps preserve the research and HF objects; the existing Claude template guards and research override; Codex eligibility prose with lifecycle/research identities and operations retained in the manifest and installer; central held-state inheritance with record/retire; all correction rows plus the three lifecycle rows; the catalog's new tasks/sources with held browser state; the validation requirements lock, the routing fixture's dependency-free manifest exemption and both coverage test hunks. Current catalog/schema tests align the approved 17-task input. All 134 selected worker rows remain; HF has a source-bound unsupported-installer exclusion. Pending reasons now distinguish present central stores from missing Tier B recording and native consumers, without activating routes. The SDK upgrade is a separately qualified overlay.
Fold statement
At
f809570b2175908e14ab6b88e5dbe394eab220bb, the absent original added lines are accounted for below. No skill/source entry, historical observation or incoming registry registration was dropped. Literal line retention differs from JSON value equality; obsolete totals and assertions are explicitly superseded.25 catalog-visible, 9,318 characters and budget charging) and Record skill provenance and native catalog limits #792's three lines (25 eligible from 26) become lines 37–42: 26 catalog-eligible from 27, held-browser/Claude-only-Creator qualifications, and explicit fresh-visibility uncertainty. UTF-8 names/descriptions/paths/newline charging remainsceil(bytes / 4);max_context_tokens = 6000is unchanged. A12's ClaudeskillListingBudgetFraction: 0.05property lives in claude.settings.template.json:399, and its directive sentence remains in the manifest'sclient_budgets; neither was dropped or moved into Codex configuration.Nonewhen absent (lines 1317/1321); no fake 6,000-token host observation replaces them.adoption/skills/skills-lifecycle.jsonis absent; this is the actual catalog. Fix skills lifecycle gates and retain skills qualification evidence #719's four absent literal lines are multiline expansions of the identical native-agent-stack source object (line 100), installed research array (194), eight source IDs (199) and research-gap string (209). Reconcile held skill inventory and plan native discovery #817's absent literal line is the identical held-browser gap string expanded at line 469. Each corresponding parsed JSON value equals its frozen source; the 1/5 and 3/4 literal-line counts reflect formatting.pin_policy/install/update/previewlines are expanded JSON with identical values. Fix skills lifecycle gates and retain skills qualification evidence #719's sums 9,637/9,318/9,318 and Record skill provenance and native catalog limits #792's 10,492/10,173/10,173 are superseded by the union's 10,645/10,326/10,326. Fix skills lifecycle gates and retain skills qualification evidence #719's olderclient_budgetssentence is retained with corrected 26-entry eligibility prose and the dated A26 amendment. Both policy mirrors become 10,645 by A26; native 0.05/6,000 settings and pinned descriptions remain unchanged.host_receipts.register_file. All 204/11/7/22/5 incoming registrations from Fix skills lifecycle gates and retain skills qualification evidence #719/Record skill provenance and native catalog limits #792/Record skill-state transitions with safe descriptors and profile views #793/Prepare held skill routing with native filters and frozen B13 gates #794/Reconcile held skill inventory and plan native discovery #817 respectively are present; 10/7/0/2/2 require fresh hashes/bytes for merged files. Every registered hash matches the retained file. This accounts for the 955/975, 17/31, 100/104 and 21/25 literal-line retention counts; no receipt or convergence-record addition was dropped, and the registry was not hand-merged.Evidence-class table
Local commands run
Initial combined failures, the unsuccessful first worker repair, the three pre-amendment budget-test failures and the missing-SDK default-Python inspect attempt remain recorded separately. The inspect repair used the already-qualified isolated interpreter without a global install or source change. A smoke is a smoke; catalog availability is not rendered description visibility, organic use or provider acceptance.
Decision record
The dated S1 decision remains unchanged. Its new acceptance amendment is the current carrier: upstream choice, lifecycle install, one real-use smoke per consuming client and native organic counters. BEFORE stays UNKNOWN with three historical-binding gaps and the fixed source-budget stop; no retrospective inference or larger-budget rerun. AFTER is pending command-center deployment and seven days of native measurement, with forward definition/carrier/lifecycle hashes captured through existing hooks.
The October 7 amendment records A26's exact metadata ceiling of 10,645 for the ruled union. Both existing policy mirrors carry it, and tests bind Claude 10,645 and Codex 10,326 so further unreviewed growth fails. The native 0.05 fraction and 6,000-token setting and every pinned description stay unchanged. Approximately 40,000 listing characters is the older decision's inference, not a fresh visibility measurement; the separate 15,000 proposal is still unapproved.
J795 review corrections
At
37d60bc205e3485c6fbdbac6808633a2634cdd3d, P2 and P3 are fixed in one subsequent repair push, with the registry committed last.skills-r2-private/sdk-01601-qualification-20261007/native-01601-readback.jsonl, digestb4ea1b8b55fc28792b1dd043ff0f449c1449edf159a272601923270f29259fb1; the final 25-name folded observation isskills-r2-private/sdk-01601-qualification-20261007/final-native-01601-readback.jsonl, digest561d9ef562b4308dc097cb386508afc8afe407184df6b0ae7b6e75267588fa4c. Their public observations and private raw bytes remain unchanged; raw contents are not published.PYTHONDONTWRITEBYTECODE=1 nice -n 19 ionice -c3 python3 -B scripts/validate.pyexited 0 (69 components, 10,643 hashed files, 4 profiles, 223 receipts). A bounded read-only review confirms the locators/digests and unchanged observation bytes. No source behavior changed and no model/test campaign or host action was repeated. Remote required CI remains pending the single landing-turn rebase; local validation is its own evidence class.Host evidence
No platform-status change, client registration, host configuration or service start. Native observations retain their exact operation/input/runtime scopes; local fixtures are not upstream or ecosystem acceptance.
Checklist