Conversation
Adopt the research loop structure, SOTA check, scope discipline, and consolidated sections from the imperative rewrite, but restore the original advisory tone throughout.
|
@claude please review this PR. Focus on correctness, security, LiteLLM/Bedrock routing impact, and whether it's safe to merge against current main. Keep it concise and prefer inline comments where it matters. |
|
Claude finished @akseljoonas's task in 1m 59s —— View job PR Review: Restructure system prompt with softer tone
Verdict: NEEDS CHANGES before merge. Three behavioral regressions; otherwise a clean, well-structured rewrite. Security / Routing impactNone. This file is a pure YAML system prompt with no routing logic, credentials, or infrastructure references. LiteLLM/Bedrock routing is unaffected. Rebase statusThis branch is missing Rule-by-rule analysis❌ HARDCODED UNAVAILABLE PACKAGES — fully dropped, no equivalentThe old prompt had an explicit mistake bullet:
The new prompt has no equivalent anywhere — not in the mistakes list, not in error recovery, not in the pre-flight check. This is the most impactful regression: an agent using
|
| Change | Status |
|---|---|
| Tone softening | ✅ No behavioral change |
| SOTA check section | ✅ Improvement |
| Scope discipline section | ✅ Improvement |
| HARDCODED UNAVAILABLE PACKAGES | ❌ Dropped, no equivalent |
| Data audit depth | |
| Task completion (non-autonomous) | |
| SILENT DATASET SUBSTITUTION | ✅ Preserved elsewhere |
| Rebase against main | ❌ Needed |
The tone rewrite itself is sound. Fix the three items above and rebase, then this is mergeable.
Summary
Test plan