AGEN-1 - trckr

← AGEN-1

feature

Created: 2026-02-03 • Updated: 2026-02-03

Relationships Loading...

# Per-Task Working Directory Override

Implement per-task `directory` field to override workflow-level working directory.

## Scope
~150 lines of code + tests

## Implementation
1. Add `directory` field to task schema validation (validator.py)
2. In `execute_single_task()`, check for task-level `directory` (executor.py)
3. If present, override workflow directory for subprocess `cwd=` parameter
4. Support template variable interpolation in directory path
5. Resolve relative paths relative to workflow directory

## YAML API
\`\`\`yaml
task:
  - build-module-a:
      directory: "{{gitroot}}/modules/a"
      prompt: "Build module A"
  
  - build-module-b:
      directory: "./modules/b"  # Relative to workflow directory
      prompt: "Build module B"
\`\`\`

## Files to Modify
- src/executor.py (~50 lines)
- src/validator.py (~30 lines)
- tests/test_executor.py (~70 lines)

## Tests Required
- Task with directory override executes in correct directory
- Template variables interpolate correctly in path
- Relative paths resolve relative to workflow directory
- Invalid/nonexistent directory raises clear error
- Workflow directory unchanged for subsequent tasks

## QA Validation Steps

### 1. Positive Test
**Action:**
\`\`\`yaml
directory: /tmp
task:
  - create-in-override:
      directory: "/tmp/subdir"
      prompt: "Create test.txt with content: executed in /tmp/subdir"
\`\`\`
**Expected:** File created at /tmp/subdir/test.txt
**Verify:** \`cat /tmp/subdir/test.txt\` shows correct content

### 2. Template Variable Test
**Action:**
\`\`\`yaml
task:
  - use-gitroot:
      directory: "{{gitroot}}/agent-yaml"
      prompt: "Create marker.txt"
\`\`\`
**Expected:** File created in git root's agent-yaml subdirectory
**Verify:** File location matches \`$GITROOT/agent-yaml/marker.txt\`

### 3. Relative Path Test
**Action:**
\`\`\`yaml
directory: /tmp/workflow
task:
  - relative-dir:
      directory: "./subdir"
      prompt: "Create relative.txt"
\`\`\`
**Expected:** File at /tmp/workflow/subdir/relative.txt
**Verify:** Path resolved relative to workflow directory

### 4. Negative Test
**Action:** Use nonexistent directory: \`directory: "/nonexistent/path"\`
**Expected:** Clear error message about directory not existing
**Verify:** Error message, workflow stops gracefully

### Pass Criteria
✅ All 4 tests pass
✅ No changes to workflow directory for subsequent tasks
✅ Unit tests pass

### Fail Criteria
❌ Any test fails
❌ Directory override affects other tasks
❌ Template variables don't interpolate

Pull Requests (comma-separated URLs)

Attachments

Comments (3)

QA Agent · 2026-02-03

## QA Verification Complete **Status:** ✅ **PASSED** ### Summary All tests pass successfully. The per-task directory override feature is fully functional and ready for production use. ### Test Results - ✅ All 665 unit/integration tests pass (77% coverage) - ✅ 10/10 directory override tests pass - ✅ 7/7 validation tests pass - ✅ All 5 end-to-end QA scenarios pass ### QA Scenarios Validated 1. ✅ Absolute path override 2. ✅ Template variable interpolation ({{gitroot}}) 3. ✅ Relative path resolution 4. ✅ Nonexistent directory error handling 5. ✅ Directory isolation between tasks ### Code Quality - Clean implementation following existing patterns - Proper isolation using .copy() to avoid side effects - Complete error handling for all strategies (fail/continue/skip) - Clear, detailed error messages ### Issues Found **None.** Implementation is production-ready. **Full QA report:** /tmp/qa-AGEN-1.md

agent · 2026-02-03

## Implementation Complete ### Changes Made **src/executor.py (~50 lines)**: - Added `os` module import at module level - Implemented task-level directory override in `execute_single_task()` (lines 155-198) - Creates a copy of `workflow_config` (`task_workflow_config`) to avoid modifying original - Interpolates template variables in directory path - Expands `~` (home directory) - Resolves relative paths relative to workflow directory - Validates directory exists (unless dry-run mode) - Handles all three error strategies (fail, continue, skip) for nonexistent directories - Updated dry-run and debug mode to show task-specific directory **src/validator.py (~15 lines)**: - Added `directory` to list of known meta fields in task validation (lines 224, 269) - Added explicit validation that `directory` must be a string (lines 297-302) **tests/test_executor.py (~220 lines)**: - New `TestDirectoryOverride` class with 10 test methods: - `test_directory_override_absolute_path` - `test_directory_override_relative_path` - `test_directory_override_with_template_variable` - `test_directory_override_nonexistent_fails` - `test_directory_override_nonexistent_continues` - `test_directory_override_nonexistent_skips` - `test_directory_override_does_not_affect_workflow_config` - `test_task_without_directory_uses_workflow_directory` - `test_directory_override_dry_run_shows_correct_path` - `test_directory_override_with_tilde_expansion` **tests/test_validator.py (~90 lines)**: - New `TestTaskDirectoryValidation` class with 7 test methods ### Test Results All 665 tests pass (18 skipped). Coverage maintained at 77%. ### Ready for QA The implementation follows the existing pattern from `loops.py` (lines 1128-1130) for directory pool feature.

triage-agent · 2026-02-03

## Feature Triage Complete **Status:** NOT IMPLEMENTED - Ready for implementation **Summary:** Per-task working directory override feature. Allows individual tasks to execute in a different directory than the workflow-level setting. **Implementation Location:** - `src/executor.py` (~50 lines) - Add directory override logic - `src/validator.py` (~30 lines) - Add directory field validation - `tests/test_executor.py` (~70 lines) - Test directory override behavior **Key Finding:** The exact pattern needed already exists in `loops.py` (lines 1128-1130) for the `directory-pool` feature: ```python task_workflow_config = workflow_config.copy() task_workflow_config["directory"] = expanded_dir ``` **Complexity:** LOW **Estimated scope:** ~150 lines total including tests **No blockers identified.** See full triage report: /tmp/triage-AGEN-1.md