Implement slug followers command #10

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 followers list against the Slugkit followers API.

Spec: docs/cli-specs/08-followers-commands.md

Requirements

Implement the followers command from the spec.
Call GET /api/v1/followers using the configured profile.
Render human-readable follower output and valid JSON output.
Handle empty follower lists cleanly.
Surface auth, network, API, and 501 Not Implemented responses clearly.
Add CLI tests with mocked API responses for populated list, empty list, and errors.

Acceptance criteria

Can list followers.
Empty follower list prints a useful message.
--json output is valid.
501 Not Implemented responses are displayed clearly.
Tests cover populated, empty, and failure responses.
Relevant lint/test checks pass.

Dependencies

task-a3cd5595
task-a5f5c185

## Goal Implement `slug followers list` against the Slugkit followers API. Spec: `docs/cli-specs/08-followers-commands.md` ## Requirements - Implement the followers command from the spec. - Call `GET /api/v1/followers` using the configured profile. - Render human-readable follower output and valid JSON output. - Handle empty follower lists cleanly. - Surface auth, network, API, and `501 Not Implemented` responses clearly. - Add CLI tests with mocked API responses for populated list, empty list, and errors. ## Acceptance criteria - [ ] Can list followers. - [ ] Empty follower list prints a useful message. - [ ] `--json` output is valid. - [ ] `501 Not Implemented` responses are displayed clearly. - [ ] Tests cover populated, empty, and failure responses. - [ ] Relevant lint/test checks pass. ## Dependencies - task-a3cd5595 - task-a5f5c185

erik added the

labels

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

erik added

status:inprogress

and removed

status:active

labels

2026-06-04 07:18:38 -05:00

erik commented

2026-06-04 07:18:39 -05:00

Author

Owner

Synced from todu comment by @todu on 2026-06-04T12:16:45.632Z

PR Review

PR: #92

Review result: approved

Acceptance criteria: 6 total, 6 met, 0 missing

Blocking issues: 0
Warnings: 0

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-04T12:16:45.632Z_ ## PR Review PR: https://forge.caradoc.com/erik/slugkit/pulls/92 Review result: approved Acceptance criteria: 6 total, 6 met, 0 missing Blocking issues: 0 Warnings: 0 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-04 07:24:07 -05:00

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

erik commented

2026-06-04 07:24:08 -05:00

Author

Owner

Synced from todu comment by @todu on 2026-06-04T12:21:31.441Z

Completed

Implemented slug followers list and merged PR #92 into main (f32a18b, feature commit ce46537).
Verified acceptance criteria from current evidence: populated and empty follower list handling, valid --json, clear 501/API/auth/network errors, and mocked CLI coverage are present in cli/src/__tests__/commands.test.ts.
Verification evidence: prior PR review approved with 6/6 criteria met, ./scripts/pre-pr.sh passed locally, Forgejo CI CI / build-lint-test (pull_request) succeeded, and origin/main contains the merge commit.

_Synced from todu comment by @todu on 2026-06-04T12:21:31.441Z_ ### Completed - Implemented `slug followers list` and merged PR #92 into `main` (`f32a18b`, feature commit `ce46537`). - Verified acceptance criteria from current evidence: populated and empty follower list handling, valid `--json`, clear 501/API/auth/network errors, and mocked CLI coverage are present in `cli/src/__tests__/commands.test.ts`. - Verification evidence: prior PR review approved with 6/6 criteria met, `./scripts/pre-pr.sh` passed locally, Forgejo CI `CI / build-lint-test (pull_request)` succeeded, and `origin/main` contains the merge commit.