Skip to content

fix: widen skill writer evidence scope#170

Open
ScriptedAlchemy wants to merge 1 commit into
masterfrom
codex/fix-skill-writer-evidence
Open

fix: widen skill writer evidence scope#170
ScriptedAlchemy wants to merge 1 commit into
masterfrom
codex/fix-skill-writer-evidence

Conversation

@ScriptedAlchemy

Copy link
Copy Markdown
Owner

Summary

  • default skill-writing CLI evidence collection to all providers instead of overriding the runner with cursor-only search
  • add skill-writing storage_scope/hermes_home support through runner, CLI, and dashboard API so it can read Hermes-profile LCM like session-reflection
  • reuse the LCM store resolver across automation tasks and add regressions for all-provider defaults and Hermes-profile skill-writer evidence
  • tighten session-reflector prompt guidance so backend facts emit numeric trust values

Verification

  • cargo fmt --all --check
  • CARGO_TARGET_DIR=/tmp/tracedecay-target/automation-evidence cargo test --test automation_skill_writer_runner_test -- --nocapture
  • CARGO_TARGET_DIR=/tmp/tracedecay-target/automation-evidence cargo test --bin tracedecay automation_run_skill_writing -- --nocapture
  • CARGO_TARGET_DIR=/tmp/tracedecay-target/automation-evidence cargo test --test automation_session_reflector_runner_test session_reflector_runner_reads_hermes_profile_lcm_with_filters -- --nocapture
  • CARGO_TARGET_DIR=/tmp/tracedecay-target/automation-evidence cargo test --test automation_session_reflector_runner_test session_reflector_runner_validates_fact_proposals_without_applying -- --nocapture
  • CARGO_TARGET_DIR=/tmp/tracedecay-target/automation-evidence cargo test --lib session_reflector_prompt_requires_numeric_trust -- --nocapture
  • CARGO_TARGET_DIR=/tmp/tracedecay-target/automation-evidence cargo run --quiet -- automation run skill-writing --dry-run true

Runtime evidence

  • Before: provider=cursor default returned zero hits for the default skill-writing query.
  • After: source CLI help shows provider default all; dry-run skill_writer_1782925717_0_ff8c35e1947f03c2 succeeded instead of no_skill_writer_evidence.

@changeset-bot

changeset-bot Bot commented Jul 1, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 77475b7

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant