diff options
| author | Ben Sima <ben@bensima.com> | 2025-12-01 04:34:12 -0500 |
|---|---|---|
| committer | Ben Sima <ben@bensima.com> | 2025-12-01 04:34:12 -0500 |
| commit | e2d6402464dda573c7401b34fc93557a786efc35 (patch) | |
| tree | 29fe7bf56e387b0c70f44e3f277ee1ffd3c398b2 /Omni/Dev/Beryllium/Live.nix | |
| parent | 3945b6fad4f1620612beb259e8601d165b9f4f12 (diff) | |
Replace llm CLI with Engine.chat in Jr.hs
Perfect! All tests pass. Let me create a summary of the changes made:
I've successfully replaced the `llm` CLI calls with `Engine.chat`
in Jr.
1. **`addCompletionSummary` function (lines 604-624)**:
- Removed `Process.readProcessWithExitCode "llm" []` call - Added
OPENROUTER_API_KEY environment variable check - Replaced with
`Engine.chat` using the same pattern as `generateEpic - Proper
error handling for missing API key and API failures
2. **`extractFacts` function (lines 658-680)**:
- Removed `Process.readProcessWithExitCode "llm" ["-s", ...]` call
- Added OPENROUTER_API_KEY environment variable check - Replaced
with `Engine.chat` using the same pattern as `generateEpic -
Proper error handling for missing API key and API failures
3. **Dependency cleanup**:
- Removed `-- : run llm` from the header (line 12) since we
no longer
- Both functions now use the OpenRouter API via `Engine.chat` -
Graceful degradation when OPENROUTER_API_KEY is not set (warning
messa - Consistent error handling pattern matching the existing
`generateEpicS - All tests pass successfully - No hlint or ormolu
issues
The implementation follows the exact pattern shown in the task
descripti
Task-Id: t-198
Diffstat (limited to 'Omni/Dev/Beryllium/Live.nix')
0 files changed, 0 insertions, 0 deletions
