umputun / cronn / 21509077922

75%

Build Type

push

github

Committed by web-flow

Commit Message

Add JSON API endpoints for job history and execution logs (#59)

* docs: add plan for JSON API history and logs endpoints

Add implementation plan for new JSON API endpoints:
- GET /api/v1/jobs/{id}/history
- GET /api/v1/jobs/{id}/executions/{exec_id}/logs

Also includes refactoring to separate JSON API code into api.go.

* feat(web): add JSON API endpoints for job history and execution logs

Create new api.go with:
- APIExecution, APIHistoryResponse, APILogsResponse types
- handleAPIJobHistory for GET /api/v1/jobs/{id}/history
- handleAPIExecutionLogs for GET /api/v1/jobs/{id}/executions/{exec_id}/logs
- Helper functions toAPIExecution and toAPIJob for type conversion
- writeJSON and writeJSONError utilities for JSON responses

Moves existing API code from handlers.go:
- APIStatusResponse, APIJob, APIStats types
- handleAPIStatus handler

* test(web): add tests for JSON API endpoints

Add comprehensive test coverage for handleAPIStatus, handleAPIJobHistory,
and handleAPIExecutionLogs handlers in new api_test.go file.

* chore: complete JSON API plan with final validation

- Remove unused json import from handlers_test.go
- Move completed plan to docs/plans/completed/

* docs: add documentation for JSON API history and logs endpoints

document the new API endpoints for job execution history
and execution logs in the README JSON API section

* fix: address codex review findings

- add ErrNotFound sentinel error in persistence package
- GetExecutionByID returns ErrNotFound for missing executions
- API handler distinguishes 404 (not found) from 500 (internal error)
- job ID mismatch returns 404 to prevent execution enumeration
- update tests to verify new error handling behavior
- use context in SQLite initialization and migration methods

Run Details

150 of 163 new or added lines in 3 files covered. (92.02%)

3 existing lines in 2 files now uncovered.

2766 of 3674 relevant lines covered (75.29%)

36.77 hits per line