feat(observability): mount unmounted emitters and generation-bound joins - #515
Merged
ScriptedAlchemy merged 11 commits intoAug 19, 2026
Conversation
The advisory GitHub review refresh owner now offers every coordinator observation to the canonical bounded Observatory producer as one capability receipt plus one receipt per exact drift interval (record_github_stack_capability / record_github_stack_drifts), through a GitHubStackObservabilityV1 lane composed at project open from the exact probe owner, project producer, and project observation database. Telemetry refusal is logged and never alters the refresh product path. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
The code-index scheduler registry carries a per-worktree Plan 26 observability lane (project session database plus bounded producer), installed by mount_code_index after the project-open delivery mount. The reconcile worker records each terminal pass through the canonical record_index emitter beside its cadence receipt, and query search emits the retrieval-pipeline families projected by observe_composition through emit_retrieval_pipeline on the bounded producer. An uninstalled lane records nothing; telemetry refusal never changes the product path. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
The native-integration invocation now records one mechanical conflict prediction per completed preflight tree merge and one independently adjudicated linked outcome per terminal apply receipt, through a new record_work_conflict_observation producer on the bounded Observatory outbox. Prediction and outcome share a deterministic prediction_ref derived from the exact preview identity; reads, refusals, already-integrated previews, and unevaluated preflights emit nothing, and telemetry refusal never changes the owner's decided result. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
Both live Work attempt timeout arms (stdio provider and app-server) now offer one operation.no_progress.terminal.v1 receipt through the bounded producer when the armed wall budget elapses and the provider is killed: the run-deadline reference is derived from the exact attempt identity, the concurrency-policy revision is the registration-pinned work topology digest, the frontier is the provably uncommitted zero, and the stall is the owner-measured elapsed wall. A zero armed budget or unmeasured stall is refused; emission never alters the timed-out product handling. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
Project open now records one adoption-eligibility census per composed capability family (eligible/enabled/available derived from the exact catalog composition stages, Known coverage) through the project-bound observation authority, fire-and-forget. Hint-outcome settlement records each pass's settled hints as one linked adoption-outcome funnel: invoked/terminal = idempotently settled hints, independently_useful = behaviorally verified acted hints only, with census coverage weakened to Partial while unresolved hints remain. Families without a composed capability and passes that settled nothing emit nothing. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
|
Compress refusal logging to single typed-outcome logs, trim narration from module and function docs, and keep only invariant comments across the A5-A9 emitter mounts. No behavior change; the generation-bound joins stay unmounted seams because no Work/code-index join is advertised. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
ScriptedAlchemy
marked this pull request as ready for review
August 19, 2026 16:11
ScriptedAlchemy
merged commit Aug 19, 2026
ac471d7
into
codex/tracedecay-total-redesign-plan
1 check passed
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
2 participants
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
Mounts the audit's unmounted observability emitters (A5–A9) and generation-bound joins (A4, in progress) with real production callers on the daemon/application path. No version bumps, no live-profile dogfood, no new telemetry DB — every emitter writes through the existing project-bound
RegisteredGlobalDb/ boundedBoundedObservabilityProducerV1authorities.Mounted production callers
observability::record_index,emit_retrieval_pipelinesrc/daemon/code_index_scheduler/registry.rs);execute_query_search_on_latestemitsobserve_compositionfamilies per real query (query_runtime.rs), via a per-worktreeCodeIndexObservabilityV1lane installed bymount_code_indexdaemon::code_index_scheduler::tests::installed_observability_lane_records_index_and_retrieval_observationsWorkConflictPredictionObservedV1,WorkConflictOutcomeLinkedV1record_work_conflict_observationbesiderecord_native_integration_transitioninsrc/daemon/service/invocation/native_integration.rs: preflight tree-merge disposition → mechanical prediction; terminal apply receipt → NativeGit-adjudicated linked outcome sharing a deterministicprediction_refobservability::work_conflict_emit::tests(6, incl. persisted linked pair)record_adoption_eligibility,record_adoption_outcomesrc/daemon/adoption_observation.rsfromproject_open_owners.rs); hint-outcome settlement records the linked funnel (src/hint_outcomes.rs, invoked from hook transcript ingest)daemon::adoption_observation::tests(3),hint_outcomes::tests(3 new)NoProgressObservedV1src/daemon/service/invocation/work_attempt_exec.rs) offer the no-progress terminal at the kill, with registration-pinned topology-policy digest and owner-measured stallobservability::no_progress_emit::tests(4) + end-to-enda_wall_exhausted_provider_seals_timed_out_and_emits_the_no_progress_terminalrecord_github_stack_drifts(+record_github_stack_capability)GitHubReviewRuntimeOwnerV1::refreshoffers every coordinator observation through aGitHubStackObservabilityV1lane composed at project open (advisory/github_runtime/owner.rs,project_open_owners/advisory_runtime.rs)github_stack_drift_observability::review_owner_observability_lane_records_capability_and_drift_receiptsGitReadAuthorityV1::join_generation,GenerationDiagnosticJoinV1::joinDisciplines
Partial/Unknown) wherever a census or adjudication is provably incomplete; unadjudicated dispositions, empty families, and unevaluated preflights deliberately emit nothing.