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
referenced
this pull request
in dgarson/clawdbot
Feb 7, 2026
…7) (#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
zooqueen
pushed a commit
to hanzoai/bot
that referenced
this pull request
Mar 6, 2026
zooqueen
pushed a commit
to hanzoai/bot
that referenced
this pull request
Mar 6, 2026
what: default bash PATH to process.env.PATH why: ensure Nix-provided tools on PATH inside sessions tests: not run Co-authored-by: Peter Steinberger <steipete@gmail.com>
Raywonder
pushed a commit
to Raywonder/openclaw
that referenced
this pull request
Mar 8, 2026
Piboonsak
pushed a commit
to Piboonsak/openclaw_github
that referenced
this pull request
Mar 22, 2026
…restart gate (openclaw#202) - Validates JSON/JSON5 syntax is valid (guards against malformed config) - Validates gateway section exists (KI-048) - Validates agents.defaults.model.primary is non-empty string (KI-050) - Validates top-level models is not null and has valid providers (KI-052, KI-047) - Exit 0 = valid, non-zero = invalid with clear error message identifying which check failed - Used as pre-restart gate in deploy and hotfix workflows Refs Piboonsak/Openclaw#202
Piboonsak
added a commit
to Piboonsak/openclaw_github
that referenced
this pull request
Mar 22, 2026
- Validates JSON/JSON5 syntax is valid (guards against malformed config) - Validates gateway section exists (KI-048) - Validates agents.defaults.model.primary is non-empty string (KI-050) - Validates top-level models is not null and has valid providers (KI-052, KI-047) - Exit 0 = valid, non-zero = invalid with clear error message identifying which check failed - Used as pre-restart gate in deploy and hotfix workflows Refs Piboonsak/Openclaw#202 Co-authored-by: OpenClaw CI <ci@openclaw.dev>
Piboonsak
pushed a commit
to Piboonsak/openclaw_github
that referenced
this pull request
Mar 22, 2026
* Initial plan * feat(deploy): add BKG save step to deploy workflow (openclaw#202) --------- Co-authored-by: openai-code-agent[bot] <242516109+Codex@users.noreply.github.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
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)