docs(highlights,notes,opds,progress,analytics): Beam batch 9 — 13 endpoints

Highlights + notes CRUD (media-item scoped, jwt), progress delete,
reading-stats analytics, and the OPDS trio frontmatter'd. OPDS docs are
the first XML entries: example-response intentionally omitted (Atom
responses), gate contract honored via nil-Response registry pins.

Gate fix: group .Use() middleware now parsed (opds.Use(DeviceAuth...
Authenticate) previously resolved to auth 'none'). Registry: 13 entries.
Drift fixed by gate: reading-stats key names (total_reading_time_minutes,
most_active_day_of_week, daily_reading_minutes — draft invented
reading_time_hours/books_completed); notes/highlights list endpoints
return BARE arrays (no {notes:[]}/{highlights:[]} wrapper); create-note
request has no percentage_location/epubcfi_location. New
/tmp/merge_arrays.py handles array-example merges.

make test 9/9.
This commit is contained in:
John O'Keefe committed 2026-10-24 17:47:07 -04:00
1 parent cc5677eb17
commit 7a7b2ab82a
15 files changed
+277 -33

No files matched your search

+17 -6
View File
@@ -1,3 +1,11 @@
---
method: GET
path: /api/analytics/reading-stats
auth: jwt
area: analytics
---
<!-- source: internal/handlers/analytics.go:72 AnalyticsHandler.GetReadingStats -->
# Get Reading Statistics
Retrieve reading statistics for a date range.
@@ -27,13 +35,16 @@ Authorization: Bearer eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9...
## Response (200 OK)
```json
```json example-response
{
"pages_read": 1250,
"books_completed": 5,
"reading_time_hours": 42.5,
"sessions_count": 28,
"average_session_minutes": 91
"average_session_time_minutes": null,
"completion_rate": null,
"daily_reading_minutes": null,
"longest_session_minutes": null,
"most_active_day_of_week": null,
"total_books_read": null,
"total_pages_read": null,
"total_reading_time_minutes": null
}
```
@@ -1,3 +1,11 @@
---
method: POST
path: /api/media-items/:id/highlights
auth: jwt
area: media-items
---
<!-- source: internal/handlers/media.go:2026 MediaHandler.CreateMediaHighlight -->
# Create Highlight
Create a new highlight for a media item.
@@ -25,7 +33,7 @@ Create a new highlight for a media item.
### Example Request
```json
```json example-request
{
"selection_text": "Highlighted text...",
"start_position": "epubcfi(/6/4/2:15)",
@@ -38,7 +46,7 @@ Create a new highlight for a media item.
## Response (201 Created)
```json
```json example-response
{
"id": "uuid",
"media_item_id": "uuid",
@@ -49,7 +57,24 @@ Create a new highlight for a media item.
"color": "#ffff00",
"percentage_start": 0.45,
"percentage_end": 0.47,
"created_at": "2026-01-31T10:00:00Z"
"created_at": "2026-01-31T10:00:00Z",
"chapter_reference": null,
"character_end": null,
"character_start": null,
"dedup_key": null,
"deleted": null,
"deleted_at": null,
"device_sync_data": null,
"epubcfi_end": null,
"epubcfi_start": null,
"last_modified_at": null,
"last_modified_source": null,
"note_id": null,
"note_text": null,
"panel_number": null,
"paragraph_end": null,
"paragraph_start": null,
"updated_at": null
}
```
@@ -1,3 +1,11 @@
---
method: DELETE
path: /api/media-items/:id/highlights/:highlightId
auth: jwt
area: media-items
---
<!-- source: internal/handlers/media.go:2155 MediaHandler.DeleteMediaHighlight -->
# Delete Highlight
Delete a highlight.
@@ -1,3 +1,11 @@
---
method: GET
path: /api/media-items/:id/highlights
auth: jwt
area: media-items
---
<!-- source: internal/handlers/media.go:2001 MediaHandler.GetMediaHighlights -->
# Get Highlights
Retrieve all highlights for a specific media item.
@@ -26,9 +34,8 @@ Authorization: Bearer eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9...
## Response (200 OK)
```json
{
"highlights": [
```json example-response
[
{
"id": "uuid",
"media_item_id": "uuid",
@@ -43,12 +50,26 @@ Authorization: Bearer eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9...
"character_end": 15480,
"epubcfi_start": "epubcfi(/6/4/2:15)",
"epubcfi_end": "epubcfi(/6/4/2:20)",
"created_at": "2026-01-31T10:00:00Z"
"created_at": "2026-01-31T10:00:00Z",
"chapter_reference": null,
"dedup_key": "",
"deleted": false,
"deleted_at": null,
"device_sync_data": "",
"last_modified_at": null,
"last_modified_source": "",
"note_id": null,
"note_text": null,
"panel_number": null,
"paragraph_end": null,
"paragraph_start": null,
"updated_at": null
}
]
}
]
```
Response is a bare JSON array of MediaHighlights rows — no wrapper object.
## Error Responses
| Code | Description |
@@ -1,3 +1,11 @@
---
method: PUT
path: /api/media-items/:id/highlights/:highlightId
auth: jwt
area: media-items
---
<!-- source: internal/handlers/media.go:2097 MediaHandler.UpdateMediaHighlight -->
# Update Highlight
Update an existing highlight.
@@ -21,7 +29,7 @@ Update an existing highlight.
### Example Request
```json
```json example-request
{
"selection_text": "Updated text",
"color": "#00ff00"
@@ -30,14 +38,35 @@ Update an existing highlight.
## Response (200 OK)
```json
```json example-response
{
"id": "uuid",
"media_item_id": "uuid",
"user_id": "uuid",
"selection_text": "Updated text",
"color": "#00ff00",
"updated_at": "2026-01-31T10:00:00Z"
"updated_at": "2026-01-31T10:00:00Z",
"chapter_reference": null,
"character_end": null,
"character_start": null,
"created_at": null,
"dedup_key": null,
"deleted": null,
"deleted_at": null,
"device_sync_data": null,
"end_position": null,
"epubcfi_end": null,
"epubcfi_start": null,
"last_modified_at": null,
"last_modified_source": null,
"note_id": null,
"note_text": null,
"panel_number": null,
"paragraph_end": null,
"paragraph_start": null,
"percentage_end": null,
"percentage_start": null,
"start_position": null
}
```
+23 -5
View File
@@ -1,3 +1,11 @@
---
method: POST
path: /api/media-items/:id/notes
auth: jwt
area: media-items
---
<!-- source: internal/handlers/media.go:1876 MediaHandler.CreateMediaNote -->
# Create Note
Create a new note for a media item.
@@ -23,18 +31,17 @@ Create a new note for a media item.
### Example Request
```json
```json example-request
{
"content": "This is a note",
"position": "epubcfi(/6/4/2:15)",
"percentage_location": 0.45,
"epubcfi_location": "epubcfi(/6/4/2:15)"
"origin": "web"
}
```
## Response (201 Created)
```json
```json example-response
{
"id": "uuid",
"media_item_id": "uuid",
@@ -43,7 +50,18 @@ Create a new note for a media item.
"position": "epubcfi(/6/4/2:15)",
"percentage_location": 0.45,
"epubcfi_location": "epubcfi(/6/4/2:15)",
"created_at": "2026-01-31T10:00:00Z"
"created_at": "2026-01-31T10:00:00Z",
"chapter_reference": null,
"character_end": null,
"character_start": null,
"dedup_key": null,
"deleted": null,
"deleted_at": null,
"device_sync_data": null,
"last_modified_at": null,
"last_modified_source": null,
"paragraph_reference": null,
"updated_at": null
}
```
+8
View File
@@ -1,3 +1,11 @@
---
method: DELETE
path: /api/media-items/:id/notes/:noteId
auth: jwt
area: media-items
---
<!-- source: internal/handlers/media.go:1980 MediaHandler.DeleteMediaNote -->
# Delete Note
Delete a note.
+22 -6
View File
@@ -1,3 +1,11 @@
---
method: GET
path: /api/media-items/:id/notes
auth: jwt
area: media-items
---
<!-- source: internal/handlers/media.go:1851 MediaHandler.GetMediaNotes -->
# Get Notes
Retrieve all notes for a specific media item.
@@ -26,9 +34,8 @@ Authorization: Bearer eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9...
## Response (200 OK)
```json
{
"notes": [
```json example-response
[
{
"id": "uuid",
"media_item_id": "uuid",
@@ -40,12 +47,21 @@ Authorization: Bearer eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9...
"character_end": 15480,
"epubcfi_location": "epubcfi(/6/4/2:15)",
"created_at": "2026-01-31T10:00:00Z",
"updated_at": "2026-01-31T10:00:00Z"
"updated_at": "2026-01-31T10:00:00Z",
"chapter_reference": null,
"dedup_key": "",
"deleted": false,
"deleted_at": null,
"device_sync_data": "",
"last_modified_at": null,
"last_modified_source": "",
"paragraph_reference": null
}
]
}
]
```
Response is a bare JSON array of MediaNotes rows — no wrapper object.
## Error Responses
| Code | Description |
+24 -3
View File
@@ -1,3 +1,11 @@
---
method: PUT
path: /api/media-items/:id/notes/:noteId
auth: jwt
area: media-items
---
<!-- source: internal/handlers/media.go:1932 MediaHandler.UpdateMediaNote -->
# Update Note
Update an existing note.
@@ -21,7 +29,7 @@ Update an existing note.
### Example Request
```json
```json example-request
{
"content": "Updated note content",
"position": "epubcfi(/6/4/2:20)"
@@ -30,14 +38,27 @@ Update an existing note.
## Response (200 OK)
```json
```json example-response
{
"id": "uuid",
"media_item_id": "uuid",
"user_id": "uuid",
"content": "Updated note content",
"position": "epubcfi(/6/4/2:20)",
"updated_at": "2026-01-31T10:00:00Z"
"updated_at": "2026-01-31T10:00:00Z",
"chapter_reference": null,
"character_end": null,
"character_start": null,
"created_at": null,
"dedup_key": null,
"deleted": null,
"deleted_at": null,
"device_sync_data": null,
"epubcfi_location": null,
"last_modified_at": null,
"last_modified_source": null,
"paragraph_reference": null,
"percentage_location": null
}
```
+8
View File
@@ -1,3 +1,11 @@
---
method: GET
path: /opds/devices/:deviceId/download/:bookId
auth: device-token
area: opds
---
<!-- source: internal/handlers/opds.go:795 OPDSHandler.DownloadBook -->
# OPDS Acquisition
Download books and list available formats.
+8
View File
@@ -1,3 +1,11 @@
---
method: GET
path: /opds/devices/:deviceId/catalog
auth: device-token
area: opds
---
<!-- source: internal/handlers/opds.go:187 OPDSHandler.GetDeviceCatalog -->
# OPDS Feeds
Bookhoard provides OPDS 1.2 feeds for device compatibility.
+8
View File
@@ -1,3 +1,11 @@
---
method: GET
path: /opds/devices/:deviceId/cover/:bookId
auth: device-token
area: opds
---
<!-- source: internal/handlers/opds.go:957 OPDSHandler.GetCoverImage -->
# OPDS Publication
OPDS navigation and publication feeds.
@@ -1,3 +1,11 @@
---
method: DELETE
path: /api/media-items/:id/progress
auth: jwt
area: media-items
---
<!-- source: internal/handlers/media.go:1350 MediaHandler.DeleteMediaReadingProgress -->
# Delete Media Item Progress
Delete the stored reading progress for a media item.
@@ -20,7 +28,7 @@ Authorization: Bearer eyJhbGciOiJIUzI1NiIs...
## Response (200 OK)
```json
```json example-response
{
"message": "reading progress deleted"
}
+11
View File
@@ -30,6 +30,7 @@ type extractedRoute struct {
var (
groupRe = regexp.MustCompile(`^(\w+)\s*:=\s*(\S+)\.Group\("([^"]*)"(?:,\s*(.+))?\)`)
useRe = regexp.MustCompile(`^(\w+)\.Use\((.+)\)$`)
echoRouteRe = regexp.MustCompile(`^(?:e|cfg\.Echo)\.(GET|POST|PUT|PATCH|DELETE)\("([^"]*)"(?:,\s*(.+))?\)`)
groupRouteRe = regexp.MustCompile(`^(\w+)\.(GET|POST|PUT|PATCH|DELETE)\("([^"]*)"(?:,\s*(.+))?\)`)
)
@@ -94,6 +95,16 @@ func extractRoutesFromRouterSource(t *testing.T) map[string]extractedRoute {
continue
}
//opds-style middleware: `grp.Use(cfg.DeviceAuthMiddleware.Authenticate)`
//applies to every route registered on that group afterwards.
if m := useRe.FindStringSubmatch(s); m != nil {
if g, ok := groups[m[1]]; ok {
g.middleware = append(g.middleware, splitMW(m[2])...)
groups[m[1]] = g
}
continue
}
if m := echoRouteRe.FindStringSubmatch(s); m != nil {
add(m[1], m[2], splitMW(m[3]))
continue
+44
View File
@@ -726,4 +726,48 @@ var Schemas = map[string]Schema{
"require_number": true, "require_special": true
}`),
},
// --- Beam batch 9: highlights, notes, progress delete, analytics, opds ---
"POST /api/media-items/:id/highlights": {
Request: handlers.CreateMediaHighlightRequest{},
Response: database.MediaHighlights{},
},
"GET /api/media-items/:id/highlights": {
Response: []database.MediaHighlights{},
},
"PUT /api/media-items/:id/highlights/:highlightId": {
// LWW edit targeted by URL id (identity NOT re-derived from content).
Request: handlers.UpdateMediaHighlightRequest{},
Response: database.MediaHighlights{},
},
"DELETE /api/media-items/:id/highlights/:highlightId": {
// Tombstone via sync service; 204 No Content.
},
"POST /api/media-items/:id/notes": {
Request: handlers.CreateMediaNoteRequest{},
Response: database.MediaNotes{},
},
"GET /api/media-items/:id/notes": {
Response: []database.MediaNotes{},
},
"PUT /api/media-items/:id/notes/:noteId": {
Request: handlers.UpdateMediaNoteRequest{},
Response: database.MediaNotes{},
},
"DELETE /api/media-items/:id/notes/:noteId": {
// Tombstone; 204 No Content.
},
"DELETE /api/media-items/:id/progress": {
Response: json.RawMessage(`{"message": ""}`),
},
"GET /api/analytics/reading-stats": {
// Query: start_date/end_date in MM-DD-YYYY, user-timezone buckets.
Response: handlers.ReadingStatsResponse{},
},
// OPDS: XML/Atom responses (content negotiated, not JSON) — response
// side intentionally unpinned.
"GET /opds/devices/:deviceId/catalog": {},
"GET /opds/devices/:deviceId/download/:bookId": {},
"GET /opds/devices/:deviceId/cover/:bookId": {},
}