feat: Add DestructiveHint to delete_project_item tool annotation

[bryankthompson]

Summary

Adds DestructiveHint: true annotation to the delete_project_item tool to be consistent with other delete operations that properly indicate destructive behavior.

Changes

• Added DestructiveHint: jsonschema.Ptr(true) to delete_project_item tool in pkg/github/projects.go
• Aligned field formatting for consistency

Why This Matters

The delete_project_item tool permanently removes a project item, making it a destructive operation. Other delete tools in the codebase (delete_file, delete_workflow_run_logs) correctly have this annotation set. This ensures LLMs receive consistent semantic metadata about tool behavior:

Tool	Current Annotations
delete_file	ReadOnlyHint: false, DestructiveHint: true ✅
delete_workflow_run_logs	ReadOnlyHint: false, DestructiveHint: true ✅
delete_project_item	ReadOnlyHint: false ❌ → Fixed

Testing

• Code follows existing annotation patterns in the codebase
• Uses same jsonschema.Ptr(true) pattern as other destructive tools

🤖 Generated with Claude Code

[Copilot]

Pull request overview

This PR adds the DestructiveHint: true annotation to the delete_project_item tool to properly indicate its destructive behavior, maintaining consistency with other delete operations in the codebase like delete_file and delete_workflow_run_logs.

Key Changes:

• Added DestructiveHint: jsonschema.Ptr(true) to delete_project_item tool annotations
• Aligned annotation field formatting for consistency (Title, ReadOnlyHint, DestructiveHint)

[Copilot]

The toolsnap file for delete_project_item needs to be updated to reflect the addition of DestructiveHint. After making this code change, you need to run UPDATE_TOOLSNAPS=true go test ./... to update the snapshot file at pkg/github/toolsnaps/delete_project_item.snap. The snapshot should include "destructiveHint": true in the annotations object, similar to delete_file.snap and delete_workflow_run_logs.snap. The updated snapshot file must be committed along with this code change.

[bryankthompson]

Thanks for the review feedback! I've run UPDATE_TOOLSNAPS=true go test ./pkg/github/... -run DeleteProjectItem and committed the updated snapshot file with the destructiveHint: true annotation.

The snapshot at pkg/github/__toolsnaps__/delete_project_item.snap now matches the code change.

[SamMorrowDrums]

Thank you for this, this was a good spot.