erik/slugkit
1
0
Fork 0
Code Issues 6 Pull requests Projects Releases 6 Packages Wiki Activity Actions

feat: add media API routes #88

Merged
erik merged 1 commit from feat/task-d520567e-media-api-routes into main 2026-06-03 21:51:38 -05:00
Conversation 1 Commits 1 Files changed 9 +600 -23
erik commented 2026-06-03 15:03:28 -05:00
Owner
Copy link

Summary

Implement authenticated media upload, metadata lookup, and deletion routes backed by the existing media storage service and SQLite metadata.

Task

#task-d520567e

Changes

  • Add /api/v1/media upload and /api/v1/media/{id} get/delete routes
  • Wire media routes to injectable S3-compatible storage/config and existing SQLite media metadata service
  • Add shared media error responses for validation, not found, storage, oversized upload, and unsupported MIME type failures
  • Document media routes and schemas in OpenAPI with spec operation IDs
  • Add media API route tests and OpenAPI coverage

Testing

  • Unit tests added/updated
  • Manual testing performed

Checklist

  • ./scripts/pre-pr.sh passes
  • Documentation updated (OpenAPI)
  • No unrelated changes included
## Summary Implement authenticated media upload, metadata lookup, and deletion routes backed by the existing media storage service and SQLite metadata. ## Task #task-d520567e ## Changes - Add `/api/v1/media` upload and `/api/v1/media/{id}` get/delete routes - Wire media routes to injectable S3-compatible storage/config and existing SQLite media metadata service - Add shared media error responses for validation, not found, storage, oversized upload, and unsupported MIME type failures - Document media routes and schemas in OpenAPI with spec operation IDs - Add media API route tests and OpenAPI coverage ## Testing - [x] Unit tests added/updated - [ ] Manual testing performed ## Checklist - [x] `./scripts/pre-pr.sh` passes - [x] Documentation updated (OpenAPI) - [x] No unrelated changes included
feat: add media API routes
All checks were successful
CI / build-lint-test (pull_request) Successful in 34s
Details
553e3a498b 
Task: #task-d520567e
erik commented 2026-06-03 15:04:41 -05:00
Author
Owner
Copy link

PR Review

Summary

Reviewed PR #88 against task task-d520567e and docs/specs/07-media-routes.md. The PR adds authenticated media upload, get, and delete routes wired to the existing media storage service, SQLite metadata, shared error shapes, OpenAPI docs, and tests.

Acceptance criteria

  • Can upload media through /api/v1/media as documented.
  • Upload creates storage object and database record.
  • Response includes the documented media metadata and public URL.
  • Can delete media through the documented route.
  • Unsupported type and oversized uploads fail with shared error responses.
  • Every media route appears in OpenAPI with documented operation IDs.
  • Relevant lint/test checks pass.

Blocking issues

None.

Warnings

None.

Verification

  • ./scripts/pre-pr.sh passed locally.
  • Forgejo CI status for CI / build-lint-test (pull_request) is success.

Verdict

Approved. Ready for human merge approval.

## PR Review ### Summary Reviewed PR #88 against task `task-d520567e` and `docs/specs/07-media-routes.md`. The PR adds authenticated media upload, get, and delete routes wired to the existing media storage service, SQLite metadata, shared error shapes, OpenAPI docs, and tests. ### Acceptance criteria - [x] Can upload media through `/api/v1/media` as documented. - [x] Upload creates storage object and database record. - [x] Response includes the documented media metadata and public URL. - [x] Can delete media through the documented route. - [x] Unsupported type and oversized uploads fail with shared error responses. - [x] Every media route appears in OpenAPI with documented operation IDs. - [x] Relevant lint/test checks pass. ### Blocking issues None. ### Warnings None. ### Verification - `./scripts/pre-pr.sh` passed locally. - Forgejo CI status for `CI / build-lint-test (pull_request)` is `success`. ### Verdict Approved. Ready for human merge approval.
erik merged commit 952deb6b84 into main 2026-06-03 21:51:38 -05:00
erik deleted branch feat/task-d520567e-media-api-routes 2026-06-03 21:51:38 -05:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
activitypub

   
admin

   
api

   
articles

   
auth

   
bug

   
cleanup

   
cli

   
comments

   
compatibility

   
config

   
contacts

   
database

   
deployment

   
design

   
dev-env

   
docs

   
documentation

   
email

   
enhancement

   
feature

   
federation

   
feed

   
homepage

   
implementation

   
integration

   
media

   
openapi

   
priority:high

   
priority:low

   
priority:medium

   
proof

   
public-routes

   
public-ui

   
release

   
safety

   
social

   
sources

   
status:active

   
status:canceled

   
status:done

   
status:inprogress

   
status:waiting

   
syndication

   
tailwind

   
template

   
test

   
web
No labels
activitypub
admin
api
articles
auth
bug
cleanup
cli
comments
compatibility
config
contacts
database
deployment
design
dev-env
docs
documentation
email
enhancement
feature
federation
feed
homepage
implementation
integration
media
openapi
priority:high
priority:low
priority:medium
proof
public-routes
public-ui
release
safety
social
sources
status:active
status:canceled
status:done
status:inprogress
status:waiting
syndication
tailwind
template
test
web
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
erik/slugkit!88
No description provided.