PLOW-44 - trckr

← PLOW-44

feature

Created: 2025-12-21 • Updated: 2025-12-21

Relationships Loading...

## Summary

Add support for configuring notifications via a config file.

## Implementation

### Config File Location

`~/.config/trckr-plow/config.yml`

### Config Schema

```yaml
notify:
  enabled: true  # Enable notifications (default: false)
  server: https://ntfy.roboalch.com  # ntfy server URL
  topic: all  # ntfy topic name
```

### Loading Logic

Add to `src/trckr_plow/config/loader.py` or create `src/trckr_plow/settings.py`:

```python
from pathlib import Path
import yaml
from trckr_plow.notify import NotifyConfig

def load_notify_config() -> NotifyConfig:
    """Load notification config from file or return defaults."""
    config_path = Path.home() / ".config" / "trckr-plow" / "config.yml"
    
    if not config_path.exists():
        return NotifyConfig()
    
    try:
        with open(config_path) as f:
            data = yaml.safe_load(f) or {}
        
        notify_data = data.get("notify", {})
        return NotifyConfig(
            enabled=notify_data.get("enabled", False),
            server=notify_data.get("server", "https://ntfy.roboalch.com"),
            topic=notify_data.get("topic", "all"),
        )
    except Exception:
        return NotifyConfig()
```

## Files to Modify

- `src/trckr_plow/config/loader.py` or create `src/trckr_plow/settings.py`
- `tests/test_config.py` - Add tests for notify config loading

## Acceptance Criteria

- [ ] load_notify_config() returns NotifyConfig with defaults if no file
- [ ] Parses config.yml correctly when present
- [ ] Handles malformed YAML gracefully (returns defaults)
- [ ] Unit tests cover all cases
- [ ] Pure Python implementation (no agents)

Pull Requests (comma-separated URLs)

Attachments

Comments (1)

agent · 2025-12-21

## Implementation Verification The notification config file support feature was found already implemented in the codebase (as part of PLOW-45 work). ### Files Involved - `src/trckr_plow/settings.py` - Contains `load_notify_config()` function - `tests/test_settings.py` - Contains 14 comprehensive unit tests ### Acceptance Criteria Verification | Criteria | Status | |----------|--------| | load_notify_config() returns NotifyConfig with defaults if no file | ✅ Verified | | Parses config.yml correctly when present | ✅ Verified via multiple tests | | Handles malformed YAML gracefully (returns defaults) | ✅ Verified | | Unit tests cover all cases | ✅ 14 tests covering all scenarios | | Pure Python implementation (no agents) | ✅ Verified - no agent dependencies | ### Test Results All 14 settings tests pass. Full test suite: 793 tests passed.