feat: trust scoring with feedback API

[ajianaz]

Summary

Add helpful/unhelpful feedback mechanism for trust scoring (#718).

Problem

Uteke memories have an importance field (0.0–1.0, default 0.5) used by salience boost and orphan detection. Currently importance can only be set directly or recomputed globally — there's no way to signal that a specific memory was useful or not in a given context. Hermes solves this with a feedback loop: +0.05 for helpful, -0.10 for unhelpful (asymmetric — penalizing bad memories more than rewarding good ones).

Changes

uteke-core/src/lib.rs

• Uteke::feedback_helpful(id) → importance += 0.05, returns new value
• Uteke::feedback_unhelpful(id) → importance -= 0.10, returns new value
• Internal feedback_adjust(id, delta) with clamp to [0.0, 1.0]

uteke-cli/src/cli.rs

• uteke feedback <id> {helpful|unhelpful} subcommand
• FeedbackAction enum

uteke-cli/src/commands/mod.rs

• Handler with JSON (--json) and human-readable output

uteke-server/src/{handlers.rs, types.rs, main.rs}

• POST /memory/feedback — { id, feedback: "helpful"|"unhelpful" }
• Returns { id, feedback, delta, importance }
• UUID validation, 400 for invalid feedback value

Usage

CLI:

uteke feedback <uuid> helpful     # importance += 0.05
uteke feedback <uuid> unhelpful   # importance -= 0.10
uteke --json feedback <uuid> helpful

API:

curl -X POST http://localhost:8767/memory/feedback \
  -H 'Content-Type: application/json' \
  -d '{"id": "uuid", "feedback": "helpful"}'
# → {"id": "...", "feedback": "helpful", "delta": 0.05, "importance": 0.55}

Design Decisions

Decision	Rationale
Asymmetric delta (±0.05 vs -0.10)	Adopted from Hermes. Bad memories should be demoted faster than good ones are promoted.
Clamp to [0.0, 1.0]	Prevents out-of-range importance from repeated feedback.
Separate helpful/unhelpful (not a generic delta)	Clearer API contract. Future: per-namespace delta config.
Returns new importance	Caller can track trust trajectory without a separate GET.

Closes #718

[github-actions]

🔍 Cora AI Code Review

❌ Blocked — critical issues found.

🔴 Error (1)

• crates/uteke-core/src/lib.rs:676 — In feedback_adjust, query_row returns Err(QueryReturnedNoRows) when the memory ID doesn't exist. This is mapped to Error::db("feedback_adjust read", e) which is a generic database error. In the server handler this surfaces as a 500 Internal Server Error rather than a 404, and in the CLI it shows "Feedback failed: ..." with no indication the ID was not found. The handler validates UUID format but never checks existence, so a well-formed but non-existent UUID produces a misleading error.

---

Review powered by cora-cli · BYOK · MIT