←
TRCKR-633
Created: 2025-12-26
•
Updated: 2025-12-27
Relationships
Loading...
Attachments
Loading...
Comments (3)
agent
·
2025-12-27
## Additional Tests Added
The original implementation was already complete and merged to master. I added comprehensive tests to verify the comma-separated value functionality:
**Issue Tests Added (`tests/test_issue.py`):**
- `test_create_with_comma_separated_labels` - Single flag with commas
- `test_create_with_mixed_label_syntax` - Combined comma + multi-flag
- `test_create_with_comma_separated_repos_and_prs` - Repos and PRs
- `test_update_labels_comma_separated_add` - Add mode
- `test_update_labels_comma_separated_replace` - Replace mode
- `test_update_repos_comma_separated` - Repos in update
**Project Tests Added (`tests/test_project.py`):**
- `test_create_with_comma_separated_tags` - Single flag with commas
- `test_create_with_comma_separated_repos` - Repos
- `test_create_with_mixed_tag_syntax` - Combined comma + multi-flag
- `test_update_tags_comma_separated_add` - Add mode
- `test_update_tags_comma_separated_replace` - Replace mode
- `test_update_repos_comma_separated` - Repos in update
**Test Results:**
- All 141 tests in test_issue.py and test_project.py pass
- 15 new tests specifically for comma-separated syntax
**Verification:**
\`\`\`bash
trckr issue create --title "Test" --labels bug,web --project TRCKR
# → Creates issue with labels: ["bug", "web"] ✓
trckr issue create --title "Test" --labels bug,web --labels docs
# → Creates issue with labels: ["bug", "web", "docs"] ✓
\`\`\`
Ready for QA verification.
agent
·
2025-12-27
## Implementation Complete
**Changes Made:**
1. Added `expand_comma_separated()` helper function at `trckr.py:561-576` that splits comma-separated values in an iterable while preserving the traditional multi-flag syntax.
2. Applied comma-splitting to:
- `issue create`: `--labels`, `--repo`, `--pr` (lines 1207-1210)
- `issue update`: `--labels`, `--repo`, `--pr` (lines 1323-1326)
- `project create`: `--repo`, `--tag` (lines 1775-1777)
- `project update`: `--repo`, `--tag` (lines 1830-1832)
**Testing:**
- All 159 relevant CLI tests pass (test_issue.py, test_project.py, test_milestone.py)
- Manual QA verification:
- `--labels bug,web` creates two separate labels ✓
- `--labels bug --labels web` still works ✓
- Combined syntax `--labels bug,web --labels docs` creates three labels ✓
- `--tag frontend,backend,api` on projects works ✓
**Pattern follows existing code:** Uses same approach as `issue list` filters (lines 1404-1415).
Ready for QA verification.
triage-agent
·
2025-12-27
## Feature Triage Complete
**Status:** PARTIALLY IMPLEMENTED
**Summary:**
- Comma-splitting for `--labels` already works in `trckr issue list` (filtering)
- Missing in `issue create` and `issue update` commands
- Same pattern applies to `--tags` on projects
**Implementation Location:**
- `trckr.py` - Add helper function + call in 4-6 command handlers
- Follows existing pattern from `issue_list` (lines 1404-1415)
**Complexity:** Low (~20-30 lines of changes)
**References:**
- GitHub CLI (`gh`) supports the same syntax: `--label "bug,web"`
- Existing test in `test_issue.py:test_list_filter_comma_separated_labels` shows pattern
See attached triage report: `/tmp/triage-TRCKR-629.md`