fix: forgejo-cli --labels now passes integers instead of strings (fixes #48) #55

Merged

jasper merged 1 commit from fix/forgejo-labels-string-int into main

2026-06-03 23:08:00 -04:00

jasper commented

2026-06-03 23:06:38 -04:00

Contributor

Summary

Fixes #48 — the --labels flag was passing string values to the Forgejo API, which expects integers.

Changes

forgejo-cli/scripts/forgejo-cli: Synced repo copy with installed version (full feature set). Fixed line 480: labels are now parsed as int() with .isdigit() guard.
forgejo-cli/SKILL.md: Added --labels pitfall section, updated Known Gaps table (removed the fixed entry), added issue create --labels example, bumped to v1.1.1.

Testing

Parsing verified: [int(x.strip()) for x in '8,9'.split(',') if x.strip().isdigit()] → [8, 9] all ints
Created test issue #54 with --labels 7 — returned 201 with enhancement label attached (no 422)
Closed test issue after verification

## Summary Fixes #48 — the `--labels` flag was passing string values to the Forgejo API, which expects integers. ## Changes - **`forgejo-cli/scripts/forgejo-cli`**: Synced repo copy with installed version (full feature set). Fixed line 480: labels are now parsed as `int()` with `.isdigit()` guard. - **`forgejo-cli/SKILL.md`**: Added `--labels` pitfall section, updated Known Gaps table (removed the fixed entry), added `issue create --labels` example, bumped to v1.1.1. ## Testing - Parsing verified: `[int(x.strip()) for x in '8,9'.split(',') if x.strip().isdigit()]` → `[8, 9]` all ints - Created test issue #54 with `--labels 7` — returned 201 with `enhancement` label attached (no 422) - Closed test issue after verification

jasper added 1 commit

2026-06-03 23:06:39 -04:00

fix: forgejo-cli --labels now passes integers instead of strings (fixes #48 ) 90b0c6af90

The --labels flag was parsing label IDs as strings (e.g. ['8']), causing a 422
error because the Forgejo API expects integers (e.g. [8]).

- Added int() conversion with .isdigit() guard at line 480
- Synced repo script to match installed version (full feature set)
- Updated SKILL.md: added labels pitfall, updated Known Gaps, bumped to v1.1.1

jasper merged commit 89b19b6d24 into main

2026-06-03 23:08:00 -04:00

jasper referenced this pull request from a commit

2026-06-03 23:08:02 -04:00

fix: forgejo-cli --labels now passes integers instead of strings (fixes #48) (#55)