Commit dccb0ad
committed
refactor: implement execution abstraction, unified exception handling, and SSH config abstraction
- Add BlockExecutor protocol with LocalExecutor and RemoteExecutor implementations
- Create ExecutorFactory for dependency injection and testability
- Implement ShellflowExceptionHandler for consistent error handling across all execution paths
- Abstract SSH configuration with SSHConfigProvider protocol and multiple provider implementations
- Refactor run_script function to use new abstractions (previously reduced from 254 to 111 lines)
- Maintain 100% test coverage with all 143 unit tests and 17 BDD scenarios passing
Closes architectural debt items C02, C03, and C04.1 parent bc6785d commit dccb0ad
3 files changed
Lines changed: 495 additions & 244 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
224 | 224 | | |
225 | 225 | | |
226 | 226 | | |
| 227 | + | |
| 228 | + | |
| 229 | + | |
| 230 | + | |
| 231 | + | |
| 232 | + | |
| 233 | + | |
| 234 | + | |
| 235 | + | |
| 236 | + | |
| 237 | + | |
| 238 | + | |
| 239 | + | |
| 240 | + | |
227 | 241 | | |
228 | 242 | | |
229 | 243 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | | - | |
| 3 | + | |
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
| |||
0 commit comments