Claude finished @windmill-internal-app[bot]'s task —— View job • claude/issue-6594-20250911-2029 • [Create PR ➔](https://github.com/windmill-labs/windmill/compare/main...claude/issue-6594-20250911-2029?quick_pull=1&title=Issue%20%236594%3A%20Changes%20from%20Claude&body=This%20PR%20addresses%20issue%20%236594%0A%0AGenerated%20with%20%5BClaude%20Code%5D(https%3A%2F%2Fclaude.ai%2Fcode)

✅ Task completed successfully!

Todo List

Search for Monaco editor components handling preprocessors
Locate the reset button implementation in the topbar
Understand preprocessor function signature format
Modify reset logic to use preprocessor signature instead of "main"
Run frontend validation checks
Commit changes and create draft PR

Summary

I've successfully fixed the reset button for preprocessor scripts. The button now correctly resets to the preprocessor function signature when on the preprocessor tab, instead of always resetting to the main function signature.

Changes Made

Added selectedTab prop to EditorBar: Tracks which tab (main or preprocessor) is currently active
Modified clearContent function: Now detects when the preprocessor tab is active and uses the appropriate preprocessor template
Added getPreprocessorCode helper: Returns the correct preprocessor signature for TypeScript/Deno/Bun and Python scripts
Updated ScriptEditor: Passes the selectedTab prop to EditorBar
Fixed type definitions: Updated WorkspaceScriptPicker and PickHubScript to support the 'preprocessor' kind

Result

✅ All validation checks passed
✅ Code committed with appropriate co-author attribution
✅ Draft PR created: #6595

The fix ensures that when users are working on preprocessor scripts and click the reset button, they get the appropriate preprocessor function signature rather than losing their context by getting the main function signature.

improve reset button for preprocessors #6594

Description

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions