erik/slugkit

Fork 0

Implement slug comments commands #7

New issue

Closed

opened 2026-05-19 19:51:44 -05:00 by erik · 2 comments

erik commented

2026-05-19 19:51:44 -05:00

Owner

Goal

Implement slug comment listing and moderation commands.

Spec: docs/cli-specs/11-comments-commands.md

Requirements

Implement post comments listing, pending comments listing, approve, and hide commands from the spec.
Support --status pending|approved|hidden|all for post comment listing.
Display target site/profile before moderation mutations.
Support valid JSON output where specified.
Surface validation, auth, not-found, network, API, and 501 Not Implemented responses clearly.
Add CLI tests with mocked API responses for listing, filtering, approve/hide, missing resources, and unsupported behavior.

Acceptance criteria

Can list comments for a post.
Can list pending comments.
Can approve a comment.
Can hide a comment.
Missing posts/comments display a clear error.
JSON output is valid when requested.
501 Not Implemented responses are displayed clearly.
Tests cover success and common API errors.
Relevant lint/test checks pass.

Dependencies

task-a3cd5595
task-64b4aa7e

## Goal Implement `slug` comment listing and moderation commands. Spec: `docs/cli-specs/11-comments-commands.md` ## Requirements - Implement post comments listing, pending comments listing, approve, and hide commands from the spec. - Support `--status pending|approved|hidden|all` for post comment listing. - Display target site/profile before moderation mutations. - Support valid JSON output where specified. - Surface validation, auth, not-found, network, API, and `501 Not Implemented` responses clearly. - Add CLI tests with mocked API responses for listing, filtering, approve/hide, missing resources, and unsupported behavior. ## Acceptance criteria - [ ] Can list comments for a post. - [ ] Can list pending comments. - [ ] Can approve a comment. - [ ] Can hide a comment. - [ ] Missing posts/comments display a clear error. - [ ] JSON output is valid when requested. - [ ] `501 Not Implemented` responses are displayed clearly. - [ ] Tests cover success and common API errors. - [ ] Relevant lint/test checks pass. ## Dependencies - task-a3cd5595 - task-64b4aa7e

erik added the

labels

2026-05-19 19:51:44 -05:00

erik added

status:inprogress

and removed

status:active

labels

2026-06-03 22:42:54 -05:00

erik commented

2026-06-03 22:48:17 -05:00

Author

Owner

Synced from todu comment by @todu on 2026-06-04T03:44:32.753Z

PR Review

PR: #89

Review result: approved

Acceptance criteria: 9 total, 9 met, 0 missing

Blocking issues: 0
Warnings: 0

Notes:

Reviewed against the user-approved command-structure override: slug comment list|approve|hide.

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.

_Synced from todu comment by @todu on 2026-06-04T03:44:32.753Z_ ## PR Review PR: https://forge.caradoc.com/erik/slugkit/pulls/89 Review result: approved Acceptance criteria: 9 total, 9 met, 0 missing Blocking issues: 0 Warnings: 0 Notes: - Reviewed against the user-approved command-structure override: `slug comment list|approve|hide`. 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

2026-06-03 23:25:57 -05:00

closed this issue
added
status:done
and removed
status:inprogress
labels

erik commented

2026-06-03 23:25:57 -05:00

Author

Owner

Synced from todu comment by @todu on 2026-06-04T04:23:33.144Z

Completed

Implemented slug comment list, slug comment approve, and slug comment hide for post/pending comment listing and moderation.
Verified PR #89 was approved with 9/9 acceptance criteria met, ./scripts/pre-pr.sh passed, and Forgejo CI CI / build-lint-test (pull_request) succeeded.
Verified the PR was merged into main as commit 14a9247 (feat: add comment moderation CLI).

_Synced from todu comment by @todu on 2026-06-04T04:23:33.144Z_ ### Completed - Implemented `slug comment list`, `slug comment approve`, and `slug comment hide` for post/pending comment listing and moderation. - Verified PR #89 was approved with 9/9 acceptance criteria met, `./scripts/pre-pr.sh` passed, and Forgejo CI `CI / build-lint-test (pull_request)` succeeded. - Verified the PR was merged into `main` as commit `14a9247` (`feat: add comment moderation CLI`).