fix(semantic): complete production activation and idle reuse - #748
Conversation
# Conflicts: # crates/tracedecay-domain/src/configuration.rs
…plan-reopened' into codex/707-qualification-latest # Conflicts: # crates/tracedecay-global-db/src/configuration/semantic.rs
…plan-reopened' into codex/707-qualification-latest
…plan-reopened' into codex/707-qualification-latest
…plan-reopened' into codex/707-qualification-latest
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e496d16dcd
ℹ️ 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Performance Comparison
|
…plan-reopened' into codex/707-qualification-latest # Conflicts: # crates/tracedecay-usecases/src/semantic_runtime/production.rs # crates/tracedecay-usecases/src/semantic_runtime/production/application_status.rs # crates/tracedecay-usecases/src/semantic_runtime/redundancy.rs
…plan-reopened' into codex/707-qualification-latest
…ification-latest # Conflicts: # crates/tracedecay/src/daemon/project_open_owners/query_authority_upgrade.rs
…plan-reopened' into codex/707-qualification-latest # Conflicts: # crates/tracedecay-cli/src/commands/profile_storage.rs # crates/tracedecay-maintenance/src/retention/orphan_stores/tests.rs # crates/tracedecay-maintenance/src/retention/storage_report.rs # crates/tracedecay/src/config.rs # crates/tracedecay/src/daemon/profile_identity.rs
Resolves semantic_runtime conflicts by keeping the landed #707 status mechanism (application_status_from_projection + lifecycle overlay + DaemonSemanticRuntimeBackendV1::from_production) and porting #748 test contracts; keeps #748 linked-activation redundancy authority and drops its now-caller-less project_committed_semantic_activation accessor.
The #748 author landed their own fixes for both remaining review findings while this integration was in flight. Adjudication keeps their implementations — the revision-bound committed-fallback core mount (stricter CAS than the interim fence relaxation here, which is reverted) and the closure-scoped activation-receipt snapshot — and retains the end-to-end regression tests added here, adapted to those shapes.
7cca98d
into
codex/tracedecay-total-redesign-plan-reopened
|
This PR's content is fully merged into #707's branch All three Codex review findings are resolved on the integration branch:
Conflict adjudication kept the landed #707 mechanisms (3-arg Closing as fully absorbed into #707. |
Integrates the completed semantic qualification fixes into #707.
Behavior:
Production evidence:
No threshold, timeout, memory limit, or admission budget was raised.