feat: TraceDecay V2 delivery (PR8–PR13) + daemon performance, memory, and reliability overhaul - #707
feat: TraceDecay V2 delivery (PR8–PR13) + daemon performance, memory, and reliability overhaul#707ScriptedAlchemy wants to merge 3299 commits into
Conversation
🦋 Changeset detectedLatest commit: 3d955ff 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  Install the bundled Linux runtime beside the binary
For Linux targets, this archive now contains tracedecay/release-please-config.json Lines 22 to 25 in fc20c4f Advance the SDK version in release PRs
The new npm publication job derives its version from the packed tracedecay/dashboard/src/workspaces/delivery/DeliveryPage.tsx Lines 80 to 82 in fc20c4f Scope Delivery queries to the selected project
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 Include Git remote identity in the registration digest
When AGENTS.md reference: AGENTS.md:L159-L161 tracedecay/plugin/cursor-native-extension/src/extension.ts Lines 187 to 189 in fc20c4f  Batch startup diagnostics instead of dropping the tail
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 b1e41b712 ("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>
|
Continuation audit at
Ordering: #1035 is protocol correctness; #1034/#1036 are measured resource cuts; #1037 is cleanup. None should displace #1026/#753/lifecycle/CI from the current undraft critical path unless measurement demonstrates an existing budget breach. No timeout/freshness/graph-seat relaxation implied. #707 stays DRAFT. |
Remove the unused diversity-stage trait and redundant directory, first-run, and maintenance accessors. Derive equivalent defaults and reuse canonical curator validation, artifact hashing, and lowercase hex decoding. Preserve byte identity, typed failures, and existing behavioral tests. Add focused receipt-prefix checks for malformed input and ignored suffixes. No dependency changes. Validation: locked all-target checks for tracedecay and tracedecay-cli with test-helpers; 48 focused runtime, artifact, receipt, and automation tests passed through Hauler (cc-9334 through cc-9338).
…redesign-plan-reopened
The floor sweep counts every full-sha quotation of the candidate; prose refers to the canonical metadata for the exact head. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Integrate the current V2 branch while preserving exact-scope lifecycle, source identity and bounded generation work. Keep durable orphan pagination with quarantine journal recovery, remove duplicate recovery planning when a live directory is discovered first, and pass caller cancellation through both recursive recovery routes. Preserve journals and final durability. Retain staging-release work through the existing terminal operation owner, including its build permit and exact store leases. Adopt the shared runtime configuration pin and projection cache while keeping exact scope ownership. Regenerate canonical SDK and dashboard outputs from the combined authorities. Validation: scoped all-target Rust checks passed; direct off-thread shutdown and five retention regressions passed. Dashboard build/typecheck and 1744 tests passed; SDK typecheck, 34 tests, 11 codegen tests and both generated contract checks passed. Broader runtime acceptance continues separately.
Integrate the current V2 branch floor while preserving owned recovery and registration lifetimes. Keep inventory filesystem work outside the global builder lock, materialize private SQLite report copies before validation, and fence source-family identity around copying. Ignore empty lexical sources before admission and compare real retained artifacts through the canonical bounded lexical lane. Consolidate the Cline journey assertions and use current unborn-history frontier semantics. Validation: widened all-target Rust check passed without diagnostics; full maintenance suite passed 140 tests, host admission passed 26 tests, and four focused snapshot/query/session-lookup tests passed. Further runtime and workspace acceptance remains in progress; this is not a release approval. PR #707 stays draft and unmerged.
The Linux test lane has not finished inside its 180-minute budget since the perf-profile cutover, and every run since 2026-09-06 was a cold build. Two causes, both measured on run 34138693824 and the cache list: 1. Four cargo invocations in the lane each resolved their own feature graph. `-p tracedecay-cli` drops the code-index grammar tier and `-p tracedecay-search-eval` resolves 52 dependencies differently from `--workspace --features tracedecay/test-helpers`, so the host-CLI fixture (20 min), the parity helpers (20 min) and one isolated unit test (7 min) recompiled the heaviest crates before `cargo test-ci` compiled the suites (56 min) and ran them (22 min). Every step now uses the workspace selection: the fixture and the parity feature-off build are cache hits, the isolated width test runs through the `test-ci` alias (nextest gives it its own process, which is all the `#[ignore]` protected against), and the parity verify uses the alias too. The full suite runs first and uploads its junit before the optional steps, and the dev-profile feature-gated checks run on Linux only (macOS compiles the same graph at half the speed: 62 vs 33 min). `build-controlled-workload-hotpath-helpers.py` gains `--workspace` and `--features` for this; its default `-p` behaviour is unchanged for the release workflow. 2. The repository's 10 GB Actions cache was full: 4.6 GB of it were entries saved by seven merged child PRs on their own `refs/pull/N/merge` refs, which no other ref can restore, plus per-job copies of the same dev-profile dependencies. The Linux test cache had been evicted, so each run rebuilt dependencies too. The dev-profile jobs (`debug-cli`, `pr-dogfood`, `dashboard`) now share one `ci-dev-<os>` rust-cache entry, the PR-closed cleanup workflow deletes the closed PR's caches, and the kache layer is removed from this workflow: its single `kache-<os>-<lockfile>` key was saved by whichever dev-profile job finished first, never by the perf-profile test lane, only on success, and no `kache-*` entry existed at all, so it cost a 5-minute `cargo install` per cold job and cache budget for no hits.
…plan-reopened' into codex/tracedecay-total-redesign-plan-reopened
Compute the baseline Git tree before preview so the fixture does not mutate the index TREE cache between preview and apply. Preserve exact checksum, authorization, stale-preview, and unchanged-HEAD assertions. Exercise restored graph serving through the canonical bounded reader rather than demanding the full decoded generation that partitioned restart no longer constructs. Check alpha, exact generation and retained store identity, current freshness, and zero decodes after the query; retain stale/corrupt and retirement coverage. Formatting and independent review passed. The two corrected runtime tests are running as cc9470. The hosted all-feature compiler-depth failure remains a separate follow-up; this checkpoint does not claim full CI acceptance.
Integrate the current V2 branch CI changes alongside the retained graph and Git preview fixture corrections. Preserve full-suite selection, isolated width/parity acceptance, and Linux/macOS-only Hotpath execution. Both workflows passed actionlint; the helper passed Python syntax parsing. No runtime source changed in this merge. PR #707 remains unmerged.
The all-feature CLI build exceeds the compiler layout-depth limit while laying out the maintenance worker. Boxing its outer future does not cut the nested instrumented schema-convergence future out of that layout. Type-erase that child at the existing worker boundary and await it normally. Keep the same retained task, database borrow, permit, cancellation and shutdown joins; do not raise the recursion limit. Formatting and independent review passed. Runtime worker checks and the exact all-feature build are queued; full acceptance remains pending.
Stale repository state and revoked policy discovered after durable apply admission must produce validated AbortedNoChange receipts with Failed termination. Keep direct authority PolicyDenied checks at their earlier boundary instead of expecting that error from an admitted transaction. Assert byte-exact preservation of index, HEAD and changed working content on the stale path, and retain the policy-denial preservation checks. Formatting passed. The focused Git authority journey is queued alongside the maintenance worker regressions.
2 participants
Replacement review for #421 after its accidental merge was reverted from master in 52a9aab. This commit has the exact tree of #421 head d4c67d4. Do not merge until explicitly authorized.
Issues this PR fixes
Refs #753 (source-identity + restart journey tests landed on branch at eae9328; keep draft until full product journey/CI)
Refs #792 (journey acceptance pending)
Refs #837 (journey acceptance pending)
Refs #838 (journey acceptance pending)
Refs #855 (journey acceptance pending)
Fixes #862
Fixes #861
Fixes #810
Fixes #845
Fixes #856
Fixes #853
Fixes #799
Fixes #836
Fixes #830
Partially addressed here, tracked to completion on their own issues: #852, #800, #863, #843, #842, #848.
🤖 Generated with Claude Code
#887 dead-surface deletion inventory
build_statecompatibility wrapper was deleted; the liveget_gc_metasuppression was removed.clean_c_comment/clean_c_doc_commentare retained because feat: TraceDecay V2 delivery (PR8–PR13) + daemon performance, memory, and reliability overhaul #707 now has production callers in C, C++, GLSL, Go, and Objective-C extractors.cargo check --workspace --all-targets;cargo clippy --workspace --all-targets -- -D warnings; both pass on the integrated branch.#888/#890/#891/#892 workflow cleanup
using-tracedecaybootstrap was deleted, detailed mechanics moved to live tool help, and unique safety/domain rules remain narrowly routed.no_skill_needed, and keep only actual host size/syntax limits.