feat(WOR-TSK-200): implement execute command with timeout support - #111
Merged
Conversation
added 8 commits
December 16, 2025 03:52
- Add SharedBuffer for CLI output capture - Add CLI JSON response types for deserialization - Implement validate_params with mutual exclusion validation - Implement resolve_timeouts from config with parameter overrides - Implement convert_params_to_args for CLI args conversion - Implement parse_execute_response for JSON parsing - Implement execute NAPI function with tokio::time::timeout - Return ETIMEOUT error code on timeout expiration
- Add execute module export in commands/mod.rs - Re-export execute function in lib.rs for public API
- Remove dead_code attribute from ExecuteApiResponse (now in use) - Add allow unused_imports for execute type re-exports - Update module documentation for Story 6.3 completion
- Add SharedBuffer tests for output capture mechanism - Add validation tests including mutual exclusion - Add timeout resolution tests with config defaults - Add args conversion tests for CLI parameter mapping - Add JSON response parsing tests with error scenarios - Add type conversion tests for NAPI compatibility - Add ExecuteParams builder pattern tests
- Add execute function to binding.js - Add ExecuteParams, ExecuteData, ExecuteApiResponse types to binding.d.ts - Add PackageExecutionResult, ExecuteSummary types to binding.d.ts
- Add execute to function imports and exports - Update module documentation with execute command
- Add comprehensive API documentation for all functions - Document status, init, changeset, bump, and execute commands - Add TypeScript interface definitions and examples - Add error handling documentation with error codes - Add Quick Start guide and usage examples
miguelramos
added
♨️ ACTION: pull request
✨ TYPE: Feature
added 2 commits
December 16, 2025 03:56
- Reorder imports in tests.rs to match rustfmt conventions - Reorder imports in types/mod.rs to match rustfmt conventions
- Add cast_precision_loss allow with justification for f64 conversion - Replace map_or with is_some_and for filter_package check - Collapse nested if statements for timeout validation - Use numeric separator in test literal (100_000) - Fix f64 comparisons in tests using epsilon comparison
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.