Skip to content

simplify(proximity): remove discarded risk metadata - #4549

Merged
ScriptedAlchemy merged 10 commits into
masterfrom
simplify/proximity-constant-risk
Oct 27, 2026
Merged

ScriptedAlchemy merged 10 commits into
masterfrom
simplify/proximity-constant-risk

Conversation

@ScriptedAlchemy

@ScriptedAlchemy ScriptedAlchemy commented Oct 25, 2026 •

Copy link
Copy Markdown
Owner

Why

Direct encounter reads computed graph impact and freshness scores that their callers discarded. Keep exact encounters and named relationships without that scoring work.

What changed

  • Remove risk payloads, tier fields, and threshold-only revision hashes.
  • Preserve contribution identity and the published threshold admission mapping.
  • Regenerate dashboard and SDK contracts. Related activity no longer claims an exact overlap.

Scope

Refs #4416. Parent is master.
The persisted numeric setting remains. Its replacement and a metered real-producer no-impact proof remain open. No speedup is claimed.

Blast Radius

Encounter identity, permission, generation, and expiry checks remain. Existing durable feedback stores contain projected findings rather than the removed transient payload.

Verification

  • Focused Bazel proximity tests passed 6 tests. Advisory contract integration passed 1 test.
  • Loom consumer passed 36 tests. Typecheck, contracts generation/check, and affected Rust formatting passed.
  • The inherited broad acceptance run passed 138 tests and failed the unrelated LSP document-symbol capability test. That known-red test was not rerun.

Devin Review

@changeset-bot

changeset-bot Bot commented Oct 25, 2026 •

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: fbfb85c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

ScriptedAlchemy and others added 2 commits October 25, 2026 16:10
Collect encounter addresses and named relationships without graph-impact or freshness-score work. Preserve contribution identity and the published threshold admission mapping. Describe related activity without claiming an exact overlap.

Refs #4416
@ScriptedAlchemy
ScriptedAlchemy force-pushed the simplify/proximity-constant-risk branch from 958731a to e75a760 Compare October 25, 2026 16:11
ScriptedAlchemy and others added 8 commits October 25, 2026 14:26
Fleet logs show five complete 48,000-function generations take 53–65 seconds each. The default medium deadline kills the final refresh near 300 seconds despite steady successful publication. Declare the measured aggregate workload through the existing Cargo-to-Bazel size metadata, preserving every refresh, memory bound, and assertion.

Verified: regenerated the maintained BUILD files and generator --check reports all 60 outputs current. Runtime verification remains with the new CI run.

Evidence: https://github.com/ScriptedAlchemy/tracedecay/actions/runs/40237000628/job/121323060188
A real NativeLink restart with an eight-second outage reproduces exit 34 using the default five retries (3.50s critical path). Ten retries reconnect and complete the same remote action (18.99s critical path). Keep the existing timeout, backoff cap, remote-only compilation, and test failure semantics.
# Conflicts:
#	.github/workflows/ci.yml
The packaged Cursor and Codex journey reused one session ID across two
providers. Canonical Git evidence correctly refused that lineage, leaving
background convergence blocked before the advisory request timed out.
Give each host its own session and reuse the Codex identity in its rollout
and Stop event. Preserve the existing terminal-cycle and deadline checks.
@ScriptedAlchemy
ScriptedAlchemy enabled auto-merge (squash) October 27, 2026 02:50
@ScriptedAlchemy
ScriptedAlchemy merged commit 1e7e8ae into master Oct 27, 2026
17 checks passed
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