Skip to content

docs: route learning capture through editable source owners - #808

Merged
ss-o merged 1 commit into
mainfrom
feature-806-learning-capture-routing
Oct 16, 2026
Merged

ss-o merged 1 commit into
mainfrom
feature-806-learning-capture-routing

Conversation

@ss-o

@ss-o ss-o commented Oct 16, 2026

Copy link
Copy Markdown
Member

Summary

Refs #806. Correct the learning-capture destination matrix to name editable knowledge-domain sources instead of generated consumers. Regenerate the maintained runbook from knowledge/domains/governance/learning-capture.md.

flowchart LR
    A[Verified learning candidate] --> B[Find declared editable owner]
    B --> C[Update source within authority]
    C --> D[Regenerate and verify consumers]
Loading

Distinguish provisional local findings, locally applied corrections and published promotion. Require a useful candidate's disposition in existing evidence without manufacturing extra artifacts for No durable learning. Preserve project-native ownership until approved delivery is complete and treat missing pilot records as evidence gaps, not zero outcomes.

Verification

  • Knowledge delivery: all 57 generated consumers current.
  • Knowledge coverage: all 366 files accounted for.
  • All 21 knowledge tests pass; the earlier full policy and routing verification passed 176 tests.
  • Matrix source owners match delivery records; local Markdown links resolve. The earlier pinned Markdown Trunk check passed both changed files.
  • The diff changes only the editable learning-capture source and its generated runbook. No policy pins, runtime settings or instruction manifests change.

Agent handoff

The destination and disposition correction is ready for current-head hosted checks and review. Keep #806 open to retain the outstanding retrospective pilot evaluation: that evaluation was not performed here, and no rollout-wide or cross-runtime adoption claim is made.

@ss-o
ss-o requested a review from a team as a code owner October 16, 2026 23:01
@ss-o

ss-o commented Oct 16, 2026

Copy link
Copy Markdown
Member Author

Requested copilot-pull-request-reviewer[bot] once through the REST requested-reviewers endpoint at 2026-10-16T23:02:53Z on head e2fcafe65b583c117968d920834ac36f5c6cdff6, after all 12 current-head checks passed. The response had no requested reviewers; the existing tsc team request predates this attempt and is not a Copilot registration. No Copilot timeline event, check run or review is present. Review request not registered; cause unconfirmed. Awaiting the maintainer’s ADR-0026 fallback election before posting a review of record or merging.

@ss-o ss-o left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fallback review under ADR-0026: Copilot request not registered on e2fcafe

Findings

No actionable findings. Reviewed the complete editable-source and generated-runbook diff against f4ce1685 and checked its owner links against the delivery records.

Note

This is a maintainer-elected self-review, not an independent review or approval to merge.

Checklist and validation

  • Source ownership and generation: PASS. The matrix names the declared organization-policy and implementation-pattern sources, routes generated destinations through delivery records, and preserves project-native ownership until approved delivery. All 57 generated consumers are current.
  • Completion and evidence semantics: PASS. Provisional capture is not presented as promotion; useful candidates receive a disposition in existing evidence; silent No durable learning remains valid without a new artifact. Missing pilot records remain unknown outcomes, not zero counts.
  • Links and coverage: PASS. Local Markdown targets resolve and source owners agree with delivery records. All 366 repository files are accounted for. Only the source and its generated runbook change.
  • Deterministic checks: PASS. All 21 knowledge tests pass locally, with earlier 176 policy/routing tests and pinned Markdown lint evidence retained. All 12 current-head hosted checks pass, including agent-instruction validation, routing, links, policy and CodeQL. Instruction evidence, routing evidence.
  • Scope and privacy: PASS. No runtime settings, policy pins or manifests change; public guidance does not introduce private environment details.

Limits and follow-up

The single configured Copilot request did not register; request evidence distinguishes the existing team request from a Copilot registration. Cause is unconfirmed. The maintainer explicitly elected this fallback for the current head. No unresolved review threads exist. The retrospective pilot evaluation was not performed; #806 remains open for that work. This does not prove cross-runtime adoption. GitHub Markdown rendering is checked; narrow-screen visual rendering is unverified.

@ss-o
ss-o merged commit 1be1470 into main Oct 16, 2026
12 checks passed
@ss-o
ss-o deleted the feature-806-learning-capture-routing branch October 16, 2026 23:13
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