-
-
Notifications
You must be signed in to change notification settings - Fork 1
feat(commands): use positional args for org/project selection #155
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Replace --org and --project flags with GitHub-style positional arguments for issue list and project list commands. New syntax: - `sentry issue list <org>/<project>` - explicit target - `sentry issue list <org>/` - all projects in org - `sentry issue list <project>` - find project across orgs - `sentry project list <org>` - list projects in org Also changes cross-org collision aliases from colon to slash separator (e.g., `o1/d` instead of `o1:d`) for consistency. BREAKING CHANGE: --org and --project flags removed from issue list and --org flag removed from project list commands.
Semver Impact of This PR🟡 Minor (new features) 📋 Changelog PreviewThis is how your changes will appear in the changelog. New Features ✨
Bug Fixes 🐛
Documentation 📚
Internal Changes 🔧
🤖 This preview updates automatically when you update the PR. |
Codecov Results 📊✅ Patch coverage is 97.92%. Project has 1959 uncovered lines. Files with missing lines (28)
Coverage diff@@ Coverage Diff @@
## main #PR +/-##
==========================================
+ Coverage 69.39% 69.68% +0.29%
==========================================
Files 56 56 —
Lines 6416 6462 +46
Branches 0 0 —
==========================================
+ Hits 4452 4503 +51
- Misses 1964 1959 -5
- Partials 0 0 —Generated by Codecov Action |
- Fix JSDoc cross-org example notation (org1:dashboard -> org1/dashboard) - Add -n short-hand flag for --limit in project list - Simplify map callback in project list - Change internal alias map key notation from : to / - Simplify findProjectsBySlug filter pattern - Remove ASCII art section dividers from resolve-target.ts - Add ProjectSpecificationType constant for discriminated union - Change /cli handling from auto-detect to project-search - Add prohibited comment styles rule to AGENTS.md
|
- Update project list tests to use positional org arg instead of --org flag - Update issue list tests to use org/project positional format - Regenerate SKILL.md
- Add placeholder properties to positional args for better help text - Add shortcut aliases: -q for query, -s for sort, -n for limit (issue list) - Add shortcut aliases: -p for platform (project list) - Add status filtering examples to issue docs (via --query) - Regenerate SKILL.md with proper argument names
- Add tests for ProjectSpecificationType constant values - Add tests for findProjectsBySlug function with various scenarios - Test handling of multiple orgs with matching projects - Test empty results when no projects match - Test graceful handling of 403 errors (permission denied)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
- Remove ProjectSpecificationType tests (testing constants is useless) - Fix footer tip to show correct project view syntax
Summary
Replace verbose
--organd--projectflags with GitHub-style positional arguments forissue listandproject listcommands, inspired by theghCLI.Before:
After:
Changes
parseOrgProjectArg()utility for parsing<org>/<project>patternsfindProjectsBySlug()API function to search projects across orgsissue listto use positional argument instead of flagsproject listto use positional argument instead of flag:to/(e.g.,o1/dinstead ofo1:d)Breaking Changes
--organd--projectflags removed fromsentry issue list--orgflag removed fromsentry project list