-
Notifications
You must be signed in to change notification settings - Fork 0
docs: add ecosystem ownership map and G-17..G-22 gap entries #1696
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
seonghobae
wants to merge
37
commits into
main
Choose a base branch
from
docs/gap-baseline-ownership-map-and-g17-g22-20260902
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
2009bcb
test(scheduler): close current-main RCA coverage gaps
seonghobae 563a766
test(scheduler): close remaining current-main coverage gaps
seonghobae 8b26dab
fix(agent-mention): stack safe OpenCode slash aliases
seonghobae cd6a986
fix(agent-mention): exclude a trailing query string from the opencode…
claude de821df
fix(agent-mention): give each mention alternative its own boundary, f…
claude f26b3bd
fix(agent-mention): exclude a percent-encoded path or URI scheme sepa…
claude db106d5
test(ci): close main's post-#1546 scheduler coverage regression
claude c46445b
fix(agent-mention): make % and : boundary exclusions direction-specific
claude 6920f07
fix(agent-mention): reject colon- and percent-delimited path segments
claude 85c2469
docs(gap-baseline): record post-#1546 scheduler coverage regression
claude 4b42ce9
fix(agent-mention): widen the trailing colon exclusion to cover a sla…
claude 93a782c
chore(stack): move current mention router onto #1567
seonghobae c9713a8
chore(stack): follow #1567 gap-baseline update
seonghobae 6f40a06
test(ci): document nested REST fixture helpers
seonghobae 0a045f5
chore(stack): follow #1567 docstring repair
seonghobae 10410b9
Merge remote-tracking branch 'origin/main' into fix-opencode-agent-oc…
claude 5534edd
Merge branch 'main' into fix/opencode-agent-oc-alias-stacked-1554
opencode-agent[bot] e851764
Merge branch 'main' into fix/opencode-agent-oc-alias-stacked-1554
claude 6dbf647
Merge branch 'main' into fix/opencode-agent-oc-alias-stacked-1554
claude 680508d
docs: add ecosystem ownership map and G-17..G-22 gap entries
claude fe3d09f
docs: correct the KV-rename orphaning claim after PR review
claude fb2283f
Merge remote-tracking branch 'origin/main' into docs/gap-baseline-own…
claude 528a584
docs: qualify contextual-orchestrator#1017 cross-repo references
claude 44c2117
docs: delink cross-repo refs from inline code, refine ownership map c…
claude bf52be4
merge(main): reconcile gap baseline ownership map with protected main
seonghobae b7cbfc2
docs: fix §2.4 cross-reference and restore noema's shared-runtime role
claude 34d6c1e
merge(main): reconcile PR #1696 after control-plane merges
seonghobae 2ec20ec
docs: add exact-head evidence for G-17 through G-22
seonghobae b281f7e
docs: codify CWL DEVELOPMENT PHILOSOPHY v2026-09-02B
seonghobae d417d57
docs: link G-17..G-22 evidence to governing policy
seonghobae a3be3a9
docs(gap): bind governance evidence to canonical single writer
seonghobae 2cb5952
docs(gap): remove competing governance authority copy
seonghobae 750f02a
docs: flag sandbox-isolation ownership conflict, defer resolution
claude 8bff7e7
docs: make §2.4 ownership-map entries navigable owner/repo links
claude 345d37c
docs(gaps): reconcile canonical baseline with protected main
seonghobae d7355be
docs(gaps): reconcile canonical baseline with protected main
seonghobae 342fac8
docs(gap-baseline): reconcile canonical writer with protected main
seonghobae File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
69 changes: 69 additions & 0 deletions
69
docs/doctoring/gap-baseline-g17-g22-evidence-2026-09-02.md
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,69 @@ | ||
| # G-17 through G-22 evidence and decision trace | ||
|
|
||
| Status: Proposed evidence for PR #1696. This note is not production authority until the PR merges to protected `main`. | ||
|
|
||
| ## Scope | ||
|
|
||
| This doctoring note records the evidence boundary behind G-17 through G-22 in `docs/product-technical-gap-baseline.md`. It distinguishes observed repository state from organization-level engineering requirements so that a later Agent can revalidate each claim without treating an open PR as released authority. | ||
|
|
||
| The canonical repository policy path is [`docs/product-goal-directive.md`](../product-goal-directive.md). `ContextualWisdomLab/.github#1692` is the single-writer Draft that carries **CWL DEVELOPMENT PHILOSOPHY v2026-09-02B** into that canonical path and its doctoring record; until #1692 merges, that revision remains Proposed rather than protected-main production authority. PR #1696 therefore does not create or retain a second governing-policy file. Its G-17 through G-22 evidence remains Draft/Proposed and must not merge ahead of the canonical policy prerequisite if a row depends on v2026-09-02B wording that is not yet present on protected `main`. | ||
|
|
||
| ## Exact-head evidence snapshot | ||
|
|
||
| - `ContextualWisdomLab/.github` protected `main`: `5935c8153722fe6b53bafd579b74f8f097303959` after PR #1715 at this repair pass. Branch protection still requires the established security, coverage, Noema, and OpenCode contexts. | ||
| - Canonical policy single writer: `ContextualWisdomLab/.github#1692`, Draft, exact head `4430864470a4ddb7a8c1692e3ad28708e37d47b5` at this repair pass. It changes `docs/product-goal-directive.md` plus `docs/doctoring/product-goal-directive.md` and explicitly carries the 2026-09-02B revision without introducing a competing authority path. | ||
| - `ContextualWisdomLab/contextual-orchestrator` protected `main`: `8839081659df587b19642be17b9114f9dee8b666`. | ||
| - `ContextualWisdomLab/contextual-orchestrator#1017`: open, not merged, exact head `fe043f4e6db8b24a6ab719fc5801bbbf40e046ae` at the prior owner audit. Its provider-name endpoint repair therefore remains **Proposed** and must not be described as protected-main production authority. The open PR also contains adjacent routing-category/naming work; consumers must wait for an immutable released owner version rather than copying its source. | ||
| - Current protected-main search still finds provider identity used in provider-specific telemetry/discovery code such as `contextual_orchestrator/openrouter_uptime.py`. That is not automatically a routing-policy violation: the violation criterion is provider identity controlling selection, endpoint rewrite, or failover where a declared capability should do so. Each match must be classified at its actual responsibility boundary. | ||
|
devin-ai-integration[bot] marked this conversation as resolved.
|
||
| - Protected-main persistence evidence still includes legacy one-word schema vocabulary in the orchestration persistence fixtures (`seq`, `kind`, `key`, `payload`) and the agent-pool contract still exposes one-word fields such as `priority`/`disabled`. G-22 is therefore a migration/contract gap, not permission to rename storage destructively. | ||
|
devin-ai-integration[bot] marked this conversation as resolved.
|
||
|
|
||
| ## G-18 — model execution timeout versus transport failure | ||
|
|
||
| CWL DEVELOPMENT PHILOSOPHY v2026-09-02B, being integrated through the canonical [`docs/product-goal-directive.md`](../product-goal-directive.md) single-writer PR #1692, requires the default **model execution timeout** across application, Agent, and Gateway to be `null`; a reasoning, streaming, or tool-call operation must not be terminated merely because elapsed model time crossed a generic ceiling. It separately requires provider communication failure to terminate upstream and requires attribution among user cancellation, provider termination, and an explicitly configured administrator timeout. | ||
|
|
||
| These are different failure domains. A `null` model-execution deadline does **not** require retaining a dead socket forever. Transport implementations must observe provider/connection termination and propagate communication failures; connection lifecycle and liveness handling remain transport responsibilities. RFC 9112 explicitly separates HTTP connection failures/timeouts and graceful connection closure from application semantics, and does not require either endpoint to have a fixed persistent-connection timeout. Consequently, the repair criterion is: | ||
|
|
||
| 1. no implicit elapsed-time ceiling for a healthy, progressing model operation; | ||
| 2. provider/network termination or communication failure propagates immediately and releases resources; | ||
| 3. an administrator may configure a model-specific timeout, with get/set/clear/restore, units, priority/inheritance, validation, and audit; | ||
| 4. cancellation cause is observable as user cancel, provider end/failure, or configured administrator timeout; | ||
| 5. clearing an administrator timeout restores the inherited/null model-execution policy rather than inventing a paid or hidden fallback. | ||
|
devin-ai-integration[bot] marked this conversation as resolved.
|
||
|
|
||
| This resolves the apparent contradiction between long-running inference and resource safety without weakening the governing no-elapsed-time-termination rule. | ||
|
|
||
| ## G-19 — p95 <= 20 ms is an internal SLO, not an external universal threshold | ||
|
|
||
| The `p95 <= 20 ms` requirement is an explicit ContextualWisdomLab engineering SLO in CWL DEVELOPMENT PHILOSOPHY v2026-09-02B, whose canonical repository integration is `ContextualWisdomLab/.github#1692` at [`docs/product-goal-directive.md`](../product-goal-directive.md). It is **not** claimed to be a universal HCI standard or a threshold derived from the cited papers. Peer-reviewed latency research instead supports the narrower premise that interaction latency below the traditional 100 ms guideline can still be perceptible and affect interaction; Forch et al. measured approximately 60 ms perception thresholds in a simple mouse task, while Attig et al. reviewed evidence that sub-100 ms latency can matter. | ||
|
|
||
| Accordingly, G-19 requires each UI-owning product to define the measured page/action boundary, workload, sample design, environment, cold/warm-cache policy, and failure denominator, then prove the organization SLO with executed k6/E2E evidence. The current central baseline records the absence of such evidence **in this audit ledger**; it is not an exhaustive proof that no repository anywhere has ever run a latency test. A product that has current executed evidence should link its exact head/run and make the gap row narrower rather than suppress the SLO. | ||
|
|
||
| ## G-20 — i18n topology is a governance requirement | ||
|
|
||
| DB-backed, versioned translation resources; screen-key-scoped fetch/cache; separation of UI translations from ontology labels; and review/approval/deploy/rollback authority are organization architecture requirements in CWL DEVELOPMENT PHILOSOPHY v2026-09-02B, whose canonical repository integration is `ContextualWisdomLab/.github#1692` at [`docs/product-goal-directive.md`](../product-goal-directive.md). They are not presented as a W3C mandate. The gap is that this central baseline currently has no verified canonical-owner release/API evidence for that shared responsibility. Until an owner is verified and released, products preserve the boundary with ports/ACLs/test doubles and do not copy an unreleased owner source tree or download a full browser catalog as a workaround. | ||
|
|
||
| ## G-21 — Rust-first scope is hot-path and risk based | ||
|
|
||
| The Rust-first rule in CWL DEVELOPMENT PHILOSOPHY v2026-09-02B, whose canonical repository integration is `ContextualWisdomLab/.github#1692` at [`docs/product-goal-directive.md`](../product-goal-directive.md), does not authorize a wholesale rewrite of every Python orchestration module. The governing scope is mathematical/psychometric/EDA/data-science core and performance/security-critical runtime, including vector/matrix algebra, token size, CPU multithreading, GPU work, and other measured hot paths. Python remains allowed only for a validated Python-only ML runtime without practical Rust parity, with an ADR that records evidence, bounded scope, and removal conditions. G-21 therefore calls for profiling and boundary identification before migration; an unmeasured `orchestrator.py` rewrite would itself violate the policy. | ||
|
|
||
| ## G-22 — schema migration safety | ||
|
|
||
| The two-semantic-word naming rule in CWL DEVELOPMENT PHILOSOPHY v2026-09-02B, whose canonical repository integration is `ContextualWisdomLab/.github#1692` at [`docs/product-goal-directive.md`](../product-goal-directive.md), applies to organization-owned DB objects and fields, but migration must preserve persisted data and released consumer contracts. Repair therefore requires a RED naming/migration contract first, an item-safe migration or compatibility layer, GREEN owner CI, and only then an immutable owner release and consumer bump. Existing one-word external/released boundary names are translated at the anti-corruption boundary until the owner version changes; they are not silently rewritten in consumers. | ||
|
|
||
| ## Revalidation checklist | ||
|
|
||
| Before merging or later marking any row complete: | ||
|
|
||
| 1. Re-fetch protected `main` for both `.github` and each canonical owner. | ||
| 2. Re-fetch the exact PR head, reviews/threads, required checks, and release/tag evidence; an open PR remains Proposed. | ||
| 3. Verify `ContextualWisdomLab/.github#1692` has merged the 2026-09-02B text into canonical `docs/product-goal-directive.md` before merging any #1696 row whose requirement depends on that revision. | ||
| 4. Re-run the repository/code/API search used by the row and record the exact head/module or remove the claim if it no longer reproduces. | ||
| 5. For G-19/G-20, replace central "evidence not recorded" wording with concrete owner evidence as soon as a current run/release exists. | ||
| 6. For G-18/G-21/G-22, land owner RED -> fix -> integrated GREEN -> immutable release -> consumer version bump; do not copy branch source into consumers. | ||
|
devin-ai-integration[bot] marked this conversation as resolved.
|
||
|
|
||
| ## References | ||
|
|
||
| Attig, C., Rauh, N., Franke, T., & Krems, J. F. (2017). System latency guidelines then and now—Is zero latency really considered necessary? In D. Harris (Ed.), *Engineering psychology and cognitive ergonomics: Cognition and design* (Lecture Notes in Computer Science, Vol. 10276, pp. 3–14). Springer. https://doi.org/10.1007/978-3-319-58475-1_1 | ||
|
|
||
| Fielding, R., Nottingham, M., & Reschke, J. (Eds.). (2022). *HTTP/1.1* (RFC 9112). Internet Engineering Task Force. https://www.rfc-editor.org/rfc/rfc9112.html | ||
|
|
||
| Forch, V., Franke, T., Rauh, N., & Krems, J. F. (2017). Are 100 ms fast enough? Characterizing latency perception thresholds in mouse-based interaction. In D. Harris (Ed.), *Engineering psychology and cognitive ergonomics: Cognition and design* (Lecture Notes in Computer Science, Vol. 10276, pp. 45–56). Springer. https://doi.org/10.1007/978-3-319-58475-1_4 | ||
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.