feat: TraceDecay V2 delivery (PR8–PR13) + daemon performance, memory, and reliability overhaul - #707
Conversation
🦋 Changeset detectedLatest commit: 35c241a The changes in this PR will be included in the next version bump. Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
Post-reopen correction pushed in a23f86a: source-neutral background reconcile no longer advances the scheduler epoch and cancel in-flight text activation; mounted hook overflow uses the explicit invalidating path. Evidence: RED reproduced 0/1, GREEN 1/1 for ordinary_background_reconcile_does_not_supersede_in_flight_text_work; cargo check --lib --locked passed. |
💡 Codex Reviewtracedecay/.github/workflows/release.yml Lines 408 to 411 in fc20c4f For Linux targets, this archive now contains tracedecay/release-please-config.json Lines 22 to 25 in fc20c4f The new npm publication job derives its version from the packed tracedecay/dashboard/src/workspaces/delivery/DeliveryPage.tsx Lines 80 to 82 in fc20c4f When the scope bar selects a project other than the active project, this query keeps the same cache key and continues fetching the unprefixed active-project route. The dashboard already provides tracedecay/src/tracedecay/lifecycle/registry.rs Lines 118 to 122 in fc20c4f When AGENTS.md reference: AGENTS.md:L159-L161 tracedecay/plugin/cursor-native-extension/src/extension.ts Lines 187 to 189 in fc20c4f At activation or language-client restart, ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
fix(clippy): clear the workspace clippy failures on #707
Removing the re-export shims left their module doc comments orphaned in `context` and `graph`. The rest are pre-existing lints that only fire under `--all-features`, which compiles the `test-transport` support surface as non-test code: - the registered-test-server constructor gets the file's existing explicit `expect_used` allow, since `lib.rs` denies it only outside `cfg(test)`; - `http_application_registry` is read by the `cfg(test)` capacity journey and only reads as dead under `test-transport` alone, so the allow is gated to `not(test)` rather than renamed to a discard; - the hotpath cadence helpers keep `#[inline(always)]` with an explicit allow, preserving the author's intent instead of silently downgrading it. The `daemon_suite` restart wait is now the original helper recovered from the PR #707 head rather than a reconstruction: it additionally proves the runtime readiness receipt authorizes the exact configuration the SDK selected.
The relocation fix that anchored this test above the package allocated a `PathBuf` only to hand out a reference, which `-D warnings` rejects as `unnecessary_to_owned` and which failed the Clippy gate on #707. `validate_requested_workload` takes `&Path`, and `CARGO_MANIFEST_DIR` is a literal, so `Path::new` yields a `&'static Path` with no allocation and no temporary to borrow from.
|
#733 landed Clippy-red. Helper branch (not merged, shared checkout untouched):
Clears the #733 Clippy denials:
Adopt when you want: I will not land this on #707. |
|
Clippy helper is now on the Codex worktrees (still not landed on this PR head):
Same change as |
|
Isolated post-#721 verification (worktree Green (non-vacuous) on merge
Not green
Clippy helper remains on this tree at |
Ports the missing pieces of commit b1e41b7 ("fix(build): land the batching commit's missing consumers") onto #707: HookCompletedReadinessDistributions moved into tracedecay-agent-hosts with pub(crate) fields, so input_rows_processed, input_rows_dropped_at_cap, and events_considered are no longer reachable outside the crate. Adds the three public accessors. source_event and input_rows_received (also added by the source commit) are not ported: nothing on this branch reads them through an accessor today - the root-crate benchmark still compares via serde_json::to_value(&readiness_distributions)["field"], which does not need them. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
5d8f2a1 to
c7fcf3f
Compare
…d6ed chore(vendor): stacked vendor-only PR for #707 — generated vendor churn, land first
…rules 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>
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>
Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
…plan-reopened' into codex/tracedecay-total-redesign-plan-reopened
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. |
Order 25 — perf routing (2026-09-16; #707 held)
/fast/tmp/td-1391-deferred-mount/fast/tmp/td-1385-token-cutover/fast/tmp/td-707-perf-profiles/fast/tmp/td-1394-seal-admitPerf tables
| #1397 @
407a7ba897owner 304d4726 | search-qual residue | audit e339e1a2 | HOLD — deletion-first. Contradicts audit (types tautological DecisionPolicy). Rework per PR body hold: keep model-free eval/provenance; delete methodology + policy fields/callers + false-pass report + dead deps/fixtures; rename validate_workload; 67/34/33 + real failed count; NL metric / protected hard; provenance evidence-only; nDCG explained; drop new tautology enums; fix digest; 2 reviewers. Escape: close #1397 → clean smaller deletion PR. || #1396 @
dd4a8ec09f| clone table batch | Fable f6d91bcf | HOLD (PASS+). Collision-branch tests; typed conflict policy (no raw SQL); search_quality_suite v16 non-vacuous; r4/r8 ≥3-run before/after; no merge without measured win; successor out of scope. Owner 055650a8/fast/tmp/td-1396-clone-batch. || #1400 | ci beta retarget | — | CLOSED unmerged. #707 held; automatic beta publication not authorized. Do not reopen without explicit release authorization. |
| #1401 @
395a090713| Hawk fail-closed | coordinator hold | HOLD. Keep delete fake seed + no continue-on-error. Invalid: Hawk out of PR CI for backlog red. Rework fail-closed affected-scope PR check (changed-surface authority); no fake bundle; no hidden #707 reds; scheduled full workspace supplements only. Escape: close rather than optionalize. Owner a5193afd/fast/tmp/td-1401-hawk. |Order 25 census 2026-09-16T08:41Z (tip
82cb66a2e5)Merges this turn: 0 (no exact-head PR had APPROVED reviews + green evidence).
Closes this turn: 0 new (evidence #1367/#1369/#1378/#1381 already closed earlier; #1394 already closed as #1389 donor).
#707: draft=true MERGEABLE HELD.
b539a34d08draft7ed395cd04ready-not-draft70da34eb1214609f4f668285bf92da2d5e3e8506c1f39dce14dd4a8ec09f407a7ba897b1757ef2ce395a090713ae5b00b5d7735038d3f4Privacy history audit 2a59924a pending — no history rewrite without explicit destructive approval. #863 remains open.
| #1414 | #1410 NonCanonical | 0ba3cc10 | DRAFT. Typed
NonCanonicalCloneBodyOrder; acceptance unchanged; keep draft until 2 reviews+CI. |