Skip to content

fix: tighten skill generation policy prompts#2014

Draft
RerankerGuo wants to merge 1 commit into
MemTensor:mainfrom
RerankerGuo:fix/tighten-skill-generation-policy
Draft

fix: tighten skill generation policy prompts#2014
RerankerGuo wants to merge 1 commit into
MemTensor:mainfrom
RerankerGuo:fix/tighten-skill-generation-policy

Conversation

@RerankerGuo

Copy link
Copy Markdown

Fixes #1380.

Summary

  • Tighten the skill creation evaluator prompt so low-reuse tasks are rejected before skill generation.
  • Add explicit filters for simple tool installation plus basic config, exploratory revert/abandon outcomes, and discussion/comparison-only outputs.
  • Add the requested experienced-developer self-check before returning shouldGenerate=true.
  • Reinforce generator guidance to prefer reusable patterns, abstract diagnostic mistakes, and keep project/version-specific details as examples unless required.
  • Add a prompt-policy regression test for the new evaluator and generator guidance.

Validation

  • Direct Node prompt-policy string check passed.
  • Attempted: npm test -- --run tests/skill-prompt-policy.test.ts; blocked because vitest is not installed in this worktree.
  • Attempted: npm run build; blocked because tsc is not installed in this worktree.
  • Attempted: make format; blocked because poetry is not installed in this environment.

@Memtensor-AI Memtensor-AI changed the base branch from main to dev-v2.0.22 July 1, 2026 13:14
@Memtensor-AI

Copy link
Copy Markdown
Collaborator

Automated Test Results: PASSED

Cloud test-engine full-audit rerun against dev-v2.0.22 completed successfully after clearing the cloud workspace disk.

  • Run: tr-1eb15b94-4a0 on cloud test-engine 10012
  • memos_local_openclaw/unit: 2 passed, 0 failed, 0 skipped

Manual code review is still required before merge.

@CarltonXiang CarltonXiang deleted the branch MemTensor:main July 3, 2026 07:25
@syzsunshine219 syzsunshine219 reopened this Jul 3, 2026
@syzsunshine219 syzsunshine219 added the needs-audit Requires manual audit before merge label Jul 3, 2026
@syzsunshine219 syzsunshine219 changed the base branch from dev-v2.0.22 to main July 3, 2026 08:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-audit Requires manual audit before merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Skill 自动生成:evaluator 评估过于宽松,导致低复用任务也被生成 skill

4 participants