Remove verify_approval tool, file preprocessing, and fix agent stability

[larryro]

Summary

• Remove the verify_approval agent tool and file preprocessing feature (beforeGenerateHook, FILE_PREPROCESSING_INSTRUCTIONS, related config/UI)
• Use version-aware agent naming ({name}:v{version}) to isolate SDK state between agent versions, preventing corrupted state from one version affecting others
• Add missing error field to messageMetadataValidator so the query return type matches the table schema, and fix the retry system message to reflect success on completion

Test plan

• Verify existing agents work without verify_approval tool
• Confirm agent version changes don't carry over stale SDK state
• Check retry messages display correctly after successful continuation
• Run bun run --filter @tale/platform test for affected test files

Summary by CodeRabbit

Release Notes

• Removed Features

  • Removed the verify_approval tool from integration agent capabilities
  • Removed file preprocessing feature and related UI controls from custom agent configuration

• Improvements

  • Enhanced error tracking and logging for agent responses
  • Improved error recovery handling in workflow execution management

[greptile-apps]

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

[coderabbitai]

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

This PR removes support for two custom agent features: file preprocessing (filePreprocessingEnabled) and the verify_approval tool. The file preprocessing removal eliminates the field from data models, schemas, mutations, UI components, and instruction generation logic. The verify_approval tool removal deletes the tool implementation and removes it from the tool registry, system defaults, and integration agent configuration. Additionally, the PR enhances error handling in agent response generation to capture and persist error information in message metadata, and updates agent debug tags to include version numbers.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

• feat(platform): resolve agent from thread metadata and fix workflow output extraction #828: Modifies triggerWorkflowCompletionResponse and creates hook handles via filePreprocessingEnabled, which this PR removes entirely.
• feat(platform): add document_retrieve agent tool #714: Reassigns tool names between categories in the same tool-selector.tsx component, addressing the removal of verify_approval from tool categorization.
• refactor: use savedMessageId directly instead of re-fetching messages #803: Modifies the same triggerWorkflowCompletionResponse call in internal_mutations.ts that this PR alters for hook removal.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 28.57% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately captures the primary changes: removing verify_approval tool, removing file preprocessing, and fixing agent stability issues. It directly relates to the changeset.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch feat/remove-verify-approval-and-file-preprocessing

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}