docs: add panic contract doc comment to prepareClaudeToolsForAllowedList

[Copilot]

prepareClaudeToolsForAllowedList panics on an invariant violation but lacked a doc comment describing this contract, relying solely on the BUG: message prefix to satisfy the panicinlibrarycode linter. Other exempted panic sites in the repo use explicit "Panics if …" doc comments.

Changes

• pkg/workflow/claude_tools.go — adds a doc comment to prepareClaudeToolsForAllowedList using the "Panics if" phrasing per repo convention:

// prepareClaudeToolsForAllowedList expands neutral tool definitions into Claude-specific
// format. Panics if tools already contains a "claude" section key, since callers must only
// ever pass neutral tool definitions at this stage (an internal invariant violation).
func (e *ClaudeEngine) prepareClaudeToolsForAllowedList(tools map[string]any) map[string]any {

No behavior change.

• Fixes [Code Quality] Add missing panic doc contract to prepareClaudeToolsForAllowedList #48969

[Copilot]

Pull request overview

Adds the missing panic contract documentation for prepareClaudeToolsForAllowedList, resolving #48969 without changing behavior.

Changes:

• Documents the method’s purpose and panic condition.
• Uses the repository’s established “Panics if” convention.

Show a summary per file

File	Description
pkg/workflow/claude_tools.go	Adds the panic contract doc comment.

Review details

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

• Files reviewed: 1/1 changed files
• Comments generated: 0
• Review effort level: Medium