Skip to content

feat: add safety_baseline flag to prepend safety preamble to system prompts#537

Closed
TakaoWing wants to merge 1 commit intonrslib:mainfrom
TakaoWing:batch/takt-safety-preamble
Closed

feat: add safety_baseline flag to prepend safety preamble to system prompts#537
TakaoWing wants to merge 1 commit intonrslib:mainfrom
TakaoWing:batch/takt-safety-preamble

Conversation

@TakaoWing
Copy link
Copy Markdown

Summary

  • Added safety_baseline boolean flag to TAKT piece YAML schema
  • When enabled, prepends 1-2 line safety constraint before persona in system prompts
  • Existing pieces without the flag retain current behavior (backward compatible)

Issues

  • Closes #ptyctl-8f7mh (bd: ptyctl-8f7mh)

Test plan

  • Piece with safety_baseline: true prepends safety constraint
  • Piece without flag or safety_baseline: false unchanged
  • Type checks pass (tsc --noEmit)

Generated with Claude Code

Adds `safety_baseline: true` flag to TAKT piece YAML schema. When
enabled, prepends a 1-line safety constraint before the persona
definition in every movement system prompt. Existing pieces without the
flag are unaffected (default: false).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@nrslib
Copy link
Copy Markdown
Owner

nrslib commented Mar 21, 2026

提案ありがとうございます。方向性は理解できますが、現状の情報だけではまだ取り込み判断ができません。

CONTRIBUTING.md にある必須事項との関係で、まず以下を確認させてください。

  • 事前に議論した Issue があれば共有してください
  • 新しい挙動に対するテストを追加してください
  • review summary を PR コメントに投稿してください
  • Closes #ptyctl-8f7mh (bd: ptyctl-8f7mh) が何を指しているのか説明してください
    このリポジトリの Issue 参照としては確認できませんでした

あわせて、変更内容についても背景を知りたいです。

  • この safety preamble を追加したい具体的な理由
  • harmful code をどの範囲で想定しているか
  • defensive / authorized security work をどう扱う想定か

背景と要件が分かれば再レビューしやすくなります。必要であれば、先に Issue で整理してから再提案してください。

@nrslib
Copy link
Copy Markdown
Owner

nrslib commented Apr 5, 2026

お返事いただけない状態のようなので一旦Close します

@nrslib nrslib closed this Apr 5, 2026
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.

2 participants