Conversation
what: default bash PATH to process.env.PATH why: ensure Nix-provided tools on PATH inside sessions tests: not run
steipete
added a commit
that referenced
this pull request
Jan 6, 2026
Contributor
|
Marked ready and merged (squash). Added a thank-you changelog entry in Unreleased Fixes: . |
Contributor
|
Marked ready and merged (squash). Added a thank-you changelog entry in Unreleased Fixes: CHANGELOG.md. |
dgarson
added a commit
to dgarson/clawdbot
that referenced
this pull request
Feb 7, 2026
…7) (openclaw#202) Implements a comprehensive, configurable scoring system for determining which experiences should become long-term memories. ## New Module: scoring/ - **types.ts**: Full type definitions for factors, breakdowns, configs - **factors.ts**: Five factor scorers (novelty, impact, relational, temporal, userIntent) - **scorer.ts**: Composite scorer with tool/pattern overrides, threshold profiles - **defaults.ts**: Default weights, profiles (standard/aggressive/conservative), overrides - **index.ts**: Barrel exports ## Factor Scoring Model - **Novelty (0.25)**: Detects repetitive vs. new tool usage patterns - **Impact (0.30)**: Classifies tool types (high/medium/low impact), error detection, meta keywords - **Relational (0.15)**: Entity detection via tags, keywords, file paths, URLs - **Temporal (0.10)**: Time-of-day awareness, burst detection, pacing analysis - **User Intent (0.20)**: Explicit marks, manual captures, intent keywords ## Override System - Tool-specific rules: fixedScore, minScore, maxScore, weightMultiplier - Pattern-based rules: error boost, large result detection, tag matching - User intent override: highest priority ## Integration - Updated evaluate.ts with evaluateRelevance() bridge function - Updated config.ts with scoring config resolution (backwards compatible) - Re-exports for convenience: shouldCaptureMultiFactor, isHighValueMultiFactor, etc. - breakdownToTrace() for compact trace event embedding ## Tests - 48 tests across factors.test.ts and scorer.test.ts - All passing
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
2 participants
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.
Human Description (from Josh)
"in nix mode then path gets all fucked up and tools that are built and in the bash closure for that nix thing dont show up. then clawdis is sad. im tired and dont really konw whats going on"
Feature Intent (low‑friction)
In Nix-managed installs, tools are added to the gateway process PATH. The bash tool was ignoring that PATH and defaulting to a hardcoded system path, so Nix-provided tools (e.g., picnic/summarize) appeared missing inside tool execution. This change makes the bash tool inherit the gateway process PATH, so tools that are already available to Clawdbot remain available during command execution.
Process summary:
process.env.PATHto align tool execution with the gateway environment.Prompt History (UTC, user prompts)