feat: move post like and boost routes #83

Merged

erik merged 1 commit from feat/task-6cf351a2-move-post-engagement-routes into main

2026-06-02 10:11:07 -05:00

erik commented

2026-06-02 09:30:53 -05:00

Owner

Summary\n- Move post like and boost actor-list API routes to /posts/{slug}/likes and /posts/{slug}/boosts.\n- Update OpenAPI operation IDs to posts.likes.list and posts.boosts.list.\n- Update API docs/specs and tests to assert old engagement-scoped actor-list routes are no longer documented or routed.\n\n## Verification\n- npm exec --workspace @slugkit/template-site -- vitest run --config ../../vitest.config.ts src/api/tests/engagement.test.ts src/api/tests/routes.test.ts\n- npm test\n- npm run lint\n- ./scripts/pre-pr.sh\n\nTask: #task-6cf351a2

## Summary\n- Move post like and boost actor-list API routes to /posts/{slug}/likes and /posts/{slug}/boosts.\n- Update OpenAPI operation IDs to posts.likes.list and posts.boosts.list.\n- Update API docs/specs and tests to assert old engagement-scoped actor-list routes are no longer documented or routed.\n\n## Verification\n- npm exec --workspace @slugkit/template-site -- vitest run --config ../../vitest.config.ts src/api/__tests__/engagement.test.ts src/api/__tests__/routes.test.ts\n- npm test\n- npm run lint\n- ./scripts/pre-pr.sh\n\nTask: #task-6cf351a2

erik added 1 commit

2026-06-02 09:30:53 -05:00

feat: move post like and boost routes

CI / build-lint-test (pull_request) Successful in 30s

Details

9432c1719e

Task: #task-6cf351a2

erik commented

2026-06-02 09:32:04 -05:00

Author

Owner

PR Review

Summary

Reviewed PR #83 for task task-6cf351a2. The changes move post like and boost actor-list routes from the engagement summary namespace to direct post-resource routes, update OpenAPI/docs, and add tests for new routes, old-route removal, auth/error operation IDs, and unchanged response shapes.

Acceptance criteria

GET /api/v1/posts/{slug}/likes lists post like actors.
GET /api/v1/posts/{slug}/boosts lists post boost actors.
GET /api/v1/posts/{slug}/engagement/likes is no longer documented or routed.
GET /api/v1/posts/{slug}/engagement/boosts is no longer documented or routed.
OpenAPI uses posts.likes.list and posts.boosts.list.
GET /api/v1/posts/{slug}/engagement remains available for aggregate counts.
Relevant tests and checks pass.

Blocking issues

None.

Warnings

None.

Verification reviewed

Targeted API/OpenAPI tests passed.
npm test passed.
npm run lint passed.
./scripts/pre-pr.sh passed.
Forgejo CI status is success for commit 9432c1719ebcebba3cdb3dc08b83f41d492c55b8.

Verdict

Approved. Ready for human merge approval.

PR Review Worker Result

pr: open
review: approved
pr_comment: posted
task_comment: posted
blocking_issues: 0
warnings: 0
criteria_total: 7
criteria_met: 7
criteria_missing: 0

## PR Review ### Summary Reviewed PR #83 for task `task-6cf351a2`. The changes move post like and boost actor-list routes from the engagement summary namespace to direct post-resource routes, update OpenAPI/docs, and add tests for new routes, old-route removal, auth/error operation IDs, and unchanged response shapes. ### Acceptance criteria - [x] `GET /api/v1/posts/{slug}/likes` lists post like actors. - [x] `GET /api/v1/posts/{slug}/boosts` lists post boost actors. - [x] `GET /api/v1/posts/{slug}/engagement/likes` is no longer documented or routed. - [x] `GET /api/v1/posts/{slug}/engagement/boosts` is no longer documented or routed. - [x] OpenAPI uses `posts.likes.list` and `posts.boosts.list`. - [x] `GET /api/v1/posts/{slug}/engagement` remains available for aggregate counts. - [x] Relevant tests and checks pass. ### Blocking issues None. ### Warnings None. ### Verification reviewed - Targeted API/OpenAPI tests passed. - `npm test` passed. - `npm run lint` passed. - `./scripts/pre-pr.sh` passed. - Forgejo CI status is `success` for commit `9432c1719ebcebba3cdb3dc08b83f41d492c55b8`. ### Verdict Approved. Ready for human merge approval. PR Review Worker Result - pr: open - review: approved - pr_comment: posted - task_comment: posted - blocking_issues: 0 - warnings: 0 - criteria_total: 7 - criteria_met: 7 - criteria_missing: 0

erik referenced this pull request

2026-06-02 09:36:17 -05:00

Move post like and boost routes #79

erik merged commit 968d757ea4 into main

2026-06-02 10:11:07 -05:00

erik deleted branch feat/task-6cf351a2-move-post-engagement-routes

2026-06-02 10:11:07 -05:00

erik referenced this pull request from a commit

2026-06-02 10:11:09 -05:00

Merge pull request 'feat: move post like and boost routes' (#83) from feat/task-6cf351a2-move-post-engagement-routes into main

erik referenced this pull request

2026-06-02 10:13:51 -05:00