Repository navigation
Bench/phase b1.2 auto kg - #9
Merged
Merged
Conversation
… miss The B1.1 run only needs the already-cached corpus/query embeddings; the unconditional top-level import required installing a heavy dependency that wasn't otherwise needed for this run.
Audited the real auth flow (OAuth start/success points, email login success point, cookie domain-sharing convention, provider-detection precedent already used in Header/AppSidebar) before designing a cookie + user_preferences based approach for valori-ui.
…n (insert needs user_id column privilege)
Audited the marketing site's real component library (home.mdx's shortcode composition, the Section/Container rhythm primitives, the architecture-diagram box/connector idiom, nav data, sitemap, and the root layout's openGraph inheritance gotcha) before designing /jev as a standalone page following the solutions/rag precedent, reusing the existing visual language rather than introducing a new one.
varshith-Git
added a commit
that referenced
this pull request
Sep 30, 2026
Registry latest is still 0.2.2, from before the AssertionsTab feature and useSearch removal landed (both already merged via PR #9). See ui/studio/CHANGELOG.md for the accurate delta since 0.2.2. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
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
1 participant
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
What does this change do, and why?
Determinism Impact
FxpScalar)If arithmetic behavior changes, explain why and what compatibility fixtures were updated.
no_std/ Kernel ABIOnly applies if
crates/valori-kernelorcrates/valori-corewere touched.use std::added insidevalori-kernel/src/; std-only code is gated behind#[cfg(feature = "std")]cargo build -p valori-kernel --target wasm32-unknown-unknownpasses locallyStandalone + Cluster Parity
Only applies if this adds or changes an HTTP endpoint or a
KernelEvent.server.rsandcluster_server.rsraft.client_write(), not a direct engine lockcargo test -p valori-node --test route_paritypasses (or the allowlist was updated with a reason)Cross-Architecture Behavior
Tested on:
Tests
cargo test -p valori-kernel -p valori-nodepassescargo fmt --all -- --checkandcargo clippy --workspace --all-targets -- -D warningspassInclude reproduction / validation cases below.
UI (if applicable)
Notes
If this relates to forensic or evaluator tracks, confirm that logic is kernel-appropriate.
Anything deferred to a follow-up, and which phase/issue owns it.