fix(sdk): repair two pre-existing test failures in packages/brain - #324
Open
sebbsssss wants to merge 6 commits into
Open
fix(sdk): repair two pre-existing test failures in packages/brain#324sebbsssss wants to merge 6 commits into
sebbsssss wants to merge 6 commits into
Conversation
Promote: login status position (#fix/dashboard-login-status-position)
Implement the Clude neural design homepage (index.html) with a full mobile-responsive pass: collapsing nav, fluid hero type scale, single -column memory/feature grids, restacked benchmark bars (full-width on phones), and tighter code blocks. Adds required assets (memory-net.js particle sphere + logos). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
feat(web): mobile-responsive Clude neural homepage
installMcpConfig has emitted ['@clude/sdk', 'mcp-serve'] since the 3.0.1 package rename (2b0d9ac); the published npm package is @clude/sdk and 'clude' does not exist on the registry. The test expectation was stale. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
scoreImportanceOnWrite nudged internal sources (dream/reflection) down by 0.05, but inferConcepts grants reflection/emergence a self_insight concept purely from the source, adding a +0.05 concepts bonus that exactly cancels the nudge. An auto-scored reflection write tied its external equivalent (0.714 == 0.714) instead of scoring below it. Raise the penalty to 0.08 so internal writes always land strictly below the same external write, per the source-monitoring design intent. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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
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
Fixes the two test failures that have been present on `origin/main` (verified failing with a clean tree on 2026-07-02, unrelated to any in-flight branch).
1. `setup-zero-config.test.ts` — stale MCP args expectation
`installMcpConfig` emits `['@clude/sdk', 'mcp-serve']` and has since the 3.0.1 package rename (`2b0d9ac7`). `@clude/sdk@3.2.0` is the published npm package; `clude` is a 404 on the registry, so the code is correct and the test expectation was stale. Fix: updated the test expectation.
2. `memory-integrity.test.ts` — internal-source nudge cancelled by concept bonus
Not a rounding issue: `inferConcepts` grants `reflection`/`emergence` a `self_insight` concept purely from the source, so an auto-scored internal write earns the +0.05 concepts bonus, which exactly cancelled the −0.05 internal nudge (0.70 base + 0.014 length + 0.05 − 0.05 = 0.714 on both sides — an exact float tie). Fix: raised the internal-source penalty to 0.08 so it strictly dominates the concept bonus internal sources can self-trigger; internal writes now always score below the equivalent external write. The existing test is kept unchanged — it now guards exactly this cancellation case.
Behavioral note: auto-scored internal writes (dream/reflection/etc.) drop by a further 0.03; explicit `importance` still always wins.
Test plan
🤖 Generated with Claude Code